Add VA_ARGS expansion for Clang support. Remove user-defined copy constructor (implicitly defined, unnecessary and destroys the implicit copy operator when compiling with pedantic warnings on newer compilers.

Author: ninetailedtori

src/include/millennium/backend_init.h

@@ -83,9 +83,6 @@ class BackendCallbacks : public Singleton<BackendCallbacks>
         PluginTypeSchema(const std::string& name, eBackendLoadEvents ev) : pluginName(name), event(ev)
         {
         }
-        PluginTypeSchema(const PluginTypeSchema& other) : pluginName(other.pluginName), event(other.event)
-        {
-        }
     };
 
     using EventCallback = std::function<void()>;

src/include/millennium/logger.h

@@ -102,5 +102,16 @@ extern OutputLogger Logger;
 #endif
 
 #ifndef LOG_ERROR
-#define LOG_ERROR(fmt, ...) Logger.ErrorTrace(fmt, __FILE__, __LINE__, PRETTY_FUNCTION, ##__VA_ARGS__)
+#define LOG_ERROR(...) Logger.ErrorTrace(FIRST(__VA_ARGS__), __FILE__, __LINE__, PRETTY_FUNCTION REST(__VA_ARGS__))
+#define FIRST(...) FIRST_HELPER(__VA_ARGS__, throwaway)
+#define FIRST_HELPER(first, ...) first
+#define REST(...) REST_HELPER(NUM(__VA_ARGS__), __VA_ARGS__)
+#define REST_HELPER(qty, ...) REST_HELPER2(qty, __VA_ARGS__)
+#define REST_HELPER2(qty, ...) REST_HELPER_##qty(__VA_ARGS__)
+#define REST_HELPER_ONE(first)
+#define REST_HELPER_TWOORMORE(first, ...) , __VA_ARGS__
+#define NUM(...) \
+    SELECT_10TH(__VA_ARGS__, TWOORMORE, TWOORMORE, TWOORMORE, TWOORMORE,\
+        TWOORMORE, TWOORMORE, TWOORMORE, TWOORMORE, ONE, throwaway)
+#define SELECT_10TH(a1, a2, a3, a4, a5, a6, a7, a8, a9, a10, ...) a10
 #endif