Skip to content
Snippets Groups Projects

move buffer initialization to buffer_create

Merged Martin Lueders requested to merge move_buffer_init into main
All threads resolved!

Description

Move the buffer initialization to the create_buffer routines. The initialization to zero is controlled by an optional argument.

News snippet

Move the buffer initialization to the create_buffer routines.

Checklist

  • I have checked that my code follows the Octopus coding standards
  • I have added tests for all the new features added in this request.

Merge request reports

Pipeline #720846829 passed

Pipeline passed for da3e2b7e on move_buffer_init

Merged by Nicolas Tancogne-DejeanNicolas Tancogne-Dejean 2 years ago (Dec 15, 2022 3:55pm UTC)

Loading

Pipeline #723729455 created

Pipeline created for 3b976ac6 on main

Deployed to ma‎in‎ 22 hours ago

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Martin Lueders marked this merge request as draft

    marked this merge request as draft

  • Martin Lueders added 1 commit

    added 1 commit

    • 9c5b8852 - introduced input option to force initialization

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • f0c5ccbc - test added (and fix temp. broken)

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • e3de33ad - test added (and fix temp. broken)

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • dbbe1cb5 - test added (and fix temp. broken)

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • 44b9b7da - test added (and fix temp. broken)

    Compare with previous version

  • Martin Lueders added 34 commits

    added 34 commits

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • e76245cc - changed number of processors for test

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    Compare with previous version

  • Sebastian Ohlmann
  • Sebastian Ohlmann requested review from @sohlmann

    requested review from @sohlmann

  • Martin Lueders added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • 136ac522 - renamed variable, and re-introduced bugfix

    Compare with previous version

  • Martin Lueders added 1 commit

    added 1 commit

    • da3e2b7e - fixed variable documentation

    Compare with previous version

  • Martin Lueders marked this merge request as ready

    marked this merge request as ready

  • @nicolastd As Sebastian is on paternal leave now, maybe you could have a look at this MR and approve when you are happy.

  • Nicolas Tancogne-Dejean approved this merge request

    approved this merge request

  • Nicolas Tancogne-Dejean resolved all threads

    resolved all threads

  • mentioned in commit 3b976ac6

  • @martin.lueders This all looked good to me so I merged it.

  • Please register or sign in to reply
    Loading