]> git.vpit.fr Git - perl/modules/Test-Valgrind.git/commitdiff
Switch to qw<>
authorVincent Pit <vince@profvince.com>
Thu, 25 Aug 2011 15:19:14 +0000 (17:19 +0200)
committerVincent Pit <vince@profvince.com>
Thu, 25 Aug 2011 15:19:14 +0000 (17:19 +0200)
24 files changed:
Makefile.PL
lib/Test/Valgrind.pm
lib/Test/Valgrind/Action.pm
lib/Test/Valgrind/Action/Captor.pm
lib/Test/Valgrind/Action/Suppressions.pm
lib/Test/Valgrind/Action/Test.pm
lib/Test/Valgrind/Command.pm
lib/Test/Valgrind/Command/Aggregate.pm
lib/Test/Valgrind/Command/Perl.pm
lib/Test/Valgrind/Command/PerlScript.pm
lib/Test/Valgrind/Component.pm
lib/Test/Valgrind/Parser.pm
lib/Test/Valgrind/Parser/Suppressions/Text.pm
lib/Test/Valgrind/Parser/Text.pm
lib/Test/Valgrind/Parser/XML.pm
lib/Test/Valgrind/Parser/XML/Twig.pm
lib/Test/Valgrind/Report.pm
lib/Test/Valgrind/Session.pm
lib/Test/Valgrind/Suppressions.pm
lib/Test/Valgrind/Tool.pm
lib/Test/Valgrind/Tool/memcheck.pm
t/10-good.t
t/92-pod-coverage.t
t/lib/Test/Valgrind/Test/Action.pm

index b30c074f6fc873da6008b97dfc5ea504cc6030b4..96979982eafcb47c9ded1941cfbf23cb42ca56f0 100644 (file)
@@ -7,7 +7,7 @@ use ExtUtils::MakeMaker;
 BEGIN {
  eval { require Config };
  die 'OS unsupported' if $@;
- Config->import(qw/%Config/);
+ Config->import(qw<%Config>);
  eval { require File::Spec };
  die 'OS unsupported' if $@;
 }
