summaryrefslogtreecommitdiff
path: root/buildstream/_frontend
diff options
context:
space:
mode:
authorBenjamin Schubert <ben.c.schubert@gmail.com>2019-03-20 17:46:01 +0000
committerTristan Van Berkom <tristan.vanberkom@codethink.co.uk>2019-03-24 21:24:24 +0900
commite0ca1ce9980e2b779f3852add4a241657689f738 (patch)
tree6595b4d63ee36d4b9b85652c9f8e07ca04ce6f41 /buildstream/_frontend
parent9e85e36e4be36617a9e4e8298c7a6dff1766b3f0 (diff)
downloadbuildstream-e0ca1ce9980e2b779f3852add4a241657689f738.tar.gz
plugin.py: make _unique_id accessible to the core
_unique_id is set at instantiation and never modified afterwards. Since the guildelines for the project is to never modify directly an object's state, accessing _unique_id as an attribute is safe. Moreover this will save us some cost of calling functions.
Diffstat (limited to 'buildstream/_frontend')
-rw-r--r--buildstream/_frontend/app.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/buildstream/_frontend/app.py b/buildstream/_frontend/app.py
index f89202154..53a342899 100644
--- a/buildstream/_frontend/app.py
+++ b/buildstream/_frontend/app.py
@@ -531,7 +531,7 @@ class App():
queue = job.queue
# Get the last failure message for additional context
- failure = self._fail_messages.get(element._get_unique_id())
+ failure = self._fail_messages.get(element._unique_id)
# XXX This is dangerous, sometimes we get the job completed *before*
# the failure message reaches us ??