1368106 - Use the new MOZ_DIAGNOSTIC_ASSERT_ENABLED instead of several defines

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Comment 1

[Sylvestre Ledru [:Sylvestre]]

I am fixing the issue which were reported here:
https://bugzilla.mozilla.org/show_bug.cgi?id=1357799#c13

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Attachment: Bug 1368106 - Use the new MOZ_DIAGNOSTIC_ASSERT_ENABLED instead of several defines

Review commit: https://reviewboard.mozilla.org/r/143314/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/143314/

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8871833 [details]
Bug 1368106 - Use the new MOZ_DIAGNOSTIC_ASSERT_ENABLED instead of several defines

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/143314/diff/1-2/

Comment 4

[Mike Hommey [:glandium]]

Comment on attachment 8871833 [details]
Bug 1368106 - Use the new MOZ_DIAGNOSTIC_ASSERT_ENABLED instead of several defines

https://reviewboard.mozilla.org/r/143314/#review147740

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f26f1c5652fc
Use the new MOZ_DIAGNOSTIC_ASSERT_ENABLED instead of several defines r=glandium

Comment 6

[Carsten Book [:Tomcat]]

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