maybe?

Signed-off-by: Rosen Penev <[email protected]>

Author: neheb

include/exiv2/basicio.hpp

@@ -310,6 +310,9 @@ class EXIV2API FileIo : public BasicIo {
         Nonzero if failure.
    */
   int open(const std::string& mode);
+#ifdef _WIN32
+  int open(const std::wstring& mode);
+#endif
   /*!
     @brief Open the file using the default access mode of "rb".
         This method can also be used to "reopen" a file which will flush

include/exiv2/image.hpp

@@ -582,6 +582,9 @@ class EXIV2API ImageFactory {
     @throw Error If the image type is not supported.
    */
   static Image::UniquePtr create(ImageType type, const std::string& path);
+#ifdef _WIN32
+  static Image::UniquePtr create(ImageType type, const std::wstring& path);
+#endif
   /*!
     @brief Create an Image subclass of the requested type by creating a
         new image in memory.

src/basicio.cpp

@@ -73,10 +73,11 @@ class FileIo::Impl {
   //! Mode of operation
   enum OpMode { opRead, opWrite, opSeek };
   // DATA
-  fs::path path_;          //!< (Standard) path
-  std::string openMode_;   //!< File open mode
-  FILE* fp_{};             //!< File stream pointer
-  OpMode opMode_{opSeek};  //!< File open mode
+  fs::path path_;           //!< (Standard) path
+  std::string openMode_;    //!< File open mode
+  std::wstring wOpenMode_;  //!< File open mode (wide)
+  FILE* fp_{};              //!< File stream pointer
+  OpMode opMode_{opSeek};   //!< File open mode
 
 #if defined _WIN32
   HANDLE hFile_{};  //!< Duplicated fd
@@ -118,6 +119,19 @@ int FileIo::Impl::switchMode(OpMode opMode) {
 
   bool reopen = true;
   switch (opMode) {
+#ifdef _WIN32
+    case opRead:
+      // Flush if current mode allows reading, else reopen (in mode "r+b"
+      // as in this case we know that we can write to the file)
+      if (wOpenMode_.front() == L'r' || wOpenMode_.at(1) == L'+')
+        reopen = false;
+      break;
+    case opWrite:
+      // Flush if current mode allows writing, else reopen
+      if (wOpenMode_.front() != L'r' || wOpenMode_.at(1) == L'+')
+        reopen = false;
+      break;
+#else
     case opRead:
       // Flush if current mode allows reading, else reopen (in mode "r+b"
       // as in this case we know that we can write to the file)
@@ -129,6 +143,7 @@ int FileIo::Impl::switchMode(OpMode opMode) {
       if (openMode_.front() != 'r' || openMode_.at(1) == '+')
         reopen = false;
       break;
+#endif
     case opSeek:
       reopen = false;
       break;
@@ -465,7 +480,11 @@ size_t FileIo::tell() const {
 
 size_t FileIo::size() const {
   // Flush and commit only if the file is open for writing
+#ifdef _WIN32
+  if (p_->fp_ && (p_->wOpenMode_.front() != L'r' || p_->wOpenMode_.at(1) == L'+')) {
+#else
   if (p_->fp_ && (p_->openMode_.front() != 'r' || p_->openMode_.at(1) == '+')) {
+#endif
     std::fflush(p_->fp_);
 #ifdef _MSC_VER
     // This is required on msvcrt before stat after writing to a file
@@ -481,26 +500,34 @@ size_t FileIo::size() const {
 
 int FileIo::open() {
   // Default open is in read-only binary mode
+#ifdef _WIN32
+  return open(L"rb");
+#else
   return open("rb");
+#endif
 }
 
 int FileIo::open(const std::string& mode) {
   close();
   p_->openMode_ = mode;
   p_->opMode_ = Impl::opSeek;
-#ifdef _WIN32
-  wchar_t wmode[10];
-  MultiByteToWideChar(CP_UTF8, 0, mode.c_str(), -1, wmode, 10);
-  if (_wfopen_s(&p_->fp_, p_->path_.c_str(), wmode))
-    return 1;
-#else
   p_->fp_ = ::fopen(path().c_str(), mode.c_str());
   if (!p_->fp_)
     return 1;
-#endif
   return 0;
 }
 
+#ifdef _WIN32
+int FileIo::open(const std::wstring& mode) {
+  close();
+  p_->wOpenMode_ = mode;
+  p_->opMode_ = Impl::opSeek;
+  if (_wfopen_s(&p_->fp_, p_->path_.c_str(), mode.c_str()))
+    return 1;
+  return 0;
+}
+#endif
+
 bool FileIo::isopen() const {
   return p_->fp_ != nullptr;
 }