On Tue, Nov 28, 2017 at 04:38:12PM -0500, Tom Lane wrote: > Thomas Munro <thomas.mu...@enterprisedb.com> writes: > > On Wed, Nov 29, 2017 at 9:47 AM, Tom Lane <t...@sss.pgh.pa.us> wrote: > >> I think that'd be taking it too far, especially given that the dependency > >> on a typedefs list means that the git hook might have a different idea > >> of what's correctly indented than the committer does. It'd be very hard > >> to debug such discrepancies and figure out what would satisfy the hook. > > > Is there no way to get reasonable indentation that doesn't depend on > > that typedefs list? > > Perhaps, but not with the tool we've got. > > It's well known that C is unparseable without knowing which identifiers > are typedefs, so it doesn't exactly surprise me that it might not be > sanely indentable without knowing that. But I've not thought hard about > it, nor looked for alternate tools.
For people curious about the C typedef parsing details: http://calculist.blogspot.com/2009/02/c-typedef-parsing-problem.html https://stackoverflow.com/questions/243383/why-cant-c-be-parsed-with-a-lr1-parser -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + As you are, so once was I. As I am, so you will be. + + Ancient Roman grave inscription +