Closed
Bug 1046156
Opened 11 years ago
Closed 11 years ago
"bind" entire AndroidLog module to a tag
Categories
(Firefox for Android Graveyard :: General, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 34
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
7.35 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
It'd be handy to be able to "bind" the entire AndroidLog module to a tag, so you don't have to specify a tag each time you call one of its functions:
let Log = Cu.import("resource://gre/modules/AndroidLog.jsm", {}).AndroidLog.bind("MyModule");
Log.d("This is a debug message.");
// Outputs "D/GeckoMyModule(#####): This is a debug message."
The "bind" function isn't Function.bind, as this is an object, not a function. But it does for the object what Function.bind does for its individual functions. So I figured it would make sense. Commence bikeshedding at your leisure!
(I once implemented a preferences API that let you access prefs via both a root Prefs object and Prefs("branch") instances, which is roughly equivalent to what this provides; but that API was a bit subtle, too clever; this one seems better.)
Attachment #8464727 -
Flags: review?(mark.finkle)
Comment 1•11 years ago
|
||
Comment on attachment 8464727 [details] [diff] [review]
patch v1: "bind" entire AndroidLog module
nice
Attachment #8464727 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 2•11 years ago
|
||
Comment 3•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•