r191370 - Sema: Simplify code a bit, plug a potential leak.
Daniel Marjamäki
Daniel.Marjamaki at evidente.se
Wed Sep 25 08:41:21 PDT 2013
Nice! Your changes look good to me.
When looking at the code, it seems to me that NumArgsToCheck is redundant. Am I missing something? I include a diff.
The tests runs fine with my change.
..................................................................................................................
Daniel Marjamäki
Senior Engineer
Evidente ES East AB
Warfvinges väg 34 SE-112 51 Stockholm Sweden
Mobile:
+46 (0)709 12 42 62
E-mail:
Daniel.Marjamaki at evidente.se
www.evidente.se
-------------- next part --------------
A non-text attachment was scrubbed...
Name: remove-NumArgsToCheck.diff
Type: text/x-patch
Size: 1037 bytes
Desc: remove-NumArgsToCheck.diff
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130925/840e099e/attachment.bin>
More information about the cfe-commits
mailing list