Merge pull request #2289 from IntelPython/use-simplified-python-constructors

Update empty Python data structure constructors

Author: ndgrigorian

docs/doc_sources/conf.py.in

@@ -134,14 +134,14 @@ if generate_multiversion == "ON":
     try:
         html_context
     except NameError:
-        html_context = dict()
+        html_context = {}
     html_context["display_lower_left"] = True
     templates_path = ["_templates"]
     html_context["current_version"] = release
     html_context["version"] = version
 
     # POPULATE LINKS TO OTHER VERSIONS
-    html_context["versions"] = list()
+    html_context["versions"] = []
 
     # Populate the list of documented versions from the doc_versions.txt
     versions = []

dpctl/_sycl_device.pyx

@@ -1210,7 +1210,7 @@ cdef class SyclDevice(_SyclDevice):
         sg_sizes = DPCTLDevice_GetSubGroupSizes(
             self._device_ref, &sg_sizes_len)
         if (sg_sizes is not NULL and sg_sizes_len > 0):
-            res = list()
+            res = []
             for i in range(sg_sizes_len):
                 res.append(sg_sizes[i])
             DPCTLSize_t_Array_Delete(sg_sizes)

dpctl/_sycl_device_factory.pyx

@@ -416,7 +416,7 @@ cdef class _DefaultDeviceCache:
     cdef dict __device_map__
 
     def __cinit__(self):
-        self.__device_map__ = dict()
+        self.__device_map__ = {}
 
     cdef get_or_create(self):
         """Return instance of SyclDevice and indicator if cache

dpctl/_sycl_queue_manager.pyx

@@ -35,7 +35,7 @@ cdef class _DeviceDefaultQueueCache:
     cdef dict __device_queue_map__
 
     def __cinit__(self):
-        self.__device_queue_map__ = dict()
+        self.__device_queue_map__ = {}
 
     def get_or_create(self, key):
         """Return instance of SyclQueue and indicator if cache

dpctl/_sycl_timer.py

@@ -190,7 +190,7 @@ def __init__(
         self.queue = None
         self.host_times = []
         self.bracketing_events = []
-        self._context_data = list()
+        self._context_data = []
         if device_timer is None:
             device_timer = "queue_barrier"
         if device_timer == "queue_barrier":

dpctl/tests/helper/_helper.py

@@ -43,7 +43,7 @@ def create_invalid_capsule():
     return ctor(id(ctor), b"invalid", 0)
 
 
-def get_queue_or_skip(args=tuple()):
+def get_queue_or_skip(args=()):
     try:
         q = dpctl.SyclQueue(*args)
     except dpctl.SyclQueueCreationError:

dpctl/tests/test_sycl_device.py

@@ -405,7 +405,7 @@ def test_peer_device_arg_validation(method):
         dev = dpctl.SyclDevice()
     except dpctl.SyclDeviceCreationError:
         pytest.skip("No default device available")
-    bad_dev = dict()
+    bad_dev = {}
     callable = getattr(dev, method)
     with pytest.raises(TypeError):
         callable(bad_dev)

dpctl/tests/test_sycl_event.py

@@ -70,7 +70,7 @@ def test_create_event_from_capsule():
 
 def test_invalid_constructor_arg():
     with pytest.raises(TypeError):
-        dpctl.SyclEvent(list())
+        dpctl.SyclEvent([])
 
 
 def test_wait_with_event():

dpctl/tests/test_sycl_queue.py

@@ -209,7 +209,7 @@ def test_queue_invalid_property():
     with pytest.raises(ValueError):
         dpctl.SyclQueue(property=4.5)
     with pytest.raises(ValueError):
-        dpctl.SyclQueue(property=["abc", tuple()])
+        dpctl.SyclQueue(property=["abc", ()])
 
 
 def test_queue_capsule():
@@ -345,13 +345,13 @@ def test_queue_memops():
     q.prefetch(m1, 512)
     q.mem_advise(m1, 512, 0)
     with pytest.raises(TypeError):
-        q.memcpy(m1, list(), 512)
+        q.memcpy(m1, [], 512)
     with pytest.raises(TypeError):
-        q.memcpy(list(), m2, 512)
+        q.memcpy([], m2, 512)
     with pytest.raises(TypeError):
-        q.prefetch(list(), 512)
+        q.prefetch([], 512)
     with pytest.raises(TypeError):
-        q.mem_advise(list(), 512, 0)
+        q.mem_advise([], 512, 0)
 
 
 @pytest.fixture(scope="session")

dpctl/tests/test_sycl_usm.py

@@ -265,7 +265,7 @@ def test_usm_type_exceptions():
     with pytest.raises(TypeError):
         m.get_usm_type(syclobj=Ellipsis)
     with pytest.raises(TypeError):
-        m.get_usm_type_enum(syclobj=list())
+        m.get_usm_type_enum(syclobj=[])
 
 
 def test_sycl_usm_array_interface(memory_ctor):
@@ -419,7 +419,7 @@ def invalid_version(suai_iface):
     # data validation
     def invalid_data(suai_iface):
         "Set data to invalid"
-        suai_iface["data"] = tuple()
+        suai_iface["data"] = ()
         return suai_iface
 
     v = View(