From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | pgadmin-hackers(at)postgresql(dot)org, Peter Geoghegan <peter(dot)geoghegan86(at)gmail(dot)com> |
Subject: | Re: pgAdmin III commit: Revert the previous change that introduced sysSetti |
Date: | 2011-02-17 11:33:27 |
Message-ID: | AANLkTikntcE_v1usNy6MFTVFvJpPEVkW7mNQwerc5Va9@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Peter;
Just so you're aware of this...
On Thu, Feb 17, 2011 at 11:31 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Revert the previous change that introduced sysSettings::WriteBool(),
> and restore the previous Write() overload. Whilst the change was, in
> theory, good, it failed to take into account that users of previous
> versions of pgAdmin would have true/false values stored in the
> Windows registry as REG_SZ values of "true" or "false", not REG_DWORDs
> as the new code would write. Needs more thought, and thorough testing
> on Windows when we re-visit this.
>
> Branch
> ------
> master
>
> Details
> -------
> http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=aedd49e2b7bfd3104b5a9f30a4e70c0ea2892170
>
> Modified Files
> --------------
> pgadmin/include/utils/sysSettings.h | 6 +++++-
> pgadmin/utils/sysSettings.cpp | 11 ++++++-----
> 2 files changed, 11 insertions(+), 6 deletions(-)
>
>
> --
> 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
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2011-02-17 13:14:48 | Re: pgAdmin III commit: Revert the previous change that introduced sysSetti |
Previous Message | Dave Page | 2011-02-17 11:31:30 | pgAdmin III commit: Revert the previous change that introduced sysSetti |