parser / properties / avoid call to isinstance for perf imprv

Author: p1-ra

openapi_python_client/parser/properties/__init__.py

@@ -9,6 +9,7 @@
     "property_from_data",
 ]
 
+from itertools import chain
 from typing import Any, ClassVar, Dict, Generic, Iterable, Iterator, List, Optional, Set, Tuple, TypeVar, Union
 
 import attr
@@ -352,12 +353,7 @@ def build_union_property(
 ) -> Tuple[Union[UnionProperty, PropertyError], Schemas]:
     sub_properties: List[Property] = []
 
-    sub_data: List[Union[oai.Schema, oai.Reference]] = []
-    for _data in [data.anyOf, data.oneOf]:
-        if isinstance(_data, Iterable):
-            sub_data.extend(_data)
-
-    for i, sub_prop_data in enumerate(sub_data):
+    for i, sub_prop_data in enumerate(chain(data.anyOf or [], data.oneOf or [])):
         sub_prop, schemas = property_from_data(
             name=f"{name}_type{i}",
             required=required,
@@ -446,7 +442,7 @@ def _property_from_data(
 
     sub_data: List[Union[oai.Schema, oai.Reference]] = []
     for _data in [data.allOf, data.anyOf, data.oneOf]:
-        if isinstance(_data, Iterable):
+        if _data:
             sub_data.extend(_data)
 
     # A union of a single reference should just be passed through to that reference (don't create copy class)