Closed
Bug 189684
Opened 22 years ago
Closed 22 years ago
nsrootidl.idl:69: Case mismatch between `PRInt32' i:/build/mozilla/extensions/cookie/nsICookieManager2.idl:61: and `PRint32'
Categories
(Core :: Networking: Cookies, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: timeless)
Details
Attachments
(1 file)
603 bytes,
patch
|
darin.moz
:
review+
dbaron
:
superreview+
|
Details | Diff | Splinter Review |
nsrootidl.idl:69: Case mismatch between `PRInt32'
i:/build/mozilla/extensions/cookie/nsICookieManager2.idl:61: and `PRint32'
i:/build/mozilla/extensions/cookie/nsICookieManager2.idl:61: (Identifiers should
be case-consistent after initial declaration)
Attachment #111973 -
Flags: superreview?(dbaron)
Attachment #111973 -
Flags: review?(dbaron)
Attachment #111973 -
Flags: superreview?(dbaron)
Attachment #111973 -
Flags: superreview+
Attachment #111973 -
Flags: review?(dbaron)
Attachment #111973 -
Flags: review?
Actually, why not just use |long|?
darin asked that in bug 185689 comment 4.
danm answered in bug 185689 comment 6.
I don't care, nor do i want to be stuck in the middle. i just want the error to
go away.
Status: NEW → ASSIGNED
ok
Comment 5•22 years ago
|
||
Comment on attachment 111973 [details] [diff] [review]
correct case
r=darin
Attachment #111973 -
Flags: review? → review+
checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
timeless: this is a code checkin, do you have a bonzai link for the query or a
cvs checkin output you could paste here and verify?
Comment 8•22 years ago
|
||
benc:
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=nsICookieManager2.idl&root=/cvsroot&subdir=mozilla/extensions/cookie&command=DIFF_FRAMESET&rev1=1.1&rev2=1.2
marking verified
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•