build: hoist CMARK_THREADING to a build-only config

This macro does not impact the public API surface and the user should not have to care. This removes the macro from the configuration headers.
kkebo · Jan 5, 2024 · aef4761 · aef4761
1 parent c788cd0
commit aef4761
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -21,6 +21,7 @@ include(FindAsan)
 if(CMARK_THREADING)
   set(THREADS_PREFER_PTHREAD_FLAG YES)
   include(FindThreads)
+  add_compile_definitions(CMARK_THREADING)
 endif()
 include(GNUInstallDirs)
 

diff --git a/Package.swift b/Package.swift
@@ -11,9 +11,12 @@ import PackageDescription
     // link time.
     let cSettings: [CSetting] = [
         .define("CMARK_GFM_STATIC_DEFINE", .when(platforms: [.windows])),
+        .define("CMARK_THREADING"),
     ]
 #else
-    let cSettings: [CSetting] = []
+    let cSettings: [CSetting] = [
+        .define("CMARK_THREADING"),
+    ]
 #endif
 
 let package = Package(

diff --git a/src/config.h.in b/src/config.h.in
@@ -5,8 +5,6 @@
 extern "C" {
 #endif
 
-#cmakedefine CMARK_THREADING
-
 #ifdef __cplusplus
 }
 #endif

diff --git a/src/include/cmark-gfm_config.h b/src/include/cmark-gfm_config.h
@@ -12,8 +12,6 @@
 extern "C" {
 #endif
 
-#define CMARK_THREADING
-
 #ifdef __cplusplus
 }
 #endif
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,8 +5,6 @@ @@
     extern "C" {
     #endif
-    #cmakedefine CMARK_THREADING
     #ifdef __cplusplus
     }
     #endif
@@ Expand Down @@