From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>,Ryan Murphy <ryanfmurphy(at)gmail(dot)com> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Patch: initdb: "'" for QUOTE_PATH (non-windows) |
Date: | 2016-08-18 03:18:33 |
Message-ID: | C97B3D0C-A6A6-402F-867B-BE8270D185F6@anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On August 17, 2016 8:15:56 PM PDT, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>+ { /* pg_ctl command w path, properly quoted */
>+ PQExpBuffer pg_ctl_path = createPQExpBuffer();
>+ printfPQExpBuffer(pg_ctl_path, "%s%spg_ctl",
>+ bin_dir,
>+ (strlen(bin_dir) > 0) ? DIR_SEP : ""
>+ );
>+ appendShellString(start_db_cmd, pg_ctl_path->data);
>+ destroyPQExpBuffer(pg_ctl_path);
>+ }
>
>This is not really project-style to have an independent block. Usually
>those are controlled by for, while or if.
Besides the comment positioning I'd not say that that is against the usual style, there's a number of such blocks already. Don't think it's necessarily needed here though...
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2016-08-18 03:18:38 | Re: Increasing timeout of poll_query_until for TAP tests |
Previous Message | Michael Paquier | 2016-08-18 03:15:56 | Re: Patch: initdb: "'" for QUOTE_PATH (non-windows) |