index a5883e4377b1d8aaacda9e1ac341502b34b16e92..322ab89292a8fe06511f1ecd981772e916c88f8a 100644 (file)
@@ -166,7 +166,7 @@ sub analyse {
  my $sess = eval {
   Test::Valgrind::Session->new(
    min_version => $tool->requires_version,
-   map { $_ => delete $args{$_} } qw/extra_supps no_def_supp/
+   map { $_ => delete $args{$_} } qw<extra_supps no_def_supp>
   );
  };
  unless ($sess) {
index 62c46c993a998ddf1fad71cc729f5e9cbce56dae..94991ec4e2e92e4bb4878b0023fed833392b348c 100644 (file)
@@ -23,7 +23,7 @@ Actions are called each time a tool encounter an error and decide what to do wit
 
 =cut
 
-use base qw/Test::Valgrind::Component Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Component Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 755f5b9eae0c7dd8bfca297d2118b023bb699c32..a29f916b2e76c8c645f1bfee1fc645337cb3d78b 100644 (file)
@@ -25,7 +25,7 @@ It's not meant to be used directly as an action.
 
 use File::Spec ();
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 8f048967981a5edb07cab03117cf913826dc28f9..8a801ecbf55f2860f6fca496db562ee387a7a2d0 100644 (file)
@@ -21,7 +21,7 @@ This action just writes the contents of the suppressions reports received into t
 
 =cut
 
-use base qw/Test::Valgrind::Action Test::Valgrind::Action::Captor/;
+use base qw<Test::Valgrind::Action Test::Valgrind::Action::Captor>;
 
 =head1 METHODS
 
@@ -45,7 +45,7 @@ sub new {
 
  my %validated;
 
- for (qw/name target/) {
+ for (qw<name target>) {
   my $arg = delete $args{$_};
   $class->_croak("'$_' is expected to be a plain scalar")
                                                    unless $arg and not ref $arg;
@@ -54,7 +54,7 @@ sub new {
 
  my $self = $class->SUPER::new(%args);
 
- $self->{$_} = $validated{$_} for qw/name target/;
+ $self->{$_} = $validated{$_} for qw<name target>;
 
  $self;
 }
@@ -82,7 +82,7 @@ sub start {
 
  $self->SUPER::start($sess);
 
- delete @{$self}{qw/status supps diagnostics/};
+ delete @{$self}{qw<status supps diagnostics>};
 
  $self->save_fh(\*STDOUT => '>' => undef);
  $self->save_fh(\*STDERR => '>' => undef);
index 70991deffdfe1deaaf5cacb93ede6d6b41b0bbf5..36ac416064b86f8be93425bd4ab51ba76d53efe3 100644 (file)
@@ -23,7 +23,7 @@ This action uses C<Test::Builder> to plan and pass or fail tests according to th
 
 use Test::Builder;
 
-use base qw/Test::Valgrind::Action Test::Valgrind::Action::Captor/;
+use base qw<Test::Valgrind::Action Test::Valgrind::Action::Captor>;
 
 =head1 METHODS
 
@@ -158,7 +158,7 @@ sub finish {
   seek $fh, $self->{capture_pos}, 0;
   $tb->diag($_) while <$fh>;
   close $fh or $self->_croak('close(capture[' . fileno($fh) . "]): $!");
-  delete @{$self}{qw/capture capture_pos/};
+  delete @{$self}{qw<capture capture_pos>};
  }
 
  my $failed = 0;
index 15e47b94cc36d38817f21c265de03a51d6dda56c..81c615de24fb9dd33f7466924c7fa0fabaa911b3 100644 (file)
@@ -23,7 +23,7 @@ Commands gather information about the target of the analysis. They should also p
 
 =cut
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index c526d8aff396904741a4fd6e6fc802efabce52ac..10a20b9e1336016ab8aba4c98d60b24a7cd62d11 100644 (file)
@@ -23,7 +23,7 @@ This command groups several commands together, which the session will run under
 
 use Scalar::Util ();
 
-use base qw/Test::Valgrind::Command Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Command Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 38c238b58ab7e387ca66a8aa49d1d32a5692ad7b..bdc8ed390e082843baf1ea63624dd6b2e5aa9c21 100644 (file)
@@ -26,7 +26,7 @@ use Env::Sanctify ();
 
 use Test::Valgrind::Suppressions;
 
-use base qw/Test::Valgrind::Command Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Command Test::Valgrind::Carp>;
 
 =head1 METHODS
 
@@ -270,7 +270,7 @@ my @cards = List::Util::shuffle(0 .. 51);
 {
  package Test::Valgrind::Test::Fake;
 
- use base qw/strict/;
+ use base qw<strict>;
 }
 
-eval 'use Time::HiRes qw/usleep/';
+eval 'use Time::HiRes qw<usleep>';
index 0aa3f9e907e566fc4955fcc9e114ae82ff04b951..29eedf462562581bec2facd1516da642b95d9941 100644 (file)
@@ -21,7 +21,7 @@ This command is meant to abstract the argument list handling of a C<perl> script
 
 =cut
 
-use base qw/Test::Valgrind::Command::Perl Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Command::Perl Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 15c7cb9b5742994e4fc1e76e13fc18c7e98ff40a..540f14742d9ba75335f1fa598979fcb190c173e7 100644 (file)
@@ -23,7 +23,7 @@ This class is the base for all others that act as components that can be started
 
 use Scalar::Util ();
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 087a199d5cdc5a09a01d99136400cc7536573e9b..0614e2b17e563557b1c4d57d38b150af4244273a 100644 (file)
@@ -21,7 +21,7 @@ This class is the base for L<Test::Valgrind> parsers.
 
 =cut
 
-use base qw/Test::Valgrind::Component Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Component Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index 6b87cf1e57157a66872d48d989acb52a9052047b..642770efee4ea61320a53e42da3621fd577f68fc 100644 (file)
@@ -23,7 +23,7 @@ This is a L<Test::Valgrind::Parser::Text> object that can extract suppressions f
 
 use Test::Valgrind::Suppressions;
 
-use base qw/Test::Valgrind::Parser::Text Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Parser::Text Test::Valgrind::Carp>;
 
 =head1 METHODS
 
@@ -79,11 +79,11 @@ sub parse {
 
    my %call; # Frames to append (if the value is 1) or to prepend (if it's 0)
    if ($t eq 'm') {       # malloc can also be called by calloc or realloc
-    $call{$_} = 1 for qw/calloc realloc/;
+    $call{$_} = 1 for qw<calloc realloc>;
    } elsif ($t eq 're') { # realloc can also call malloc or free
-    $call{$_} = 0 for qw/malloc free/;
+    $call{$_} = 0 for qw<malloc free>;
    } elsif ($t eq 'c') {  # calloc can also call malloc
-    $call{$_} = 0 for qw/malloc/;
+    $call{$_} = 0 for qw<malloc>;
    }
 
    my $c = $_;
@@ -138,7 +138,7 @@ This program is free software; you can redistribute it and/or modify it under th
 
 package Test::Valgrind::Report::Suppressions;
 
-use base qw/Test::Valgrind::Report/;
+use base qw<Test::Valgrind::Report>;
 
 sub kinds { shift->SUPER::kinds(), 'Suppression' }
 
index f17f09ecb29add9ee24be388520044f2e7177f8b..6320c8b19a7b64d3eae13ea431c632a9342c019d 100644 (file)
@@ -21,7 +21,7 @@ This is a L<Test::Valgrind::Parser> object that can extract suppressions from C<
 
 =cut
 
-use base qw/Test::Valgrind::Parser/;
+use base qw<Test::Valgrind::Parser>;
 
 =head1 METHODS
 
index d7d232c310ae71b3c0c335d76629fb1b17155243..be1aa2e8c685b618d6b1d63bad2145de2b50a702 100644 (file)
@@ -21,7 +21,7 @@ This is a base class for L<Test::Valgrind::Parser> objects that can parse C<valg
 
 =cut
 
-use base qw/Test::Valgrind::Parser/;
+use base qw<Test::Valgrind::Parser>;
 
 =head1 METHODS
 
index e919008fec2df9b9a497483b86747a153d490d0f..6e294824fdf9330766a87b9d28417c458cb73535 100644 (file)
@@ -24,7 +24,7 @@ It is able to parse the XML output from C<valgrind> up to protocol version 4 and
 
 use Scalar::Util ();
 
-use base qw/Test::Valgrind::Parser::XML Test::Valgrind::Carp XML::Twig/;
+use base qw<Test::Valgrind::Parser::XML Test::Valgrind::Carp XML::Twig>;
 
 BEGIN { XML::Twig->add_options('Stash'); }
 
@@ -62,7 +62,7 @@ It is reset before and after the parsing phase, so it's effectively only availab
 =cut
 
 eval "sub $_ { \@_ <= 1 ? \$_[0]->{$_} : (\$_[0]->{$_} = \$_[1]) }"
-                                              for qw/_session protocol_version/;
+                                              for qw<_session protocol_version>;
 
 # We must store the session in ourselves because it's only possible to pass
 # arguments to XML::Twig objects by a global stash.
@@ -125,7 +125,7 @@ sub handle_error {
  $data->{stack} = [ map $_->listify_frame,
                                        $node->kid('stack')->children('frame') ];
 
- for (qw/leakedbytes leakedblocks/) {
+ for (qw<leakedbytes leakedblocks>) {
   my $kid = ($xwhat || $node)->first_child($_);
   next unless $kid;
   $data->{$_} = int $kid->text;
@@ -194,7 +194,7 @@ our $VERSION = '1.12';
 
 BEGIN { require XML::Twig; }
 
-use base qw/XML::Twig::Elt Test::Valgrind::Carp/;
+use base qw<XML::Twig::Elt Test::Valgrind::Carp>;
 
 sub kid {
  my ($self, $what) = @_;
@@ -212,7 +212,7 @@ sub listify_frame {
   map {
    my $x = $frame->first_child($_);
    $x ? $x->text : undef
-  } qw/ip obj fn dir file line/
+  } qw<ip obj fn dir file line>
  ];
 }
 
index 44d1c77c9119b0a3a56ff35bc64f5bb519b383df..46f44b83f5049b255905af8e9ff1ef40d781aa36 100644 (file)
@@ -25,7 +25,7 @@ The C<Diag> kind is reserved for diagnostics.
 
 =cut
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head2 C<< new kind => $kind, id => $id, data => $data >>
 
index 6c2a16c3fd2f190b957708769d9e886d0b325c23..951713f9f3978296f5fb2669e7b175641d48dd8b 100644 (file)
@@ -30,7 +30,7 @@ use POSIX (); # SIGKILL
 
 use version ();
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head1 METHODS
 
@@ -153,7 +153,7 @@ Read-only accessor for the C<no_def_supp> option.
 
 =cut
 
-eval "sub $_ { \$_[0]->{$_} }" for qw/valgrind no_def_supp/;
+eval "sub $_ { \$_[0]->{$_} }" for qw<valgrind no_def_supp>;
 
 =head2 C<extra_supps>
 
@@ -295,7 +295,7 @@ Read-only accessor for the C<command> associated to the current run.
 
 my @members;
 BEGIN {
- @members = qw/action tool command parser/;
+ @members = qw<action tool command parser>;
  for (@members) {
   eval "sub $_ { \@_ <= 1 ? \$_[0]->{$_} : (\$_[0]->{$_} = \$_[1]) }";
   die if $@;
@@ -388,7 +388,7 @@ sub start {
 
  my %args = @_;
 
- for (qw/action tool command/) {
+ for (qw<action tool command>) {
   my $base = 'Test::Valgrind::' . ucfirst;
   my $value = $args{$_};
   $self->_croak("Invalid $_") unless Scalar::Util::blessed($value)
@@ -396,7 +396,7 @@ sub start {
   $self->$_($args{$_})
  }
 
- delete @{$self}{qw/last_status exit_code/};
+ delete @{$self}{qw<last_status exit_code>};
 
  $self->tool->start($self);
  $self->parser($self->parser_class->new)->start($self);
@@ -428,7 +428,7 @@ sub report {
 
  return unless defined $report;
 
- for my $handler (qw/tool command/) {
+ for my $handler (qw<tool command>) {
   $report = $self->$handler->filter($self, $report);
   return unless defined $report;
  }
index 1790243a00d82f0ca254d66edfc2f28f6c14634f..99e79783a5ee9ebd0e65ac8e480c75e8b66abd4d 100644 (file)
@@ -21,7 +21,7 @@ This module is an helper for generating suppressions.
 
 =cut
 
-use base qw/Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index d91ab8102da0fa96ff25ff5d832c7089a8045d78..4ecb2e476d4cc761ca9e30529d52f4b18d550626 100644 (file)
@@ -24,7 +24,7 @@ They are expected to function both in suppressions generation and in analysis mo
 
 =cut
 
-use base qw/Test::Valgrind::Component Test::Valgrind::Carp/;
+use base qw<Test::Valgrind::Component Test::Valgrind::Carp>;
 
 =head1 METHODS
 
index abb7b6ea086521206aa8110216a6cd307cc92146..8e8371c460ea6924c76ec100c6067df0aa22cfcd 100644 (file)
@@ -21,7 +21,7 @@ This class contains the information required by the session for running the C<me
 
 =cut
 
-use base qw/Test::Valgrind::Tool/;
+use base qw<Test::Valgrind::Tool>;
 
 =head1 METHODS
 
@@ -159,11 +159,11 @@ This program is free software; you can redistribute it and/or modify it under th
 
 package Test::Valgrind::Tool::memcheck::Report;
 
-use base qw/Test::Valgrind::Report/;
+use base qw<Test::Valgrind::Report>;
 
 our $VERSION = '1.12';
 
-my @kinds = qw/
+my @kinds = qw<
  InvalidFree
  MismatchedFree
  InvalidRead
@@ -179,7 +179,7 @@ my @kinds = qw/
  Leak_IndirectlyLost
  Leak_PossiblyLost
  Leak_StillReachable
-/;
+>;
 push @kinds, __PACKAGE__->SUPER::kinds();
 
 my %kinds_hashed = map { $_ => 1 } @kinds;
index 2534e5161f2ad4d16336bc8ae769f7804c1c3268..d3ffb905c599d08e5037a323b2703030cd2c3958 100644 (file)
@@ -15,7 +15,7 @@ if ($@) {
 {
  package Test::Valgrind::Test::Fake;
 
- use base qw/strict/;
+ use base qw<strict>;
 }
 
 plan tests => 1;
index a225486c8c3e61360feade855a5f6b23681d8311..74b67f4aaae7409a2115d1f68fe0df8da0bde180 100644 (file)
@@ -42,7 +42,7 @@ pod_coverage_ok('Test::Valgrind::Parser::Text', $trustparents);
 pod_coverage_ok('Test::Valgrind::Parser::XML', $trustparents);
 pod_coverage_ok('Test::Valgrind::Parser::XML::Twig', {
  %$trustparents,
- also_private => [ qr/^handle_/, qw/stash/ ],
+ also_private => [ qr/^handle_/, qw<stash> ],
 });
 
 pod_coverage_ok('Test::Valgrind::Report');
index 0b409400df761198bcd18820e02661d11b1b3196..8b41a68d6239bff23f9c0b5dc196a8312fcd3c25 100644 (file)
@@ -3,7 +3,7 @@ package Test::Valgrind::Test::Action;
 use strict;
 use warnings;
 
-use base qw/Test::Valgrind::Action::Test/;
+use base qw<Test::Valgrind::Action::Test>;
 
 my $extra_tests;