fix: do not edit memcpy

Author: jharlow-intel

dpctl/_sycl_queue.pyx

@@ -592,7 +592,9 @@ cdef DPCTLSyclEventRef _copy_impl(
         )
 
     if dep_events_count == 0 or dep_events is NULL:
-        ERef = DPCTLQueue_CopyData(q._queue_ref, c_dst_ptr, c_src_ptr, byte_count)
+        ERef = DPCTLQueue_CopyData(
+            q._queue_ref, c_dst_ptr, c_src_ptr, byte_count
+        )
     else:
         ERef = DPCTLQueue_CopyDataWithEvents(
             q._queue_ref,

dpctl/tests/test_sycl_queue_memcpy.py

@@ -155,40 +155,6 @@ def test_memcpy_async():
     assert dst_buf2 == src_buf
 
 
-def test_copy_copy_host_to_host():
-    try:
-        q = dpctl.SyclQueue()
-    except dpctl.SyclQueueCreationError:
-        pytest.skip("Default constructor for SyclQueue failed")
-
-    src_buf = b"abcdefghijklmnopqrstuvwxyz"
-    dst_buf = bytearray(len(src_buf))
-
-    q.copy(dst_buf, src_buf, len(src_buf))
-
-    assert dst_buf == src_buf
-
-
-def test_copy_async():
-    try:
-        q = dpctl.SyclQueue()
-    except dpctl.SyclQueueCreationError:
-        pytest.skip("Default constructor for SyclQueue failed")
-
-    src_buf = b"abcdefghijklmnopqrstuvwxyz"
-    n = len(src_buf)
-    dst_buf = bytearray(n)
-    dst_buf2 = bytearray(n)
-
-    e = q.copy_async(dst_buf, src_buf, n)
-    e2 = q.copy_async(dst_buf2, src_buf, n, [e])
-
-    e.wait()
-    e2.wait()
-    assert dst_buf == src_buf
-    assert dst_buf2 == src_buf
-
-
 def test_memcpy_type_error():
     try:
         q = dpctl.SyclQueue()

libsyclinterface/source/dpctl_sycl_queue_interface.cpp

@@ -742,9 +742,9 @@ DPCTLQueue_CopyDataWithEvents(__dpctl_keep const DPCTLSyclQueueRef QRef,
             }
 
             // Bind queue::copy with uint8_t so Count is interpreted as bytes.
-            auto ev = Q->copy(static_cast<const std::uint8_t *>(Src),
-                              static_cast<std::uint8_t *>(Dest), Count,
-                              dep_events);
+            auto ev =
+                Q->copy(static_cast<const std::uint8_t *>(Src),
+                        static_cast<std::uint8_t *>(Dest), Count, dep_events);
             return wrap<event>(new event(std::move(ev)));
         } catch (const std::exception &ex) {
             error_handler(ex, __FILE__, __func__, __LINE__);

libsyclinterface/tests/test_sycl_queue_interface.cpp

@@ -374,8 +374,8 @@ TEST(TestDPCTLSyclQueueInterface, CheckMemOpsZeroQRef)
     ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_CopyData(QRef, p1, p2, n_bytes));
     ASSERT_FALSE(bool(ERef));
 
-    ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_CopyDataWithEvents(
-                                QRef, p1, p2, n_bytes, NULL, 0));
+    ASSERT_NO_FATAL_FAILURE(
+        ERef = DPCTLQueue_CopyDataWithEvents(QRef, p1, p2, n_bytes, NULL, 0));
     ASSERT_FALSE(bool(ERef));
 
     ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_Prefetch(QRef, p1, n_bytes));
@@ -439,8 +439,8 @@ TEST_P(TestDPCTLQueueMemberFunctions, CheckMemOpsNullPtr)
     ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_CopyData(QRef, p1, p2, n_bytes));
     ASSERT_FALSE(bool(ERef));
 
-    ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_CopyDataWithEvents(
-                                QRef, p1, p2, n_bytes, NULL, 0));
+    ASSERT_NO_FATAL_FAILURE(
+        ERef = DPCTLQueue_CopyDataWithEvents(QRef, p1, p2, n_bytes, NULL, 0));
     ASSERT_FALSE(bool(ERef));
 
     ASSERT_NO_FATAL_FAILURE(ERef = DPCTLQueue_Prefetch(QRef, p1, n_bytes));