Fix log macro doesn't work without format args; copy lib in Windows properly

Author: kzhdev

CHANGELOG

@@ -2,7 +2,8 @@
 - Change include folder structure from include/slick_socket/ to include/slick/
 - Change namespace from slick_socket to slick::socket
 - Add GitHub CI workflow
-- Fix MacOS build by using kqueue/kevent
+- Fix MacOS build
+- Fix example log macros doesn't work without format args
 
 #1.0.0.0 - [09/26/2025]
 - Initial Release

CMakeLists.txt

@@ -8,15 +8,15 @@ set(BUILD_VERSION 1.0.0.1)
 project(slick_socket VERSION ${BUILD_VERSION} LANGUAGES C CXX)
 
 if (NOT CMAKE_BUILD_TYPE)
-    set(CMAKE_BUILD_TYPE Release)
+  set(CMAKE_BUILD_TYPE Release)
 endif()
 
 if (CMAKE_BUILD_TYPE MATCHES Debug)
-    add_definitions(-DDEBUG)
+  add_definitions(-DDEBUG)
 endif()
 
 if (CMAKE_BUILD_TYPE MATCHES Release)
-    add_definitions(-DNDEBUG)
+  add_definitions(-DNDEBUG)
 endif()
 
 option(ENABLE_ASAN "Enable AddressSanitizer" OFF)
@@ -86,27 +86,36 @@ endif()
 
 # Automatically run install after build in Release mode
 if(CMAKE_BUILD_TYPE STREQUAL "Release")
-  add_custom_target(dist_slick_socket ALL
+  if (WIN32)
+    add_custom_target(dist_slick_socket ALL
       COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_CURRENT_SOURCE_DIR}/include ${CMAKE_BINARY_DIR}/dist/include
+      COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_BINARY_DIR}/lib/${CMAKE_BUILD_TYPE}/slick_socket.lib ${CMAKE_BINARY_DIR}/dist/lib/slick_socket.lib
       COMMENT "Copying slick_socket headers to dist/include"
       VERBATIM
-  )
+    )
+  else()
+    add_custom_target(dist_slick_socket ALL
+      COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_CURRENT_SOURCE_DIR}/include ${CMAKE_BINARY_DIR}/dist/include
+      COMMENT "Copying slick_socket headers to dist/include"
+      VERBATIM
+    )
+  endif()
 
   if (PROJECT_IS_TOP_LEVEL)
-      if (WIN32)
-          add_custom_target(package_slick_socket ALL
-              COMMAND ${CMAKE_COMMAND} -E tar "cfv" "${CMAKE_BINARY_DIR}/dist/slick_socket_${BUILD_VERSION}.zip" --format=zip "include" "lib"
-              WORKING_DIRECTORY "${CMAKE_BINARY_DIR}/dist"
-              COMMENT "Creating zip archive"
-          )
-      else()
-          add_custom_target(package_slick_socket ALL
-              COMMAND ${CMAKE_COMMAND} -E tar "cfv" "${CMAKE_BINARY_DIR}/dist/slick_socket_${BUILD_VERSION}.zip" --format=zip "include"
-              WORKING_DIRECTORY "${CMAKE_BINARY_DIR}/dist"
-              COMMENT "Creating zip archive"
-          )
-      endif()
-
-      add_dependencies(package_slick_socket dist_slick_socket)
+    if (WIN32)
+      add_custom_target(package_slick_socket ALL
+        COMMAND ${CMAKE_COMMAND} -E tar "cfv" "${CMAKE_BINARY_DIR}/dist/slick_socket_${BUILD_VERSION}.zip" --format=zip "include" "lib"
+        WORKING_DIRECTORY "${CMAKE_BINARY_DIR}/dist"
+        COMMENT "Creating zip archive"
+      )
+    else()
+      add_custom_target(package_slick_socket ALL
+        COMMAND ${CMAKE_COMMAND} -E tar "cfv" "${CMAKE_BINARY_DIR}/dist/slick_socket_${BUILD_VERSION}.zip" --format=zip "include"
+        WORKING_DIRECTORY "${CMAKE_BINARY_DIR}/dist"
+        COMMENT "Creating zip archive"
+      )
+    endif()
+
+    add_dependencies(package_slick_socket dist_slick_socket)
   endif()
 endif()

examples/logger.h

@@ -1,11 +1,25 @@
 #pragma once
 
+// NOTE: This is a simple example logger for demonstration purposes.
+// For production use, consider using a proper logging library like spdlog or slick_logger.
+
 #include <iostream>
 #include <chrono>
 #include <format>
 
-#define LOG_DEBUG(fmt, ...) std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now()) << "[DEBUG] " << std::format(fmt __VA_OPT__(,) __VA_ARGS__) << std::endl
-#define LOG_INFO(fmt, ...) std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now()) << "[INFO] " << std::format(fmt __VA_OPT__(,) __VA_ARGS__) << std::endl
-#define LOG_WARN(fmt, ...) std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now()) << "[WARNING] " << std::format(fmt __VA_OPT__(,) __VA_ARGS__) << std::endl
-#define LOG_ERROR(fmt, ...) std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now()) << "[ERROR] " << std::format(fmt __VA_OPT__(,) __VA_ARGS__) << std::endl
-#define LOG_TRACE(fmt, ...) std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now()) << "[TRACE] " << std::format(fmt __VA_OPT__(,) __VA_ARGS__) << std::endl
+namespace {
+    // Single function using vformat handles both cases: with and without arguments
+    template<typename... Args>
+    void log_impl(const std::string& level, const std::string& fmt, Args&&... args)
+    {
+        std::cout << std::format("{:%Y-%m-%d %H:%M:%S} ", std::chrono::system_clock::now())
+                  << "[" << level << "] " << std::vformat(fmt, std::make_format_args(args...)) << std::endl;
+    }
+}
+
+// Works with any number of arguments
+#define LOG_DEBUG(fmt, ...) log_impl("DEBUG", fmt, ##__VA_ARGS__)
+#define LOG_INFO(fmt, ...) log_impl("INFO", fmt, ##__VA_ARGS__)
+#define LOG_WARN(fmt, ...) log_impl("WARNING", fmt, ##__VA_ARGS__)
+#define LOG_ERROR(fmt, ...) log_impl("ERROR", fmt, ##__VA_ARGS__)
+#define LOG_TRACE(fmt, ...) log_impl("TRACE", fmt, ##__VA_ARGS__)