Closed Bug 951110 Opened 8 years ago Closed 8 years ago

AssertValidDepedentString() should inform readers about dependent *substrings*

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: hsivonen, Assigned: hsivonen)

Details

Attachments

(1 file)

NS_ASSERTION(mData[mLength] == 0, "nsTDependentString must wrap only null-terminated strings");
has unfriendly assertion text.
Attached patch Tweak the textSplinter Review
Assertion text like this would have saved me some trouble.
Attachment #8348658 - Flags: review?(benjamin)
Attachment #8348658 - Flags: review?(benjamin) → review+
https://hg.mozilla.org/mozilla-central/rev/c245cc9c1b43
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.