Mentions légales du service

Skip to content
Snippets Groups Projects

HandOn: rename DisplayMultiply() DisplaySumOfMultiply()

@steff @vrouvrea

I've just begun to read the hands on (still in procedural...) but one of the modifications I have made is impactful for most hands on so I deemed it better to share it immediately.

There is a function which was called display_sum() which I renamed display_multiply()... which is not really better. I have renamed it DisplaySumOfMultiply(, which albeit ugly is correct. The notebook has been corrected as well (a display_sum slipped through and was unduly kept).

Others modifications are much more minor:

  • I heeded a warning from CMake and changed the type of the variable which indicated which C++ standard was used.
  • A typo was corrected.

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
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading