Fix includes.

GitOrigin-RevId: 7b62de39f2d62ac8c14c24fb83041c717ba49873

benchmark/bench_queue.cpp

@@ -8,9 +8,10 @@
 #include "td/utils/common.h"
 #include "td/utils/logging.h"
 #include "td/utils/MpscPollableQueue.h"
+#include "td/utils/port/sleep.h"
+#include "td/utils/port/thread.h"
 #include "td/utils/queue.h"
 #include "td/utils/Random.h"
-#include "td/utils/port/sleep.h"
 
 // TODO: check system calls
 // TODO: all return values must be checked

td/mtproto/PingConnection.cpp

@@ -6,8 +6,8 @@
 //
 #include "td/mtproto/PingConnection.h"
 
-#include "td/mtproto/AuthKey.h"
 #include "td/mtproto/AuthData.h"
+#include "td/mtproto/AuthKey.h"
 #include "td/mtproto/mtproto_api.h"
 #include "td/mtproto/NoCryptoStorer.h"
 #include "td/mtproto/PacketInfo.h"

td/mtproto/TlsInit.cpp

@@ -7,13 +7,15 @@
 #include "td/mtproto/TlsInit.h"
 
 #include "td/utils/as.h"
+#include "td/utils/common.h"
 #include "td/utils/crypto.h"
-#include "td/utils/port/Clocks.h"
+#include "td/utils/logging.h"
 #include "td/utils/Random.h"
 #include "td/utils/Span.h"
+#include "td/utils/Time.h"
 
 #include <algorithm>
