But I don't think you can pass a Flag() to something that wants a boolean, so this isn't too useful...
There are various ways to streamline the use of MyBranches values in conditionals.…
For example, the expression
if true { 3 }won't compile unless you add anelseblock to convince the compiler that it has the same type in all situations.
This is one advantage of using…
Not necessarily, mostly just musing/wondering what Rust's behavior actually is when there is no else clause... if every expression/statement has a value, what is the value of an if let when…
Either of the last two is fine by me
Done (6eeeb1c).
p.s. it would be sort of nice, and reasonably intuitive, if the default else-branch of an if let was just else {false}.
If this is…
So I guess I am specifically suggesting replacing this block with
if let Ok(spec) = SpecifiedValue::try_from(value.get_clone_untracked()) { valid.set(true) …
I've pushed changes that should resolve all the remaining conversations. If you agree, you can go ahead and review! Otherwise, let me know what other changes are needed.
I've switched to Proposal 1c (84bfdef), which we adopted during today's meeting. The SpecifiedValue structure is read-only, courtesy of the readonly…
I don't actually see anything that needs to be done, do you? It seems like the defaults are all fine...
After looking it over during today's meeting, I agree that it seems possible to set up…
Your thoughts on just a method-only interface with an opaque underlying datatype that we can iterate on if we see fit without changing any client code?
At this point, I can't predict whether…
I've removed if_present, replacing it with matches!(/* ... */, Present { .. }) where it was used (c58fed0)
The idea of using a parameterized trait is nice, but in this case I'd prefer to use methods named set and set_if_ok. The former always updates the set point, while the latter only updates the…
This is not the place to debate Husht […]
Agreed. I think you get the tradeoff at this point, so let me know whether you want to have is_present.
Now a point might be that try_from doesn't give you a SpecifiedValue, it gives you a Result<SpecifiedValue, SomeKindaError>.
Exactly.
But then wouldn't it be most natural just to…