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
  • !133

[implicit] Remove the shouldWriteOutput check inside writeOutput

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Timo Koch requested to merge feature/remove-double-write-output-check into master Mar 24, 2016
  • Overview 0
  • Commits 1
  • Pipelines 0
  • Changes 1

When calling writeOutput one would expect that the output is actually written to file. Instead this might not have happened in the old version. This commit makes the implicit problem consistent with the sequential models. Before calling writeOutput one should ask if shouldWriteOutput is true. This is already consistently done all over dumux, so this commit removes the double checking.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feature/remove-double-write-output-check