Horton

Re: Review of VS 2010 support patches

Andrew Dunstan <andrew@dunslane.net>

View thread
From: Andrew Dunstan <andrew@dunslane.net>
To: Magnus Hagander <magnus@hagander.net>
Cc: Brar Piening <brar@gmx.de>, Craig Ringer <craig@postnewspapers.com.au>, PG Hackers <pgsql-hackers@postgresql.org>
Date: 2011-08-10T13:25:24Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Enable compiling with the mingw-w64 32 bit compiler.

  2. Unify spelling of "canceled", "canceling", "cancellation"


On 08/10/2011 09:03 AM, Magnus Hagander wrote:
> On Sun, Jul 31, 2011 at 03:25, Andrew Dunstan<andrew@dunslane.net>  wrote:
>>
>> On 07/06/2011 08:26 PM, Brar Piening wrote:
>>> -------- Original Message  --------
>>> Subject: Re: [HACKERS] Review of VS 2010 support patches
>>> From: Andrew Dunstan<andrew@dunslane.net>
>>> To: Brar Piening<brar@gmx.de>
>>> Date: 06.07.2011 22:58
>>>
>>>>> I'll remove my versions from the patch (v9 probably) if those files get
>>>>> commited.
>>>>>
>>>>
>>>> I'm just doing some final testing and preparing to commit the new pgflex
>>>> and pgbison.
>>>
>>> The attached patch includes documentation changes and excludes my versions
>>> of pgbison.pl and pgflex.pl which have been replaced by Andrews' versions
>>> that are already commited.
>>>
>>> As before "perltidy_before.patch" has to be applied first and
>>> "VS2010v9.patch" second.
>>>
>>>
>> I just started looking at this a bit. One small question: why are we using
>> "use base qw(foo);" instead of "use parent qw(foo);" which I understand is
>> preferred these days?
> I am no perl expert, but I see we are using this already today - in
> code written by you in one case ;) I'd assume it was just following
> the same standard... If the other way is the way to do it today, I see
> no reason not to change it to use that.
>

Umm, where are we using it today?

    [andrew@emma pg_head]$ grep -r -P 'use\s+base' .
    ./doc/src/sgml/release-old.sgml:   what lexer you use based on the
    platform you use.
    ./doc/src/sgml/charset.sgml:     encoding to use based on the
    specified or default locale.
    ./src/backend/commands/aggregatecmds.c:         * Old style: use
    basetype parameter.  This supports aggregates of
    ./autom4te.cache/output.0:# Required to use basename.
    ./autom4te.cache/output.0:# Required to use basename.
    ./configure:# Required to use basename.
    ./configure:# Required to use basename.
    [andrew@emma pg_head]$


cheers

andrew