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

> On Tue, Jul 7, 2009 at 1:29 AM, Chris Lattner<[email protected]>  
> 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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to