[common] Fix some compilation warnings

Remove excessive semicolons.
Remove unused variables.
Use python3.6 friendly alias instead of text param.

Author: lslusarczyk

source/adapters/level_zero/v2/command_list_cache.cpp

@@ -142,7 +142,7 @@ void command_list_cache_t::addCommandList(
     raii::ze_command_list_handle_t cmdList) {
   // TODO: add a limit?
   std::unique_lock<ur_mutex> Lock(ZeCommandListCacheMutex);
-  auto [it, _] = ZeCommandListCache.try_emplace(desc);
+  auto it = ZeCommandListCache.try_emplace(desc).first;
   it->second.emplace(std::move(cmdList));
 }
 

source/common/umf_helpers.hpp

@@ -56,11 +56,11 @@ using provider_unique_handle_t =
         return UMF_RESULT_ERROR_NOT_SUPPORTED;                                 \
     }
 
-DEFINE_CHECK_OP(get_ipc_handle_size);
-DEFINE_CHECK_OP(get_ipc_handle);
-DEFINE_CHECK_OP(put_ipc_handle);
-DEFINE_CHECK_OP(open_ipc_handle);
-DEFINE_CHECK_OP(close_ipc_handle);
+DEFINE_CHECK_OP(get_ipc_handle_size)
+DEFINE_CHECK_OP(get_ipc_handle)
+DEFINE_CHECK_OP(put_ipc_handle)
+DEFINE_CHECK_OP(open_ipc_handle)
+DEFINE_CHECK_OP(close_ipc_handle)
 
 #define UMF_ASSIGN_OP(ops, type, func, default_return)                         \
     ops.func = [](void *obj, auto... args) {                                   \

source/loader/layers/sanitizer/asan_buffer.cpp

@@ -164,7 +164,8 @@ ur_result_t MemBuffer::getHandle(ur_device_handle_t Device, char *&Handle) {
 }
 
 ur_result_t MemBuffer::free() {
-    for (const auto &[_, Ptr] : Allocations) {
+    for (auto &&allocation : Allocations) {
+        char *Ptr = allocation.second;
         ur_result_t URes =
             getContext()->interceptor->releaseMemory(Context, Ptr);
         if (URes != UR_RESULT_SUCCESS) {

source/loader/ur_lib.cpp

@@ -50,8 +50,8 @@ void context_t::parseEnvEnabledLayers() {
 }
 
 void context_t::initLayers() {
-    for (auto &[layer, _] : layers) {
-        layer->init(&urDdiTable, enabledLayerNames, codelocData);
+    for (auto &&layer : layers) {
+        layer.first->init(&urDdiTable, enabledLayerNames, codelocData);
     }
 }
 

test/conformance/cts_exe.py

@@ -28,7 +28,7 @@
     result = subprocess.Popen([args.test_command, '--gtest_brief=1',        # nosec B603
                                f'--devices_count={args.devices_count}',
                                f'--platforms_count={args.platforms_count}'],
-                               stdout=subprocess.PIPE, stderr=subprocess.STDOUT, text=True)
+                               stdout=subprocess.PIPE, stderr=subprocess.STDOUT, universal_newlines=True)
 
     pat = re.compile(r'\[( )*FAILED( )*\]')
     output_list = []

test/layers/validation/fixtures.hpp

@@ -128,7 +128,7 @@ struct valAllDevicesTest : valPlatformTest {
 
 // We use this to avoid segfaults in the mock adapter when we're doing stuff
 // like double releases in the leak detection tests.
-inline ur_result_t genericSuccessCallback(void *) { return UR_RESULT_SUCCESS; };
+inline ur_result_t genericSuccessCallback(void *) { return UR_RESULT_SUCCESS; }
 
 // This returns valid (non-null) handles that we can safely leak.
 inline ur_result_t fakeContext_urContextCreate(void *pParams) {