From b00b37fb0c70a3761ac39c3459876c1d37ed6569 Mon Sep 17 00:00:00 2001 From: Patrick Peglar Date: Wed, 15 Nov 2023 13:50:33 +0000 Subject: [PATCH] Clarify result testing in metadata operations decorator. --- lib/iris/common/_split_attribute_dicts.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/iris/common/_split_attribute_dicts.py b/lib/iris/common/_split_attribute_dicts.py index 875ee3102f..41ce321a8f 100644 --- a/lib/iris/common/_split_attribute_dicts.py +++ b/lib/iris/common/_split_attribute_dicts.py @@ -105,7 +105,8 @@ def _inner_function(*args, **kwargs): result = operation(*args, **kwargs) - # Convert 'pairedkeys' dicts in the result back to split-attributes form. + # Convert known specific cases of 'pairedkeys' dicts in the result, and convert + # those back into split-attribute dictionaries. if isinstance(result, Mapping): # Fix a result which is a single dictionary -- for "combine" result = _convert_pairedkeys_dict_to_splitattrs(result) @@ -117,6 +118,7 @@ def _inner_function(*args, **kwargs): _convert_pairedkeys_dict_to_splitattrs(right), ) result = result.__class__([left, right]) + # ELSE: leave other types of result unchanged. E.G. None, bool return result