Re: [PATCH] rust/revocable: add try_with() convenience method

From: Benno Lossin
Date: Thu Mar 13 2025 - 11:39:38 EST


On Thu Mar 13, 2025 at 4:08 PM CET, Alexandre Courbot wrote:
> On Thu Mar 13, 2025 at 11:19 PM JST, Benno Lossin wrote:
>> Would it make sense to not use `Result` here and continue with `Option`?
>
> We would have to return an Option<Result<R>> in this case. The current
> code folds the closure's Result into the one of the guard's acquisition
> for convenience.
>
> Actually, I don't think I have ever used try_access() a single time
> without converting its returned Option into a Result. Wouldn't it make
> sense to do the opposite, i.e. make try_access() return Err(ENXIO) when
> the guard cannot be acquired and document this behavior?

Sure, if you're always doing

let guard = rev.try_access().ok_or(ENXIO)?;

Then it makes sense from my view, maybe Danilo has some other argument
for why `Option` is better.

---
Cheers,
Benno