User-Agent: Mozilla/5.0 (X11; U; AIX 5.1; en-US; rv:184.108.40.206) Gecko/20080806 Firefox/220.127.116.11 Build Identifier: MOZILLA_1_8_BRANCH build bustage due to illegal C code MOZILLA_1_8_BRANCH is not required a C99 C compiler, so c++ style comments are illegal in .c files Reproducible: Always Steps to Reproduce: 1. compile on AIX 4.3.3 or AIX 5.1 with VisualAge 18.104.22.168 Actual Results: compilation error Expected Results: successfull compilation
Created attachment 334075 [details] [diff] [review] replaces c++ with c style comment only Trunk and fx 3.0 requires a C99 compiler, so no problems with c++ style comments in c files.
Severity: minor → normal
Comment on attachment 334075 [details] [diff] [review] replaces c++ with c style comment only This was already fixed on 1.9/trunk. No functional changes, just a fix to make it compile with more compilers.
Not a blocker, will look at the approval requests
Attachment #334075 - Flags: approval22.214.171.124? → approval126.96.36.199+
(In reply to comment #2) > This was already fixed on 1.9/trunk by http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/parser/expat/lib/moz_extensions.c&mark=1.10
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Whiteboard: [c-n: 1.8.1 branch only]
Checking in parser/expat/lib/moz_extensions.c; /cvsroot/mozilla/parser/expat/lib/moz_extensions.c,v <-- moz_extensions.c new revision: 188.8.131.52; previous revision: 184.108.40.206 done ->FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: checkin-needed → fixed220.127.116.11
Resolution: --- → FIXED
Whiteboard: [c-n: 1.8.1 branch only] → [branch only]
Status: RESOLVED → VERIFIED
Keywords: fixed18.104.22.168 → verified22.214.171.124
You need to log in before you can comment on or make changes to this bug.