http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4768

           Summary: spamc/libspamc.c uses C++ comment characters
           Product: Spamassassin
           Version: 3.1.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: major
          Priority: P5
         Component: Building & Packaging
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


spamc/libspamc.c uses C++/C99 comment characters "//", instead of "/*" and "*/".
 This causes problems for:

1.  Any compiler that does not handle these.

2.  Any compiler that can handle them, but needs a special option to enable this
feature.

This is essentially a duplicate of #4577, which includes a patch.  I've raised
this as a separate report because #4577 mistakenly reports that it only affects
IBM xlc on AIX -- I've run in to the problem on Solaris using Sun's Forte
compiler, and the problem is going to affect any environment with a picky 
compiler.

Please (a) apply the patch in #4577, and (b) consider an automated test for
SpamAssassin that does a slightly better version of "find . -name \*.[ch] -print
| xargs grep '//'" and fails if it finds any.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to