This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 02/89] Introduce gimple_switch and use it in various places
- From: Florian Weimer <fweimer at redhat dot com>
- To: David Malcolm <dmalcolm at redhat dot com>, Trevor Saunders <tsaunders at mozilla dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 23 Apr 2014 09:17:06 +0200
- Subject: Re: [PATCH 02/89] Introduce gimple_switch and use it in various places
- Authentication-results: sourceware.org; auth=none
- References: <1398099480-49147-1-git-send-email-dmalcolm at redhat dot com> <1398099480-49147-3-git-send-email-dmalcolm at redhat dot com> <20140421224535 dot GA19895 at tsaunders-iceball dot corp dot tor1 dot mozilla dot com> <1398186820 dot 26834 dot 100 dot camel at surprise> <1398197846 dot 26834 dot 134 dot camel at surprise>
On 04/22/2014 10:17 PM, David Malcolm wrote:
or indeed, something like:
else if (gimple_switch switch_stmt =
dyn_cast <gimple_switch> (stmt))
{
to avoid an 83-character-wide line :)
Hope that's the appropriate way to split such a line; I can never
remember if one is supposed to put the linebreak before or after the =.
It's supposed to come before, so that the equal sign is at the start of
the line. Operators are also supposed to be at the start and not the
end. At least the operator part is covered in the coding standards:
<http://www.gnu.org/prep/standards/html_node/Formatting.html>
--
Florian Weimer / Red Hat Product Security Team