[cfe-dev] c++ templates

Andrew Sutton andrew.n.sutton at gmail.com
Thu Nov 13 13:05:40 PST 2008


>
>
>  Attached is a very (VERY) preliminary stab at the beginning of template
>> parsing for clang. It's not complete, it doesn't do anything, there's no AST
>> stuff, I haven't even considered scopes and name resolution, etc., etc. It
>> just recognizes "template <...>". I had hoped to spend some spare time, an
>> hour or so a day (whenever possible) to continue working on this. Please
>> bear in mind that a) I've never worked on a real compiler before and b) I
>> probably don't fully understand the requirements of what I'm working on.
>>
> I hate to be nitpicking instead of giving constructive comments, but a lot
> of the patch is just changes in whitespace. While I agree that all the
> trailing whitespace in the Clang source is a bit annoying, those changes
> make the patch very hard to read.
>

I'll lay that at the feet of my text editor. I guess I didn't realize how
much actually changed. Here's the same diff - hopefully all the relevant
parts - without  the WS changes. I cut all the seemingly trivial changes by
hand. Hopefully, I didn't cut anything important.

Andrew Sutton
andrew.n.sutton at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20081113/3c77e06e/attachment-0005.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: templates-2.diff
Type: application/octet-stream
Size: 8442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20081113/3c77e06e/attachment-0005.obj>


More information about the cfe-dev mailing list