Presubmit fix: c++ code style

Author: olehkuznetsov

core/cc/interval_list.h

@@ -162,8 +162,8 @@ inline void CustomIntervalList<T>::replace(const T& i) {
       // i sits within a single interval. Split it into two.
       //           ┏━━━━━━━━━━━━━━┓
       //           ┗━━━━━━━━━━━━━━┛
-      //━━━━━━━━━━━┳═─═─═─═─═─═─═─┳━━━━━━━━━━━
-      //━━━━━━━━━━━┻─═─═─═─═─═─═─═┻━━━━━━━━━━━
+      // ━━━━━━━━━━━┳═─═─═─═─═─═─═─┳━━━━━━━━━━━
+      // ━━━━━━━━━━━┻─═─═─═─═─═─═─═┻━━━━━━━━━━━
       auto interval = mIntervals.begin() + first;
       mIntervals.insert(interval, *interval);
       last++;
@@ -172,16 +172,16 @@ inline void CustomIntervalList<T>::replace(const T& i) {
       // Trim end of first interval.
       //           ┏━━━━━━━━━━━━━━━━
       //           ┗━━━━━━━━━━━━━━━━
-      //━━━━━━━━━━━┳═─═─╗
-      //━━━━━━━━━━━┻─═─═┘
+      // ━━━━━━━━━━━┳═─═─╗
+      // ━━━━━━━━━━━┻─═─═┘
       auto interval = mIntervals.begin() + first;
       interval->adjust(interval->start(), i.start());
       first++;  // Don't erase the first interval.
     }
     if (trimHead) {
       // Trim front of last interval.
-      //━━━━━━━━━━━━━━━━┓
-      //━━━━━━━━━━━━━━━━┛
+      // ━━━━━━━━━━━━━━━━┓
+      // ━━━━━━━━━━━━━━━━┛
       //           ┌═─═─┳━━━━━━━━━━━
       //           ╚─═─═┻━━━━━━━━━━━
       auto interval = mIntervals.begin() + last;

core/cc/linux/process_name.cpp

@@ -17,8 +17,8 @@
 #include <libgen.h>
 #include <unistd.h>
 
-#include <cstring>
 #include <cstdint>
+#include <cstring>
 #include <string>
 
 namespace core {

core/vulkan/vk_debug_marker_layer/cc/vk_api_emitter.h

@@ -33,7 +33,7 @@ class VkApiEmitter : ThreadlocalEmitterBase {
 
   void StartTracing() override;
   void SetupTracing(
-      const typename perfetto::DataSourceBase::SetupArgs&) override{};
+      const typename perfetto::DataSourceBase::SetupArgs&) override {};
   void StopTracing() override;
   void EmitDebugUtilsObjectName(uint64_t vk_device, int32_t object_type,
                                 uint64_t handle, const char* name);
@@ -50,7 +50,7 @@ class VkApiEmitter : ThreadlocalEmitterBase {
         : vk_device_(vk_device),
           object_type_(object_type),
           handle_(handle),
-          name_(name){};
+          name_(name) {};
 
     uint64_t vk_device_;
     int32_t object_type_;

core/vulkan/vk_memory_tracker_layer/cc/memory_tracker_layer_impl.h

@@ -417,7 +417,7 @@ class HostAllocation {
         alignment(alignment_),
         scope(scope_),
         caller_api(caller_api_),
-        allocator_type(allocator_type_){};
+        allocator_type(allocator_type_) {};
   VulkanMemoryEventPtr GetVulkanMemoryEvent();
 
  private:

gapii/cc/vulkan_extras.cpp

@@ -1045,12 +1045,11 @@ uint32_t VulkanSpy::SpyOverride_vkAllocateMemory(
       device, pAllocateInfo, pAllocator, pMemory);
   auto l_physical_device =
       mState.PhysicalDevices[mState.Devices[device]->mPhysicalDevice];
-  if (0 !=
-      (l_physical_device->mMemoryProperties
-           .mmemoryTypes[pAllocateInfo->mmemoryTypeIndex]
-           .mpropertyFlags &
-       ((uint32_t)(
-           VkMemoryPropertyFlagBits::VK_MEMORY_PROPERTY_HOST_COHERENT_BIT)))) {
+  if (0 != (l_physical_device->mMemoryProperties
+                .mmemoryTypes[pAllocateInfo->mmemoryTypeIndex]
+                .mpropertyFlags &
+            ((uint32_t)(VkMemoryPropertyFlagBits::
+                            VK_MEMORY_PROPERTY_HOST_COHERENT_BIT)))) {
     // This is host-coherent memory. Some drivers actually allocate these pages
     // on-demand. This forces all of the pages to be created. This is needed as
     // our coherent memory tracker relies on page-faults which interferes with

gapii/cc/vulkan_layer_extras.h

@@ -115,7 +115,7 @@ typedef struct {
 
 #define VK_VERSION_MAJOR(version) ((uint32_t)(version) >> 22)
 #define VK_VERSION_MINOR(version) (((uint32_t)(version) >> 12) & 0x3ff)
-#define VK_VERSION_PATCH(version) ((uint32_t)(version)&0xfff)
+#define VK_VERSION_PATCH(version) ((uint32_t)(version) & 0xfff)
 
 #if defined(_WIN32)
 #define VK_LAYER_EXPORT __declspec(dllexport)

replay2/core_utils/non_copyable.h

@@ -20,7 +20,7 @@ namespace replay2 {
 
 class NonCopyable {
    public:
-    NonCopyable(void){};
+    NonCopyable(void) {};
     NonCopyable(const NonCopyable& rhs) = delete;
     NonCopyable& operator=(const NonCopyable& rhs) = delete;
 };