Closed Bug 782919 Opened 7 years ago Closed 7 years ago

Enum class replacement needs to have default zero-arg constructor

Categories

(Core :: MFBT, defect, minor)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: ayg, Assigned: ayg)

References

Details

Attachments

(1 file)

On compilers that don't support enum class, we define it as a class and define our own constructors.  These override the default constructors, so

  Enum res;

no longer works.  We need a default constructor with zero arguments that does nothing.
Attached patch PatchSplinter Review
Attachment #652067 - Flags: review?(ehsan)
Attachment #652067 - Flags: review?(ehsan) → review+
Sorry, was looking at the wrong push. Relanded:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d2db7e3879a
https://hg.mozilla.org/mozilla-central/rev/7d2db7e3879a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.