Shane F. Carr
Shane F. Carr
Action: create an alias.
@eggrobin Could you take a look at the bullet points in this issue and determine what needs to be done?
@eggrobin Please take a look at this; thanks!
Discussion: - We're not fully sure whether we need both `c` and `e` in `PluralOperands` - To punt this problem, make `PluralOperands` be `non_exhaustive` and add a function `PluralOperands::from_ivwftc()` for...
The other option is we just change all call sites to do `Default::default()` and set the fields they need. Would that be more palatable?
The advantage of option 2 is that we don't need any more APIs or experimental features. Between 1 and 3, I slightly prefer 3 because it doesn't involve importing an...
After discussing with @markusicu and @echeran, the point was brought up that these fields should really not generally be things that users of ICU4X should know about or mutate. I'm...
- @zbraniecki - It seems useful to allow people to mutate their plural operands. - @sffc - Some of the operands, like v, w, f, and t, are not independent....
What to do next: - Make `icu_plurals::rules` experimental - Add a type `icu_plurals::rules::RawPluralOperands`, which is exhaustive with public fields - Add conversions between PluralOperands and RawPluralOperands - Update code that...
Blocked on #1856