[cfe-commits] [PATCH 3/5] [clang.py] Implement Type.arguments()

Tobias Grosser tobias at grosser.es
Sun Feb 19 00:49:07 PST 2012


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



More information about the cfe-commits mailing list