Closed Bug 1586605 Opened 5 years ago Closed 5 years ago

"inline PRTime SecondsToPRTime(uint32_t t_sec)" seems abnormal complex

Categories

(Core :: Networking, enhancement)

enhancement
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: Sylvestre, Assigned: Sylvestre)

References

(Blocks 1 open bug, )

Details

Attachments

(1 file)

http://sylvestre.ledru.info/reports/fx-scan-build/report-nsAboutProtocolUtils.h-SecondsToPRTime-4-7.html#EndPath

Although the value stored to 't_usec' is used in the enclosing expression, the value is never actually read from 't_usec'

Or I am missing something!

Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/db24db8f5f54
Simplify the SecondsToPRTime function r=valentin
Type: defect → enhancement
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: