// construction/destruction
// ----------------------------------------------------------------------------
-wxStringInputStream::wxStringInputStream(const wxString& s)
+// TODO: Do we want to include the null char in the stream? If so then
+// just add +1 to m_len in the ctor
+wxStringInputStream::wxStringInputStream(const wxString& s)
#if wxUSE_UNICODE
: m_str(s), m_buf(wxMBConvUTF8().cWX2MB(s).release()), m_len(strlen(m_buf))
#else
- : m_str(s), m_buf((char*)s.c_str()), m_len(s.length())
+ : m_str(s), m_buf((char*)(const char*)s.c_str()), m_len(s.length())
#endif
{
#if wxUSE_UNICODE
return wxInvalidOffset;
}
- if ( ofs < 0 || wx_static_cast(size_t, ofs) >= m_len )
+ if ( ofs < 0 || ofs > wx_static_cast(wxFileOffset, m_len) )
return wxInvalidOffset;
- m_pos = wx_static_cast(size_t, ofs);
+ // FIXME: this can't be right
+ m_pos = wx_truncate_cast(size_t, ofs);
return ofs;
}
// the literal length
m_str->Append(wxString(p, m_conv, size));
- // return number of bytes actually written
+ // update position
m_pos += size;
+ // return number of bytes actually written
return size;
}