Closed Bug 399340 Opened 17 years ago Closed 17 years ago

Case mismatch in nsIMsgNewsFolder.idl (PRint32/PRInt32)

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bugzilla, Assigned: bugzilla)

References

(
URL
)

Details

Attachments

(1 file)

PRint32 -> PRInt32
1.28 KB, patch
mnyromyr
: review+
neil
: superreview+
Details | Diff | Splinter Review 
See URL, PRint32 is used, but should be PRInt32. Patch to come.
This fixes the compiler warning.
Assignee: nobody → aqualon
Status: NEW → ASSIGNED
Attachment #284336 - Flags: superreview?(neil)
Attachment #284336 - Flags: review?(mnyromyr)
That's the VC8 warning:

../../../dist/idl\nsrootidl.idl:69: Case mismatch between `PRInt32'
d:/seamonkeysource/mozilla/mailnews/news/public/nsIMsgNewsFolder.idl:70: and `PRint32'
d:/seamonkeysource/mozilla/mailnews/news/public/nsIMsgNewsFolder.idl:70: (Identifiers should be case-consistent after initial declaration)
Severity: normal → minor
OS: Windows XP → All
Hardware: PC → All
Attachment #284336 - Flags: review?(mnyromyr) → review+
Comment on attachment 284336 [details] [diff] [review]
PRint32 -> PRInt32

Compiler warning? I'm surprised, I thought .idl was processed by xpidlgen
Attachment #284336 - Flags: superreview?(neil) → superreview+
(In reply to comment #3)
> (From update of attachment 284336 [details] [diff] [review])
> Compiler warning? I'm surprised, I thought .idl was processed by xpidlgen
They are, I saw this warning at stderr while stdout went to grep and thougt it was something from VC itself. But it really was from xpidl.
Landed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: