diff options
author | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-03-20 17:46:01 +0000 |
---|---|---|
committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2019-03-24 21:24:24 +0900 |
commit | e0ca1ce9980e2b779f3852add4a241657689f738 (patch) | |
tree | 6595b4d63ee36d4b9b85652c9f8e07ca04ce6f41 /buildstream/_frontend | |
parent | 9e85e36e4be36617a9e4e8298c7a6dff1766b3f0 (diff) | |
download | buildstream-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.py | 2 |
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 ?? |