(blind) fix for gcc 3.4 warning for an always true comparison in an assert

[wxWidgets.git] / src / common / wfstream.cpp

diff --git a/src/common/wfstream.cpp b/src/common/wfstream.cpp
index e4da5d94dfdd4d7c21ae3f2d281fecf264e037f3..296f205b2e3b1f4c703c590e960d29d6302a1e5e 100644 (file)
--- a/src/common/wfstream.cpp
+++ b/src/common/wfstream.cpp
@@ -5,11 +5,11 @@
 // Modified by:
 // Created:     11/07/98
 // RCS-ID:      $Id$
 // Modified by:
 // Created:     11/07/98
 // RCS-ID:      $Id$

-// Copyright:   (c) Guilhem Lavaux 
-// Licence:     wxWindows license
+// Copyright:   (c) Guilhem Lavaux
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "wfstream.h"
 #endif
 
 #pragma implementation "wfstream.h"
 #endif
 


@@ -20,15 +20,11 @@
   #pragma hdrstop
 #endif
 
   #pragma hdrstop
 #endif
 

-#ifndef WX_PRECOMP
-  #include "wx/defs.h"
-#endif
-

 #if wxUSE_STREAMS && wxUSE_FILE
 
 #include <stdio.h>
 #if wxUSE_STREAMS && wxUSE_FILE
 
 #include <stdio.h>

-#include <wx/stream.h>
-#include <wx/wfstream.h>
+#include "wx/stream.h"
+#include "wx/wfstream.h"

 
 // ----------------------------------------------------------------------------
 // wxFileInputStream
 
 // ----------------------------------------------------------------------------
 // wxFileInputStream


@@ -37,61 +33,74 @@
 wxFileInputStream::wxFileInputStream(const wxString& fileName)
   : wxInputStream()
 {
 wxFileInputStream::wxFileInputStream(const wxString& fileName)
   : wxInputStream()
 {

-  m_file = new wxFile(fileName, wxFile::read);
-  m_file_destroy = TRUE;
-  m_i_streambuf->SetBufferIO(1024);
+    m_file = new wxFile(fileName, wxFile::read);
+    m_file_destroy = TRUE;

 }
 
 wxFileInputStream::wxFileInputStream()
   : wxInputStream()
 {
 }
 
 wxFileInputStream::wxFileInputStream()
   : wxInputStream()
 {

-  m_file_destroy = FALSE;
-  m_file = NULL;
+    m_file_destroy = FALSE;
+    m_file = NULL;

 }
 
 wxFileInputStream::wxFileInputStream(wxFile& file)
 {
 }
 
 wxFileInputStream::wxFileInputStream(wxFile& file)
 {

-  m_file = &file;
-  m_file_destroy = FALSE;
-  m_i_streambuf->SetBufferIO(1024);
+    m_file = &file;
+    m_file_destroy = FALSE;

 }
 
 wxFileInputStream::wxFileInputStream(int fd)
 {
 }
 
 wxFileInputStream::wxFileInputStream(int fd)
 {

-  m_file = new wxFile(fd);
-  m_file_destroy = TRUE;
-  m_i_streambuf->SetBufferIO(1024);
+    m_file = new wxFile(fd);
+    m_file_destroy = TRUE;

 }
 
 wxFileInputStream::~wxFileInputStream()
 {
 }
 
 wxFileInputStream::~wxFileInputStream()
 {

-  if (m_file_destroy)
-    delete m_file;
-}
-
-char wxFileInputStream::Peek()
-{
-  return 0;
+    if (m_file_destroy)
+        delete m_file;

 }
 
 }
 

