From 08cc9481653abc3a6a8bdc2419dcb8d71e39ca31 Mon Sep 17 00:00:00 2001
From: LoveSy <shana@zju.edu.cn>
Date: Thu, 13 Jun 2024 13:47:11 +0000
Subject: [PATCH] Merge FMT_MODULE and FMT_IMPORT_STD

---
 include/fmt/args.h       |  2 +-
 include/fmt/base.h       |  8 ++---
 include/fmt/chrono.h     |  2 +-
 include/fmt/compile.h    |  2 +-
 include/fmt/format-inl.h | 11 +++---
 include/fmt/format.h     |  7 ++--
 include/fmt/os.h         |  8 ++---
 include/fmt/ostream.h    |  2 +-
 include/fmt/printf.h     |  2 +-
 include/fmt/ranges.h     |  2 +-
 include/fmt/std.h        |  4 +--
 include/fmt/xchar.h      |  5 ++-
 src/fmt.cc               |  2 ++
 src/os.cc                | 74 +++++++++++++++++++++-------------------
 14 files changed, 64 insertions(+), 67 deletions(-)

diff --git a/include/fmt/args.h b/include/fmt/args.h
index d45b965bc51a..31a60e8faf1a 100644
--- a/include/fmt/args.h
+++ b/include/fmt/args.h
@@ -8,7 +8,7 @@
 #ifndef FMT_ARGS_H_
 #define FMT_ARGS_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <functional>  // std::reference_wrapper
 #  include <memory>      // std::unique_ptr
 #  include <vector>
diff --git a/include/fmt/base.h b/include/fmt/base.h
index 396d268faaab..0e9758b70b2f 100644
--- a/include/fmt/base.h
+++ b/include/fmt/base.h
@@ -8,19 +8,19 @@
 #ifndef FMT_BASE_H_
 #define FMT_BASE_H_
 
+#if defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#  define FMT_MODULE
+#endif
+
 // c headers are preferable for performance reasons
 #ifndef FMT_MODULE
 #  include <limits.h>  // CHAR_BIT
 #  include <stdio.h>   // FILE
 #  include <string.h>  // strlen
-#endif
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
 // <cstddef> is also included transitively from <type_traits>.
 #  include <cstddef>      // std::byte
 #  include <type_traits>  // std::enable_if
-#elif !defined(FMT_MODULE)
-import std;
 #endif
 
 // The fmt library version in the form major * 10000 + minor * 100 + patch.
diff --git a/include/fmt/chrono.h b/include/fmt/chrono.h
index 3406a4a673f2..c93123fd3353 100644
--- a/include/fmt/chrono.h
+++ b/include/fmt/chrono.h
@@ -8,7 +8,7 @@
 #ifndef FMT_CHRONO_H_
 #define FMT_CHRONO_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <algorithm>
 #  include <chrono>
 #  include <cmath>    // std::isfinite
diff --git a/include/fmt/compile.h b/include/fmt/compile.h
index a98e1bc0531b..b2afc2c309f4 100644
--- a/include/fmt/compile.h
+++ b/include/fmt/compile.h
@@ -8,7 +8,7 @@
 #ifndef FMT_COMPILE_H_
 #define FMT_COMPILE_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <iterator>  // std::back_inserter
 #endif
 
diff --git a/include/fmt/format-inl.h b/include/fmt/format-inl.h
index db6d74d4c95f..0750e7fcb947 100644
--- a/include/fmt/format-inl.h
+++ b/include/fmt/format-inl.h
@@ -8,17 +8,14 @@
 #ifndef FMT_FORMAT_INL_H_
 #define FMT_FORMAT_INL_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
-#  include <algorithm>
-#  include <cmath>
-#  include <exception>
-#endif
-
 #ifndef FMT_MODULE
+#  include <algorithm>
 #  include <cerrno>  // errno
 #  include <climits>
+#  include <cmath>
+#  include <exception>
 
