I don't believe e-prefix adds sufficient value. Mutable statics *must* always be s-prefixed. If this is not the case I will gladly bring the codebase into compliance.
On Fri, Jun 29, 2018 at 11:36 AM, smaug <sm...@welho.com> wrote: > On 06/29/2018 05:58 PM, Boris Zbarsky wrote: >> >> On 6/29/18 10:30 AM, Nathan Froyd wrote: >>> >>> Given the language-required qualification for >>> `enum class` and a more Rust-alike syntax, I would feel comfortable >>> with saying CamelCase `enum class` is the way to go. >> >> >> For what it's worth, I agree. The point of the "e" prefix is to highlight >> that you have an enumeration and add some poor-man's namespacing for a >> potentially large number of common-looking names, and the language-required >> qualification already handles both of those. >> > > That works if we're consistently naming static variables and such using > s-prefix etc, so that > class Foo1 > { > static int sBar; > }; > Foo1::sBar > > is clearly different to > enum class Foo2 > { > Bar > }; > > Foo2::Bar > > > (personally I'd still prefer using e-prefix always with enums, class or not. > Foo1::sBar vs Foo2::eBar is easier to distinguish than Foo1::sBar vs > Foo2::Bar) > > -Olli > > > > >> -Boris > > > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform