Closed
Bug 1586674
Opened 5 years ago
Closed 5 years ago
static inline uint32_t PRTimeToSeconds(PRTime aTimeUsec) can be simplified
Categories
(Core :: DOM: Navigation, defect)
Core
DOM: Navigation
Tracking
()
RESOLVED
FIXED
mozilla71
| Tracking | Status | |
|---|---|---|
| firefox71 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Just like in bug 1586605
http://sylvestre.ledru.info/reports/fx-scan-build/report-nsDocShell.h-PRTimeToSeconds-2-173.html#EndPath
Although the value stored to 'aTimeUsec' is used in the enclosing expression, the value is never actually read from 'aTimeUsec'
|
Assignee | |
Comment 1•5 years ago
|
||
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec0f7787850e
Simplify 'static inline uint32_t PRTimeToSeconds(PRTime aTimeUsec)' r=valentin
|
|
Assignee | |
Comment 3•5 years ago
|
||
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5de53dab979a
Simplify 'static inline uint32_t PRTimeToSeconds(PRTime aTimeUsec)' in CacheEntry.h too r=valentin
|
||
Comment 5•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
|
||
Updated•5 years ago
|
Assignee: nobody → sledru
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•