Resolve "Add missing ENABLE_AMR"
All threads resolved!
All threads resolved!
This MR does:
- add missing ENABLE_AMR preprocessor directive
- add two missing commens on AMR options
Closes #528 (closed)
Edited by frey_m
Merge request reports
Activity
Filter activity
assigned to @frey_m
- Resolved by frey_m
@frey_m : why do have
Options::amr
only ifENABLE_AMR=TRUE
?Would it not be easier to have this bool always available and set it to false if
ENABLE_AMR
is not defined and true by default otherwise?Also can you add a description of the bool in Options.h?
Edited by snuverink_j
mentioned in commit 66b6ab16
mentioned in issue #529 (closed)
Please register or sign in to reply