- The meaning I intended was "the value specified by the empty string," rather than "the canonically specified absent value." I think this works fine in both of the places where the function is…
In assembly.rs, you put all of the sub-namespaces under a top-level item on a single line […], whereas here this has been spread across three lines. They should be consistent in format.
Good…
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)