This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] support argument dependent lookup (ADL)


>>>>> "Sami" == Sami Wagiaalla <swagiaal@redha.com> writes:

Sami> I have added tests for all of the above situations.

Thanks.

Sami>     2010-04-09  Sami Wagiaalla  <swagiaal@redhat.com>
Sami>     	PR C++/7943:
Sami>     	* valops.c (find_overload_match): Handle fsym == NULL case.
Sami>     	Add int no_adl argument.
Sami>     	(find_oload_champ_namespace_loop): Call make_symbol_overload_list_adl
[...]

This ChangeLog entry and the one in the patch are different.

Sami> +			  write_exp_elt_sym (NULL); /* Place holder */

A nit, "placeholder" is one word.
Also, end with a period & double space.

Sami> +  while (TYPE_CODE (type) == TYPE_CODE_PTR || TYPE_CODE (type) == TYPE_CODE_REF
Sami> +         || TYPE_CODE (type) == TYPE_CODE_ARRAY)
Sami> +    type = TYPE_TARGET_TYPE (type);

I think you need a check_typedef in the loop.  Otherwise in some
situations with typedefs I think you can end up with type==NULL.

This is ok with those changes.  Thanks. 

Could you submit a followup patch that adds a NEWS entry?

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]