Closed Bug 321847 Opened 20 years ago Closed 20 years ago

pldhash.c includes nscore.h which uses c++ comments

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: timeless, Assigned: benjamin)

Details

pldhash.c In file included from /mnt/scratch/mozhack/mozilla/xpcom/glue/pldhash.h:45, from /mnt/scratch/mozhack/mozilla/xpcom/glue/pldhash.c:48: ../../dist/include/xpcom/nscore.h:257:1: warning: C++ style comments are not allowed in ISO C90 ../../dist/include/xpcom/nscore.h:257:1: warning: (this will be reported only once per input file) does pldhash.c need nscore.h?
plevent.c
timeless: I bet cvsblame would help assign this better. You wanted bsmedberg, not dougt. /be
Assignee: dougt → benjamin
That block is #if 0 What the hell? Why would the preprocessor tokenize blocks that are ifdefed out?
Fixed on trunk.
Status: UNCONFIRMED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Integrated tokenizer and pre-processor, someone didn't think to suppress warnings in #if 0 code. File a bug against GCC? /be
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.