Here is another version of my initdb shell quoting patch. I have removed the unnecessary {} block. I also ran pgindent on the code prior to creating the patch.
On Thu, Aug 18, 2016 at 3:50 PM, Ryan Murphy <ryanfmur...@gmail.com> wrote: > > > On Thu, Aug 18, 2016 at 3:44 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: > >> Ryan Murphy <ryanfmur...@gmail.com> writes: >> > On Thu, Aug 18, 2016 at 3:22 PM, Robert Haas <robertmh...@gmail.com> >> wrote: >> >>>> + { /* pg_ctl command w path, properly quoted */ >> >>>> + PQExpBuffer pg_ctl_path = createPQExpBuffer(); >> >>>> + printfPQExpBuffer(pg_ctl_path, "%s%spg_ctl", >> >> >> I think it's worth reducing the scope of variables when that's as >> >> simple as putting them inside a block that you have to create anyway, >> >> but I'm skeptical about the idea that one would create a block just to >> >> reduce the scope of the variables. I don't think that's our usual >> >> practice, and I would expect the compiler to detect where the variable >> >> is referenced first and last anyway. >> >> > I enjoy adding the blocks for explicit variable scoping and for quick >> > navigation in vim (the % key navigates between matching {}'s). But I >> want >> > to fit in with the style conventions of the project. >> >> Another point here is that code like this will look quite a bit different >> after pgindent gets done with it --- that comment will not stay where >> you put it, for example. Some of our worst formatting messes come from >> code wherein somebody adhered to their own favorite layout style without >> any thought for how it would play with pgindent. >> >> regards, tom lane >> > > Ahh, I didn't know about pgindent, but now I see it in /src/tools. I can > run that on my code before submitting. > > I found these > <https://www.postgresql.org/message-id/1221125165.5637.12.camel@abbas-laptop> > links <https://www.postgresql.org/docs/devel/static/source-format.html> > about the style convention and will make sure my patch fits the conventions > before submitting it. > >
0001-initdb-quote-shell-args-in-final-pg_ctl-command.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers