use strict;
use warnings;
+sub skipall {
+ my ($msg) = @_;
+ require Test::More;
+ Test::More::plan(skip_all => $msg);
+}
+
use Config qw/%Config/;
BEGIN {
- if (!$Config{useithreads}) {
- require Test::More;
- Test::More->import;
- plan(skip_all => 'This perl wasn\'t built to support threads');
- }
+ my $t_v = '1.67';
+ my $ts_v = '1.14';
+ skipall 'This perl wasn\'t built to support threads'
+ unless $Config{useithreads};
+ skipall "threads $t_v required to test thread safety"
+ unless eval "use threads $t_v; 1";
+ skipall "threads::shared $ts_v required to test thread safety"
+ unless eval "use threads::shared $ts_v; 1";
}
-use threads; # Before Test::More
-use threads::shared;
-
-use Test::More;
+use Test::More; # after threads
use Variable::Magic qw/wizard cast dispell getdata VMG_THREADSAFE VMG_OP_INFO_NAME VMG_OP_INFO_OBJECT/;
-if (VMG_THREADSAFE) {
+BEGIN {
+ skipall 'This Variable::Magic isn\'t thread safe' unless VMG_THREADSAFE;
plan tests => 2 * (4 * 18 + 1) + 2 * (4 * 13 + 1);
my $v = $threads::VERSION;
diag "Using threads $v" if defined $v;
$v = $threads::shared::VERSION;
diag "Using threads::shared $v" if defined $v;
-} else {
- plan skip_all => 'This Variable::Magic isn\'t thread safe';
}
my $destroyed : shared = 0;
sig => $sig,
get => sub { ++$c; 0 },
set => sub {
- my $name = $_[-1];
- $name = $name->name if $op_info == VMG_OP_INFO_OBJECT;
- is $name, 'sassign', "opname for op_info $op_info in thread $tid is correct";
+ my $op = $_[-1];
+ if ($op_info == VMG_OP_INFO_OBJECT) {
+ is_deeply { class => ref($op), name => $op->name },
+ { class => 'B::BINOP', name => 'sassign' },
+ "op object in thread $tid is correct";
+ } else {
+ is $op, 'sassign', "op name in thread $tid is correct";
+ }
0
},
- free => sub { ++$destroyed; 0 },
+ free => sub { lock $destroyed; ++$destroyed; 0 },
op_info => $op_info
};
is($@, '', "wizard in thread $tid doesn't croak");
for my $dispell (1, 0) {
for my $sig (undef, Variable::Magic::gensig()) {
- $destroyed = 0;
+ {
+ lock $destroyed;
+ $destroyed = 0;
+ }
+
my @t = map { threads->create(\&try, $dispell, $sig, $_) }
(VMG_OP_INFO_NAME) x 2, (VMG_OP_INFO_OBJECT) x 2;
$_->join for @t;
- is($destroyed, (1 - $dispell) * 4, 'destructors');
+
+ {
+ lock $destroyed;
+ is $destroyed, (1 - $dispell) * 4, 'destructors';
+ }
}
}