X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/3bb0b01c290489c73816dfaccf16568757c1a731..878711c01c1b9ad5b97d35f379a048b8ce1bfb49:/src/common/zstream.cpp?ds=sidebyside diff --git a/src/common/zstream.cpp b/src/common/zstream.cpp index 8e444b4b5f..f1380d5b64 100644 --- a/src/common/zstream.cpp +++ b/src/common/zstream.cpp @@ -27,11 +27,12 @@ #include "wx/intl.h" #include "wx/log.h" -// When using configure, the path must be "zlib.h" I don't know -// what other ports (wxMac, wxMotif without configure) need here. - -#if defined(__WXMSW__) && !defined(USING_CONFIGURE) - #include "..\zlib\zlib.h" +// normally, the compiler options should contain -I../zlib, but it is +// apparently not the case for all MSW makefiles and so, unless we use +// configure (which defines __WX_SETUP_H__) or it is explicitly overridden by +// the user (who can define wxUSE_ZLIB_H_IN_PATH), we hardcode the path here +#if defined(__WXMSW__) && !defined(__WX_SETUP_H__) && !defined(wxUSE_ZLIB_H_IN_PATH) + #include "../zlib/zlib.h" #else #include "zlib.h" #endif @@ -45,8 +46,6 @@ wxZlibInputStream::wxZlibInputStream(wxInputStream& stream) : wxFilterInputStream(stream) { - int err; - // I need a private stream buffer. m_inflate = new z_stream_s; @@ -54,7 +53,7 @@ wxZlibInputStream::wxZlibInputStream(wxInputStream& stream) m_inflate->zfree = (free_func)0; m_inflate->opaque = (voidpf)0; - err = inflateInit(m_inflate); + int err = inflateInit(m_inflate); if (err != Z_OK) { inflateEnd(m_inflate); delete m_inflate; @@ -72,6 +71,8 @@ wxZlibInputStream::~wxZlibInputStream() { inflateEnd(m_inflate); delete m_inflate; + + delete [] m_z_buffer; } size_t wxZlibInputStream::OnSysRead(void *buffer, size_t size) @@ -84,26 +85,37 @@ size_t wxZlibInputStream::OnSysRead(void *buffer, size_t size) while (m_inflate->avail_out > 0) { if (m_inflate->avail_in == 0) { - m_parent_i_stream->Read(m_z_buffer, m_z_size); + m_parent_i_stream->Read(m_z_buffer, wxMin(m_z_size, size)); m_inflate->next_in = m_z_buffer; m_inflate->avail_in = m_parent_i_stream->LastRead(); - if (m_parent_i_stream->LastError() != wxStream_NOERROR) - return (size - m_inflate->avail_in); + wxStreamError err = m_parent_i_stream->GetLastError(); + if ( err != wxSTREAM_NO_ERROR && err != wxSTREAM_EOF) + { + m_lasterror = err; + return 0; // failed to read anything + } + + if ( m_inflate->avail_in == 0 ) + { + // EOF + m_lasterror = wxSTREAM_EOF; + break; + } } err = inflate(m_inflate, Z_FINISH); if (err == Z_STREAM_END) - return (size - m_inflate->avail_in); + return (size - m_inflate->avail_out); } - return size-m_inflate->avail_in; + return size-m_inflate->avail_out; } ////////////////////// // wxZlibOutputStream ////////////////////// -wxZlibOutputStream::wxZlibOutputStream(wxOutputStream& stream) +wxZlibOutputStream::wxZlibOutputStream(wxOutputStream& stream, int level) : wxFilterOutputStream(stream) { int err; @@ -114,7 +126,16 @@ wxZlibOutputStream::wxZlibOutputStream(wxOutputStream& stream) m_deflate->zfree = (free_func)0; m_deflate->opaque = (voidpf)0; - err = deflateInit(m_deflate, Z_DEFAULT_COMPRESSION); + if ( level == -1 ) + { + level = Z_DEFAULT_COMPRESSION; + } + else + { + wxASSERT_MSG(level >= 0 && level <= 9, wxT("wxZlibOutputStream compression level must be between 0 and 9!")); + } + + err = deflateInit(m_deflate, level); if (err != Z_OK) { deflateEnd(m_deflate); return; @@ -137,11 +158,12 @@ wxZlibOutputStream::~wxZlibOutputStream() err = deflate(m_deflate, Z_FINISH); if (err != Z_STREAM_END) { - wxLogDebug( _T("wxZlibOutputStream: an error occured while closing the stream.\n") ); + wxLogDebug( wxT("wxZlibOutputStream: an error occured while closing the stream.\n") ); return; } deflateEnd(m_deflate); + delete m_deflate; delete[] m_z_buffer; } @@ -159,6 +181,19 @@ void wxZlibOutputStream::Sync() return; } + // Fixed by "Stefan Csomor" + while( m_deflate->avail_out == 0 ) + { + m_parent_o_stream->Write(m_z_buffer, m_z_size ); + m_deflate->next_out = m_z_buffer; + m_deflate->avail_out = m_z_size; + err = deflate(m_deflate, Z_FULL_FLUSH); + if (err != Z_OK) { + return; + } + } + // End + m_parent_o_stream->Write(m_z_buffer, m_z_size-m_deflate->avail_out); m_deflate->next_out = m_z_buffer; m_deflate->avail_out = m_z_size; @@ -175,7 +210,7 @@ size_t wxZlibOutputStream::OnSysWrite(const void *buffer, size_t size) if (m_deflate->avail_out == 0) { m_parent_o_stream->Write(m_z_buffer, m_z_size); - if (m_parent_o_stream->LastError() != wxStream_NOERROR) + if ( !*m_parent_o_stream ) return (size - m_deflate->avail_in); m_deflate->next_out = m_z_buffer;