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

Dave,

In that case, we should not change the inheritance mode.

Kaarel,

I've attached the patch as per Dave's comment.
Can you please test it at your end?

On Thu, Jan 9, 2014 at 2:35 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
>
> 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
>

--
--

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>