More _T() in asserts...

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

diff --git a/src/common/http.cpp b/src/common/http.cpp
index 08c4df058c2a38484e840c389def352ce0b4d426..18beaf43cf816fe5b0ba69d5149bf45ff438de94 100644 (file)
--- a/src/common/http.cpp
+++ b/src/common/http.cpp
@@ -46,7 +46,7 @@ wxHTTP::wxHTTP()
   m_addr = NULL;
   m_read = FALSE;
 
   m_addr = NULL;
   m_read = FALSE;
 

-  SetNotify(REQ_LOST);
+  SetNotify(GSOCK_LOST_FLAG);

 }
 
 wxHTTP::~wxHTTP()
 }
 
 wxHTTP::~wxHTTP()


@@ -120,8 +120,8 @@ bool wxHTTP::ParseHeaders()
   m_read = TRUE;
 
   while (1) {
   m_read = TRUE;
 
   while (1) {

-    m_error = GetLine(this, line);
-    if (m_error != wxPROTO_NOERR)
+    m_perr = GetLine(this, line);
+    if (m_perr != wxPROTO_NOERR)

       return FALSE;
 
     if (line.Length() == 0)
       return FALSE;
 
     if (line.Length() == 0)


@@ -154,7 +154,7 @@ bool wxHTTP::Connect(const wxString& host)
   if (!addr->Hostname(host)) {
     delete m_addr;
     m_addr = NULL;
   if (!addr->Hostname(host)) {
     delete m_addr;
     m_addr = NULL;

-    m_error = wxPROTO_NETERR;
+    m_perr = wxPROTO_NETERR;

     return FALSE;
   }
 
     return FALSE;
   }
 


@@ -166,22 +166,20 @@ bool wxHTTP::Connect(const wxString& host)
 
 bool wxHTTP::Connect(wxSockAddress& addr, bool WXUNUSED(wait))
 {
 
 bool wxHTTP::Connect(wxSockAddress& addr, bool WXUNUSED(wait))
 {

-  struct sockaddr *raw_addr;
-  size_t len;
-
-  m_addr = (wxSockAddress *)(addr.GetClassInfo()->CreateObject());
-
-  addr.Build(raw_addr, len);
-  m_addr->Disassemble(raw_addr, len);
+  if (m_addr) {
+    delete m_addr;
+    m_addr = NULL;
+    Close();
+  }

 
 

+  m_addr = (wxSockAddress *) addr.Clone();

   return TRUE;
 }
 
 bool wxHTTP::BuildRequest(const wxString& path, wxHTTP_Req req)
 {
   wxChar *tmp_buf;
   return TRUE;
 }
 
 bool wxHTTP::BuildRequest(const wxString& path, wxHTTP_Req req)
 {
   wxChar *tmp_buf;

-  wxChar buf[200];
-  const wxWX2MBbuf pathbuf;
+  wxChar buf[200]; // 200 is arbitrary.

   wxString tmp_str;
 
   switch (req) {
   wxString tmp_str;
 
   switch (req) {


@@ -197,14 +195,14 @@ bool wxHTTP::BuildRequest(const wxString& path, wxHTTP_Req req)
   Notify(FALSE);
 
   tmp_str = wxURL::ConvertToValidURI(path);
   Notify(FALSE);
 
   tmp_str = wxURL::ConvertToValidURI(path);

-  wxSprintf(buf, _T("%s %s\n\r"), tmp_buf, tmp_str.GetData());
-  pathbuf = wxConvLibc.cWX2MB(buf);
-  Write(pathbuf, strlen(pathbuf));
+  wxSprintf(buf, _T("%s %s HTTP/1.0\n\r"), tmp_buf, tmp_str.GetData());
+  const wxWX2MBbuf pathbuf = wxConvLibc.cWX2MB(buf);
+  Write(pathbuf, strlen(MBSTRINGCAST pathbuf));

   SendHeaders();
   Write("\n\r", 2);
 
   SendHeaders();
   Write("\n\r", 2);
 

-  m_error = GetLine(this, tmp_str);
-  if (m_error != wxPROTO_NOERR) {
+  m_perr = GetLine(this, tmp_str);
+  if (m_perr != wxPROTO_NOERR) {

     RestoreState();
     return FALSE;
   }
     RestoreState();
     return FALSE;
   }


@@ -228,7 +226,7 @@ bool wxHTTP::BuildRequest(const wxString& path, wxHTTP_Req req)
   case 200:
     break;
   default:
   case 200:
     break;
   default:

-    m_error = wxPROTO_NOFILE;
+    m_perr = wxPROTO_NOFILE;

     RestoreState();
     return FALSE;
   }
     RestoreState();
     return FALSE;
   }


@@ -242,12 +240,29 @@ class wxHTTPStream : public wxSocketInputStream {
 public:
   wxHTTP *m_http;
   size_t m_httpsize;
 public:
   wxHTTP *m_http;
   size_t m_httpsize;

+  unsigned long m_read_bytes;

 
   wxHTTPStream(wxHTTP *http) : wxSocketInputStream(*http), m_http(http) {}
   size_t StreamSize() const { return m_httpsize; }
   virtual ~wxHTTPStream(void) { m_http->Abort(); }
 
   wxHTTPStream(wxHTTP *http) : wxSocketInputStream(*http), m_http(http) {}
   size_t StreamSize() const { return m_httpsize; }
   virtual ~wxHTTPStream(void) { m_http->Abort(); }

+
+protected:
+  size_t OnSysRead(void *buffer, size_t bufsize);

 };
 
 };
 

+size_t wxHTTPStream::OnSysRead(void *buffer, size_t bufsize)
+{
+  size_t ret;
+
+  if (m_httpsize > 0 && m_read_bytes >= m_httpsize)
+    return 0;
+
+  ret = wxSocketInputStream::OnSysRead(buffer, bufsize);
+  m_read_bytes += ret;
+
+  return ret;
+}
+

 bool wxHTTP::Abort(void)
 {
   return wxSocketClient::Close();
 bool wxHTTP::Abort(void)
 {
   return wxSocketClient::Close();


@@ -258,7 +273,7 @@ wxInputStream *wxHTTP::GetInputStream(const wxString& path)
   wxHTTPStream *inp_stream = new wxHTTPStream(this);
 
   if (!m_addr || m_connected) {
   wxHTTPStream *inp_stream = new wxHTTPStream(this);
 
   if (!m_addr || m_connected) {

-    m_error = wxPROTO_CONNERR;
+    m_perr = wxPROTO_CONNERR;

     return NULL;
   }
 
     return NULL;
   }
 


@@ -268,11 +283,15 @@ wxInputStream *wxHTTP::GetInputStream(const wxString& path)
   if (!BuildRequest(path, wxHTTP_GET))
     return NULL;
 
   if (!BuildRequest(path, wxHTTP_GET))
     return NULL;
 

-  wxPrintf(_T("Len = %s\n"), WXSTRINGCAST GetHeader(_T("Content-Length")));

   if (!GetHeader(_T("Content-Length")).IsEmpty())
     inp_stream->m_httpsize = wxAtoi(WXSTRINGCAST GetHeader(_T("Content-Length")));
   if (!GetHeader(_T("Content-Length")).IsEmpty())
     inp_stream->m_httpsize = wxAtoi(WXSTRINGCAST GetHeader(_T("Content-Length")));

+  else
+    inp_stream->m_httpsize = (size_t)-1;
+
+  inp_stream->m_read_bytes = 0;
+
+  Notify(FALSE);

 
 

-  SetFlags(WAITALL);

   return inp_stream;
 }
 
   return inp_stream;
 }