diff options
| author |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-10-16 22:35:36 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-10-17 23:19:38 +0100 |
| commit | 6d19eb3226b59922c0f888e33b28443635151501 (patch) | |
| tree | b4c26034bb01fb7bfb7e917be0d811be992db1e2 | |
| parent | a68de8ace62eaba23856bfb301efbbe1824322aa (diff) | |
| download | bitbake-6d19eb3226b59922c0f888e33b28443635151501.tar.gz | |
data_smart: Micro optimise _remove handling
This brings _remove handling into line with _append/_prepend with regard
to the parsing flag to getVarFlag.
This is an internal flag and the only times this is used is through getVar
during renameVar operations and when processing ?= operations to see if
a variable is set. In either case we don't need to process remove operations.
Therefore take the minor speedup and skip processing for parsing=True.
[YOCTO #10945]
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | lib/bb/data_smart.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/bb/data_smart.py b/lib/bb/data_smart.py index 0a8488ca1..7b2c0a894 100644 --- a/lib/bb/data_smart.py +++ b/lib/bb/data_smart.py @@ -805,7 +805,7 @@ class DataSmart(MutableMapping): cachename = var + "[" + flag + "]" value = self.expand(value, cachename) - if value and flag == "_content" and local_var is not None and "_remove" in local_var: + if value and flag == "_content" and local_var is not None and "_remove" in local_var and not parsing: removes = [] self.need_overrides() for (r, o) in local_var["_remove"]: |