nit changes

Author: nikhilbhatia08

resource_detectors/process_detector_utils.cc

@@ -12,6 +12,7 @@
 #  include <windows.h>
 #  include <psapi.h>
 #  include <shellapi.h>
+#  pragma comment(lib, "shell32.lib")
 // clang-format on
 #else
 #  include <sys/types.h>
@@ -92,7 +93,7 @@ std::vector<std::string> GetCommandWithArgs(const int32_t &pid)
   LPWSTR *argvW = CommandLineToArgvW(GetCommandLineW(), &argc);
   if (!argvW)
   {
-    return {};
+    return {};  // returns an empty vector if CommandLineToArgvW fails
   }
 
   std::vector<std::string> args;

resource_detectors/test/process_detector_test.cc

@@ -12,8 +12,6 @@
 #  include <process.h>
 #  include <windows.h>
 #  include <psapi.h>
-#  include <shellapi.h>
-#  pragma comment(lib, "shell32.lib")
 #  define getpid _getpid
 // clang-format on
 #else
@@ -173,16 +171,18 @@ TEST(ProcessDetectorUtilsTest, GetCommandWithArgsTest)
   {
     args = {};
   }
-
-  for (int i = 0; i < argc; i++)
+  else
   {
-    // Convert UTF-16 to UTF-8
-    int size_needed = WideCharToMultiByte(CP_UTF8, 0, argvW[i], -1, NULL, 0, NULL, NULL);
-    if (size_needed > 0)
+    for (int i = 0; i < argc; i++)
     {
-      std::string arg(size_needed - 1, 0);
-      WideCharToMultiByte(CP_UTF8, 0, argvW[i], -1, &arg[0], size_needed, NULL, NULL);
-      args.push_back(arg);
+      // Convert UTF-16 to UTF-8
+      int size_needed = WideCharToMultiByte(CP_UTF8, 0, argvW[i], -1, NULL, 0, NULL, NULL);
+      if (size_needed > 0)
+      {
+        std::string arg(size_needed - 1, 0);
+        WideCharToMultiByte(CP_UTF8, 0, argvW[i], -1, &arg[0], size_needed, NULL, NULL);
+        args.push_back(arg);
+      }
     }
   }