Skip to content
Snippets Groups Projects
Commit ac8cba05 authored by Steffen Müthing's avatar Steffen Müthing
Browse files

[!340] Changed hasBackupRestoreFacilities for GeometryGrid [bugfix]

Merge branch 'issue/geometrygrid_backuprestore_coordfunction' into 'master'

ref:core/dune-grid

### Summary

Changes GeometryGrid capabilities to be true for hasBackupRestoreFacilities
only if the CoordFunction is default constructible

### Details

Instantiating the BackupRestoreFacilities::restore for
`GeometryGrid<HostGrid, CoordFunction>` with a CoordFunction that is not
default constructible leads to an error. This is in contrast to the capability
hasBackupRestoreFacilities that returns true for this grid.

This MR updates the capabilities to be false if the CoordsFunction is not
default constructible and adds a tag-dispatching with a runtime-error in case
someone instantiates the BackupRestoreFacilities with such a CoordsFunction
nevertheless.

So, the BackupRestoreFacilities can be instantiated without a compile-time
error and one can use the hasBackupRestoreFacilities for a compile-time check
whether this should work or not.

### See Also

Some discussions about how to restore a non-default-constructible
CoordFunction can be found in [!339]

See merge request [!340]

  [!339]: gitlab.dune-project.org/NoneNone/merge_requests/339
  [!340]: gitlab.dune-project.org/core/dune-grid/merge_requests/340
parents 797a1846 6b409e9d
Branches
Tags
Loading
Checking pipeline status
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment