Re: Re: [COMMITTERS] pgsql: Get rid of the need for manual maintenance of the initial

Tom Lane wrote:
> I'm not nearly good enough in Perl to be sure about the semantics
> of this loop.  Is it possible that it's changing the global contents
> of the @$data structure, rather than just hacking a local copy of
> each row before pushing some values into @fmgr?
>   

That's exactly what it does. The loop variable is an alias. See 
perlsyn(1) for details.

These two lines appear to be suspicious:
   $row->{bki_values} =~ s/"[^"]*"/"xxx"/g;   @{$row}{@attnames} = split /\s+/, $row->{bki_values};

Something like:
   (my $bkival = $row->{bki_values}) =~ s/"[^"]*"/"xxx"/g;   my $atts = {};   @{$atts}{@attnames} = split /\s+/,
$bkival;

might work better.

cheers

andrew