From 5e6ea7a66704b78180520cb8f1fccaed5d9ea7b1 Mon Sep 17 00:00:00 2001 From: Philippe Proulx Date: Wed, 8 Apr 2020 13:05:55 -0400 Subject: [PATCH] Revert "bt2: _EventConst.__getitem__(): use a single temporary variable" I did not want to merge this and pressed the wrong button on Gerrit. Sorry. Change-Id: I11cc739c126131560ba31d2a1f3f01b7e961a837 Reviewed-on: https://review.lttng.org/c/babeltrace/+/3354 Reviewed-by: Philippe Proulx CI-Build: Philippe Proulx --- src/bindings/python/bt2/bt2/event.py | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/bindings/python/bt2/bt2/event.py b/src/bindings/python/bt2/bt2/event.py index e18f12cf..6a6c1185 100644 --- a/src/bindings/python/bt2/bt2/event.py +++ b/src/bindings/python/bt2/bt2/event.py @@ -108,26 +108,26 @@ class _EventConst(object._UniqueObject): def __getitem__(self, key): utils._check_str(key) - root_field = self.payload_field + payload_field = self.payload_field - if root_field is not None and key in root_field: - return root_field[key] + if payload_field is not None and key in payload_field: + return payload_field[key] - root_field = self.specific_context_field + specific_context_field = self.specific_context_field - if root_field is not None and key in root_field: - return root_field[key] + if specific_context_field is not None and key in specific_context_field: + return specific_context_field[key] - root_field = self.common_context_field + common_context_field = self.common_context_field - if root_field is not None and key in root_field: - return root_field[key] + if common_context_field is not None and key in common_context_field: + return common_context_field[key] if self.packet: - root_field = self.packet.context_field + packet_context_field = self.packet.context_field - if root_field is not None and key in root_field: - return root_field[key] + if packet_context_field is not None and key in packet_context_field: + return packet_context_field[key] raise KeyError(key) -- 2.34.1