[cfe-dev] [PATCH] C++0x unicode string and character literals now with test cases
craig.topper at gmail.com
Tue Jul 19 00:08:40 PDT 2011
Here's a new patch that fixes the line lengths. Also widens to the
token kind in the Identifier table because 8-bits wasn't enough
anymore so kw_unknown_anytype was being truncated from 256 to 0.
On Mon, Jul 18, 2011 at 7:13 PM, Craig Topper <craig.topper at gmail.com> wrote:
> On Mon, Jul 18, 2011 at 1:16 PM, Eric Christopher <echristo at apple.com> wrote:
>> On Jul 18, 2011, at 11:50 AM, Craig Topper wrote:
>>> Thanks for the review! I'll fix the line lengths tonight. Do you know
>>> who is more familiar with the code?
>> Chris or Doug would be the two people most likely to approve it.
>> Had a couple of thoughts that I figured I'd ask about:
>> a) the enum for character and string is almost identical, any reason
>> not to make them identical
> I mainly didn't make them identical cause I put them inside the
> CharacterLiteral and StringLiteral classes. Wasn't sure were an
> appropriate home would be if they were shared.
>> b) any reason not to have the string literal creation machinery just
>> default to ascii and not need the changes in most of the use cases?
> I don't follow.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 78944 bytes
Desc: not available
More information about the cfe-dev