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

On Thu, Jan 9, 2014 at 4:41 AM, Ashesh Vashi
<ashesh(dot)vashi(at)enterprisedb(dot)com>wrote:

> Looking good to me.
> If nobody has any objection, then I can check-in.
>
> NOTE:
> I don't know the reason for the class sysSettings to inherit the wxConfig
> privately in original implementation.
> It was done by Dave in this commit:
> cfbae7d378766ab8cf138123186a024afffc061e.
>

That was a long time ago, but if memory serves it was intentionally done to
force the use of the sysSettings members, and not wxConfig, because some of
them have similar names (because there are only so many ways to name some
generic functions), but do additional things that we need; and in the past
some code had crept in that used wxConfig directly, bypassing our
additional code and subtly breaking things.

>
>
>
> On Thu, Jan 9, 2014 at 5:42 AM, Kaarel Moppel <kaarel(dot)moppel(at)gmail(dot)com>wrote:
>
>> 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>
>>>
>>
>>
>
>
> --
> --
>
> 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>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company