Closed Bug 1648334 Opened 4 years ago Closed 4 years ago

Use less unsafe in gecko_logger.

Categories

(Core :: XPCOM, task)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: emilio, Assigned: emilio)

Details

Attachments

(1 file)

No description provided.

We were storing LogModule::mName as an &'static str in the map... That's
not fine.

The Arc shenanigans were also more complicated than they need to be IMO.

This uses a plain atomic bool to keep the fast path snappy.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6e4c9d585501 Use less unsafe in gecko_logger. r=froydnj,valentin
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: