[cfe-commits] r81123 - in /cfe/trunk: lib/AST/ExprCXX.cpp lib/Sema/SemaDeclCXX.cpp test/SemaTemplate/default-expr-arguments.cpp

[Anders Carlsson]

6 sep 2009 kl. 10.35 skrev Sebastian Redl:

> Anders Carlsson wrote:
>> Author: andersca
>> Date: Sun Sep  6 11:54:02 2009
>> New Revision: 81123
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=81123&view=rev
>> Log:
>> Reapply 81096, now with a fix. Spot the bug:
>>
>>      for (unsigned i = numargs; i < NumArgs; ++i)
>>        Args[0] = 0;
>>
> Aside from the actual bug, wouldn't it also be a good idea to rename  
> one
> of the two numargs variables?

Turns out the code didn't handle variadic constructors, so I cleaned  
up the code in r81178.

Anders

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20090907/713134ba/attachment.html>