diff --git a/map/framework.cpp b/map/framework.cpp index dee7e53770..8f798884ee 100644 --- a/map/framework.cpp +++ b/map/framework.cpp @@ -1256,7 +1256,7 @@ void Framework::InitSearchEngine() params.m_numThreads = 1; m_searchEngine.reset(new search::Engine(const_cast(m_model.GetIndex()), GetDefaultCategories(), *m_infoGetter, - make_unique(), params)); + make_unique(), params)); } catch (RootException const & e) { diff --git a/search/processor_factory.hpp b/search/processor_factory.hpp index f4b5ecc1c6..83538df79f 100644 --- a/search/processor_factory.hpp +++ b/search/processor_factory.hpp @@ -12,14 +12,14 @@ class CountryInfoGetter; namespace search { -class SearchProcessorFactory +class ProcessorFactory { public: - virtual ~SearchProcessorFactory() = default; + virtual ~ProcessorFactory() = default; - virtual unique_ptr BuildProcessor(Index & index, CategoriesHolder const & categories, - vector const & suggests, - storage::CountryInfoGetter const & infoGetter) + virtual unique_ptr Build(Index & index, CategoriesHolder const & categories, + vector const & suggests, + storage::CountryInfoGetter const & infoGetter) { return make_unique(index, categories, suggests, infoGetter); } diff --git a/search/search_engine.cpp b/search/search_engine.cpp index 1a540edc19..7bb5ac4cdd 100644 --- a/search/search_engine.cpp +++ b/search/search_engine.cpp @@ -124,8 +124,8 @@ Engine::Params::Params(string const & locale, size_t numThreads) // Engine ------------------------------------------------------------------------------------------ Engine::Engine(Index & index, CategoriesHolder const & categories, - storage::CountryInfoGetter const & infoGetter, - unique_ptr factory, Params const & params) + storage::CountryInfoGetter const & infoGetter, unique_ptr factory, + Params const & params) : m_categories(categories), m_shutdown(false) { InitSuggestions doInit; @@ -135,7 +135,7 @@ Engine::Engine(Index & index, CategoriesHolder const & categories, m_contexts.resize(params.m_numThreads); for (size_t i = 0; i < params.m_numThreads; ++i) { - auto processor = factory->BuildProcessor(index, m_categories, m_suggests, infoGetter); + auto processor = factory->Build(index, m_categories, m_suggests, infoGetter); processor->SetPreferredLocale(params.m_locale); m_contexts[i].m_processor = move(processor); } diff --git a/search/search_engine.hpp b/search/search_engine.hpp index c4e1a7de51..2462a4c616 100644 --- a/search/search_engine.hpp +++ b/search/search_engine.hpp @@ -93,7 +93,7 @@ public: // Doesn't take ownership of index. Takes ownership of categoriesR. Engine(Index & index, CategoriesHolder const & categories, - storage::CountryInfoGetter const & infoGetter, unique_ptr factory, + storage::CountryInfoGetter const & infoGetter, unique_ptr factory, Params const & params); ~Engine(); diff --git a/search/search_integration_tests/helpers.cpp b/search/search_integration_tests/helpers.cpp index 5457b559c7..d12126777f 100644 --- a/search/search_integration_tests/helpers.cpp +++ b/search/search_integration_tests/helpers.cpp @@ -12,8 +12,8 @@ namespace search SearchTest::SearchTest() : m_platform(GetPlatform()) , m_scopedLog(LDEBUG) - , m_engine(make_unique(), - make_unique(), Engine::Params()) + , m_engine(make_unique(), make_unique(), + Engine::Params()) { } diff --git a/search/search_quality/features_collector_tool/features_collector_tool.cpp b/search/search_quality/features_collector_tool/features_collector_tool.cpp index 5c8cccc0e4..59cabc2c3d 100644 --- a/search/search_quality/features_collector_tool/features_collector_tool.cpp +++ b/search/search_quality/features_collector_tool/features_collector_tool.cpp @@ -217,8 +217,7 @@ int main(int argc, char * argv[]) } classificator::Load(); - TestSearchEngine engine(move(infoGetter), make_unique(), - Engine::Params{}); + TestSearchEngine engine(move(infoGetter), make_unique(), Engine::Params{}); vector mwms; platform::FindAllLocalMapsAndCleanup(numeric_limits::max() /* the latest version */, diff --git a/search/search_quality/search_quality_tool/search_quality_tool.cpp b/search/search_quality/search_quality_tool/search_quality_tool.cpp index 1596472c88..a49384af61 100644 --- a/search/search_quality/search_quality_tool/search_quality_tool.cpp +++ b/search/search_quality/search_quality_tool/search_quality_tool.cpp @@ -380,8 +380,7 @@ int main(int argc, char * argv[]) Engine::Params params; params.m_locale = FLAGS_locale; params.m_numThreads = FLAGS_num_threads; - TestSearchEngine engine(move(infoGetter), make_unique(), - Engine::Params{}); + TestSearchEngine engine(move(infoGetter), make_unique(), Engine::Params{}); vector mwms; if (!FLAGS_mwm_list_path.empty()) diff --git a/search/search_tests_support/test_search_engine.cpp b/search/search_tests_support/test_search_engine.cpp index bd41d42c31..c871635669 100644 --- a/search/search_tests_support/test_search_engine.cpp +++ b/search/search_tests_support/test_search_engine.cpp @@ -9,7 +9,7 @@ namespace search namespace tests_support { TestSearchEngine::TestSearchEngine(unique_ptr infoGetter, - unique_ptr<::search::SearchProcessorFactory> factory, + unique_ptr<::search::ProcessorFactory> factory, Engine::Params const & params) : m_platform(GetPlatform()) , m_infoGetter(move(infoGetter)) @@ -17,7 +17,7 @@ TestSearchEngine::TestSearchEngine(unique_ptr infoGe { } -TestSearchEngine::TestSearchEngine(unique_ptr<::search::SearchProcessorFactory> factory, +TestSearchEngine::TestSearchEngine(unique_ptr<::search::ProcessorFactory> factory, Engine::Params const & params) : m_platform(GetPlatform()) , m_infoGetter(storage::CountryInfoReader::CreateCountryInfoReader(m_platform)) diff --git a/search/search_tests_support/test_search_engine.hpp b/search/search_tests_support/test_search_engine.hpp index c32b94bbc5..c2d2938f67 100644 --- a/search/search_tests_support/test_search_engine.hpp +++ b/search/search_tests_support/test_search_engine.hpp @@ -27,10 +27,8 @@ class TestSearchEngine : public Index { public: TestSearchEngine(unique_ptr infoGetter, - unique_ptr factory, - Engine::Params const & params); - TestSearchEngine(unique_ptr<::search::SearchProcessorFactory> factory, - Engine::Params const & params); + unique_ptr factory, Engine::Params const & params); + TestSearchEngine(unique_ptr<::search::ProcessorFactory> factory, Engine::Params const & params); ~TestSearchEngine() override; inline void SetLocale(string const & locale) { m_engine.SetLocale(locale); }