Closed Bug 548237 Opened 15 years ago Closed 15 years ago

C++ comments in C unit mailnews/extensions/fts3/src/fts3_porter.c

Categories

(MailNews Core :: Database, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED WONTFIX

People

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

Details

Attachments

(1 file)

Replacing C++ comments with C comments fixes compilation for strict ANSI conformance. xlc_r -o fts3_porter.o -c -DMOZILLA_INTERNAL_API -DMOZ_SUITE=1 -DOSTYPE=\"AIX4. 3\" -DOSARCH=AIX -I/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src -I. -I../../../../mozilla/dist/include/xpcom -I../../../../mozilla/dist/include/stri ng -I../../../../mozilla/dist/include/sqlite3 -I../../../../mozilla/dist/include /storage -I../../../../mozilla/dist/include -I../../../../mozilla/dist/include /fts3tok `/home/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist/bin/nspr-config --prefix=/home/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist --includedir=/hom e/ulink/src/comm-1.9.1/obj-sm2-aix43/mozilla/dist/include/nspr --cflags` -q flag=w:w -DNDEBUG -DTRIMMED -O2 -qarch=604 -qmaxmem=-1 -qalias=noansi -DMO ZILLA_1_9_1_BRANCH=1 -DMOZILLA_VERSION=\"1.9.1.9pre\" -DMOZILLA_VERSION_U=1.9.1. 9pre -DAIX=1 -DHAVE_SYS_INTTYPES_H=1 -DNSCAP_DISABLE_DEBUG_PTR_TYPES=1 -DD_INO=d _ino -DSTDC_HEADERS=1 -DHAVE_ST_BLKSIZE=1 -DHAVE_SIGINFO_T=1 -DHAVE_INT16_T=1 -D HAVE_INT32_T=1 -DHAVE_INT64_T=1 -DHAVE_INT64=1 -DHAVE_UINT=1 -DHAVE_UINT_T=1 -DH AVE_UINT16_T=1 -DHAVE_DIRENT_H=1 -DHAVE_MEMORY_H=1 -DHAVE_UNISTD_H=1 -DHAVE_NL_T YPES_H=1 -DHAVE_MALLOC_H=1 -DHAVE_X11_XKBLIB_H=1 -DHAVE_SYS_STATVFS_H=1 -DHAVE_S YS_STATFS_H=1 -DHAVE_LIBC_R=1 -DHAVE_LIBM=1 -DHAVE_LIBDL=1 -DHAVE_LIBC_R=1 -DFUN CPROTO=15 -DHAVE_XSHM=1 -D_REENTRANT=1 -DHAVE_RANDOM=1 -DHAVE_STRERROR=1 -DHAVE_ LCHOWN=1 -DHAVE_FCHMOD=1 -DHAVE_SNPRINTF=1 -DHAVE_MEMMOVE=1 -DHAVE_RINT=1 -DHAVE _STAT64=1 -DHAVE_LSTAT64=1 -DHAVE_TRUNCATE64=1 -DHAVE_FLOCKFILE=1 -DHAVE_LOCALTI ME_R=1 -DHAVE_STRTOK_R=1 -DHAVE_RES_NINIT=1 -DHAVE_LANGINFO_CODESET=1 -DHAVE_I18 N_LC_MESSAGES=1 -DMOZ_SUITE=1 -DMOZ_BUILD_APP=suite -DMOZ_X11=1 -DMOZ_WIDGET_GTK 2=1 -DMOZ_DISTRIBUTION_ID=\"org.mozilla\" -DOJI=1 -DIBMBIDI=1 -DMOZ_VIEW_SOURCE= 1 -DMOZ_XPINSTALL=1 -DMOZ_JSLOADER=1 -DNS_PRINTING=1 -DNS_PRINT_PREVIEW=1 -DMOZ_ NO_XPCOM_OBSOLETE=1 -DMOZ_XTF=1 -DMOZ_UPDATE_CHANNEL=default -DMOZ_PLACES=1 -DMO Z_FEEDS=1 -DMOZ_STORAGE=1 -DMOZ_HELP_VIEWER=1 -DMOZ_LOGGING=1 -DMOZ_USER_DIR=\". mozilla\" -DMOZ_STATIC_MAIL_BUILD=1 -DHAVE_INTTYPES_H=1 -DMOZ_XUL=1 -DMOZ_PROFIL ELOCKING=1 -DMOZ_RDF=1 -DMOZ_MORK=1 -DMOZ_MORKREADER=1 -DMOZ_DLL_SUFFIX=\".so\" -DXP_UNIX=1 -DUNIX_ASYNC_DNS=1 -D_COMM_CONFIG_H_ -DMOZILLA_CLIENT /home/ulink/s rc/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 74 0.5: 1506-046 (S) Syntax error. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 74 0.8: 1506-045 (S) Undeclared identifier This. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 0.7: 1506-046 (S) Syntax error. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 0.10: 1506-058 (S) Label cases has already been defined on line 744 of "/home/ul ink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c". "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 2.7: 1506-068 (S) Operation between types "char*" and "int*" is not allowed. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 2.14: 1506-277 (S) Syntax error: possible missing ';' or ','? "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 4.10: 1506-058 (S) Label cases has already been defined on line 744 of "/home/ul ink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c". "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 5.7: 1506-068 (S) Operation between types "char*" and "int*" is not allowed. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 75 5.14: 1506-277 (S) Syntax error: possible missing ';' or ','? "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 76 0.3: 1506-046 (S) Syntax error. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 76 0.6: 1506-045 (S) Undeclared identifier at. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 76 8.3: 1506-046 (S) Syntax error. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 77 0.15: 1506-260 (S) Octal integer constant 0a4cf is not valid. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 76 8.6: 1506-045 (S) Undeclared identifier Anything. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 77 1.80: 1506-209 (S) Character constants must end before the end of a line. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 77 1.66: 1506-076 (W) Character constant 're leaving it' has more than one characte r. No more than rightmost 4 characters are used. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 77 3.8: 1506-045 (S) Undeclared identifier We. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 77 4.6: 1506-058 (S) Label cases has already been defined on line 744 of "/home/uli nk/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c". "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 91 7.38: 1506-076 (W) Character constant 't be a delimiter because in a ' has more than one character. No more than rightmost 4 characters are used. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 91 7.78: 1506-209 (S) Character constants must end before the end of a line. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 91 7.77: 1506-073 (E) Empty character constant. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 93 0.79: 1506-209 (S) Character constants must end before the end of a line. "/home/ulink/src/comm-1.9.1/mailnews/extensions/fts3/src/fts3_porter.c", line 93 0.20: 1506-076 (W) Character constant 't terminate. However, if we terminate, l en still contains' has more than one character. No more than rightmost 4 charact ers are used. gmake[6]: *** [fts3_porter.o] Error 1
Assignee: nobody → ul.mcamafia
Status: NEW → ASSIGNED
Attachment #428686 - Flags: review?(djripper)
Component: Search → Database
QA Contact: search → database
Comment on attachment 428686 [details] [diff] [review] Replacing C++ comments with C comments for ANSI conformance I'd rather we just change the file to be C++. Since none of the supported platform builders get angry about C++ comments, it's likely a better long-term strategy. We do, of course, need to make sure to mark the functions exposed to SQLite as extern "C". Please direct any reviews to me for this code.
Attachment #428686 - Flags: review?(djripper) → review-
(In reply to comment #2) > (From update of attachment 428686 [details] [diff] [review]) > Since none of the supported > platform builders get angry about C++ comments, both GCC and MSVC accept this non ANSI conformant C code, even with --pedantic I can add "-qcpluscmt" in CPPFLAGS in Makefile.in or compile in C99 mode.
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: