Closed Bug 12435 Opened 21 years ago Closed 21 years ago

nsNntpUrl.cpp build fix for solaris/native compilers

Categories

(MailNews Core :: Backend, defect, P3)

Sun
Solaris
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: tor, Assigned: sspitzer)

Details

Your recent change to nsNntpUrl.cpp (1.35) broke the solaris/native
build.  The solaris compiler is strict, requiring this patch to compile:

Index: nsNntpUrl.cpp
===================================================================
RCS file: /cvsroot/mozilla/mailnews/news/src/nsNntpUrl.cpp,v
retrieving revision 1.35
diff -u -r1.35 nsNntpUrl.cpp
--- nsNntpUrl.cpp       1999/08/24 22:39:40     1.35
+++ nsNntpUrl.cpp       1999/08/25 01:35:09
@@ -288,7 +288,7 @@

     nsCString newsgroupURI(kNewsMessageRootURI);
     newsgroupURI.Append("/");
-    if (userName || (userName != "")) {
+    if (userName || (userName != (const char *)"")) {
        newsgroupURI.Append(userName);
        newsgroupURI.Append("@");
     }
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Target Milestone: M10
marking m10.

thanks for the patch.

I've checked the fix in.

marking fixed.
QA Contact: lchiang → tor
tor@cs.brown.edu - I've changed the QA Contact to you.  Can you verify this once
it's checked in?  Thanks
Status: RESOLVED → VERIFIED
Verified on a 8/25 CVS pull.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.