[cfe-commits] [PATCH] Remove redundancy surrounding CharLiteralParser

Sean Silva silvas at purdue.edu
Tue Feb 28 18:27:55 PST 2012


I saw some code duplication around the use of CharLiteralParser, and
hopefully this fixes it; nothing major, it just looks like some APIs
changed and a little redundancy sprouted up.

There's some very similar duplication surrounding NumericLiteralParser and
StringLiteralParser that I'll fix up in a future patch soon, but I thought
I'd test the waters first.

I git diff'd against c568543. `make test` ran with no unexpected failures.

--Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120228/fb4375f5/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: CharLiteralParser.patch
Type: text/x-patch
Size: 4684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120228/fb4375f5/attachment.bin>


More information about the cfe-commits mailing list