Try to fix issue with escape_json

Author: Deniskore

binlog_json_parser/CMakeLists.txt

@@ -5,9 +5,18 @@ project(binlog_json_parser)
 set(CMAKE_CXX_STANDARD 23)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
+include(CheckIPOSupported)
+check_ipo_supported(RESULT IPO_SUPPORTED OUTPUT IPO_OUTPUT)
+
+if(IPO_SUPPORTED)
+    set(CMAKE_INTERPROCEDURAL_OPTIMIZATION TRUE)
+    message(STATUS "Interprocedural optimization (IPO/LTO) enabled globally.")
+else()
+    message(STATUS "IPO/LTO is not supported: ${IPO_OUTPUT}")
+endif()
+
 # Check if the build type is Release
 if(CMAKE_BUILD_TYPE STREQUAL "Release")
-
     # Set optimization level to -O3 for release builds
     if(NOT CMAKE_CXX_FLAGS_RELEASE MATCHES "-O")
         set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -O3")
@@ -32,24 +41,9 @@ if(CMAKE_BUILD_TYPE STREQUAL "Release")
         message(WARNING "The -march option will not be set because the system is not x86 or x64.")
     endif()
 
-    # Check for LTO support
-    include(CheckCXXCompilerFlag)
-
-    check_cxx_compiler_flag("-flto" COMPILER_SUPPORTS_LTO)
-
-    if(COMPILER_SUPPORTS_LTO)
-        message(STATUS "Link Time Optimization (LTO) is supported by the compiler.")
-        set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -flto")
-        set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -flto")
-        set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -flto")
-    else()
-        message(WARNING "Link Time Optimization (LTO) is not supported by the compiler.")
-    endif()
-
     # Export compile flags to a file
     file(WRITE "${CMAKE_BINARY_DIR}/compile_flags.txt" "CXXFLAGS: ${CMAKE_CXX_FLAGS_RELEASE}\n")
     file(APPEND "${CMAKE_BINARY_DIR}/compile_flags.txt" "LINKER_FLAGS: ${CMAKE_EXE_LINKER_FLAGS}\n")
-
 endif()
 
 add_library(mysqljsonparse SHARED mysqljsonparse.cpp mysql_json_parser.cpp)

binlog_json_parser/mysql_json_parser.cpp

@@ -120,24 +120,28 @@ static bool read_variable_length(const char *data, size_t data_length,
 
 std::string escape_json(const std::string &s) {
   std::ostringstream o;
-  for (auto c = s.cbegin(); c != s.cend(); c++) {
-    switch (*c) {
-      case '"': o << "\\\""; break;
+
+  for (const unsigned char uc : s) {
+    switch (uc) {
+      case '"':  o << "\\\""; break;
       case '\\': o << "\\\\"; break;
-      case '\b': o << "\\b"; break;
-      case '\f': o << "\\f"; break;
-      case '\n': o << "\\n"; break;
-      case '\r': o << "\\r"; break;
-      case '\t': o << "\\t"; break;
+      case '\b': o << "\\b";   break;
+      case '\f': o << "\\f";   break;
+      case '\n': o << "\\n";   break;
+      case '\r': o << "\\r";   break;
+      case '\t': o << "\\t";   break;
+
       default:
-        if (*c <= '\x1f') {
+        if (uc <= 0x1F) {
           o << "\\u"
-            << std::hex << std::setw(4) << std::setfill('0') << static_cast<int>(*c);
+            << std::hex << std::setw(4) << std::setfill('0') << static_cast<int>(uc)
+            << std::dec;
         } else {
-          o << *c;
+          o << static_cast<char>(uc);
         }
     }
   }
+
   return o.str();
 }