From: | "MauMau" <maumau307(at)gmail(dot)com> |
---|---|
To: | "Rajeev rastogi" <rajeev(dot)rastogi(at)huawei(dot)com> |
Cc: | <pgsql-hackers(at)postgresql(dot)org>, "Magnus Hagander" <magnus(at)hagander(dot)net> |
Subject: | Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path |
Date: | 2014-02-22 00:46:15 |
Message-ID: | 10DE394009FB4117B17C8807B92FDB62@maumau |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Rajeev,
From: "Rajeev rastogi" <rajeev(dot)rastogi(at)huawei(dot)com>
Changed the patch as per your suggestion.
Now only one version of make_absolute_path there defined in src/port/path.c
Found one small memory leak in the existing function
make_absolute_path(miscinit.c),
fixed the same.
Thanks for refactoring. I confirmed that the revised patch applies to HEAD
cleanly, the source files built without extra warnings, and the original
intended problem was solved.
Please make small cosmetic changes so that make_absolute_path() follows the
style of other parts. Then I'll make this ready for committer.
(1)
Add the function name in the comment as in:
/*
* make_absolute_path
*
* ...existing function descripton
*/
(2)
Add errno description as in:
fprintf(stderr, _("could not get current working directory: %s\n",
strerror(errno)));
Regards
MauMau
From | Date | Subject | |
---|---|---|---|
Next Message | Tomas Vondra | 2014-02-22 01:07:23 | Re: pg_stat_tmp files for dropped databases |
Previous Message | Thom Brown | 2014-02-22 00:13:18 | pg_stat_tmp files for dropped databases |