-size_t wxFileInputStream::StreamSize() const
+size_t wxFileInputStream::GetSize() const

 {
 {

-  return m_file->Length();
+    return m_file->Length();

 }
 
 size_t wxFileInputStream::OnSysRead(void *buffer, size_t size)
 {
 }
 
 size_t wxFileInputStream::OnSysRead(void *buffer, size_t size)
 {

-  return m_file->Read(buffer, size);
+    off_t ret = m_file->Read(buffer, size);
+
+    // NB: we can't use a switch here because HP-UX CC doesn't allow
+    //     switching over long long (which off_t is in 64bit mode)
+
+    if ( !ret )
+    {
+        // nothing read, so nothing more to read
+        m_lasterror = wxSTREAM_EOF;
+    }
+    else if ( ret == wxInvalidOffset )
+    {
+        m_lasterror = wxSTREAM_READ_ERROR;
+        ret = 0;
+    }
+    else
+    {
+        // normal case
+        m_lasterror = wxSTREAM_NO_ERROR;
+    }
+
+    return ret;

 }
 
 off_t wxFileInputStream::OnSysSeek(off_t pos, wxSeekMode mode)
 {
 }
 
 off_t wxFileInputStream::OnSysSeek(off_t pos, wxSeekMode mode)
 {

-  return m_file->Seek(pos, mode);
+    return m_file->Seek(pos, mode);

 }
 
 off_t wxFileInputStream::OnSysTell() const
 {
 }
 
 off_t wxFileInputStream::OnSysTell() const
 {

-  return m_file->Tell();
+    return m_file->Tell();

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -100,76 +109,257 @@ off_t wxFileInputStream::OnSysTell() const
 
 wxFileOutputStream::wxFileOutputStream(const wxString& fileName)
 {
 
 wxFileOutputStream::wxFileOutputStream(const wxString& fileName)
 {

-  m_file = new wxFile(fileName, wxFile::write);
-  m_file_destroy = TRUE;
-  m_o_streambuf->SetBufferIO(1024);
+    m_file = new wxFile(fileName, wxFile::write);
+    m_file_destroy = TRUE;
+
+    if (!m_file->IsOpened())
+    {
+        m_lasterror = wxSTREAM_WRITE_ERROR;
+    }
+    else
+    {
+        if (m_file->Error())
+            m_lasterror = wxSTREAM_WRITE_ERROR;
+    }

 }
 
 wxFileOutputStream::wxFileOutputStream(wxFile& file)
 {
 }
 
 wxFileOutputStream::wxFileOutputStream(wxFile& file)
 {

-  m_file = &file;
-  m_file_destroy = FALSE;
-  m_o_streambuf->SetBufferIO(1024);
+    m_file = &file;
+    m_file_destroy = FALSE;

 }
 
 wxFileOutputStream::wxFileOutputStream()
 }
 
 wxFileOutputStream::wxFileOutputStream()

-  : wxOutputStream()
+                  : wxOutputStream()

 {
 {

-  m_o_streambuf->SetBufferIO(1024);
-  m_file_destroy = FALSE;
-  m_file = NULL;
+    m_file_destroy = FALSE;
+    m_file = NULL;

 }
 
 wxFileOutputStream::wxFileOutputStream(int fd)
 {
 }
 
 wxFileOutputStream::wxFileOutputStream(int fd)
 {

-  m_file = new wxFile(fd);
-  m_file_destroy = TRUE;
-  m_o_streambuf->SetBufferIO(1024);
+    m_file = new wxFile(fd);
+    m_file_destroy = TRUE;

 }
 
 wxFileOutputStream::~wxFileOutputStream()
 {
 }
 
 wxFileOutputStream::~wxFileOutputStream()
 {

-  if (m_file_destroy) {
-    Sync();
-    delete m_file;
-  }
+    if (m_file_destroy)
+    {
+        Sync();
+        delete m_file;
+    }

 }
 
 size_t wxFileOutputStream::OnSysWrite(const void *buffer, size_t size)
 {
 }
 
 size_t wxFileOutputStream::OnSysWrite(const void *buffer, size_t size)
 {

-  size_t ret = m_file->Write(buffer, size);
-  m_lasterror = wxStream_EOF; // TODO
-  return ret;
+    size_t ret = m_file->Write(buffer, size);
+
+    m_lasterror = m_file->Error() ? wxSTREAM_WRITE_ERROR : wxSTREAM_NO_ERROR;
+
+    return ret;

 }
 
 off_t wxFileOutputStream::OnSysTell() const
 {
 }
 
 off_t wxFileOutputStream::OnSysTell() const
 {

-  return m_file->Tell();
+    return m_file->Tell();

 }
 
 off_t wxFileOutputStream::OnSysSeek(off_t pos, wxSeekMode mode)
 {
 }
 
 off_t wxFileOutputStream::OnSysSeek(off_t pos, wxSeekMode mode)
 {

-  return m_file->Seek(pos, mode);
+    return m_file->Seek(pos, mode);

 }
 
 void wxFileOutputStream::Sync()
 {
 }
 
 void wxFileOutputStream::Sync()
 {

-  wxOutputStream::Sync();
-  m_file->Flush();
+    wxOutputStream::Sync();
+    m_file->Flush();

 }
 
 }
 

-size_t wxFileOutputStream::StreamSize() const
+size_t wxFileOutputStream::GetSize() const

 {
 {

-  return m_file->Length();
+    return m_file->Length();

 }
 
 // ----------------------------------------------------------------------------
 // wxFileStream
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------
 // wxFileStream
 // ----------------------------------------------------------------------------

+

 wxFileStream::wxFileStream(const wxString& fileName)
 wxFileStream::wxFileStream(const wxString& fileName)

- : wxFileInputStream(fileName), wxFileOutputStream(*wxFileInputStream::m_file)
+            : wxFileInputStream(fileName)
+{
+    wxFileOutputStream::m_file = wxFileInputStream::m_file;
+}
+
+// ----------------------------------------------------------------------------
+// wxFFileInputStream
+// ----------------------------------------------------------------------------
+
+wxFFileInputStream::wxFFileInputStream(const wxString& fileName)
+  : wxInputStream()
+{
+    m_file = new wxFFile(fileName, _T("rb"));
+    m_file_destroy = TRUE;
+}
+
+wxFFileInputStream::wxFFileInputStream()
+  : wxInputStream()
+{
+    m_file_destroy = FALSE;
+    m_file = NULL;
+}
+
+wxFFileInputStream::wxFFileInputStream(wxFFile& file)
+{
+    m_file = &file;
+    m_file_destroy = FALSE;
+}
+
+wxFFileInputStream::wxFFileInputStream(FILE *file)
+{
+    m_file = new wxFFile(file);
+    m_file_destroy = TRUE;
+}
+
+wxFFileInputStream::~wxFFileInputStream()
+{
+    if (m_file_destroy)
+        delete m_file;
+}
+
+size_t wxFFileInputStream::GetSize() const
+{
+    return m_file->Length();
+}
+
+size_t wxFFileInputStream::OnSysRead(void *buffer, size_t size)
+{
+    off_t ret;
+
+    ret = m_file->Read(buffer, size);
+
+    if (m_file->Eof())
+        m_lasterror = wxSTREAM_EOF;
+    if (ret == wxInvalidOffset)
+    {
+        m_lasterror = wxSTREAM_READ_ERROR;
+        ret = 0;
+    }
+
+    return ret;
+}
+
+off_t wxFFileInputStream::OnSysSeek(off_t pos, wxSeekMode mode)
+{
+#ifdef __VMS
+#pragma message disable intsignchange
+#endif
+   return ( m_file->Seek(pos, mode) ? m_file->Tell() : wxInvalidOffset );
+#ifdef __VMS
+#pragma message enable intsignchange
+#endif
+}
+
+off_t wxFFileInputStream::OnSysTell() const
+{
+    return m_file->Tell();
+}
+
+// ----------------------------------------------------------------------------
+// wxFFileOutputStream
+// ----------------------------------------------------------------------------
+
+wxFFileOutputStream::wxFFileOutputStream(const wxString& fileName)
+{
+    m_file = new wxFFile(fileName, _T("w+b"));
+    m_file_destroy = TRUE;
+
+    if (!m_file->IsOpened())
+    {
+        m_lasterror = wxSTREAM_WRITE_ERROR;
+    }
+    else
+    {
+        if (m_file->Error())
+            m_lasterror = wxSTREAM_WRITE_ERROR;
+    }
+}
+
+wxFFileOutputStream::wxFFileOutputStream(wxFFile& file)
+{
+    m_file = &file;
+    m_file_destroy = FALSE;
+}
+
+wxFFileOutputStream::wxFFileOutputStream()
+  : wxOutputStream()
+{
+    m_file_destroy = FALSE;
+    m_file = NULL;
+}
+
+wxFFileOutputStream::wxFFileOutputStream(FILE *file)
+{
+    m_file = new wxFFile(file);
+    m_file_destroy = TRUE;
+}
+
+wxFFileOutputStream::~wxFFileOutputStream()

 {
 {

+    if (m_file_destroy)
+    {
+        Sync();
+        delete m_file;
+    }
+}
+
+size_t wxFFileOutputStream::OnSysWrite(const void *buffer, size_t size)
+{
+    size_t ret = m_file->Write(buffer, size);
+    if (m_file->Error())
+        m_lasterror = wxSTREAM_WRITE_ERROR;
+    else
+        m_lasterror = wxSTREAM_NO_ERROR;
+    return ret;
+}
+
+off_t wxFFileOutputStream::OnSysTell() const
+{
+    return m_file->Tell();
+}
+
+off_t wxFFileOutputStream::OnSysSeek(off_t pos, wxSeekMode mode)
+{
+#ifdef __VMS
+#pragma message disable intsignchange
+#endif
+    return ( m_file->Seek(pos, mode) ? m_file->Tell() : wxInvalidOffset );
+#ifdef __VMS
+#pragma message enable intsignchange
+#endif
+}
+
+void wxFFileOutputStream::Sync()
+{
+    wxOutputStream::Sync();
+    m_file->Flush();
+}
+
+size_t wxFFileOutputStream::GetSize() const
+{
+    return m_file->Length();
+}
+
+// ----------------------------------------------------------------------------
+// wxFFileStream
+// ----------------------------------------------------------------------------
+
+wxFFileStream::wxFFileStream(const wxString& fileName)
+             : wxFFileInputStream(fileName)
+{
+    wxFFileOutputStream::m_file = wxFFileInputStream::m_file;

 }
 
 #endif
   // wxUSE_STREAMS && wxUSE_FILE
 }
 
 #endif
   // wxUSE_STREAMS && wxUSE_FILE

+