From 457c50c7d71bbf393480d4a5f0fd6c5c2f768c60 Mon Sep 17 00:00:00 2001
From: Alexey Milovidov <milovidov@yandex-team.ru>
Date: Sat, 18 Jan 2020 23:19:10 +0300
Subject: [PATCH] Code cleanups around

---
 dbms/src/Common/RemoteHostFilter.cpp |  5 +++--
 dbms/src/Common/RemoteHostFilter.h   | 11 +++++++----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/dbms/src/Common/RemoteHostFilter.cpp b/dbms/src/Common/RemoteHostFilter.cpp
index 16aaac35dbe..4c4aa3bca81 100644
--- a/dbms/src/Common/RemoteHostFilter.cpp
+++ b/dbms/src/Common/RemoteHostFilter.cpp
@@ -1,12 +1,13 @@
 #include <re2/re2.h>
-#include <Common/RemoteHostFilter.h>
 #include <Poco/URI.h>
-#include <Formats/FormatFactory.h>
 #include <Poco/Util/AbstractConfiguration.h>
+#include <Formats/FormatFactory.h>
+#include <Common/RemoteHostFilter.h>
 #include <Common/StringUtils/StringUtils.h>
 #include <Common/Exception.h>
 #include <IO/WriteHelpers.h>
 
+
 namespace DB
 {
 namespace ErrorCodes
diff --git a/dbms/src/Common/RemoteHostFilter.h b/dbms/src/Common/RemoteHostFilter.h
index 86743891051..48d9b2bda7c 100644
--- a/dbms/src/Common/RemoteHostFilter.h
+++ b/dbms/src/Common/RemoteHostFilter.h
@@ -1,17 +1,19 @@
 #pragma once
 
+#include <string>
 #include <vector>
 #include <unordered_set>
-#include <Poco/URI.h>
-#include <Poco/Util/AbstractConfiguration.h>
 
 
+namespace Poco { class URI; }
+namespace Poco { namespace Util { class AbstractConfiguration; } }
+
 namespace DB
 {
 class RemoteHostFilter
 {
 /**
- * This class checks if url is allowed.
+ * This class checks if URL is allowed.
  * If primary_hosts and regexp_hosts are empty all urls are allowed.
  */
 public:
@@ -25,6 +27,7 @@ private:
     std::unordered_set<std::string> primary_hosts;      /// Allowed primary (<host>) URL from config.xml
     std::vector<std::string> regexp_hosts;              /// Allowed regexp (<hots_regexp>) URL from config.xml
 
-    bool checkForDirectEntry(const std::string & str) const; /// Checks if the primary_hosts and regexp_hosts contain str. If primary_hosts and regexp_hosts are empty return true.
+    /// Checks if the primary_hosts and regexp_hosts contain str. If primary_hosts and regexp_hosts are empty return true.
+    bool checkForDirectEntry(const std::string & str) const;
 };
 }