-#include <cstdlib>
+#include <cstring>
 
 namespace td {
 namespace mtproto {

td/mtproto/TlsInit.h

@@ -8,6 +8,10 @@
 
 #include "td/net/TransparentProxy.h"
 
+#include "td/actor/actor.h"
+
+#include "td/utils/port/IPAddress.h"
+#include "td/utils/port/SocketFd.h"
 #include "td/utils/Slice.h"
 #include "td/utils/Status.h"
 

td/mtproto/TlsReaderByteFlow.cpp

@@ -6,6 +6,7 @@
 //
 #include "td/mtproto/TlsReaderByteFlow.h"
 
+#include "td/utils/Slice.h"
 #include "td/utils/Status.h"
 
 namespace td {

td/mtproto/Transport.cpp

@@ -13,6 +13,7 @@
 #include "td/utils/crypto.h"
 #include "td/utils/format.h"
 #include "td/utils/logging.h"
+#include "td/utils/misc.h"
 #include "td/utils/Random.h"
 #include "td/utils/Status.h"
 

td/telegram/BackgroundManager.cpp

@@ -29,8 +29,10 @@
 
 #include "td/utils/buffer.h"
 #include "td/utils/common.h"
+#include "td/utils/format.h"
 #include "td/utils/logging.h"
 #include "td/utils/misc.h"
+#include "td/utils/Slice.h"
 #include "td/utils/tl_helpers.h"
 
 #include <algorithm>

td/telegram/ConfigManager.cpp

@@ -25,6 +25,7 @@
 #include "td/mtproto/AuthKey.h"
 #include "td/mtproto/crypto.h"
 #include "td/mtproto/RawConnection.h"
+#include "td/mtproto/TransportType.h"
 
 #if !TD_EMSCRIPTEN  //FIXME
 #include "td/net/HttpQuery.h"
@@ -42,6 +43,7 @@
 #include "td/utils/JsonBuilder.h"
 #include "td/utils/logging.h"
 #include "td/utils/misc.h"
+#include "td/utils/Parser.h"
 #include "td/utils/port/Clocks.h"
 #include "td/utils/Random.h"
 #include "td/utils/Time.h"
@@ -53,8 +55,6 @@
 #include <memory>
 #include <utility>
 
-#include <ctime>
-
 namespace td {
 
 int VERBOSITY_NAME(config_recoverer) = VERBOSITY_NAME(INFO);

td/telegram/Global.cpp

@@ -118,7 +118,7 @@ double Global::get_dns_time_difference() const {
     return dns_flag ? dns_diff : server_diff;
   }
   if (dns_flag) {
-    return std::max(dns_diff, server_diff);
+    return max(dns_diff, server_diff);
   }
   if (td_db_) {
     return server_diff;

td/telegram/Photo.h

@@ -21,6 +21,7 @@
 #include "td/utils/buffer.h"
 #include "td/utils/common.h"
 #include "td/utils/StringBuilder.h"
+#include "td/utils/Variant.h"
 
 namespace td {
 

td/telegram/PhotoSizeSource.hpp

@@ -6,7 +6,7 @@
 //
 #pragma once
 
-#include "td/telegram/Photo.h"
+#include "td/telegram/PhotoSizeSource.h"
 
 #include "td/utils/tl_helpers.h"
 

td/telegram/StickersManager.cpp

@@ -22,8 +22,8 @@
 #include "td/telegram/files/FileManager.h"
 #include "td/telegram/files/FileType.h"
 #include "td/telegram/Global.h"
-#include "td/telegram/logevent/LogEvent.h"
 #include "td/telegram/LanguagePackManager.h"
+#include "td/telegram/logevent/LogEvent.h"
 #include "td/telegram/MessagesManager.h"
 #include "td/telegram/misc.h"
 #include "td/telegram/net/DcId.h"

td/telegram/Td.cpp

@@ -89,8 +89,12 @@
 
 #include "td/db/binlog/BinlogEvent.h"
 
+#include "td/mtproto/crypto.h"
+#include "td/mtproto/DhHandshake.h"
+#include "td/mtproto/Handshake.h"
 #include "td/mtproto/HandshakeActor.h"
 #include "td/mtproto/RawConnection.h"
+#include "td/mtproto/TransportType.h"
 #include "td/mtproto/utils.h"  // for create_storer, fetch_result, etc, TODO
 
 #include "td/utils/buffer.h"
@@ -99,7 +103,9 @@
 #include "td/utils/MimeType.h"
 #include "td/utils/misc.h"
 #include "td/utils/PathView.h"
+#include "td/utils/port/IPAddress.h"
 #include "td/utils/port/path.h"
+#include "td/utils/port/SocketFd.h"
 #include "td/utils/Random.h"
 #include "td/utils/Slice.h"
 #include "td/utils/Status.h"

td/telegram/UpdatesManager.h

@@ -6,11 +6,13 @@
 //
 #pragma once
 
+#include "td/telegram/ChannelId.h"
+#include "td/telegram/ChatId.h"
 #include "td/telegram/DialogId.h"
 #include "td/telegram/PtsManager.h"
-
 #include "td/telegram/td_api.h"
 #include "td/telegram/telegram_api.h"
+#include "td/telegram/UserId.h"
 
 #include "td/actor/actor.h"
 #include "td/actor/PromiseFuture.h"

td/telegram/net/ConnectionCreator.cpp

@@ -20,6 +20,7 @@
 #include "td/telegram/TdDb.h"
 
 #include "td/mtproto/Ping.h"
+#include "td/mtproto/ProxySecret.h"
 #include "td/mtproto/RawConnection.h"
 #include "td/mtproto/TlsInit.h"
 

td/telegram/net/MtprotoHeader.cpp

@@ -6,8 +6,8 @@
 //
 #include "td/telegram/net/MtprotoHeader.h"
 
-#include "td/telegram/LanguagePackManager.h"
 #include "td/telegram/JsonValue.h"
+#include "td/telegram/LanguagePackManager.h"
 
 #include "td/tl/tl_object_store.h"
 

td/telegram/net/Proxy.h

@@ -136,7 +136,7 @@ class Proxy {
       parse(port_, parser);
       secret_ = mtproto::ProxySecret::from_link(parser.template fetch_string<Slice>()).move_as_ok();
     } else {
-      LOG_CHECK(type_ == Proxy::Type::None) << static_cast<int32>(type_);
+      CHECK(type_ == Proxy::Type::None);
     }
   }
 

td/tl/tl_json.h

@@ -7,6 +7,7 @@
 #pragma once
 
 #include "td/utils/base64.h"
+#include "td/utils/common.h"
 #include "td/utils/format.h"
 #include "td/utils/JsonBuilder.h"
 #include "td/utils/misc.h"
@@ -17,6 +18,7 @@
 #include "td/telegram/td_api.h"
 #include "td/telegram/td_api.hpp"
 
+#include <functional>
 #include <type_traits>
 
 namespace td {

tdutils/td/utils/ConcurrentHashTable.h

@@ -8,6 +8,8 @@
 
 #include "td/utils/common.h"
 #include "td/utils/HazardPointers.h"
+#include "td/utils/logging.h"
+#include "td/utils/port/thread_local.h"
 
 #include <atomic>
 #include <condition_variable>

tdutils/td/utils/EpochBasedMemoryReclamation.h

@@ -9,7 +9,9 @@
 #include "td/utils/common.h"
 #include "td/utils/logging.h"
 #include "td/utils/port/sleep.h"
-#include "td/utils/port/thread.h"
+
+#include <atomic>
+#include <memory>
 
 namespace td {
 

tdutils/td/utils/Gzip.cpp

@@ -13,6 +13,7 @@ char disable_linker_warning_about_empty_file_gzip_cpp TD_UNUSED;
 
 #include <cstring>
 #include <limits>
+#include <utility>
 
 #include <zlib.h>
 

tdutils/td/utils/Hash.h

@@ -12,6 +12,8 @@
 #include <absl/hash/hash.h>
 #endif
 
+#include <utility>
+
 namespace td {
 // A simple wrapper for absl::flat_hash_map, std::unordered_map and probably some our implementaion of hash map in
 // the future

tdutils/td/utils/MpmcWaiter.h

@@ -7,7 +7,6 @@
 #pragma once
 
 #include "td/utils/common.h"
-#include "td/utils/logging.h"
 #include "td/utils/port/thread.h"
 
 #include <atomic>

tdutils/td/utils/ThreadSafeCounter.h

@@ -6,8 +6,7 @@
 //
 #pragma once
 
-#include "td/utils/int_types.h"
-#include "td/utils/port/thread.h"
+#include "td/utils/common.h"
 #include "td/utils/port/thread_local.h"
 
 #include <array>

tdutils/td/utils/optional.h

@@ -6,6 +6,7 @@
 //
 #pragma once
 
+#include "td/utils/common.h"
 #include "td/utils/Status.h"
 
 #include <type_traits>

tdutils/td/utils/port/MemoryMapping.cpp

@@ -6,6 +6,7 @@
 //
 #include "td/utils/port/MemoryMapping.h"
 
+#include "td/utils/logging.h"
 #include "td/utils/misc.h"
 
 // TODO:

tdutils/td/utils/port/SocketFd.cpp

@@ -7,6 +7,7 @@
 #include "td/utils/port/SocketFd.h"
 
 #include "td/utils/common.h"
+#include "td/utils/format.h"
 #include "td/utils/logging.h"
 #include "td/utils/misc.h"
 #include "td/utils/port/PollFlags.h"

tdutils/td/utils/port/StdStreams.cpp

@@ -8,8 +8,8 @@
 
 #include "td/utils/logging.h"
 #include "td/utils/misc.h"
-#include "td/utils/port/detail/NativeFd.h"
 #include "td/utils/port/detail/Iocp.h"
+#include "td/utils/port/detail/NativeFd.h"
 #include "td/utils/port/PollFlags.h"
 #include "td/utils/port/thread.h"
 #include "td/utils/Slice.h"

tdutils/td/utils/port/detail/Iocp.h

@@ -13,9 +13,10 @@
 #include "td/utils/common.h"
 #include "td/utils/Context.h"
 #include "td/utils/port/detail/NativeFd.h"
-#include "td/utils/port/thread.h"
 #include "td/utils/Status.h"
 
+#include <memory>
+
 namespace td {
 namespace detail {
 

tdutils/td/utils/port/signals.cpp

@@ -24,6 +24,7 @@
 
 #include <cerrno>
 #include <cstdint>
+#include <cstdlib>
 #include <cstring>
 #include <ctime>
 #include <limits>

tdutils/td/utils/port/stacktrace.cpp

@@ -58,7 +58,7 @@ void print_backtrace_gdb(void) {
       return;
     }
 #if defined(PR_SET_PTRACER)
-    // We can't use EventFd because we are in a signal handler
+    // We can't use event fd because we are in a signal handler
     int fds[2];
     bool need_set_ptracer = true;
     if (pipe(fds) < 0) {

tdutils/test/ConcurrentHashMap.cpp

@@ -6,14 +6,12 @@
 //
 #include "td/utils/benchmark.h"
 #include "td/utils/ConcurrentHashTable.h"
-#include "td/utils/HazardPointers.h"
 #include "td/utils/port/thread.h"
 #include "td/utils/SpinLock.h"
 #include "td/utils/tests.h"
 
-#include <algorithm>
 #include <atomic>
-#include <cstdio>
+#include <mutex>
 
 #if !TD_THREAD_UNSUPPORTED
 
@@ -29,8 +27,8 @@
 
 #if TD_WITH_JUNCTION
 #include <junction/ConcurrentMap_Grampa.h>
-#include <junction/ConcurrentMap_Linear.h>
 #include <junction/ConcurrentMap_Leapfrog.h>
+#include <junction/ConcurrentMap_Linear.h>
 #endif
 
 namespace td {

tdutils/test/EpochBasedMemoryReclamation.cpp

@@ -4,6 +4,7 @@
 // Distributed under the Boost Software License, Version 1.0. (See accompanying
 // file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
 //
+#include "td/utils/common.h"
 #include "td/utils/EpochBasedMemoryReclamation.h"
 #include "td/utils/logging.h"
 #include "td/utils/port/thread.h"

tdutils/test/HazardPointers.cpp

@@ -9,6 +9,7 @@
 #include "td/utils/logging.h"
 #include "td/utils/port/thread.h"
 #include "td/utils/Random.h"
+#include "td/utils/Slice.h"
 #include "td/utils/tests.h"
 
 #include <atomic>

tdutils/test/crypto.cpp

@@ -8,6 +8,7 @@
 #include "td/utils/benchmark.h"
 #include "td/utils/common.h"
 #include "td/utils/crypto.h"
+#include "td/utils/logging.h"
 #include "td/utils/Slice.h"
 #include "td/utils/tests.h"
 #include "td/utils/UInt.h"

tdutils/test/misc.cpp

@@ -27,6 +27,7 @@
 #include "td/utils/port/wstring_convert.h"
 #include "td/utils/Random.h"
 #include "td/utils/Slice.h"
+#include "td/utils/Status.h"
 #include "td/utils/StringBuilder.h"
 #include "td/utils/tests.h"
 #include "td/utils/Time.h"

tdutils/test/port.cpp

@@ -8,8 +8,11 @@
 #include "td/utils/logging.h"
 #include "td/utils/misc.h"
 #include "td/utils/port/FileFd.h"
+#include "td/utils/port/IoSlice.h"
 #include "td/utils/port/path.h"
 #include "td/utils/port/signals.h"
+#include "td/utils/port/thread.h"
+#include "td/utils/port/thread_local.h"
 #include "td/utils/Slice.h"
 #include "td/utils/tests.h"
 

test/mtproto.cpp

@@ -34,9 +34,9 @@
 #include "td/utils/base64.h"
 #include "td/utils/common.h"
 #include "td/utils/logging.h"
+#include "td/utils/port/Clocks.h"
 #include "td/utils/port/IPAddress.h"
 #include "td/utils/port/SocketFd.h"
-#include "td/utils/Parser.h"
 #include "td/utils/Random.h"
 #include "td/utils/Status.h"
 #include "td/utils/Time.h"