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