Beefy Boxes and Bandwidth Generously Provided by pair Networks
Clear questions and runnable code
get the best and fastest answer

Re^2: Data::Diver polluting $@ (local)

by tye (Sage)
on Nov 01, 2012 at 13:35 UTC ( #1001827=note: print w/replies, xml ) Need Help??

in reply to Re: Data::Diver polluting $@
in thread Data::Diver polluting $!

Although I mostly agree, I'll try to upload a new release of Data::Diver in short order that adds local($@) any places that it uses eval.

And thanks for mentioning $@ as it saved me from figuring out what was really going on, because I was immediately pretty sure that the problem wasn't with $!, but I didn't immediately realize what the real problem was.

- tye        

Replies are listed 'Best First'.
Re^3: Data::Diver polluting $@ (truth)
by Anonymous Monk on Sep 26, 2015 at 00:04 UTC

    Here is another, docs lead me to believe these should all work but Dive doesn't

    #!/usr/bin/perl -- use strict; use warnings; use Data::Diver qw/ Dive DiveRef DiveDie /; sub say { print @_, "\n"; } say DiveDie( DiveRef( { qw/ key val / }, \'key' ) ); say DiveDie( Dive( { qw/ key val / }, \'key' ) ); __END__ SCALAR(0x3f90cc) Key not present in hash using SCALAR(0xabe37c) on HASH(0x3f90dc) (from + Data::Diver).

    Problem is in sub Dive, when you're testing if $ref is a hash with eval, you forgot the 1, this line
    -        } elsif(  eval { exists $ref->{$key} }  ) {
    +        } elsif(  eval { exists $ref->{$key}; 1 }  ) {

    I've tested it it works , your test suite doesn't test for this so maybe add to Data-Diver-1.0101/t/base.t the following

    ok( 'val', Dive( { qw/ key val / }, \'key' ) ); ok( 'val', Dive( { qw/ key val / }, 'key' ) ); ok( 'val', DiveVal( { qw/ key val / }, \'key' ) ); ok( 'val', DiveVal( { qw/ key val / }, 'key' ) );

    Also remember the Replacment for Data::Diver due missing license

    For the honor

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1001827]
[shmem]: same with undef as last element
[msh210]: oh, good point. It's the best kind of correct
[shmem]: msh210: but I'd not call it a lie outright. That's when you do know the truth.
[msh210]: I don't have sendmail (I'm on MS Windows), so does one of you think you can report the bug?
[shmem]: msh210: 'tis about time to tell the sad truth to perl5porters - via perlbug
[msh210]: probably whoever wrote that actually knows the truth
[shmem]: msh210: you can save the perlbug text in a file and send that via your mail agent of choice
[msh210]: hm, okay, I'll try, thanks
[shmem]: ...not as an attachment, it's probably better to c&p that into the mail body

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (12)
As of 2017-04-24 12:49 GMT
Find Nodes?
    Voting Booth?
    I'm a fool:

    Results (439 votes). Check out past polls.