Closed Bug 1294940 Opened 8 years ago Closed 8 years ago

Add ConstUTF8CharsZ class for "const char *" typed UTF8.

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(5 files)

Separated from bug 987069. I want to use ConstUTF8CharsZ class also in bug 1283710 and depending bugs.
Blocks: 1283710
Assignee: nobody → arai.unmht
Blocks: 1295017
No longer blocks: 1283710
Comment on attachment 8781238 [details] [diff] [review] Part 1: Add ConstUTF8CharsZ class for const UTF8 string. These patches are almost identical to the patch in original bug tho, I'd like to ask review again about length parameter and validation part.
Attachment #8781238 - Flags: review?(jwalden+bmo)
Attachment #8781239 - Flags: review?(jwalden+bmo)
I meant, those are only the difference between the original patch. others are same or trivial change.
Blocks: 1289003
No longer blocks: 1295017
Comment on attachment 8781238 [details] [diff] [review] Part 1: Add ConstUTF8CharsZ class for const UTF8 string. Review of attachment 8781238 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/public/CharacterEncoding.h @@ +109,5 @@ > char* c_str() { return reinterpret_cast<char*>(get()); } > }; > > /* > + * A wrapper for a "const char *" that is encoded using UTF-8. const char* Stars by types are wrong throughout this, actually.
Attachment #8781238 - Flags: review?(jwalden+bmo) → review+
Attachment #8781239 - Flags: review?(jwalden+bmo) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c1c4f1f646dc8a56c813131206e3b4e32ea4f5d4 Bug 1294940 - Part 0.1: Capitalize template parameter "action" of InflateUTF8StringToBuffer. r=jwalden https://hg.mozilla.org/integration/mozilla-inbound/rev/27bfa4989cc987bad6d7ddafb07ee6d3550fcf3d Bug 1294940 - Part 0.2: Make InflateUTF8StringHelper template function instead of passing count action function. r=jwalden https://hg.mozilla.org/integration/mozilla-inbound/rev/bb091f358c2795aaa3e4cbe2b413edd283dc2c33 Bug 1294940 - Part 1: Add ConstUTF8CharsZ class for const UTF8 string. r=jwalden https://hg.mozilla.org/integration/mozilla-inbound/rev/3e0129fc331893b14103f1cb2f8799a1cf2b6772 Bug 1294940 - Part 2: Add validation for ConstUTF8CharsZ. r=jwalden https://hg.mozilla.org/integration/mozilla-inbound/rev/aa74aca78adde46c2626a8b5fa628ea22257e3d4 Bug 1294940 - Part 3: Add ConstUTF8CharsZ variant for UTF8CharsToNewTwoByteCharsZ and LossyUTF8CharsToNewTwoByteCharsZ. r=jwalden
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: