Re: [C++ Patch] PR 58305

2013-09-03 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 58305

2013-09-03 Thread Paolo Carlini
On 09/03/2013 11:32 PM, Jason Merrill wrote: On 09/03/2013 11:10 AM, Paolo Carlini wrote: ToBeDeprecated(); I'd rather handle this case in build_functional_cast. Indeed. Thanks for the great tip. A tried to find a place where I could use the same check we have in grokdeclarator, involving

Re: [C++ Patch] PR 58305

2013-09-03 Thread Jason Merrill
On 09/03/2013 11:10 AM, Paolo Carlini wrote: ToBeDeprecated(); I'd rather handle this case in build_functional_cast. Jason

Re: [C++ Patch] PR 58305

2013-09-03 Thread Paolo Carlini
.. unfortunately the issue isn't so easy because in any case we don't want to warn for typedefs of ToBeDeprecated. Paolo.

[C++ Patch] PR 58305

2013-09-03 Thread Paolo Carlini
Hi, I think it's rather clear that we should warn in this case too, that is, when we have: ToBeDeprecated(); for a deprecated ToBeDeprecated type. Note however in my patchlet the check on the TREE_CODE: if I remove it, then we also warn for the lines x = e; and y = S::f; of