From: | Vladimir Kokovic <vladimir(dot)kokovic(at)gmail(dot)com> |
---|---|
To: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Information for added functionalities |
Date: | 2012-09-19 07:46:04 |
Message-ID: | CAHsHPqdbT2HEcJ6iy3KP8pXzN4J18aemp5ghJP4OdCCTEHBhxQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
Separated in three patches:
patch1.tar.gz - pg_scanner and frmQuery.cpp with multiple result sets support
patch2.tar.gz - sysSettings::Flush
patch3.tar.gz - copy/paste table(s) with flush server settings to disk
Best regards
Vladimir Kokovic
Belgrade, Serbia, 19.September 2012
On 9/18/12, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Sat, Sep 15, 2012 at 6:53 AM, Vladimir Kokovic
> <vladimir(dot)kokovic(at)gmail(dot)com> wrote:
>> Hi,
>>
>> Separated in three patches:
>> patch1.tar.gz - pg_scanner and frmQuery.cpp with multiple result sets
>> support
>> patch2.tar.gz - sysSettings::Flush
>> patch3.tar.gz - copy/paste table(s)
>
> Please post the patches publicly, i.e. to the mailing list, on
> individual threads.
>
> I took a very brief look at 2 and 3, and noticed some issues you'll
> want to cleanup first:
>
> - There are whitespace-only changes which should be removed.
>
> - There are completely unrelated changes to some files - e.g. patch 3
> removes "$(srcdir)/" from the file paths in pgadmin/ui/module.mk, and
> there are changes to function argument handling.
>
> - There are still some parts of one new feature in the other - e.g.,
> patch 3 seems to contain code to flush server settings to disk.
>
> - I see very few comments explaining what code is supposed to do. This
> is not necessary if code is self explanatory, but is needed in more
> complex code blocks.
>
> I'm assuming that patch 1 has similar issues. Fixing them first will
> make the patches *much* smaller and will make it possible to review
> them.
>
> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
Attachment | Content-Type | Size |
---|---|---|
patch1.tar.gz | application/x-gzip | 235.8 KB |
patch2.tar.gz | application/x-gzip | 955 bytes |
patch3.tar.gz | application/x-gzip | 30.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2012-09-19 13:01:36 | Re: Information for added functionalities |
Previous Message | Begina Felicysym | 2012-09-17 21:55:47 | Odp: PL pgAdmin III commit: Update .pot and .po files with 1.16 files |