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

Ryan Flynn parseerror at gmail.com
Tue Jul 7 02:27:15 PDT 2009


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 :)

Ryan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Lexer.cpp.CharInfo-static-init-(80-cols).patch
Type: text/x-diff
Size: 5646 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20090707/1f0fb822/attachment.patch>


More information about the cfe-commits mailing list