1648938 - Convert writing-modes #defines to an enum class.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task)

Comment 1

[Philipp Zech]

I started converting the existing NS_STYLE_WRITING_MODE_* defines to an enum calling it StyleWritingMode but upon compilation realized that there already exists a struct in the same namespace (mozilla) with the same name in ServoStyleConsts.h. Are they somehow related? Is this enum even necessary, or shall the struct in general take over the former #defines' jobs?

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Yeah, this one is a bit weird, because there are two "writing-mode" things:

• The writing-mode property (which is what these constants are about).
• The WritingMode class + StyleWritingMode, which is really a bitfield with various information computed from writing-mode, direction, text-orientation... As these affect each other.

So I think the right thing to do here is calling the new enum StyleWritingModeProperty or something like this, so that it properly differentiates from one from the other.

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Alternatively we could rename StyleWritingMode / WritingMode, but those are much more widely used, and it's not clear what the right name for that would be otherwise, so it seems better / more straight-forward to rename the (not-yet-existing) enum :)

Comment 4

[Philipp Zech]

Ok, thx!

Comment 5

[Philipp Zech]

Attachment: Bug 1648938 - Convert writing-modes #defines to an enum class. r=emilio

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ee4a5d3b193f
Convert writing-modes #defines to an enum class. r=emilio

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/ee4a5d3b193f