Ugh. I dunno what the right call is here. This is a kinda horrible hack,
if slightly less horrible than the last iteration... But I don't think
anything is going to fix it short of the refactoring you have a FIXME
for... I'd probably just wait for that refactoring to land, but maybe
Doug feels differently.


http://codereview.appspot.com/4929043/diff/13001/lib/Sema/SemaDecl.cpp
File lib/Sema/SemaDecl.cpp (right):

http://codereview.appspot.com/4929043/diff/13001/lib/Sema/SemaDecl.cpp#newcode4205
lib/Sema/SemaDecl.cpp:4205: struct AoFDParameterPack {
We need to put this into an anonymous namespace. Also, "ParameterPack"
means something very specific in C++. Maybe: ActOnFDArgs? Yuck. Dunno.

http://codereview.appspot.com/4929043/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to