From fc4353fb3a4bb64050365df02ddf03ff7f0a684d Mon Sep 17 00:00:00 2001 From: Tres Seaver Date: Thu, 16 Oct 2014 16:56:01 -0400 Subject: [PATCH] Kowtow to pylint's over-fussy 'too-many-return-statements'. Incorporates feedback from @dhermes. --- gcloud/datastore/_helpers.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/gcloud/datastore/_helpers.py b/gcloud/datastore/_helpers.py index 706d24c3758c..2b23d62484c3 100644 --- a/gcloud/datastore/_helpers.py +++ b/gcloud/datastore/_helpers.py @@ -88,35 +88,35 @@ def _get_value_from_value_pb(value_pb): :returns: The value provided by the Protobuf. """ + result = None if value_pb.HasField('timestamp_microseconds_value'): microseconds = value_pb.timestamp_microseconds_value naive = (datetime.utcfromtimestamp(0) + timedelta(microseconds=microseconds)) - return naive.replace(tzinfo=pytz.utc) + result = naive.replace(tzinfo=pytz.utc) elif value_pb.HasField('key_value'): - return Key.from_protobuf(value_pb.key_value) + result = Key.from_protobuf(value_pb.key_value) elif value_pb.HasField('boolean_value'): - return value_pb.boolean_value + result = value_pb.boolean_value elif value_pb.HasField('double_value'): - return value_pb.double_value + result = value_pb.double_value elif value_pb.HasField('integer_value'): - return value_pb.integer_value + result = value_pb.integer_value elif value_pb.HasField('string_value'): - return value_pb.string_value + result = value_pb.string_value elif value_pb.HasField('entity_value'): - return Entity.from_protobuf(value_pb.entity_value) + result = Entity.from_protobuf(value_pb.entity_value) elif value_pb.list_value: - return [_get_value_from_value_pb(x) for x in value_pb.list_value] + result = [_get_value_from_value_pb(x) for x in value_pb.list_value] - else: - return None + return result def _get_value_from_property_pb(property_pb):