refactor option module loads for JSON subclass
Graham Knop [Sat, 4 Feb 2017 06:33:55 +0000 (01:33 -0500)]
lib/Config/Any/JSON.pm

index 5a432d7..950424f 100644 (file)
@@ -49,34 +49,30 @@ sub load {
     my $content = do { local $/; <$fh> };
     close $fh;
 
-    eval { require JSON::DWIW; };
-    unless( $@ ) {
+    if ( eval { require JSON::DWIW } ) {
         my $decoder = JSON::DWIW->new;
         my ( $data, $error ) = $decoder->from_json( $content );
         die $error if $error;
         return $data;
     }
-
-    eval { require JSON::XS; };
-    unless( $@ ) {
+    elsif ( eval { require JSON::XS } ) {
         my $decoder = JSON::XS->new->relaxed;
         return $decoder->decode( $content );
     }
-
-    eval { require JSON::Syck; };
-    unless( $@ ) {
+    elsif ( eval { require JSON::Syck } ) {
         return JSON::Syck::Load( $content );
     }
-
-    eval { require JSON::PP; JSON::PP->VERSION( 2 ); };
-    unless( $@ ) {
+    elsif ( eval { require JSON::PP; JSON::PP->VERSION( 2 ); } ) {
         my $decoder = JSON::PP->new->relaxed;
         return $decoder->decode( $content );
     }
-
     require JSON;
-    eval { JSON->VERSION( 2 ); };
-    return $@ ? JSON::jsonToObj( $content ) : JSON::from_json( $content );
+    if ( eval { JSON->VERSION( 2 ) } ) {
+        return JSON::from_json( $content );
+    }
+    else {
+        return JSON::jsonToObj( $content );
+    }
 }
 
 =head2 requires_any_of( )