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.)

Author: BenWiederhake

AK/Bitmap.h

@@ -37,7 +37,8 @@
 namespace AK {
 
 class Bitmap {
-    AK_MAKE_NONCOPYABLE(Bitmap)
+    AK_MAKE_NONCOPYABLE(Bitmap);
+
 public:
     // NOTE: A wrapping Bitmap won't try to free the wrapped data.
     static Bitmap wrap(u8* data, size_t size)

AK/NeverDestroyed.h

@@ -33,8 +33,9 @@ namespace AK {
 
 template<typename T>
 class NeverDestroyed {
-    AK_MAKE_NONCOPYABLE(NeverDestroyed)
-    AK_MAKE_NONMOVABLE(NeverDestroyed)
+    AK_MAKE_NONCOPYABLE(NeverDestroyed);
+    AK_MAKE_NONMOVABLE(NeverDestroyed);
+
 public:
     template<typename... Args>
     NeverDestroyed(Args... args)

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

AK/RefCounted.h

@@ -60,8 +60,9 @@ constexpr auto call_one_ref_left_if_present(...) -> FalseType
 }
 
 class RefCountedBase {
-    AK_MAKE_NONCOPYABLE(RefCountedBase)
-    AK_MAKE_NONMOVABLE(RefCountedBase)
+    AK_MAKE_NONCOPYABLE(RefCountedBase);
+    AK_MAKE_NONMOVABLE(RefCountedBase);
+
 public:
     typedef unsigned int RefCountType;
 

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();

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();

DevTools/HackStudio/Project.h

@@ -42,8 +42,9 @@ enum class ProjectType {
 };
 
 class Project {
-    AK_MAKE_NONCOPYABLE(Project)
-    AK_MAKE_NONMOVABLE(Project)
+    AK_MAKE_NONCOPYABLE(Project);
+    AK_MAKE_NONMOVABLE(Project);
+
 public:
     ~Project();
 

DevTools/HackStudio/Tool.h

@@ -34,8 +34,9 @@ namespace HackStudio {
 class FormEditorWidget;
 
 class Tool {
-    AK_MAKE_NONCOPYABLE(Tool)
-    AK_MAKE_NONMOVABLE(Tool)
+    AK_MAKE_NONCOPYABLE(Tool);
+    AK_MAKE_NONMOVABLE(Tool);
+
 public:
     virtual ~Tool() { }
 

Games/Minesweeper/Field.h

@@ -35,7 +35,8 @@ class SquareButton;
 class SquareLabel;
 
 class Square {
-    AK_MAKE_NONCOPYABLE(Square)
+    AK_MAKE_NONCOPYABLE(Square);
+
 public:
     Square();
     ~Square();
@@ -59,6 +60,7 @@ class Field final : public GUI::Frame {
     C_OBJECT(Field)
     friend class Square;
     friend class SquareLabel;
+
 public:
     Field(GUI::Label& flag_label, GUI::Label& time_label, GUI::Button& face_button, Function<void(Gfx::IntSize)> on_size_changed);
     virtual ~Field() override;

Libraries/LibC/stdlib.cpp

@@ -85,7 +85,7 @@ enum DigitConsumeDecision {
 
 template<typename T, T min_value, T max_value>
 class NumParser {
-    AK_MAKE_NONMOVABLE(NumParser)
+    AK_MAKE_NONMOVABLE(NumParser);
 
 public:
     NumParser(Sign sign, int base)