Closed Bug 824397 Opened 7 years ago Closed 7 years ago

Use NS_ENSURE_TRUE_VOID(foo) instead of NS_ENSURE_TRUE(foo, )

Categories

(Core :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: njn, Assigned: njn)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

I just updated my Ubuntu installation to 12.10 and got a new clang, and now I'm getting failures like these:

  /home/njn/moz/mi2/content/media/webm/WebMReader.cpp:1033:74: error: empty macro arguments are a C99 feature [-Werror,-Wc99-extensions]
  NS_ENSURE_TRUE((uint32_t)aSubsegmentIdx < mClusterByteRanges.Length(), );
 
They're errors in some dirs and warnings in others, so there must be some WARNINGS_AS_ERRORS action happening.  (I'm compiling with --enable-warnings-as-errors.)
Comment on attachment 695375 [details] [diff] [review]
Use NS_ENSURE_{TRUE,FALSE,SUCCESS}_VOID(foo) instead of NS_ENSURE_{TRUE,FALSE,SUCCESS}(foo, ).

This patch misses some that use /**/ (or varations) as the 2nd arg.  I'll post a new version shortly.
Attachment #695375 - Flags: review?(tterribe)
New version that converts the NS_ENSURE_TRUE(foo, /**/) cases too.
Attachment #695378 - Flags: review?(tterribe)
Attachment #695375 - Attachment is obsolete: true
Attachment #695378 - Flags: review?(tterribe) → review+
https://hg.mozilla.org/mozilla-central/rev/372d779d8f27
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.