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
Activity
requested review from @jediaz
assigned to @sgilles
mentioned in merge request !95 (closed)
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
added 4 commits
- 017ecc3c - #124 (closed) Typo.
- 7ce6021d - #124 (closed) Typo.
- 5b66316e - #124 (closed) English mistake.
- 676f56f1 - #124 (closed) English.
Toggle commit list- Resolved by DIAZ Jerome
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.
Toggle commit list- Resolved by DIAZ Jerome
added 1 commit
- 4e35d963 - #124 (closed) Use accessor instead of direct data attribute.
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
- Resolved by DIAZ Jerome
Please register or sign in to reply