Re: Review of VS 2010 support patches

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