Closed Bug 1402153 Opened 2 years ago Closed 2 years ago
Remove dead code from the crash reporting machinery
59 bytes, text/x-review-board-request
While working nsExceptionHandler.* I've noticed we have quite a bit of dead code in there, let's get rid of it.
Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
The patch is just stripping out some unused stuff, the try run is here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6f4a787a9e8830700d7d5ced4829d06a9ac00a97
I've updated the patch making a lot more non-public functions static. Not a big deal but let's be nice to our linker, it will also be easier to spot if we don't use them anymore so we don't leave more dead code in there.
Comment on attachment 8910976 [details] Bug 1402153 - Remove dead code from the crash reporter; .mielczarek https://reviewboard.mozilla.org/r/182444/#review199606
Attachment #8910976 - Flags: review?(ted) → review+
Thanks for the review, the try run is here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=680ff20d80b888a600ce0e4f8a908148a95dbe73
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/5b5154ab31fc Remove dead code from the crash reporter; r=ted.mielczarek
You need to log in before you can comment on or make changes to this bug.