Closed Bug 799000 Opened 8 years ago Closed 8 years ago

Don't double count when memory-reporting for JSObject2WrappedJSMap

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: njn, Assigned: njn)

Details

Attachments

(1 file)

DMD tells me that bug 772292 introduced double-counting of |mTable| in
JSObject2WrappedJSMap::SizeOfIncludingThis().  It's an easy fix.
Patch.  I also fixed a compile warning about a mixed signed/unsigned
comparison nearby.
Attachment #669001 - Flags: review?(terrence)
Nb: note the change from "Including" to "Excluding", which is easy to miss.
Comment on attachment 669001 [details] [diff] [review]
Don't double count when memory-reporting for JSObject2WrappedJSMap.

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

Doh!
Attachment #669001 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/3cfef9371c03
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.