[cfe-commits] patch: Lexer.cpp CharInfo static init

Chris Lattner clattner at apple.com
Tue Jul 7 10:10:15 PDT 2009


On Jul 7, 2009, at 2:27 AM, Ryan Flynn wrote:

> On Tue, Jul 7, 2009 at 1:29 AM, Chris Lattner<clattner at apple.com>  
> wrote:
>>
>> On Jul 5, 2009, at 2:06 PM, Ryan Flynn wrote:
>>
>>> addressed an outstanding TODO in Lexer.cpp to statically initialize
>>> CharInfo table.
>>> this implementation maps to the ASCII character set.
>>> i couldn't find any explicit documentation on charset support, so i
>>> hope this is ok.
>>
>> Looks good, but please stay in 80 cols :).  Please repost a patch  
>> with that
>> fix, thanks!
>>
>> -Chris
>
> Formatted for <= 80 columns. Will follow project's coding standards
> more rigorously in future :)

Thanks committed here:
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20090706/018812.html

-Chris



More information about the cfe-commits mailing list