[Mesa-dev] Preprocessor patches

[Carl Worth]

Carl Worth <cworth at cworth.org> writes:
> It probably wouldn't be too hard to fix this code to be more general. I
> might take a whack at that now that I have this test in hand.

It wasn't too hard. I just sent a v2 patch in reply to the original (and
also force-pushed a new glcpp-fixup branch to my repository).

I've attached the result of glcpp on the mixed-newlines test file from
my previous email. It looks quite lovely, (if I do say so myself).

I haven't yet added testing for this to the test suite, (I think having
a source file with a mixture of newlines in is likely to confuse the
infrastructure I have for munging the test suite files to have various
flavors of newlines.) But I'll look into adding testing for that soon.

Thanks for nudging me to make this code more robust.

-Carl

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mixed-newlines-glcpp-v2
Type: application/octet-stream
Size: 206 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140806/941ec766/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140806/941ec766/attachment.sig>