This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)
- From: Marek Polacek <polacek at redhat dot com>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: David Malcolm <dmalcolm at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Jason Merrill <jason at redhat dot com>
- Date: Fri, 2 Jun 2017 18:39:54 +0200
- Subject: Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=polacek at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 29225C057FA6
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 29225C057FA6
- References: <20170601164517.GB3413@redhat.com> <1496340501.7551.92.camel@redhat.com> <alpine.DEB.2.20.1706012210400.27473@digraph.polyomino.org.uk>
On Thu, Jun 01, 2017 at 10:13:01PM +0000, Joseph Myers wrote:
> On Thu, 1 Jun 2017, David Malcolm wrote:
>
> > The patch appears to only consider "if" and "else" clauses. Shouldn't
> > it also cover "for", "while" and "do/while"?
>
> do/while would normally get a syntax error in the problem cases.
Indeed, I think there's no point in handling "do/while" for this warning.
Marek