Hi, On 2019-01-11 09:42:19 -0500, Tom Lane wrote: > Andres Freund <and...@anarazel.de> writes: > > The pluggable storage patchset has a large struct full of callbacks, and > > a bunch of wrapper functions for calling those callbacks. While > > starting to polish the patchset, I tried to make the formatting nice. > > ... > > So, putting the parameter list, both in use and declaration, entirely > > into a new line yields decent formatting with pgindent. But it's kinda > > weird. I can't really come up with a better alternative, and after a > > few minutes it looks pretty reasonable. > > > Comments? Better alternatives? > > Use shorter method names? This sounds like an ugly workaround for > a carpal-tunnel-syndrome-inducing design.
I'm confused. What did I write about that has unreasonably long names? And if you're referring to the wider design, that all seems fairly fundamental to something needing callbacks - not exactly a first in postgres. Greetings, Andres Freund
