Re: pgAdmin III commit: Include a variant of sysSettings::Write() that take

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: pgAdmin III commit: Include a variant of sysSettings::Write() that take
Дата
Msg-id AANLkTimef6VPOnf6bruDG2vJ2pEyfeLBs5w1vyahUnDg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgAdmin III commit: Include a variant of sysSettings::Write() that take  (Peter Geoghegan <peter.geoghegan86@gmail.com>)
Список pgadmin-hackers
On Wed, Feb 16, 2011 at 6:50 PM, Peter Geoghegan
<peter.geoghegan86@gmail.com> wrote:
> On 16 February 2011 18:42, Dave Page <dpage@pgadmin.org> wrote:
>> The main issue I have with that is that we now have a bunch of
>> overloaded Write() members, and WriteBool(). If we're going to deviate
>> away from the API in wxConfig (which at least is private), then we
>> should do so consistently.
>>
>> FYI, in wxPython and wxPerl they implement the following:
>>
>> Write(key, value)        Writes a string
>> WriteInt(key, value)     Writes an integer
>> WriteFloat(key, value)   Writes a floating point number
>> WriteBool(key, value)    Writes a boo
>
> I totally agree. What about ctlListView? The fact that its
> AppendYesNoItem() member function doesn't overload AppendItem() is, as
> I said at the time, logical, because you aren't actually appending a
> true or false value - you're appending a "Yes" or a "No".

Yeah, I think that one is OK - AppendItem() is akin to Write() - ie.
string by default - where AppendYesNoItem() is (sort of) akin to
WriteBool().


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

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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: pgAdmin III commit: Include a variant of sysSettings::Write() that take
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Another set of wx2.9 updates. This patch includes t