Skip site navigation (1) Skip section navigation (2)

Re: Information for added functionalities

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 (view raw or flat)
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: patch3.tar.gz
Description: application/x-gzip (30.4 KB)
Attachment: patch2.tar.gz
Description: application/x-gzip (955 bytes)
Attachment: patch1.tar.gz
Description: application/x-gzip (235.8 KB)

In response to

Responses

pgadmin-hackers by date

Next:From: Dave PageDate: 2012-09-19 13:01:36
Subject: Re: Information for added functionalities
Previous:From: Begina FelicysymDate: 2012-09-17 21:55:47
Subject: Odp: PL pgAdmin III commit: Update .pot and .po files with 1.16 files

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group