Changeset 13256

Timestamp:

May 4, 2007, 2:14:17 PM (19 years ago)

Author:

jhoblitt

Message:

change all Nebulous::Client public methods to behave in list context

Location:

trunk

Files:

• Nebulous-Server/Changes (modified) (1 diff)
• Nebulous/Changes (modified) (1 diff)
• Nebulous/lib/Nebulous/Client.pm (modified) (18 diffs)

trunk/Nebulous-Server/Changes

@@ -2 +2 @@
 
 0.05
+    - change all Nebulous::Client public methods to behave in list context
     - rework the getmountedvol() stored procedure to attempt to work around
       what appears to be some nasty transacational isolation leak throught that

trunk/Nebulous/Changes

@@ -2 +2 @@
 
 0.05
+    - change all Nebulous::Client public methods to behave in list context
     - rework the getmountedvol() stored procedure to attempt to work around
       what appears to be some nasty transacational isolation leak throught that

trunk/Nebulous/lib/Nebulous/Client.pm

@@ -1 +1 @@
 # Copyright (c) 2004  Joshua Hoblitt
 #
-# $Id: Client.pm,v 1.33 2007-05-05 00:02:53 jhoblitt Exp $
+# $Id: Client.pm,v 1.34 2007-05-05 00:14:17 jhoblitt Exp $
 
 package Nebulous::Client;
@@ -109 +109 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -145 +145 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -189 +189 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -198 +198 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -326 +326 @@
             # don't bother to spin lock if the object doesn't exist
             if ( $response->faultstring =~ /storage object does not exist/ ) {
-                return undef;
+                return;
             }
 
@@ -346 +346 @@
     $log->debug( "leaving" );
     
-    return $locked ? 1 : undef;
+    $locked ? return 1 : return;
 }
 
@@ -375 +375 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -387 +387 @@
     $log->debug( "leaving" );
     
-    return $unlocked ? 1 : undef;
+    $unlocked ? return 1 : return;
 }
 
@@ -407 +407 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -447 +447 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -528 +528 @@
         } else {
             $log->debug( "leaving" );
-            return undef;
+            return;
         }
     }
@@ -538 +538 @@
             $log->warn( "write not allowed with multiple instances" );
             $log->debug( "leaving" );
-            return undef;
+            return;
         }
 
@@ -606 +606 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -619 +619 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -651 +651 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -684 +684 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }
 
@@ -716 +716 @@
         $log->debug( "leaving" );
 
-        return undef;
+        return;
     }