Closed Bug 1369622 Opened 3 years ago Closed 3 years ago

Followups after bug 1368932

Categories

(Core :: MFBT, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(4 files)

No description provided.
Comment on attachment 8873715 [details]
Bug 1369622 - Fix MOZ_FOR_EACH with an empty list.

https://reviewboard.mozilla.org/r/145106/#review150208
Attachment #8873715 - Flags: review?(nfroyd) → review+
Comment on attachment 8873716 [details]
Bug 1369622 - Use MOZ_ARG_COUNT instead of MOZ_PASTE_PREFIX_AND_ARG_COUNT.

https://reviewboard.mozilla.org/r/145108/#review150210
Attachment #8873716 - Flags: review?(nfroyd) → review+
Comment on attachment 8873717 [details]
Bug 1369622 - Restore static asserts for lack of arguments on some macros.

https://reviewboard.mozilla.org/r/145110/#review150214
Attachment #8873717 - Flags: review?(nfroyd) → review+
Comment on attachment 8873718 [details]
Bug 1369622 - Add a static_assert when no variadic argument is given to MOZ_CRASH_UNSAFE_PRINTF.

https://reviewboard.mozilla.org/r/145112/#review150216
Attachment #8873718 - Flags: review?(nfroyd) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bee4404d2f88
Fix MOZ_FOR_EACH with an empty list. r=froydnj
https://hg.mozilla.org/integration/autoland/rev/e51f693cdbd3
Use MOZ_ARG_COUNT instead of MOZ_PASTE_PREFIX_AND_ARG_COUNT. r=froydnj
https://hg.mozilla.org/integration/autoland/rev/f360dd9d411e
Restore static asserts for lack of arguments on some macros. r=froydnj
https://hg.mozilla.org/integration/autoland/rev/e1a83fc8ff4f
Add a static_assert when no variadic argument is given to MOZ_CRASH_UNSAFE_PRINTF. r=froydnj
You need to log in before you can comment on or make changes to this bug.