diff options
| author | Ben Wiederhake <BenWiederhake.GitHub@gmx.de> | 2020-08-26 21:52:24 +0200 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2020-08-27 10:12:04 +0200 |
| commit | 9f7ec331803917c9e73041012b5e4b568ab1198a (patch) | |
| tree | e275f60fa86a48a9a87a8e9e2bfc35f7bd21b2c7 /Applications/Piano | |
| parent | 6454969d6bf855aa8a37d6b2b1a5320951196eac (diff) | |
| download | serenity-9f7ec331803917c9e73041012b5e4b568ab1198a.zip | |
Meta: Force semi-colon after MAKE_AK_NONXXXABLE()
Before, we had about these occurrence counts:
COPY: 13 without, 33 with
MOVE: 12 without, 28 with
Clearly, 'with' was the preferred way. However, this introduced double-semicolons
all over the place, and caused some warnings to trigger.
This patch *forces* the usage of a semi-colon when calling the macro,
by removing the semi-colon within the macro. (And thus also gets rid
of the double-semicolon.)
Diffstat (limited to 'Applications/Piano')
| -rw-r--r-- | Applications/Piano/Track.h | 5 | ||||
| -rw-r--r-- | Applications/Piano/TrackManager.h | 5 |
2 files changed, 6 insertions, 4 deletions
diff --git a/Applications/Piano/Track.h b/Applications/Piano/Track.h index 99f3e1f3ab..276e7e8bdb 100644 --- a/Applications/Piano/Track.h +++ b/Applications/Piano/Track.h @@ -36,8 +36,9 @@ typedef AK::SinglyLinkedListIterator<SinglyLinkedList<RollNote>, RollNote> RollIter; class Track { - AK_MAKE_NONCOPYABLE(Track) - AK_MAKE_NONMOVABLE(Track) + AK_MAKE_NONCOPYABLE(Track); + AK_MAKE_NONMOVABLE(Track); + public: explicit Track(const u32& time); ~Track(); diff --git a/Applications/Piano/TrackManager.h b/Applications/Piano/TrackManager.h index a04daee696..79331a5586 100644 --- a/Applications/Piano/TrackManager.h +++ b/Applications/Piano/TrackManager.h @@ -34,8 +34,9 @@ #include <AK/Vector.h> class TrackManager { - AK_MAKE_NONCOPYABLE(TrackManager) - AK_MAKE_NONMOVABLE(TrackManager) + AK_MAKE_NONCOPYABLE(TrackManager); + AK_MAKE_NONMOVABLE(TrackManager); + public: TrackManager(); ~TrackManager(); |