Re: [PATCH] fix newly added server being lost when crashing

Ok, got it. Corrected version in attach now.

On Mon, Jan 6, 2014 at 12:29 PM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:

> Hi Kaarel,
>
> I think - everything looks good except:
> 1. Please use TAB instead of 4 spaces
> 2. You don't require to define Flush function after inheriting the
> wxConfig class publicly.
> Or,
> Define it only if you keep the wxConfig inherited protectively.
>
>
> On Mon, Jan 6, 2014 at 3:48 AM, Kaarel Moppel <kaarel(dot)moppel(at)gmail(dot)com>wrote:
>
>> Hi,
>>
>> Sending some 3 small fixes in separate emails for issues that me and my
>> colleagues found really annoying. This is my first try at Pgadmin and I
>> haven't done too much C++ though so in case something should be corrected
>> let me know and I'd be happy to do that.
>>
>> Cheers,
>> Kaarel
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>
>
> --
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company<http://www.enterprisedb.com>
>
>
>
> *http://www.linkedin.com/in/asheshvashi*<http://www.linkedin.com/in/asheshvashi>
>