Fixed style; added a comment about terribly wrong code [#CLICKHOUSE-2]

dbms/src/Dictionaries/HTTPDictionarySource.cpp

@@ -32,12 +32,13 @@ HTTPDictionarySource::HTTPDictionarySource(const DictionaryStructure & dict_stru
     if (update_field.empty())
         return;
 
+    /// TODO This code is totally wrong and ignorant.
+    /// What if URL contains fragment (#). What if update_field contains characters that must be %-encoded.
     std::string::size_type option = url.find("?");
-    if (option == std::string::npos) {
-        update_field = "?&" + update_field;
-    } else {
+    if (option == std::string::npos)
+        update_field = '?' + update_field;
+    else
         update_field = '&' + update_field;
-    }
 }
 
 HTTPDictionarySource::HTTPDictionarySource(const HTTPDictionarySource & other)