Last Comment Bug 679346 - js/src/jsdbgapi.cpp:1233: warning: ‘void UnsafeError(const char*, ...)’ defined but not used
: js/src/jsdbgapi.cpp:1233: warning: ‘void UnsafeError(const char*, ...)’ defin...
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Other Branch
: x86 Mac OS X
-- normal (vote)
: mozilla9
Assigned To: Jason Orendorff [:jorendorff]
: Jason Orendorff [:jorendorff]
Depends on:
  Show dependency treegraph
Reported: 2011-08-16 07:42 PDT by Jason Orendorff [:jorendorff]
Modified: 2011-08-17 04:43 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v1 (556 bytes, patch)
2011-08-16 07:59 PDT, Jason Orendorff [:jorendorff]
gal: review+
Details | Diff | Splinter Review

Description User image Jason Orendorff [:jorendorff] 2011-08-16 07:42:16 PDT
Another GCC warning snuck in. Introduced by jsdbg2, probably.
Comment 1 User image Jason Orendorff [:jorendorff] 2011-08-16 07:59:02 PDT
Created attachment 553475 [details] [diff] [review]

_GNU_SOURCE is a way for *programs* to specify which signatures they want[1] from GNU system headers that... anyway, it's not what you want here.

The compiler's preprocessor badge is __GNUC__.

[1] ...or something? if you care, read on:
Comment 2 User image Jason Orendorff [:jorendorff] 2011-08-16 08:00:42 PDT
I forgot to mention, in case the war room is reading along here: it turns out
jsdbg2 is innocent! In fact, it's perfect in every way. Please don't back it out.
Comment 3 User image Jason Orendorff [:jorendorff] 2011-08-16 09:38:21 PDT
Comment 4 User image Steve Fink [:sfink] [:s:] 2011-08-16 12:10:08 PDT
The above link describes _GNU_SOURCE, but I needed to continue my edumification at

I think I knew this, but did a lazy grep for "GNU" and grabbed the first thing that looked relevant.

Thanks for the cleanup.

Note You need to log in before you can comment on or make changes to this bug.