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 /AK/Noncopyable.h | |
| 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 'AK/Noncopyable.h')
| -rw-r--r-- | AK/Noncopyable.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/AK/Noncopyable.h b/AK/Noncopyable.h index a68f42b7ca..cf9b672da6 100644 --- a/AK/Noncopyable.h +++ b/AK/Noncopyable.h @@ -29,9 +29,9 @@ #define AK_MAKE_NONCOPYABLE(c) \ private: \ c(const c&) = delete; \ - c& operator=(const c&) = delete; + c& operator=(const c&) = delete #define AK_MAKE_NONMOVABLE(c) \ -private: \ - c(c&&) = delete; \ - c& operator=(c&&) = delete; +private: \ + c(c&&) = delete; \ + c& operator=(c&&) = delete |