Skip to content

Commit

Permalink
fix: #10 no more template export header
Browse files Browse the repository at this point in the history
Template classes are included in every compilation unit and do not need symbol visibility to be specifically managed. The '_export.h' file auto-generated by cmake is enough.
  • Loading branch information
abdes committed Aug 20, 2022
1 parent 5a7416f commit dd8ffd5
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 86 deletions.
3 changes: 0 additions & 3 deletions cmake/BuildHelpers.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
# Build Helpers to simplify target creation.
# ------------------------------------------------------------------------------

include(GenerateTemplateExportHeader)
function(asap_generate_export_headers target include_dir)
# Set API export file and macro
string(MAKE_C_IDENTIFIER ${target} TEMPLATE_TARGET_ID)
Expand All @@ -21,6 +20,4 @@ function(asap_generate_export_headers target include_dir)
# Create API export headers
generate_export_header(${target} EXPORT_FILE_NAME ${export_file}
EXPORT_MACRO_NAME ${TEMPLATE_TARGET_ID}_API)
generate_template_export_header(${target} ${TEMPLATE_TARGET_ID}
${template_export_file})
endfunction()
25 changes: 0 additions & 25 deletions cmake/GenerateTemplateExportHeader.cmake

This file was deleted.

29 changes: 0 additions & 29 deletions templates/template_api.h.in

This file was deleted.

29 changes: 0 additions & 29 deletions templates/template_msvc_api.h.in

This file was deleted.

0 comments on commit dd8ffd5

Please sign in to comment.