On 02/19/2012 05:39 AM, Gregory Szorc wrote:
> Does anyone else think that "arguments" may not be the best name?
> Perhaps "argument_types" would be more suitable...

Yes. I agree. Either name it argument_types, or return a class called 
'argument' that than has a .type() function. Returning a class is 
probably overhead, as it would only have a single function. I would just 
name the iterator argument_types().

Also, I just realized you define the __iter__(self) method. Why is that 
needed? Diagnostic.ranges() gets away without using it?

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

Reply via email to