Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 100
    • Issues 100
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 92
    • Merge requests 92
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositoriesdumux-repositories
  • dumuxdumux
  • Merge requests
  • !1309

Use Dune::is_detected

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Kilian Weishaupt requested to merge feature/dune-is_detected into master Nov 21, 2018
  • Overview 12
  • Commits 1
  • Pipelines 0
  • Changes 1

fixes #602 (closed)

Dune2.6 already provides a fallback implementation of std::is_detected (C++17). We can use this to replace the rather lengthy and complicated isValid() method.

It is substantially less typing and easier to read. However, you need two helper structs instead of one (but each one being a one-liner).

Also seems to work with gcc5.5

Edited Nov 21, 2018 by Timo Koch
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feature/dune-is_detected