-#  if !defined(FMT_STATIC_THOUSANDS_SEPARATOR) && !defined(FMT_IMPORT_STD)
+#  if !defined(FMT_STATIC_THOUSANDS_SEPARATOR)
 #    include <locale>
 #  endif
 #endif
diff --git a/include/fmt/format.h b/include/fmt/format.h
index bfdb2e0e3fe8..38ea62194627 100644
--- a/include/fmt/format.h
+++ b/include/fmt/format.h
@@ -38,7 +38,7 @@
 #  define FMT_REMOVE_TRANSITIVE_INCLUDES
 #endif
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <cmath>             // std::signbit
 #  include <cstdint>           // uint32_t
 #  include <cstring>           // std::memcpy
@@ -56,15 +56,14 @@
 #include "base.h"
 
 // Checking FMT_CPLUSPLUS for warning suppression in MSVC.
-#if FMT_HAS_INCLUDE(<bit>) && FMT_CPLUSPLUS > 201703L && \
-    !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#if FMT_HAS_INCLUDE(<bit>) && FMT_CPLUSPLUS > 201703L && !defined(FMT_MODULE)
 #  include <bit>  // std::bit_cast
 #endif
 
 // libc++ supports string_view in pre-c++17.
 #if FMT_HAS_INCLUDE(<string_view>) && \
     (FMT_CPLUSPLUS >= 201703L || defined(_LIBCPP_VERSION))
-#  if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#  if !defined(FMT_MODULE)
 #    include <string_view>
 #  endif
 #  define FMT_USE_STRING_VIEW
diff --git a/include/fmt/os.h b/include/fmt/os.h
index 5462661a026a..5344e5c2378b 100644
--- a/include/fmt/os.h
+++ b/include/fmt/os.h
@@ -10,11 +10,9 @@
 
 #ifndef FMT_MODULE
 #  include <cerrno>
-#  ifndef FMT_IMPORT_STD
-#    include <cstddef>
-#    include <cstdio>
-#    include <system_error>  // std::system_error
-#  endif
+#  include <cstddef>
+#  include <cstdio>
+#  include <system_error>  // std::system_error
 #endif
 
 #include "format.h"
diff --git a/include/fmt/ostream.h b/include/fmt/ostream.h
index 0be26709dcd0..98faef659f52 100644
--- a/include/fmt/ostream.h
+++ b/include/fmt/ostream.h
@@ -8,7 +8,7 @@
 #ifndef FMT_OSTREAM_H_
 #define FMT_OSTREAM_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <fstream>  // std::filebuf
 #endif
 
diff --git a/include/fmt/printf.h b/include/fmt/printf.h
index 3b2b9b4be8b3..ee4b06fd5e72 100644
--- a/include/fmt/printf.h
+++ b/include/fmt/printf.h
@@ -8,7 +8,7 @@
 #ifndef FMT_PRINTF_H_
 #define FMT_PRINTF_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <algorithm>  // std::max
 #  include <limits>     // std::numeric_limits
 #endif
diff --git a/include/fmt/ranges.h b/include/fmt/ranges.h
index f3377ab5c312..66329aae71ed 100644
--- a/include/fmt/ranges.h
+++ b/include/fmt/ranges.h
@@ -8,7 +8,7 @@
 #ifndef FMT_RANGES_H_
 #define FMT_RANGES_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <initializer_list>
 #  include <iterator>
 #  include <string>
diff --git a/include/fmt/std.h b/include/fmt/std.h
index 5f1f403c41ee..01635e18a43b 100644
--- a/include/fmt/std.h
+++ b/include/fmt/std.h
@@ -8,7 +8,7 @@
 #ifndef FMT_STD_H_
 #define FMT_STD_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <atomic>
 #  include <bitset>
 #  include <complex>
@@ -29,7 +29,7 @@
 #  include <version>
 #endif
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 // Checking FMT_CPLUSPLUS for warning suppression in MSVC.
 #  if FMT_CPLUSPLUS >= 201703L
 #    if FMT_HAS_INCLUDE(<filesystem>)
