Make CHECK_SDL a function

Check correct type and avoid clang-tidy trying to "simplify" the
resulting boolean condition, i.e.
>  warning: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr]

Author: Flamefire

extras/videoDrivers/SDL2/VideoSDL2.cpp

@@ -32,11 +32,17 @@
 #    include <gl4esinit.h>
 #endif
 
+namespace {
+
 /// Check that the (SDL) call returns success or print the error
 /// Can be used in conditions: if(CHECK_SDL(SDL_Foo()))
-#define CHECK_SDL(call) ((call) >= 0 || (PrintError(), false))
-
-namespace {
+bool CHECK_SDL(int sdlResult)
+{
+    if(sdlResult >= 0)
+        return true;
+    VideoSDL2::PrintError();
+    return false;
+}
 
 template<typename T>
 struct SDLMemoryDeleter

extras/videoDrivers/SDL2/VideoSDL2.h

@@ -53,9 +53,10 @@ class VideoSDL2 final : public VideoDriver
     /// Get (device-dependent!) window pointer, HWND in Windows
     void* GetMapPointer() const override;
 
-private:
     static void PrintError();
     static void PrintError(const std::string& msg);
+
+private:
     void HandlePaste();
     void UpdateCurrentSizes();
     void MoveWindowToCenter();