refactor: names is passed by value — prefer const std::vector<std::string>&

With svchost.exe yielding ~78 related processes, this copies all those strings on every call. Since names is only read inside the function, pass by const reference like pids

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: supervoidcoder

main.cpp

@@ -1727,7 +1727,7 @@ void FindProcessPorts(DWORD targetPid) {
 
 
 
-void PIDinspect(const std::vector<DWORD>& pids, std::vector<std::string> names) { // ooh guys look i'm in the void
+void PIDinspect(const std::vector<DWORD>& pids, const std::vector<std::string>& names) { // ooh guys look i'm in the void
     DWORD pid = pids[0];
 	std::string procName = GetProcessNameFromPid(pid);
 	if (IsVirtualTerminalModeEnabled()) {