Work around weird bug in Python 3.7 and 3.8 ChainMap

dargueta · Feb 14, 2023 · 55baf2d · 55baf2d
1 parent b4d17d8
commit 55baf2d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/binobj/structures.py b/binobj/structures.py
@@ -349,15 +349,17 @@ def _to_dict_whatever_possible(self) -> MutableMapping[str, Any]:
         have no dependencies and can serialize themselves (e.g. Bytes and sized int
         fields). This allows us to resolve forward references.
         """
-        dct = collections.ChainMap({}, self)
+        dct = {}
 
         for field in self.__binobj_struct__.components.values():
             try:
-                dct[field.name] = field.compute_value_for_dump(dct)
+                dct[field.name] = field.compute_value_for_dump(
+                    typing.cast(StrDict, self)
+                )
             except errors.Error:
                 continue
 
-        return dct
+        return collections.ChainMap(dct, self)
 
     @classmethod
     def from_stream(

diff --git a/tox.ini b/tox.ini
@@ -60,6 +60,7 @@ per_file_ignores =
     tests/*:D
 unused_arguments_ignore_abstract_functions = true
 unused_arguments_ignore_overload_functions = true
+unused_arguments_ignore_override_functions = true
 unused_arguments_ignore_stub_functions = true
 unused_arguments_ignore_dunder = true