Closed Bug 210694 Opened 17 years ago Closed 17 years ago

build warnings in nsImapProtocol.cpp

Categories

(MailNews Core :: Networking: IMAP, defect, minor)

x86
Windows 2000
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bugzilla, Assigned: Bienvenu)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

When I build nsImapProtocol.cpp on my gcc on Win32:

c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp: In member function
   `void nsImapProtocol::ProcessSelectedStateURL()':
c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp:1949: warning: unused
   variable `const char*hostName'
c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp: In member function
   `virtual void nsImapProtocol::NormalMessageEndDownload()':
c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp:3376: warning: compariso
n
   between signed and unsigned integer expressions
c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp: In member function
   `PRBool nsImapProtocol::CheckNeeded()':
c:/mozilla/mozilla/mailnews/imap/src/nsImapProtocol.cpp:7347: warning: compariso
n
   between signed and unsigned integer expressions

20030623
Severity: normal → minor
Blocks: buildwarning
Attached patch fix warningsSplinter Review
fixed, r/sr=mscott
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
All the original warnings indeed went away, but there is a new one:

mailnews/imap/src/nsImapProtocol.cpp:3696
Unused variable `const PRInt32 kHdrsToFetchAtOnce'

It was introduced a few days ago by bug 103401 (all usages of the variable were
removed, but the variable stayed).
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.