summaryrefslogtreecommitdiff
path: root/buildstream/element.py
diff options
context:
space:
mode:
authorTom Pollard <tom.pollard@codethink.co.uk>2019-04-12 10:27:25 +0100
committerTom Pollard <tom.pollard@codethink.co.uk>2019-04-17 16:47:27 +0100
commit68f5d92fb1f5a37800c11aa74ebd56f7f2a43ca5 (patch)
tree2e91dad8c7d03d35764e8c06dd8cc7ff2d0a45cc /buildstream/element.py
parentf3b7c2218989a93481986a2d8b8b7ba35b45bdbb (diff)
downloadbuildstream-68f5d92fb1f5a37800c11aa74ebd56f7f2a43ca5.tar.gz
element.py: Remove redundant fully private __get_build_result()
This was being accessed via a local private method that wasn't augmenting the output as such it can be moved.
Diffstat (limited to 'buildstream/element.py')
-rw-r--r--buildstream/element.py14
1 files changed, 5 insertions, 9 deletions
diff --git a/buildstream/element.py b/buildstream/element.py
index d9f32bfc8..17cc9548c 100644
--- a/buildstream/element.py
+++ b/buildstream/element.py
@@ -1056,7 +1056,10 @@ class Element(Plugin):
# (str): Detailed description of the result
#
def _get_build_result(self):
- return self.__get_build_result()
+ if self.__build_result is None:
+ self.__load_build_result()
+
+ return self.__build_result
# __set_build_result():
#
@@ -2893,18 +2896,11 @@ class Element(Plugin):
self.__build_result = self.__artifact.load_build_result()
- def __get_build_result(self):
-
- if self.__build_result is None:
- self.__load_build_result()
-
- return self.__build_result
-
def __cached_success(self):
if not self._cached():
return False
- success, _, _ = self.__get_build_result()
+ success, _, _ = self._get_build_result()
return success
def __get_cache_keys_for_commit(self):