Closed
Bug 242588
Opened 20 years ago
Closed 20 years ago
r:\mozilla\extensions\cookie\nsCookiePermission.h(65) : warning C4244: 'initializing' : conversion from 'PRInt64' to 'PRUint8', possible loss of data
Categories
(Core :: Networking: Cookies, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: dwitte)
References
()
Details
Attachments
(1 file)
|
817 bytes,
patch
|
mconnor
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
r:\mozilla\extensions\cookie\nsCookiePermission.h(65) : warning C4244:
'initializing' : conversion from 'PRInt64' to 'PRUint8', possible loss of data
60 , mCookiesLifetimePolicy(LL_MAXINT)
75 PRUint8 mCookiesLifetimePolicy; // pref for how long cookies
are stored
|
Assignee | |
Comment 1•20 years ago
|
||
|
|
Assignee | |
Comment 2•20 years ago
|
||
Comment on attachment 147650 [details] [diff] [review]
tsk tsk
i could just ask darin for r+sr here, but i want the fun of making mconnor
review it, so he doesn't do this again. :)
Attachment #147650 -
Flags: superreview?(darin)
Attachment #147650 -
Flags: review?(mconnor)
|
||
Updated•20 years ago
|
Attachment #147650 -
Flags: superreview?(darin) → superreview+
|
||
Comment 3•20 years ago
|
||
Comment on attachment 147650 [details] [diff] [review]
tsk tsk
hey, you reviewed it, Mr. C++ is easy :)
Attachment #147650 -
Flags: review?(mconnor) → review+
|
|
||
Updated•20 years ago
|
Assignee: mconnor → dwitte
|
|
Assignee | |
Comment 4•20 years ago
|
||
yeah, i know, was worth a try though ;)
checked in.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•