Closed
Bug 1047429
Opened 10 years ago
Closed 10 years ago
Remove non-structured mozlog documentation
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla34
People
(Reporter: ahal, Assigned: chmanchester)
Details
Attachments
(1 file)
2.45 KB,
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
I think there is basically no longer any reason to use the "old style" of mozlog. While we can't just remove it quite yet as a lot of things are still using it, I think we should at least remove the documentation so there's a smaller chance of new code accidentally using it (this happened recently in bug 1026710)
Assignee | ||
Comment 1•10 years ago
|
||
This removes them, but maybe just a bigger deprecated warning would be sufficient?
Attachment #8478317 -
Flags: review?(ahalberstadt)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → cmanchester
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•10 years ago
|
||
Comment on attachment 8478317 [details] [diff] [review] Remove unstructured mozlog docs to reduce risk of use in new code. Review of attachment 8478317 [details] [diff] [review]: ----------------------------------------------------------------- Thanks!
Attachment #8478317 -
Flags: review?(ahalberstadt) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c70db30dc932
Keywords: checkin-needed
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c70db30dc932
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•