Avoid calling isLoggable

RESOLVED FIXED in Firefox 36

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mfinkle, Assigned: mfinkle)

Tracking

Trunk
Firefox 36
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 8521627 [details] [diff] [review]
avoid-isloggable v0.1

isLoggable is too expensive to have in-line in each log call. This patch converts some calls to used a cached value.
Attachment #8521627 - Flags: review?(nalexander)
Comment on attachment 8521627 [details] [diff] [review]
avoid-isloggable v0.1

Review of attachment 8521627 [details] [diff] [review]:
-----------------------------------------------------------------

Sure.
Attachment #8521627 - Flags: review?(nalexander) → review+

Comment 3

4 years ago
https://hg.mozilla.org/mozilla-central/rev/d4fd43b50ba4
Assignee: nobody → mark.finkle
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
You need to log in before you can comment on or make changes to this bug.