Re: pg_bsd_indent - improvements around offsetof and sizeof

From: Andres Freund <andres(at)anarazel(dot)de>
To: Piotr Stefaniak <postgres(at)piotr-stefaniak(dot)me>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pg_bsd_indent - improvements around offsetof and sizeof
Date: 2016-05-25 20:07:34
Message-ID: 20160525200734.nuv4ppfbvzj6odeu@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2016-05-25 22:01:53 +0200, Piotr Stefaniak wrote:
> On 2016-05-25 21:13, Tom Lane wrote:
> > I'd love to see a fix for its brain damage around function pointer typedef formatting, too.
>
> Show me a few examples and I'll look into it.
>
> > I'm excited about this too, not least because it suggests that maybe bsdindent isn't quite as opaque as it appears.
>
> It's old, hacked on many times over the past few decades and historically
> just a band-aid rather than something designed from the ground up, so it's
> not easy to work with. Which is why I think that a newer tool (like
> ClangFormat) should be considered as a replacement for pg_bsd_indent.

FWIW, I looked at using clang-format at some point, and it looked like
it'd be a number of additional options to make it work for our case
without changing the code layout too much. There seemed limited
enthusiasm from the authors about accepting relevant options.

Andres

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2016-05-25 20:32:57 Re: Changed SRF in targetlist handling
Previous Message Piotr Stefaniak 2016-05-25 20:01:53 Re: pg_bsd_indent - improvements around offsetof and sizeof