Skip to content

Export API routines for MSVC compiler#176

Merged
TyBalduf merged 1 commit intodftd4:mainfrom
TyBalduf:msvc
Oct 12, 2022
Merged

Export API routines for MSVC compiler#176
TyBalduf merged 1 commit intodftd4:mainfrom
TyBalduf:msvc

Conversation

@TyBalduf
Copy link
Copy Markdown
Collaborator

  • Exports must be explictly declared for the MSVC compiler. This commit adds DLLEXPORT attributes to the subroutines needed for the api and app to compile

Signed-off-by: Ty [email protected]

This still the same pull request as #168. Had to remake due to clunky choice of branching

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 11, 2022

Codecov Report

Merging #176 (adaed32) into main (3e4ea97) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #176   +/-   ##
=======================================
  Coverage   60.12%   60.12%           
=======================================
  Files          44       44           
  Lines        3130     3130           
  Branches     1019     1019           
=======================================
  Hits         1882     1882           
  Misses        667      667           
  Partials      581      581           
Impacted Files Coverage Δ
src/dftd4/api.f90 40.67% <ø> (ø)
src/dftd4/charge.f90 64.28% <ø> (ø)
src/dftd4/cutoff.f90 51.02% <ø> (ø)
src/dftd4/damping/rational.f90 69.04% <ø> (ø)
src/dftd4/data/covrad.f90 37.50% <ø> (ø)
src/dftd4/data/en.f90 37.50% <ø> (ø)
src/dftd4/disp.f90 64.10% <ø> (ø)
src/dftd4/model.f90 79.52% <ø> (ø)
src/dftd4/ncoord.f90 88.00% <ø> (ø)
src/dftd4/output.f90 51.39% <ø> (ø)
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

* Exports must be explictly declared for the MSVC compiler.
This commit adds DLLEXPORT attributes to the subroutines needed for
the api and app to compile
@TyBalduf TyBalduf marked this pull request as ready for review October 12, 2022 15:06
@TyBalduf TyBalduf merged commit 900ec73 into dftd4:main Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant