Comment 12 for bug 442475

Why is SearchResultPtr used within gui functions? Would it not be better to make SearchResultPtrs and the result map be used only within client functions? This would save having to create and dispatch gui funcs for each of the duplicate results. Plus we should use this new data structure for more than just storage. Instead of calling ClientManager::getInstance()->findUser(cid) we could use this data structure to speed it up (findUser locks the ClientManager critical section).