Skip to content
Snippets Groups Projects

[Fix] Memory leak due to containers serialization

Merged Maxence Naud requested to merge fix/GenericOp into standardization
  • [Add] minimal std::any implementation compatible with C++14 GenericOperator parameters are accessed by reference

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • The merge request is missing some comments to make the code easily readable, other than that great merge request good job @pineapple !

    I will validate once the comments are added ;)

  • Maxence Naud added 1 commit

    added 1 commit

    Compare with previous version

  • Cyril Moineau resolved all threads

    resolved all threads

  • lgtm :)

  • Cyril Moineau approved this merge request

    approved this merge request

  • Maxence Naud added 44 commits

    added 44 commits

    Compare with previous version

  • Maxence Naud changed target branch from main to standardization

    changed target branch from main to standardization

  • Maxence Naud added 6 commits

    added 6 commits

    • d8349d45 - Dropped usage of const char* for name, use std::string instead
    • 50c7e911 - Fixed using std::string for node name
    • 1408fcb5 - Fixed type error in printf
    • a0bf5e8d - Fixed default argument for name
    • 49c815ad - Make getNodes() return a const ref
    • 5f3ac9c8 - Merge remote-tracking branch 'origin/main' into fix/GenericOp

    Compare with previous version

  • Maxence Naud added 1 commit

    added 1 commit

    • 1786b36a - [Fix] Type call for GenericOperator parameters

    Compare with previous version

  • Maxence Naud added 15 commits

    added 15 commits

    • 1786b36a...6b2df066 - 14 commits from branch standardization
    • a5914815 - Merge remote-tracking branch 'origin/main' into fix/GenericOp

    Compare with previous version

  • Maxence Naud added 1 commit

    added 1 commit

    • 94ed8b34 - __or_ did not work with MSVC so changed for classical logical or (||)

    Compare with previous version

  • Maxence Naud added 1 commit

    added 1 commit

    • 936a5dc1 - explicitly add includes in CParameter.hpp

    Compare with previous version

  • Maxence Naud added 5 commits

    added 5 commits

    • 936a5dc1...ac37b61c - 4 commits from branch standardization
    • dde40b01 - Merge remote-tracking branch 'origin/standardization' into fix/GenericOp

    Compare with previous version

  • Maxence Naud enabled an automatic merge when the pipeline for dde40b01 succeeds

    enabled an automatic merge when the pipeline for dde40b01 succeeds

  • Maxence Naud mentioned in commit 84821386

    mentioned in commit 84821386

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading