Skip to content
Snippets Groups Projects

Move include of config.h from source to some header files

Merged Simon Praetorius requested to merge feature/move-config-h-to-headers into master

Summary

Remove includes of config.h from most source files and add a corresponding include of dune-common-config.hh to some header files

With !1262 (merged) we can remove the include of config.h in all source files. Instead, we include the public config file dune-common-config.hh in those header files that depend on some config dependent preprocessor defines. We have already reduced these defines. Now only optional-dependencies that cannot be activated later are included.

ToDo

  • Check that only those header files that really need the config.hh file to be included (depend on a config variable introduced in a config file) actually are included.
  • Check that source files include private config files if additional variables are checked.
  • Needs !1314 (merged) to be merged before.
Edited by Simon Praetorius

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading