summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJesse Luehrs <doy@tozt.net>2010-11-14 16:47:30 -0600
committerJesse Luehrs <doy@tozt.net>2010-11-14 16:47:30 -0600
commit68cb1ade6177987a8950e4cb3050c1139e802fee (patch)
treeda58353394fc75776ed27ee619ab8b22482fd2e0
parentfe890fb97648df1253becf110e4e4dfa3c2c8015 (diff)
downloadeval-closure-68cb1ade6177987a8950e4cb3050c1139e802fee.tar.gz
eval-closure-68cb1ade6177987a8950e4cb3050c1139e802fee.zip
Revert "use Devel::Hints where possible"0.01
This reverts commit fe890fb97648df1253becf110e4e4dfa3c2c8015. "meh", this is a lot of effort for very little real gain
-rw-r--r--dist.ini1
-rw-r--r--lib/Eval/Closure.pm18
-rw-r--r--t/03-description.t2
-rw-r--r--t/05-memoize.t27
-rw-r--r--t/11-line-differences.t61
5 files changed, 9 insertions, 100 deletions
diff --git a/dist.ini b/dist.ini
index 620f458..75234df 100644
--- a/dist.ini
+++ b/dist.ini
@@ -7,7 +7,6 @@ copyright_holder = Jesse Luehrs
dist = Eval-Closure
[Prereqs]
-Devel::Hints = 0.22
Scalar::Util = 0
Sub::Exporter = 0
Try::Tiny = 0
diff --git a/lib/Eval/Closure.pm b/lib/Eval/Closure.pm
index ee93a7a..b03df5b 100644
--- a/lib/Eval/Closure.pm
+++ b/lib/Eval/Closure.pm
@@ -8,14 +8,11 @@ use Sub::Exporter -setup => {
# ABSTRACT: safely and cleanly create closures via string eval
use Carp;
-use Devel::Hints qw(cop_file cop_line);
use overload ();
use Memoize;
use Scalar::Util qw(reftype);
use Try::Tiny;
-use constant USE_DEVEL_HINTS => ($] >= 5.010);
-
=head1 SYNOPSIS
use Eval::Closure;
@@ -96,23 +93,14 @@ sub eval_closure {
$args{source} = _canonicalize_source($args{source});
_validate_env($args{environment} ||= {});
- if (!USE_DEVEL_HINTS) {
- $args{source} = _line_directive($args{description}) . $args{source}
- if defined $args{description};
- }
+ $args{source} = _line_directive($args{description}) . $args{source}
+ if defined $args{description};
my ($code, $e) = _clean_eval_closure(@args{qw(source environment)});
croak("Failed to compile source: $e\n\nsource:\n$args{source}")
unless $code;
- if (USE_DEVEL_HINTS) {
- if (defined $args{description}) {
- cop_file($code, $args{description});
- cop_line($code, 1);
- }
- }
-
return $code;
}
@@ -159,7 +147,7 @@ sub _validate_env {
sub _line_directive {
my ($description) = @_;
- return qq{#line 0 "$description"\n};
+ return qq{#line 1 "$description"\n};
}
sub _clean_eval_closure {
diff --git a/t/03-description.t b/t/03-description.t
index c9e8b21..97f8372 100644
--- a/t/03-description.t
+++ b/t/03-description.t
@@ -32,7 +32,7 @@ SOURCE
like(
exception { $code->() },
- qr/^foo at accessor foo \(defined at Class\.pm line 282\) line 1\n/,
+ qr/^foo at accessor foo \(defined at Class\.pm line 282\) line 2\n/,
"description is set"
);
}
diff --git a/t/05-memoize.t b/t/05-memoize.t
index e4b582b..02fd11f 100644
--- a/t/05-memoize.t
+++ b/t/05-memoize.t
@@ -8,12 +8,7 @@ use Test::Requires 'Test::Output';
use Eval::Closure;
{
- my $source = <<'SOURCE';
- sub {
- $foo * 2;
- };
- BEGIN { warn "foo\n" }
-SOURCE
+ my $source = 'BEGIN { warn "foo\n" } sub { $foo * 2 }';
my $code;
my $bar = 15;
@@ -43,12 +38,7 @@ SOURCE
}
{
- my $source = <<'SOURCE';
- sub {
- $bar * 2;
- };
- BEGIN { warn "bar\n" }
-SOURCE
+ my $source = 'BEGIN { warn "bar\n" } sub { $bar * 2 }';
my $code;
my $foo = 60;
@@ -66,8 +56,7 @@ SOURCE
my $code2;
my $baz = 23;
- { local $TODO = $] < 5.010 ? "description breaks memoization on 5.8"
- : undef;
+ { local $TODO = "description breaks memoization";
stderr_is {
$code2 = eval_closure(
source => $source,
@@ -83,12 +72,7 @@ SOURCE
}
{
- my $source = <<'SOURCE';
- sub {
- Carp::confess "baz";
- };
- BEGIN { warn "baz\n" }
-SOURCE
+ my $source = 'BEGIN { warn "baz\n" } sub { Carp::confess "baz" }';
my $code;
stderr_is {
@@ -102,8 +86,7 @@ SOURCE
"got the right description");
my $code2;
- { local $TODO = $] < 5.010 ? "description breaks memoization on 5.8"
- : undef;
+ { local $TODO = "description breaks memoization";
stderr_is {
$code2 = eval_closure(
source => $source,
diff --git a/t/11-line-differences.t b/t/11-line-differences.t
deleted file mode 100644
index 4dd3625..0000000
--- a/t/11-line-differences.t
+++ /dev/null
@@ -1,61 +0,0 @@
-#!/usr/bin/env perl
-use strict;
-use warnings;
-use Test::More;
-use Test::Requires 'Test::Output';
-
-use Eval::Closure;
-
-{
- my $code = eval_closure(
- source => 'sub { warn "foo" }',
- description => 'bar',
- );
- { local $TODO = $] < 5.010 ? "line numbers from #line are slightly different" : undef;
- stderr_is { $code->() } "foo at bar line 1.\n", "got the right line";
- }
-}
-
-{
- my $code = eval_closure(
- source => <<'SOURCE',
- sub {
-
- warn "foo";
-
- }
-SOURCE
- description => 'bar',
- );
- { local $TODO = $] < 5.010 ? "line numbers from #line are slightly different" : undef;
- stderr_is { $code->() } "foo at bar line 1.\n", "got the right line";
- }
-}
-
-{
- my $code = eval_closure(
- source => <<'SOURCE',
-
- sub {
- warn "foo";
- }
-SOURCE
- description => 'bar',
- );
- { local $TODO = $] < 5.010 ? "line numbers from #line are slightly different" : undef;
- stderr_is { $code->() } "foo at bar line 1.\n", "got the right line";
- }
-}
-
-{
- my $code = eval_closure(
- source => '$sub',
- environment => { '$sub' => \sub { warn "foo" } },
- description => 'bar',
- );
- { local $TODO = $] < 5.010 ? "#line can't adjust line numbers inside non-evaled subs" : undef;
- stderr_is { $code->() } "foo at bar line 1.\n", "got the right line";
- }
-}
-
-done_testing;