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

RESOLVED FIXED in Firefox 51

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: arai, Assigned: arai)

Tracking

Trunk
mozilla51
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(5 attachments)

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.