Page MenuHomePhabricator

Bug 1607419 - Avoid option.expect(&format in hot code. r=kvark
ClosedPublic

Authored by nical on Jan 7 2020, 10:49 AM.

Details

Summary

Easy to miss that the slow formatting code is run unconditionally.
The remaining instances are in recording and startup code.

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nical created this revision.Jan 7 2020, 10:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 7 2020, 10:49 AM
phab-bot requested review of this revision.Jan 7 2020, 10:49 AM
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: Restricted Project.
jrmuizel accepted this revision.Jan 7 2020, 3:40 PM
This revision is now accepted and ready to land.Jan 7 2020, 3:40 PM