X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FException.pm;h=6c8d0e9a69e599a01a9664c11a0fc593aa7c010b;hb=3b80fa31b60050d4c8df91457ba6fd51b579a7a6;hp=34d709e6801bc3cc4a4fa5d40c8cba0bf1349612;hpb=02ddab6d63ad4d8e52dc818d00f855de19478488;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Exception.pm b/lib/DBIx/Class/Exception.pm index 34d709e..6c8d0e9 100644 --- a/lib/DBIx/Class/Exception.pm +++ b/lib/DBIx/Class/Exception.pm @@ -3,8 +3,9 @@ package DBIx::Class::Exception; use strict; use warnings; -use Carp::Clan qw/^DBIx::Class/; -use Scalar::Util qw/blessed/; +use Carp::Clan qw/^DBIx::Class|^Try::Tiny/; +use Try::Tiny; +use namespace::clean; use overload '""' => sub { shift->{msg} }, @@ -16,8 +17,8 @@ DBIx::Class::Exception - Exception objects for DBIx::Class =head1 DESCRIPTION -Exception objects of this class are used in internally by -he default error handling of L +Exception objects of this class are used internally by +the default error handling of L to prevent confusing and/or redundant re-application of L's stack trace information. @@ -42,7 +43,7 @@ C<$stacktrace> tells it to use L instead of L. DBIx::Class::Exception->throw('Foo'); - eval { ... }; DBIx::Class::Exception->throw($@) if $@; + try { ... } catch { DBIx::Class::Exception->throw(shift) } =cut @@ -50,18 +51,16 @@ sub throw { my ($class, $msg, $stacktrace) = @_; # Don't re-encapsulate exception objects of any kind - die $msg if blessed($msg); + die $msg if ref($msg); # use Carp::Clan's croak if we're not stack tracing if(!$stacktrace) { - local $@; - eval { croak $msg }; - $msg = $@ + try { croak $msg } catch { $msg = shift }; } else { $msg = Carp::longmess($msg); } - + my $self = { msg => $msg }; bless $self => $class;