Closed Bug 979566 Opened 6 years ago Closed 6 years ago

Convert usleep to nanosleep

Categories

(Core :: Gecko Profiler, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: BenWa, Assigned: vikstrous)

Details

Attachments

(1 file, 1 obsolete file)

usleep calls nanosleep under the covers but it's doing a division in the wrapper code. If we call nanosleep directly we can avoid this division.

Viktor got time for a small patch?
Attached patch nanosleep (obsolete) — Splinter Review
https://tbpl.mozilla.org/?tree=Try&rev=6d2ff251956a
Assignee: nobody → vstanchev
Attachment #8385674 - Flags: review?(bgirard)
Attached patch nanosleepSplinter Review
https://tbpl.mozilla.org/?tree=Try&rev=56c170b5fc89
Attachment #8385674 - Attachment is obsolete: true
Attachment #8385674 - Flags: review?(bgirard)
Attachment #8386162 - Flags: review?(bgirard)
Attachment #8386162 - Flags: review?(bgirard) → review+
Flags: needinfo?(bgirard)
https://hg.mozilla.org/mozilla-central/rev/f952ac387d75
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.