Bugfix: using iosfwd instead of predefines to avoid issues on macos

Author: sunxfancy

.vscode/settings.json

@@ -97,7 +97,24 @@
         "span": "cpp",
         "typeindex": "cpp",
         "valarray": "cpp",
-        "regex": "cpp"
+        "regex": "cpp",
+        "__config": "cpp",
+        "__bit_reference": "cpp",
+        "__bits": "cpp",
+        "__debug": "cpp",
+        "__errc": "cpp",
+        "__hash_table": "cpp",
+        "__locale": "cpp",
+        "__mutex_base": "cpp",
+        "__node_handle": "cpp",
+        "__nullptr": "cpp",
+        "__split_buffer": "cpp",
+        "__string": "cpp",
+        "__threading_support": "cpp",
+        "__tree": "cpp",
+        "__tuple": "cpp",
+        "stack": "cpp",
+        "__functional_base": "cpp"
     },
     "cmake.buildDirectory": "${workspaceFolder}/build/windows"
 }

Makefile

@@ -11,14 +11,23 @@ build/linux/Makefile: Makefile
 linux: build/linux/Makefile
 	cmake --build build/linux -j `nproc`
 
-build/windows/ZeroErr.sln:
+build/windows/ZeroErr.sln: Makefile
 	mkdir -p build/windows
 	cmake.exe -B build/windows -S . \
 		-DBUILD_EXAMPLES=ON -DBUILD_TEST=ON -DDISABLE_CUDA_BUILD=OFF -T host=x64 -A x64
 
 windows: build/windows/ZeroErr.sln
 	cmake.exe --build build/windows --config Debug -j `nproc`
 
+build/macosx/Makefile: Makefile
+	mkdir -p build/macosx
+	cmake -B build/macosx -S . -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_STANDARD=11 \
+		-DBUILD_EXAMPLES=ON -DBUILD_TEST=ON -DUSE_MOLD=ON -DDISABLE_CUDA_BUILD=OFF -DENABLE_FUZZING=OFF \
+		-DCMAKE_CXX_COMPILER=clang++ -DCMAKE_C_COMPILER=clang
+
+macosx: build/macosx/Makefile
+	cmake --build build/macosx -j 4
+
 test: linux-test windows-test fuzz-test
 
 cuda-test: windows
@@ -39,6 +48,8 @@ reporter: linux
 windows-test: windows
 	cd build/windows/test && ../examples/Debug/2_log.exe --testcase="parsing test"
 
+macosx-test: macosx
+	cd build/macosx/test && ./unittest
 
 build/linux-release/Makefile: Makefile
 	mkdir -p build/linux-release

include/zeroerr/internal/debugbreak.h

@@ -175,6 +175,9 @@ __attribute__((always_inline)) __inline__ static bool isDebuggerActive() {
     return false;
 }
 #elif defined(__APPLE__)
+#include <sys/sysctl.h>
+#include <unistd.h>
+#include <iostream>
 // The following function is taken directly from the following technical note:
 // https://developer.apple.com/library/archive/qa/qa1361/_index.html
 // Returns true if the current process is being debugged (either

include/zeroerr/internal/typetraits.h

@@ -6,25 +6,10 @@
 #include <string>
 #include <tuple>  // this should be removed
 #include <type_traits>
+#include <iosfwd>
 
 ZEROERR_SUPPRESS_COMMON_WARNINGS_PUSH
 
-// those predefines can help to avoid include too many headers
-namespace std {
-template <typename T>
-class complex;
-
-template <class T, class Deleter>
-class unique_ptr;
-
-template <class T>
-class shared_ptr;
-
-template <class T>
-class weak_ptr;
-
-}  // namespace std
-
 
 namespace zeroerr {
 

include/zeroerr/log.h

@@ -12,15 +12,12 @@
 #include <map>
 #include <string>
 #include <vector>
+#include <iosfwd>
 
 ZEROERR_SUPPRESS_COMMON_WARNINGS_PUSH
 
 extern const char* ZEROERR_LOG_CATEGORY;
 
-namespace std {
-class mutex;
-}
-
 namespace zeroerr {
 
 // clang-format off

include/zeroerr/print.h

@@ -3,7 +3,7 @@
 
 #include "zeroerr/color.h"
 #include "zeroerr/internal/typetraits.h"
-
+#include <complex>
 
 #ifdef __GNUG__
 #include <cxxabi.h>

zeroerr.hpp

@@ -643,6 +643,9 @@ __attribute__((always_inline)) __inline__ static bool isDebuggerActive() {
     return false;
 }
 #elif defined(__APPLE__)
+#include <sys/sysctl.h>
+#include <unistd.h>
+#include <iostream>
 // The following function is taken directly from the following technical note:
 // https://developer.apple.com/library/archive/qa/qa1361/_index.html
 // Returns true if the current process is being debugged (either
@@ -724,25 +727,11 @@ __attribute__((always_inline)) __inline__ static bool isDebuggerActive() { retur
 #include <string>
 #include <tuple>  // this should be removed
 #include <type_traits>
+#include <iosfwd>
 
 ZEROERR_SUPPRESS_COMMON_WARNINGS_PUSH
 
 // those predefines can help to avoid include too many headers
-namespace std {
-template <typename T>
-class complex;
-
-template <class T, class Deleter>
-class unique_ptr;
-
-template <class T>
-class shared_ptr;
-
-template <class T>
-class weak_ptr;
-
-}  // namespace std
-
 
 namespace zeroerr {
 
@@ -1302,7 +1291,7 @@ ZEROERR_SUPPRESS_COMMON_WARNINGS_POP
 
 
 
-
+#include <complex>
 
 #ifdef __GNUG__
 #include <cxxabi.h>
@@ -3402,15 +3391,12 @@ std::string format(const char* fmt, T... args) {
 #include <map>
 #include <string>
 #include <vector>
+#include <iosfwd>
 
 ZEROERR_SUPPRESS_COMMON_WARNINGS_PUSH
 
 extern const char* ZEROERR_LOG_CATEGORY;
 
-namespace std {
-class mutex;
-}
-
 namespace zeroerr {
 
 // clang-format off