547955 - C++ style comments in C unit bsdiff/bsdiff.c/.h

Status: RESOLVED WONTFIX

(Toolkit :: Application Update, defect)

Description

[Uli Link (:ul-mcamafia)]

Attachment: replacing C++ comments with C-style

xlc_r -o host_bsdiff.o -c -DXP_UNIX -O2 -qarch=604 -qmaxmem=-1 -qalias=noansi  -I/home/ulink/src/comm-1.9.1/mozilla/toolkit/mozapps/update/src/updater -I/home/ulink/src/comm-1.9.1/mozilla/other-licenses/bsdiff -I.  -I../../dist/include   -I../../dist/include -I/home/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist/include/nspr    -I/home/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist/sdk/include -I/home/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist/include/nspr /home/ulink/src/comm-1.9.1/mozilla/other-licenses/bsdiff/bsdiff.c
"/home/ulink/src/comm-1.9.1/mozilla/toolkit/mozapps/update/src/updater/bspatch.h", line 35.1: 1506-046 (S) Syntax error.
"/home/ulink/src/comm-1.9.1/mozilla/toolkit/mozapps/update/src/updater/bspatch.h", line 35.4: 1506-166 (S) Definition of function We requires parentheses.
"/home/ulink/src/comm-1.9.1/mozilla/toolkit/mozapps/update/src/updater/bspatch.h", line 35.54: 1506-209 (S) Character constants must end before the end of a line.
"/home/ulink/src/comm-1.9.1/mozilla/toolkit/mozapps/update/src/updater/bspatch.h", line 35.37: 1506-076 (W) Character constant 't link with NSPR' has more than one character. No more than rightmost 4 characters are used.
...

Simply replacing the C++ style comments with C style comments.

Comment 1

[Uli Link (:ul-mcamafia)]

Attachment: replacing C++ comments with C-style

Comment 2

[Uli Link (:ul-mcamafia)]

applies to mozilla-central, mozilla-1.9.2 and mozilla-1.9.1

Comment 3

[(not currently active) Ted Mielczarek]

Comment on attachment 428432 [details] [diff] [review]
replacing C++ comments with C-style 

I would be worried about ever taking a bsdiff update, but I'm pretty sure we've already thoroughly forked this stuff.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

The patch changes toolkit/mozapps/update/src/updater/bspatch.h, but my tree only has toolkit/mozapps/update/updater/bspatch.h... what does this mean?

Comment 5

[Phil Ringnalda (:philor)]

Since the patch claims to be against 90fbd13348e0, which is a 1.9.1 rev, I'd say it means Uli patched against 1.9.1, instead of against mozilla-central after bug 542222 got rid of the /src/, and the "applies to mozilla-central" meant "... if you haven't updated your m-c tree for several weeks now."

Comment 6

[Uli Link (:ul-mcamafia)]

(In reply to comment #5)
> Since the patch claims to be against 90fbd13348e0, which is a 1.9.1 rev, I'd
> say it means Uli patched against 1.9.1, instead of against mozilla-central

That's what I did. And visual checking the source for trunk in MXR.
Seems I've missed the moved src dir.

> after bug 542222 got rid of the /src/, and the "applies to mozilla-central"
> meant "... if you haven't updated your m-c tree for several weeks now."

Porting work for trunk stops compilation long,long before coming to this dir.

Comment 7

[Uli Link (:ul-mcamafia)]

Attachment: recreated remaining patch for trunk

Here the part of the patch for trunk that wasn't fixed by
http://hg.mozilla.org/mozilla-central/rev/9b7c5d7fcbda

bspatch.h is included in bsdiff.c