wxImage::Ok() now returns false for images with 0 width or height

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

diff --git a/src/common/protocol.cpp b/src/common/protocol.cpp
index 21df7e8997284e0fed06df95342a78d3e5cd476c..2203ac231f4ee76b6f1c05f5ab67fb767c1f6c24 100644 (file)
--- a/src/common/protocol.cpp
+++ b/src/common/protocol.cpp
@@ -13,18 +13,18 @@
 #pragma implementation "protocol.h"
 #endif
 
 #pragma implementation "protocol.h"
 #endif
 

-#ifdef WXPREC
-#include <wx/wxprec.h>
-#else
-#include <wx/wx.h>
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
+#ifdef __BORLANDC__
+  #pragma hdrstop

 #endif
 
 #include "wx/protocol/protocol.h"
 #include "wx/url.h"
 #endif
 
 #include "wx/protocol/protocol.h"
 #include "wx/url.h"

+#include "wx/module.h"

 
 

-#ifdef __BORLANDC__
-#pragma hdrstop
-#endif
+#include <stdlib.h>

 
 /////////////////////////////////////////////////////////////////
 // wxProtoInfo
 
 /////////////////////////////////////////////////////////////////
 // wxProtoInfo


@@ -36,33 +36,41 @@
  * --------------------------------------------------------------
  */
 
  * --------------------------------------------------------------
  */
 

-wxProtoInfo::wxProtoInfo(const char *name, const char *serv,
+wxProtoInfo::wxProtoInfo(const wxChar *name, const wxChar *serv,

                          const bool need_host1, wxClassInfo *info)
 {
   m_protoname = name;
   m_servname = serv;
   m_cinfo = info;
   m_needhost = need_host1;
                          const bool need_host1, wxClassInfo *info)
 {
   m_protoname = name;
   m_servname = serv;
   m_cinfo = info;
   m_needhost = need_host1;

-  next = wxURL::g_protocols;
-  wxURL::g_protocols = this;
+  next = wxURL::ms_protocols;
+  wxURL::ms_protocols = this;

 }
 
 /////////////////////////////////////////////////////////////////
 // wxProtocol ///////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////
 
 }
 
 /////////////////////////////////////////////////////////////////
 // wxProtocol ///////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////
 

+#if wxUSE_SOCKETS

 IMPLEMENT_ABSTRACT_CLASS(wxProtocol, wxSocketClient)
 IMPLEMENT_ABSTRACT_CLASS(wxProtocol, wxSocketClient)

+#else
+IMPLEMENT_ABSTRACT_CLASS(wxProtocol, wxObject)
+#endif

 
 wxProtocol::wxProtocol()
 
 wxProtocol::wxProtocol()

+#if wxUSE_SOCKETS

  : wxSocketClient()
  : wxSocketClient()

+#endif

 {
 }
 
 {
 }
 

+#if wxUSE_SOCKETS

 bool wxProtocol::Reconnect()
 {
   wxIPV4address addr;
 
 bool wxProtocol::Reconnect()
 {
   wxIPV4address addr;
 

-  if (!GetPeer(addr)) {
+  if (!GetPeer(addr))
+  {

     Close();
     return FALSE;
   }
     Close();
     return FALSE;
   }


@@ -70,9 +78,60 @@ bool wxProtocol::Reconnect()
     return FALSE;
   if (!Connect(addr))
     return FALSE;
     return FALSE;
   if (!Connect(addr))
     return FALSE;

+

   return TRUE;
 }
 
   return TRUE;
 }
 

+// ----------------------------------------------------------------------------
+// Read a line from socket
+// ----------------------------------------------------------------------------
+
+// TODO ReadLine() should use buffers private to wxProtocol for efficiency!
+
+// static
+wxProtocolError wxProtocol::ReadLine(wxSocketBase *socket, wxString& result)
+{
+    result.Empty();
+    char ch, chLast = '\0';
+    while ( !socket->Read(&ch, sizeof(ch)).Error() )
+    {
+        switch ( ch )
+        {
+            case '\r':
+                // remember it, if the following is '\n', we're done
+                chLast = '\r';
+                break;
+
+            case '\n':
+                // only ends line if the previous character was '\r'
+                if ( chLast == '\r' )
+                {
+                    // EOL found
+                    return wxPROTO_NOERR;
+                }
+                //else: fall through
+
+            default:
+                // normal char
+                if ( chLast )
+                {
+                    result += chLast;
+                    chLast = '\0';
+                }
+
+                result += ch;
+        }
+    }
+
+    return wxPROTO_NETERR;
+}
+
+wxProtocolError wxProtocol::ReadLine(wxString& result)
+{
+    return ReadLine(this, result);
+}
+
+// old function which only chops '\n' and not '\r\n'

 wxProtocolError GetLine(wxSocketBase *sock, wxString& result) {
 #define PROTO_BSIZE 2048
   size_t avail, size;
 wxProtocolError GetLine(wxSocketBase *sock, wxString& result) {
 #define PROTO_BSIZE 2048
   size_t avail, size;


@@ -81,7 +140,7 @@ wxProtocolError GetLine(wxSocketBase *sock, wxString& result) {
   bool found;
 
   avail = sock->Read(tmp_buf, PROTO_BSIZE).LastCount();
   bool found;
 
   avail = sock->Read(tmp_buf, PROTO_BSIZE).LastCount();

-  if (sock->LastError() != 0 || avail == 0)
+  if (sock->Error() || avail == 0)

     return wxPROTO_NETERR;
 
   memcpy(tmp_str, tmp_buf, avail);
     return wxPROTO_NETERR;
 
   memcpy(tmp_str, tmp_buf, avail);


@@ -103,7 +162,8 @@ wxProtocolError GetLine(wxSocketBase *sock, wxString& result) {
   result = result.Left(result.Length()-1);
 
   size = ret-tmp_str+1;
   result = result.Left(result.Length()-1);
 
   size = ret-tmp_str+1;

-  sock->CreatePushbackBefore(&tmp_buf[size], avail-size);
+  sock->Unread(&tmp_buf[size], avail-size);

   return wxPROTO_NOERR;
 #undef PROTO_BSIZE
 }
   return wxPROTO_NOERR;
 #undef PROTO_BSIZE
 }

+#endif