steinbergmedia / vstgui

A user interface toolkit mainly for audio plug-ins
Other
857 stars 122 forks source link

MouseDownUpMoveEvent::ignoreFollowUpMoveAndUpEvents() should be [[nodiscard]] #254

Closed sophiapoirier closed 2 years ago

sophiapoirier commented 2 years ago

bool MouseDownUpMoveEvent::ignoreFollowUpMoveAndUpEvents() should be marked with the attribute [[nodiscard]] given that its name, starting with a verb, can easily be interpreted as a mutating action rather than a const getter. Adding [[nodiscard]] will make such misuse produce a compiler warning.