Skip to content

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

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