This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 3/3] Fix indentation issues seen by -Wmisleading-indentation
- From: Mikael Morin <mikael dot morin at sfr dot fr>
- To: David Malcolm <dmalcolm at redhat dot com>, gcc-patches at gcc dot gnu dot org, thomas at codesourcery dot com
- Date: Wed, 29 Apr 2015 14:10:33 +0200
- Subject: Re: [PATCH 3/3] Fix indentation issues seen by -Wmisleading-indentation
- Authentication-results: sourceware.org; auth=none
- Authentication-results: sfrmc.priv.atos.fr; dkim=none (no signature); dkim-adsp=none (no policy) header dot from=mikael dot morin at sfr dot fr
- References: <5536936F dot 8090004 at gmail dot com> <1430265776-8157-1-git-send-email-dmalcolm at redhat dot com> <1430265776-8157-3-git-send-email-dmalcolm at redhat dot com>
Hello,
Le 29/04/2015 02:02, David Malcolm a écrit :
> diff --git a/gcc/fortran/parse.c b/gcc/fortran/parse.c
> index 2c7c554..30e4eab 100644
> --- a/gcc/fortran/parse.c
> +++ b/gcc/fortran/parse.c
> @@ -4283,7 +4283,7 @@ parse_oacc_structured_block (gfc_statement acc_st)
> unexpected_eof ();
> else if (st != acc_end_st)
> gfc_error ("Expecting %s at %C", gfc_ascii_statement (acc_end_st));
> - reject_statement ();
> + reject_statement ();
> }
> while (st != acc_end_st);
>
I think this one is a bug; there should be braces around 'gfc_error' and
'reject_statement'.
At least that's the pattern in 'parse_oacc_loop', and how the
'unexpected_statement' function is used.
Author CC'ed in any case.
Mikael