diff --git a/include/fmt/xchar.h b/include/fmt/xchar.h
index 31bdccdb97ee..6c19bf3e722f 100644
--- a/include/fmt/xchar.h
+++ b/include/fmt/xchar.h
@@ -8,7 +8,7 @@
 #ifndef FMT_XCHAR_H_
 #define FMT_XCHAR_H_
 
-#if !defined(FMT_IMPORT_STD) && !defined(FMT_MODULE)
+#ifndef FMT_MODULE
 #  include <cwchar>
 #endif
 
@@ -16,8 +16,7 @@
 #include "format.h"
 #include "ranges.h"
 
-#if !defined(FMT_STATIC_THOUSANDS_SEPARATOR) && !defined(FMT_IMPORT_STD) && \
-    !defined(FMT_MODULE)
+#if !defined(FMT_STATIC_THOUSANDS_SEPARATOR) && !defined(FMT_MODULE)
 #  include <locale>
 #endif
 
diff --git a/src/fmt.cc b/src/fmt.cc
index 108f15312b3b..28f4bb821ad8 100644
--- a/src/fmt.cc
+++ b/src/fmt.cc
@@ -41,6 +41,8 @@ module;
 #  include <stdint.h>
 #  include <stdio.h>
 #  include <time.h>
+
+import std;
 #endif
 #include <cerrno>
 #include <climits>
diff --git a/src/os.cc b/src/os.cc
index 81177b263e48..7bc18e2a8e51 100644
--- a/src/os.cc
+++ b/src/os.cc
@@ -12,47 +12,49 @@
 
 #include "fmt/os.h"
 
-#include <climits>
+#ifndef FMT_MODULE
+#  include <climits>
 
-#if FMT_USE_FCNTL
-#  include <sys/stat.h>
-#  include <sys/types.h>
-
-#  ifdef _WRS_KERNEL    // VxWorks7 kernel
-#    include <ioLib.h>  // getpagesize
-#  endif
+#  if FMT_USE_FCNTL
+#    include <sys/stat.h>
+#    include <sys/types.h>
 
-#  ifndef _WIN32
-#    include <unistd.h>
-#  else
-#    ifndef WIN32_LEAN_AND_MEAN
-#      define WIN32_LEAN_AND_MEAN
+#    ifdef _WRS_KERNEL    // VxWorks7 kernel
+#      include <ioLib.h>  // getpagesize
 #    endif
-#    include <io.h>
 
-#    ifndef S_IRUSR
-#      define S_IRUSR _S_IREAD
-#    endif
-#    ifndef S_IWUSR
-#      define S_IWUSR _S_IWRITE
-#    endif
-#    ifndef S_IRGRP
-#      define S_IRGRP 0
-#    endif
-#    ifndef S_IWGRP
-#      define S_IWGRP 0
-#    endif
-#    ifndef S_IROTH
-#      define S_IROTH 0
-#    endif
-#    ifndef S_IWOTH
-#      define S_IWOTH 0
-#    endif
-#  endif  // _WIN32
-#endif    // FMT_USE_FCNTL
+#    ifndef _WIN32
+#      include <unistd.h>
+#    else
+#      ifndef WIN32_LEAN_AND_MEAN
+#        define WIN32_LEAN_AND_MEAN
+#      endif
+#      include <io.h>
 
-#ifdef _WIN32
-#  include <windows.h>
+#      ifndef S_IRUSR
+#        define S_IRUSR _S_IREAD
+#      endif
+#      ifndef S_IWUSR
+#        define S_IWUSR _S_IWRITE
+#      endif
+#      ifndef S_IRGRP
+#        define S_IRGRP 0
+#      endif
+#      ifndef S_IWGRP
+#        define S_IWGRP 0
+#      endif
+#      ifndef S_IROTH
+#        define S_IROTH 0
+#      endif
+#      ifndef S_IWOTH
+#        define S_IWOTH 0
+#      endif
+#    endif  // _WIN32
+#  endif    // FMT_USE_FCNTL
+
+#  ifdef _WIN32
+#    include <windows.h>
+#  endif
 #endif
 
 namespace {