Closed Bug 1097283 Opened 5 years ago Closed 5 years ago
_ABORT _OOM callsites for count/bytes mismatches
I just made this mistake myself by copying nearby code in bug 1096155. As atonement, I'm going to fix all of them!
Contains a drive-by fix in nsTextFragment.h: Presumably the append operation uses both lengths. Open to other names for AllocFailed.
Attachment #8521070 - Flags: review?(nfroyd)
Comment on attachment 8521070 [details] [diff] [review] Pound of cure + ounce of prevention Nice.
Attachment #8521070 - Flags: review?(nfroyd) → review+
gcc didn't like it. https://hg.mozilla.org/integration/mozilla-inbound/rev/540b81dfaf97
Now with 37% more build trickery. https://hg.mozilla.org/integration/mozilla-inbound/rev/2cedc0acd40b https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=78a39c45d9f5
Assignee: nobody → dmajor
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.