Mentions légales du service

Skip to content
Snippets Groups Projects

Refactoring upon hands on

  • All solutions have been reworked so that declarations and definitions are separated.
  • Two optional exercises that were problematic have been removed (#125 (closed))
  • 3 new exercises have been added (one is the reworking of one deleted in #125 (closed))

@jediaz I am not sure I have fixed all remarks stemming from previous MRs; I'll check that tomorrow.

@vrouvrea I put @jediaz as reviewer but feel free to chime in; in peculiar I would like to make sure you're ok with new exercises (34, 38 and 43).

Closes #124 (closed), #125 (closed)

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
  • DIAZ Jerome
  • DIAZ Jerome
  • DIAZ Jerome
  • DIAZ Jerome
  • DIAZ Jerome
  • DIAZ Jerome
  • GILLES Sebastien added 4 commits
  • DIAZ Jerome
  • GILLES Sebastien added 4 commits

    added 4 commits

    • 9d72ad63 - Hands-on solutions: add a comment about noreturn field.
    • 9c53705b - #124 (closed) Following Vincent suggestion, use more modern way to silence warnings about unused variables.
    • b7a88524 - #124 (closed) Exercice 44: add missing include, use the proper main, and fix newline warning.
    • d7b65e86 - #124 (closed) Also replace the static_cast<void> in notebooks.

    Compare with previous version

  • DIAZ Jerome
  • added 1 commit

    Compare with previous version

  • DIAZ Jerome
  • DIAZ Jerome
  • DIAZ Jerome
  • GILLES Sebastien added 3 commits

    added 3 commits

    • b4f877f8 - Fix wrong argument numbering in solution, and add missing quote.
    • 3164cdb0 - Add Doxygen comment for a constructor parameter.
    • 76c9e93c - Revert commit 4e35d963 that was ill advised:...

    Compare with previous version

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