Why push the hashes onto a list? You could just push the values and then refer to the list as a hash (right?) or you could just build a hashref and point $self->{valuePair} to it. If you want to have multiple values per key then just make it a hashref with lists as the values.

my $theHash;
if ($inline =~ /<valueTargetPair (.*)\/>/) {
   if ($inline =~ /value=\"(.*?)\" targetPo=\"(.*?)\"/) {
      push @{$theHash->{$1}},$2;
   }
}
$self->{valuePair} = $theHash;
[download]

___________
Eric Hodges