[PATCH,c++] introduce {class,type}_of_this functions

Jason Merrill jason@redhat.com
Fri May 6 13:43:00 GMT 2011


On 05/06/2011 07:49 AM, Nathan Froyd wrote:
> The patch below introduces simple accessors for getting at the class or
> the type of the `this' parameter.  It hides a couple of TYPE_ARG_TYPES
> usages and makes the code slightly more obvious, I think.

Hmm, when I first read the names I expected them to refer to the 'this' 
in the current function.  I think adding _parm to the end of the names 
would help.

The type_ function should also have an assert that fntype is a METHOD_TYPE.

OK with those changes.

Jason



More information about the Gcc-patches mailing list