On Wed, Aug 10, 2011 at 15:25, Andrew Dunstan <andrew@dunslane.net> wrote:
>
>
> 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]$
Meh. I am clearly not back in the game since my vacation. I didn't
realize base was a keyword... Ignore and move on, nothing to see here.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/