From 7ad8a493163e3f543117830d7f91c08747a955ee Mon Sep 17 00:00:00 2001
From: Colin Finck <c.finck@enlyze.com>
Date: Thu, 30 Nov 2023 16:33:47 +0100
Subject: [PATCH] Write a Byte Order Mark (BOM) to the CSV file.

This indicates UTF-8 and thereby fixes special characters when opening the resulting CSV in Excel.
To my knowledge, it has no negative effects in any application that works with CSV files.

Fixes #4
---
 src/csv_exporter.cpp | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/csv_exporter.cpp b/src/csv_exporter.cpp
index 6c07ad8..c441431 100644
--- a/src/csv_exporter.cpp
+++ b/src/csv_exporter.cpp
@@ -60,11 +60,18 @@ ExportCSV(const std::wstring& wstrCSVFilePath, const std::vector<S7DeviceSymbolI
         return CS7PError(L"CreateFileW failed with error " + std::to_wstring(GetLastError()));
     }
 
-    // Write the CSV output.
+    // Write the BOM to indicate UTF-8 output.
+    const unsigned char ByteOrderMark[] = { 0xEF, 0xBB, 0xBF };
     DWORD cbWritten;
+    if (!WriteFile(hFile.get(), ByteOrderMark, sizeof(ByteOrderMark), &cbWritten, nullptr))
+    {
+        return CS7PError(L"WriteFile failed for the ByteOrderMark with error " + std::to_wstring(GetLastError()));
+    }
+
+    // Write the CSV output.
     if (!WriteFile(hFile.get(), strCSV.c_str(), strCSV.size(), &cbWritten, nullptr))
     {
-        return CS7PError(L"WriteFile failed with error " + std::to_wstring(GetLastError()));
+        return CS7PError(L"WriteFile failed for the CSV content with error " + std::to_wstring(GetLastError()));
     }
 
     return std::monostate();