This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: genmatch indent generated code
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Michael Matz <matz at suse dot de>
- Cc: Jakub Jelinek <jakub at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 10 Jul 2015 10:45:25 +0200
- Subject: Re: genmatch indent generated code
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot LSU dot 2 dot 20 dot 1507091416380 dot 23227 at wotan dot suse dot de> <20150709151048 dot GT10247 at tucnak dot redhat dot com> <alpine dot LSU dot 2 dot 20 dot 1507091720320 dot 23227 at wotan dot suse dot de>
On Thu, Jul 9, 2015 at 5:23 PM, Michael Matz <matz@suse.de> wrote:
> Hi,
>
> On Thu, 9 Jul 2015, Jakub Jelinek wrote:
>
>> That violates the coding style by not using tabs ;)
>
> I knew it! Somebody would notice, pffft. Fixed in the committed version.
I also noticed it but didn't care ;) But now I notice
switch (TREE_CODE (t))
{
case SSA_NAME:
cases are indented too much, it should be
switch (TREE_CODE (t))
{
case SSA_NAME:
Richard.
>
> Ciao,
> Michael.
> PS: this still isn't fully correct, as sometimes I start the strings with
> spaces which don't count towards the indent parameter, I don't align code
> coming from (with ...) and long lines aren't broken up. Left as an
> excercise for the reader ;)