Skip to content

Conversation

hageboeck
Copy link
Contributor

@hageboeck hageboeck commented Aug 20, 2025

  • Add slides to explain value categories
  • Add one slide on the vector move pessimisation when move constructor isn't noexcept
  • Fix latex compilation

Fix #522

@hageboeck hageboeck self-assigned this Aug 20, 2025
@hageboeck hageboeck added the Advanced Content for the "advanced" course label Aug 20, 2025
@hageboeck hageboeck force-pushed the prepareAdvanced25 branch 2 times, most recently from ba68c70 to 9904d41 Compare October 9, 2025 13:34
@hageboeck hageboeck closed this Oct 9, 2025
Copy link
Contributor

@sponce sponce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice explanation. I like it.
On the other hand, we'll have a hard time to squeeze it into the already overloaded schedule of the advanced course. Maybe we'll skip parts ? Or we add officially 15mn to every day so that we are a bit more relaxed ?
anyway, beyond the scope of this MR.

void fVal(T value); fVal(a); // by value
void fRef(const T &value); fRef(a); // by reference
void fPtr(const T *value); fPtr(|{\setlength{\fboxsep}{0pt}\color{gray}\colorbox{yellow}{\textsc{&}}}|a); // by pointer
void fPtr(const T *value); fPtr(|{\setlength{\fboxsep}{0pt}\color{gray}\colorbox{yellow}{\textsc{\&}}}|a); // by pointer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

somehow this & now appears on yellow background... no clue why

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was yellow also in the past. The idea was to give a visual aid because it's the only call that requires using the address-of operator.
If it's not needed, we can remove it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I found it weird, did not think it could be deliberate, I really thought it was a bug. So yes, I would remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, removed now.

@hageboeck hageboeck reopened this Oct 13, 2025
Underline why move constructors must be noexcept using the example of a
vector that needs to reallocate.
When colour or other formatting is used in minted with "escapeinside", "&" needs
to be escaped depending on the package versions.
For one instance, the colour box isn't deemed necessary, any more, so it
was removed.
@hageboeck hageboeck merged commit 078bb61 into hsf-training:master Oct 15, 2025
3 of 4 checks passed
@hageboeck hageboeck deleted the prepareAdvanced25 branch October 15, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Advanced Content for the "advanced" course

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add slide on value categories

2 participants