From 5d6554c21132ac7ca6d191c39a22bfaf90ac39fc Mon Sep 17 00:00:00 2001
From: narspt <narspt@users.noreply.github.com>
Date: Wed, 26 Feb 2020 07:16:02 +0000
Subject: [PATCH] fix wiresx search, avoid listing same reflector multiple
 times

if match occurs multiple times along reflector name... and a bit of code cleanup...
---
 YSFGateway/YSFReflectors.cpp | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/YSFGateway/YSFReflectors.cpp b/YSFGateway/YSFReflectors.cpp
index 58e06df..5e34a35 100644
--- a/YSFGateway/YSFReflectors.cpp
+++ b/YSFGateway/YSFReflectors.cpp
@@ -313,19 +313,17 @@ std::vector<CYSFReflector*>& CYSFReflectors::search(const std::string& name)
 		reflector.erase(std::find_if(reflector.rbegin(), reflector.rend(), std::not1(std::ptr_fun<int, int>(std::isspace))).base(), reflector.end());
 		std::transform(reflector.begin(), reflector.end(), reflector.begin(), ::toupper);
 
-		// Origional match function - only matches start of string.
+		// Original match function - only matches start of string.
 		// if (trimmed == reflector.substr(0U, len))
-		//	m_search.push_back(*it);
+		// 	m_search.push_back(*it);
 		
 		// New match function searches the whole string
-		unsigned int refSrcPos;
-                for (refSrcPos=0;refSrcPos<reflector.length(); refSrcPos++)
-                {
-                        if (reflector.substr(refSrcPos,trimmed.length()) == trimmed)
-                        {
-                                m_search.push_back(*it);
-                        }
-                }
+		for (unsigned int refSrcPos = 0U; refSrcPos < reflector.length(); refSrcPos++) {
+			if (reflector.substr(refSrcPos, trimmed.length()) == trimmed) {
+				m_search.push_back(*it);
+				break;
+			}
+		}
 	}
 
 	std::sort(m_search.begin(), m_search.end(), refComparison);