many "warning: '__STDC_FORMAT_MACROS' macro redefined"

RESOLVED FIXED in mozilla27

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: keeler, Assigned: hub)

Tracking

Trunk
mozilla27
x86_64
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [c= p=1 s= u=])

Attachments

(1 attachment)

Compiling what looks like the libs subtier in media/webrtc, I'm getting tons of these warnings printed:

13:23.36 ccsip_task.o
13:23.36 In file included from <built-in>:162:
13:23.43 In file included from <command line>:49:
13:23.43 ../../../../mozilla-config.h:211:9: warning: '__STDC_FORMAT_MACROS' macro redefined
13:23.43 #define __STDC_FORMAT_MACROS
13:23.43         ^
13:23.43 <command line>:40:9: note: previous definition is here
13:23.43 #define __STDC_FORMAT_MACROS 1
13:23.43         ^
13:23.43 1 warning generated.

This is with clang 3.3
On a B2G build, I'm getting this on basically every file (not just media/webrtc files)

widget/gonk/libui/Unicode.h sets it to 1 and causes the warnings
gfx/angle/src/compiler/preprocessor/Tokenizer.cpp also sets it to 1
Oops, that's STDC_LIMIT_MACROS with gonk.
I also see this when compiling WebRTC on Mac.
OS: Linux → All
Assignee

Comment 5

6 years ago
(In reply to Randell Jesup [:jesup] from comment #2)
> Oops, that's STDC_LIMIT_MACROS with gonk.

And that was fixed with bug 911359 and I thought would fix this, but I too confused macros. Patch soon.
Assignee: nobody → hub
Status: NEW → ASSIGNED
Assignee

Comment 6

6 years ago
Proposed fix. You might need to do |touch configure| to force configure to be re-run. Only testing on a FxOS build for now.
Assignee

Updated

6 years ago
Attachment #808642 - Flags: review?(ted)
Hmm, why isn't mozilla-config.h included first for WebRTC files?
Assignee

Comment 8

6 years ago
I think the defines on the command line take precedence.
Assignee

Comment 9

6 years ago
but said defines are BEFORE the -include ../../../../../../mozilla-config.h in the command line.
Attachment #808642 - Flags: review?(ted) → review+

Updated

6 years ago
Whiteboard: [c= p= s= u=]
Assignee

Updated

6 years ago
Whiteboard: [c= p= s= u=] → [c= p=1 s= u=]
https://hg.mozilla.org/mozilla-central/rev/d7599a4f3e17
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.