Skip to content

Commit 52c5e07

Browse files
authored
Merge pull request commonmark#289 from eli-schwartz/exportfile-gfm
extensions: avoid useless duplication of headers
2 parents 9d8ebd6 + 32c8fe6 commit 52c5e07

File tree

2 files changed

+12
-21
lines changed

2 files changed

+12
-21
lines changed

extensions/CMakeLists.txt

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ include_directories(
1717
${PROJECT_BINARY_DIR}/src
1818
)
1919

20-
include (GenerateExportHeader)
21-
2220
include_directories(. ${CMAKE_CURRENT_BINARY_DIR})
2321

2422
set(CMAKE_C_FLAGS_PROFILE "${CMAKE_C_FLAGS_RELEASE} -pg")
@@ -29,6 +27,7 @@ if (CMARK_SHARED)
2927

3028
set_target_properties(${LIBRARY} PROPERTIES
3129
OUTPUT_NAME "cmark-gfm-extensions"
30+
DEFINE_SYMBOL "cmark-gfm"
3231
SOVERSION ${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}.${PROJECT_VERSION_PATCH}.gfm.${PROJECT_VERSION_GFM}
3332
VERSION ${PROJECT_VERSION})
3433

@@ -38,9 +37,6 @@ if (CMARK_SHARED)
3837
# Avoid name clash between PROGRAM and LIBRARY pdb files.
3938
set_target_properties(${LIBRARY} PROPERTIES PDB_NAME cmark-gfm-extensions_dll)
4039

41-
generate_export_header(${LIBRARY}
42-
BASE_NAME cmark-gfm-extensions)
43-
4440
list(APPEND CMARK_INSTALL ${LIBRARY})
4541
target_link_libraries(${LIBRARY} libcmark-gfm)
4642

@@ -51,6 +47,7 @@ if (CMARK_STATIC)
5147

5248
set_target_properties(${STATICLIBRARY} PROPERTIES
5349
COMPILE_FLAGS "-DCMARK_GFM_STATIC_DEFINE -DCMARK_GFM_EXTENSIONS_STATIC_DEFINE"
50+
DEFINE_SYMBOL "cmark-gfm"
5451
POSITION_INDEPENDENT_CODE ON)
5552

5653
if (MSVC)
@@ -63,11 +60,6 @@ if (CMARK_STATIC)
6360
VERSION ${PROJECT_VERSION})
6461
endif(MSVC)
6562

66-
if (NOT CMARK_SHARED)
67-
generate_export_header(${STATICLIBRARY}
68-
BASE_NAME cmark-gfm-extensions)
69-
endif()
70-
7163
list(APPEND CMARK_INSTALL ${STATICLIBRARY})
7264
endif()
7365

@@ -84,7 +76,6 @@ install(TARGETS ${CMARK_INSTALL}
8476
if (CMARK_SHARED OR CMARK_STATIC)
8577
install(FILES
8678
cmark-gfm-core-extensions.h
87-
${CMAKE_CURRENT_BINARY_DIR}/cmark-gfm-extensions_export.h
8879
DESTINATION include
8980
)
9081

extensions/cmark-gfm-core-extensions.h

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,45 +6,45 @@ extern "C" {
66
#endif
77

88
#include "cmark-gfm-extension_api.h"
9-
#include "cmark-gfm-extensions_export.h"
9+
#include "cmark-gfm_export.h"
1010
#include <stdbool.h>
1111
#include <stdint.h>
1212

13-
CMARK_GFM_EXTENSIONS_EXPORT
13+
CMARK_GFM_EXPORT
1414
void cmark_gfm_core_extensions_ensure_registered(void);
1515

16-
CMARK_GFM_EXTENSIONS_EXPORT
16+
CMARK_GFM_EXPORT
1717
uint16_t cmark_gfm_extensions_get_table_columns(cmark_node *node);
1818

1919
/** Sets the number of columns for the table, returning 1 on success and 0 on error.
2020
*/
21-
CMARK_GFM_EXTENSIONS_EXPORT
21+
CMARK_GFM_EXPORT
2222
int cmark_gfm_extensions_set_table_columns(cmark_node *node, uint16_t n_columns);
2323

24-
CMARK_GFM_EXTENSIONS_EXPORT
24+
CMARK_GFM_EXPORT
2525
uint8_t *cmark_gfm_extensions_get_table_alignments(cmark_node *node);
2626

2727
/** Sets the alignments for the table, returning 1 on success and 0 on error.
2828
*/
29-
CMARK_GFM_EXTENSIONS_EXPORT
29+
CMARK_GFM_EXPORT
3030
int cmark_gfm_extensions_set_table_alignments(cmark_node *node, uint16_t ncols, uint8_t *alignments);
3131

32-
CMARK_GFM_EXTENSIONS_EXPORT
32+
CMARK_GFM_EXPORT
3333
int cmark_gfm_extensions_get_table_row_is_header(cmark_node *node);
3434

3535
/** Sets whether the node is a table header row, returning 1 on success and 0 on error.
3636
*/
37-
CMARK_GFM_EXTENSIONS_EXPORT
37+
CMARK_GFM_EXPORT
3838
int cmark_gfm_extensions_set_table_row_is_header(cmark_node *node, int is_header);
3939

40-
CMARK_GFM_EXTENSIONS_EXPORT
40+
CMARK_GFM_EXPORT
4141
bool cmark_gfm_extensions_get_tasklist_item_checked(cmark_node *node);
4242
/* For backwards compatibility */
4343
#define cmark_gfm_extensions_tasklist_is_checked cmark_gfm_extensions_get_tasklist_item_checked
4444

4545
/** Sets whether a tasklist item is "checked" (completed), returning 1 on success and 0 on error.
4646
*/
47-
CMARK_GFM_EXTENSIONS_EXPORT
47+
CMARK_GFM_EXPORT
4848
int cmark_gfm_extensions_set_tasklist_item_checked(cmark_node *node, bool is_checked);
4949

5050
#ifdef __cplusplus

0 commit comments

Comments
 (0)