Closed Bug 1216038 Opened 4 years ago Closed 4 years ago

eliminate MakeEnumeratedRange's IntType template parameter

Categories

(Core :: MFBT, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: heycam, Assigned: heycam)

References

Details

Attachments

(1 file)

In the one argument version of MakeEnumeratedRange, we can eliminate the IntType template parameter and just deduce how big of an integer type we need for the given EnumType.  Since we're interating from EnumType(0), we know that it's safe to use an unsigned integer type.  (In the future when we can rely on std::underlying_type, we won't need the IntType parameter for the other MakeEnumeratedRange overload, either.)
Assignee: nobody → cam
Status: NEW → ASSIGNED
Attachment #8675508 - Flags: review?(nfroyd)
Blocks: 1216043
Attachment #8675508 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/1f8cd027929b
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.