Closed Bug 547955 Opened 15 years ago Closed 15 years ago

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

Categories

(Toolkit :: Application Update, defect)

PowerPC
AIX
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: ul-mcamafia, Assigned: ul-mcamafia)

References

Details

Attachments

(2 files, 1 obsolete file)

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.
Attachment #428431 - Attachment is patch: true
Attachment #428431 - Attachment mime type: application/octet-stream → text/plain
Assignee: nobody → ul.mcamafia
Attachment #428431 - Attachment is obsolete: true
Attachment #428432 - Flags: review?(ted.mielczarek)
applies to mozilla-central, mozilla-1.9.2 and mozilla-1.9.1
Blocks: 535369
Status: NEW → ASSIGNED
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.
Attachment #428432 - Flags: review?(ted.mielczarek) → review+
Keywords: checkin-needed
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?
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."
Keywords: checkin-needed
(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.
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
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: