Skip to content
Snippets Groups Projects

Warning cleaning

Merged Guillaume Pasero requested to merge wrg_cleaning into develop
All threads resolved!

Ninja (or CTest) stops reporting warnings and errors after a given limit so we need some cleaning. There are 50+ warning so far.

I have bumped diapotb SHA1 to get warning fixes in this remote module as well.

Edited by Guillaume Pasero

Merge request reports

Pipeline #3288 passed

Pipeline passed for 57d3ae07 on wrg_cleaning

Merged by Cédric TraizetCédric Traizet 5 years ago (Jan 9, 2020 8:52am UTC)

Loading

Pipeline #3294 passed

Pipeline passed for e472b712 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Guillaume Pasero added 2 commits

    added 2 commits

    • 88166cb2 - WRG: pessimizing-move, fix noexcept
    • 4c5b2b77 - WRG: nonnull-compare

    Compare with previous version

  • added 1 commit

    • 28f15fc8 - COMP: remove noexcept in header

    Compare with previous version

  • Guillaume Pasero added 2 commits

    added 2 commits

    • bd84be52 - WRG: try warning fixes from diapotb
    • aaf867d2 - WRG: try to adapt diagnostics with compiler versions

    Compare with previous version

  • added 1 commit

    • f86639d7 - WRG: use expansion-to-defined warning on apple only if AppleClang version is > 9.0.0

    Compare with previous version

  • Cédric Traizet added 2 commits

    added 2 commits

    • 1a2e5e2c - WRG: update git tags for SertitObject and GRM (warning fixes)
    • 72300de4 - CI: add Carole Amiot to the .mailmap file

    Compare with previous version

  • Should be merged to continue investigation on MultiImageFileWriter MR.

  • added 1 commit

    Compare with previous version

  • Cédric Traizet resolved all threads

    resolved all threads

  • Cédric Traizet added 2 commits

    added 2 commits

    Compare with previous version

  • added 1 commit

    • 57d3ae07 - BUG: remove DiapOTB folder, included by mistake

    Compare with previous version

  • Cédric Traizet mentioned in commit e472b712

    mentioned in commit e472b712

  • mentioned in issue #2015 (closed)

  • Please register or sign in to reply
    Loading