450864 - c++ style comment in parser/expat/lib/moz_extensions.c

Status: VERIFIED FIXED

(Core :: XML, defect)

Description

[Uli Link (:ul-mcamafia)]

User-Agent:       Mozilla/5.0 (X11; U; AIX 5.1; en-US; rv:1.8.1.16) Gecko/20080806 Firefox/2.0.0.16
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 5.0.2.9

Actual Results:  
compilation error

Expected Results:  
successfull compilation

Comment 1

[Uli Link (:ul-mcamafia)]

Attachment: 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.

Comment 2

[Peter Van der Beken [:peterv]]

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.

Comment 3

[Daniel Veditz [:dveditz]]

Not a blocker, will look at the approval requests

Comment 4

[Serge Gautherie (:sgautherie)]

(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

Comment 5

[Magnus Melin [:mkmelin]]

Checking in parser/expat/lib/moz_extensions.c;
/cvsroot/mozilla/parser/expat/lib/moz_extensions.c,v  <--  moz_extensions.c
new revision: 1.6.18.2; previous revision: 1.6.18.1
done

->FIXED