This is the mail archive of the mailing list for the GCC 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: PING: [PATCH: c++/diagnostic/18313] Warn for pointless qualifiers on return type

Dirk Mueller <> writes:

| On Tuesday, 28. November 2006 12:28, Gabriel Dos Reis wrote:
| > | If you have a better way to describe this behaviour, please tell me. I'd
| > | also be interested if the TREE_NO_WARNING on the type node has any
| > | undesired side-effect (there was none exposed by the testsuite).
| > But actually, we wanted to suppress the warning for that particular
| > decl, so we should be saying that.  Do you think you can arrange for
| > the TREE_NO_WARNING be set on the FUNCTION_DECL (or maybe on its
| > DECL_RESULT I'm not sure which is better) instead of its type?
| I'm not 100% sure I understood you correctly, but the patch below works as 
| well and survives bootstrap and regtest with no failures. 


| @@ -8612,6 +8620,9 @@ grokdeclarator (const cp_declarator *dec
|      if (!processing_template_decl)
|        cp_apply_type_quals_to_decl (type_quals, decl);
| +    if (set_no_warning)
| +        TREE_NO_WARNING (decl) = 1;
| +

Yes, this does what I was suggesting.  The revised patch if OK with
me.  Thanks for your patience.

-- Gaby

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