use Devel::Hints where possible

this will avoid breaking memoization when generating coderefs with
descriptions, and should be more robust and useful

3 files changed, 84 insertions, 6 deletions

diff --git a/t/03-description.t b/t/03-description.t
index 97f8372..c9e8b21 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 2\n/,
+        qr/^foo at accessor foo \(defined at Class\.pm line 282\) line 1\n/,
         "description is set"
     );
 }
diff --git a/t/05-memoize.t b/t/05-memoize.t
index 02fd11f..e4b582b 100644
--- a/t/05-memoize.t
+++ b/t/05-memoize.t
@@ -8,7 +8,12 @@ use Test::Requires 'Test::Output';
 use Eval::Closure;
 
 {
-    my $source = 'BEGIN { warn "foo\n" } sub { $foo * 2 }';
+    my $source = <<'SOURCE';
+    sub {
+        $foo * 2;
+    };
+    BEGIN { warn "foo\n" }
+SOURCE
 
     my $code;
     my $bar = 15;
@@ -38,7 +43,12 @@ use Eval::Closure;
 }
 
 {
-    my $source = 'BEGIN { warn "bar\n" } sub { $bar * 2 }';
+    my $source = <<'SOURCE';
+    sub {
+        $bar * 2;
+    };
+    BEGIN { warn "bar\n" }
+SOURCE
 
     my $code;
     my $foo = 60;
@@ -56,7 +66,8 @@ use Eval::Closure;
 
     my $code2;
     my $baz = 23;
-    { local $TODO = "description breaks memoization";
+    { local $TODO = $] < 5.010 ? "description breaks memoization on 5.8"
+                               : undef;
     stderr_is {
         $code2 = eval_closure(
             source      => $source,
@@ -72,7 +83,12 @@ use Eval::Closure;
 }
 
 {
-    my $source = 'BEGIN { warn "baz\n" } sub { Carp::confess "baz" }';
+    my $source = <<'SOURCE';
+    sub {
+        Carp::confess "baz";
+    };
+    BEGIN { warn "baz\n" }
+SOURCE
 
     my $code;
     stderr_is {
@@ -86,7 +102,8 @@ use Eval::Closure;
          "got the right description");
 
     my $code2;
-    { local $TODO = "description breaks memoization";
+    { local $TODO = $] < 5.010 ? "description breaks memoization on 5.8"
+                               : undef;
     stderr_is {
         $code2 = eval_closure(
             source      => $source,
diff --git a/t/11-line-differences.t b/t/11-line-differences.t
new file mode 100644
index 0000000..4dd3625
--- /dev/null
+++ b/t/11-line-differences.t
@@ -0,0 +1,61 @@
+#!/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;