From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Mon, 19 Sep 2005 23:34:07 +0000 (+0000)
Subject: only call fsync() on disk files, otherwise we get an error for pipes under Unix
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/608a34bf5b95059789cda694a9b4257e282648e7?ds=inline

only call fsync() on disk files, otherwise we get an error for pipes under Unix


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@35605 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/common/file.cpp b/src/common/file.cpp
index d72e649bc8..ae71d0ef12 100644
--- a/src/common/file.cpp
+++ b/src/common/file.cpp
@@ -334,17 +334,18 @@ size_t wxFile::Write(const void *pBuf, size_t nCount)
 // flush
 bool wxFile::Flush()
 {
-    if ( IsOpened() ) {
 #if defined(__VISUALC__) || defined(HAVE_FSYNC)
+    // fsync() only works on disk files and returns errors for pipes, don't
+    // call it then
+    if ( IsOpened() && GetKind() == wxFILE_KIND_DISK )
+    {
         if ( wxFsync(m_fd) == -1 )
         {
             wxLogSysError(_("can't flush file descriptor %d"), m_fd);
             return false;
         }
-#else // no fsync
-        // just do nothing
-#endif // fsync
     }
+#endif // fsync
 
     return true;
 }