Move the sync operation to deleter

src/engine/serialize.cpp

@@ -399,11 +399,6 @@ std::string_view ROStreamBuf::getStringView( const size_t size /* = 0 */ )
     return { reinterpret_cast<const char *>( strBeg ), static_cast<size_t>( strEnd - strBeg ) };
 }
 
-StreamFile::~StreamFile()
-{
-    close();
-}
-
 bool StreamFile::open( const std::string & fn, const std::string & mode )
 {
     _file.reset( std::fopen( fn.c_str(), mode.c_str() ) );
@@ -419,12 +414,6 @@ bool StreamFile::open( const std::string & fn, const std::string & mode )
 void StreamFile::close()
 {
     _file.reset();
-
-#ifdef __EMSCRIPTEN__
-    // clang-format off
-    EM_ASM( FS.syncfs( err => err && console.warn( "FS.syncfs() error:", err ) ) );
-    // clang-format on
-#endif
 }
 
 size_t StreamFile::size()
@@ -637,3 +626,16 @@ std::string StreamFile::getString( const size_t size /* = 0 */ )
 
     return { buf.begin(), std::find( buf.begin(), buf.end(), 0 ) };
 }
+
+int StreamFile::closeFile( std::FILE * f )
+{
+    const int res = std::fclose( f );
+
+#ifdef __EMSCRIPTEN__
+    // clang-format off
+    EM_ASM( FS.syncfs( err => err && console.warn( "FS.syncfs() error:", err ) ) );
+    // clang-format on
+#endif
+
+    return res;
+}

src/engine/serialize.h

@@ -597,7 +597,7 @@ class StreamFile final : public IStreamBase, public OStreamBase
 
     StreamFile( const StreamFile & ) = delete;
 
-    ~StreamFile() override;
+    ~StreamFile() = default;
 
     StreamFile & operator=( const StreamFile & ) = delete;
 
@@ -669,7 +669,9 @@ class StreamFile final : public IStreamBase, public OStreamBase
         }
     }
 
-    std::unique_ptr<std::FILE, int ( * )( std::FILE * )> _file{ nullptr, []( std::FILE * f ) { return std::fclose( f ); } };
+    static int closeFile( std::FILE * f );
+
+    std::unique_ptr<std::FILE, int ( * )( std::FILE * )> _file{ nullptr, closeFile };
 };
 
 namespace fheroes2