boolean logic cleanup in service.Swift[Copy|Post]Object

Change-Id: I07e74536502ec2479b22a825f684703d65924563

1 files changed, 4 insertions, 5 deletions

diff --git a/swiftclient/service.py b/swiftclient/service.py
index 0c16262..f204895 100644
--- a/swiftclient/service.py
+++ b/swiftclient/service.py
@@ -323,13 +323,12 @@ class SwiftPostObject(object):
     specified separately for each individual object.
     """
     def __init__(self, object_name, options=None):
-        if not isinstance(object_name, string_types) or not object_name:
+        if not (isinstance(object_name, string_types) and object_name):
             raise SwiftError(
                 "Object names must be specified as non-empty strings"
             )
-        else:
-            self.object_name = object_name
-            self.options = options
+        self.object_name = object_name
+        self.options = options
 
 
 class SwiftCopyObject(object):
@@ -340,7 +339,7 @@ class SwiftCopyObject(object):
     destination and fresh_metadata should be set in options
     """
     def __init__(self, object_name, options=None):
-        if not isinstance(object_name, string_types) or not object_name:
+        if not (isinstance(object_name, string_types) and object_name):
             raise SwiftError(
                 "Object names must be specified as non-empty strings"
             )