Re: [PATCH] Function-like cast notation support

[Tom Tromey <tromey@redhat.com>]

>>>>> "Sérgio" == Sérgio Durigan Júnior <sergiodj@redhat.com> writes:

Sérgio> The following patch adds support for function-like cast notation
Sérgio> on GDB.

This looks good.

It could use a NEWS entry.  I suggest waiting until my NEWS change goes
in, then adding it to the C++ improvements section.

Sérgio> Currently it only works for primitive data type (int,
Sérgio> char, double, etc), but I intend to send another patch soon that
Sérgio> will implement this feature for classes as well.

Hmm, you seem to have implemented this already...

Sérgio> +typebase_single
Sérgio>  	:	TYPENAME
Sérgio>  			{ $$ = $1.type; }

... because the lexer can return TYPENAME for a class name.

Sérgio> +		{ "void"      "void"		"'a'"		"void" }

I didn't realize this was supposed to work... learn something every day
:-)

This patch is ok.  Thanks.

For future reference, it is worth noting in the email how and whether
you regression tested it.  I assume that everybody does this, but it is
nice to be explicit about it.  At least in my case, this occasionally
helps me realize that I forgot.

Tom