// Purpose: Socket handler classes
// Authors: Guilhem Lavaux, Guillermo Rodriguez Garcia
// Created: April 1997
-// Updated: July 1999
+// Updated: September 1999
// Copyright: (C) 1999, 1998, 1997, Guilhem Lavaux
+// (C) 1999, Guillermo Rodriguez Garcia
// RCS_ID: $Id$
// License: see wxWindows license
/////////////////////////////////////////////////////////////////////////////
#include "wx/sckaddr.h"
#include "wx/socket.h"
+
+#if defined(__WXMSW__) || defined(__WXPM__)
+#define PROCESS_EVENTS() wxYield()
+#elif defined(__WXGTK__)
+#include <gtk/gtk.h>
+#define PROCESS_EVENTS() gtk_main_iteration()
+#endif
+
+
// --------------------------------------------------------------
// ClassInfos
// --------------------------------------------------------------
wxSocketBase::wxSocketBase(wxSocketBase::wxSockFlags _flags,
wxSocketBase::wxSockType _type) :
wxEvtHandler(),
- m_socket(NULL), m_flags(_flags), m_type(_type),
- m_neededreq(0),
- m_lcount(0), m_timeout(600),
- m_unread(NULL), m_unrd_size(0), m_unrd_cur(0),
- m_cbk(NULL), m_cdata(NULL),
+ m_socket(NULL), m_id(-1),
+ m_flags(_flags), m_type(_type),
+ m_neededreq(0), m_notify_state(FALSE),
m_connected(FALSE), m_establishing(FALSE),
- m_notify_state(FALSE), m_id(-1),
- m_defering(NO_DEFER), m_error(FALSE),
- m_states()
+ m_reading(FALSE), m_writing(FALSE),
+ m_error(FALSE), m_lcount(0), m_timeout(600), m_states(),
+ m_unread(NULL), m_unrd_size(0), m_unrd_cur(0),
+ m_defering(NO_DEFER), m_defer_buffer(NULL), m_defer_timer(NULL),
+ m_cbk(NULL), m_cdata(NULL)
{
}
wxSocketBase::wxSocketBase() :
wxEvtHandler(),
- m_socket(NULL), m_flags(WAITALL | SPEED), m_type(SOCK_UNINIT),
- m_neededreq(0),
- m_lcount(0), m_timeout(600),
- m_unread(NULL), m_unrd_size(0), m_unrd_cur(0),
- m_cbk(NULL), m_cdata(NULL),
+ m_socket(NULL), m_id(-1),
+ m_flags(NONE), m_type(SOCK_UNINIT),
+ m_neededreq(0), m_notify_state(FALSE),
m_connected(FALSE), m_establishing(FALSE),
- m_notify_state(FALSE), m_id(-1),
- m_defering(NO_DEFER), m_error(FALSE),
- m_states()
+ m_reading(FALSE), m_writing(FALSE),
+ m_error(FALSE), m_lcount(0), m_timeout(600), m_states(),
+ m_unread(NULL), m_unrd_size(0), m_unrd_cur(0),
+ m_defering(NO_DEFER), m_defer_buffer(NULL), m_defer_timer(NULL),
+ m_cbk(NULL), m_cdata(NULL)
{
}
// wxSocketBase basic IO operations
// --------------------------------------------------------------
-// GRG: I have made some changes to wxSocket internal event
-// system; now, all events (INPUT, OUTPUT, CONNECTION, LOST)
-// are always internally monitored; but users will only be
-// notified of these events they are interested in. So we
-// no longer have to change the event mask with SetNotify()
-// in internal functions like DeferRead, DeferWrite, and
-// the like. This solves a lot of problems.
-
-// GRG: I added m_error handling to IO operations. Now,
-// wxSocketBase::Error() correctly indicates if the last
-// operation from {Read, Write, ReadMsg, WriteMsg, Peek,
-// Unread, Discard} failed. Note that now, every function
-// that updates m_lcount, also updates m_error. While I
-// was at it, also fixed an UGLY bug in ReadMsg.
+// All IO operations {Read, Write, ReadMsg, WriteMsg, Peek,
+// Unread, Discard} update m_error and m_lcount.
+//
+// TODO: Should Connect, Accept and AcceptWith update m_error?
class _wxSocketInternalTimer: public wxTimer
{
}
};
-int wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes)
+wxUint32 wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes)
{
// Timer for timeout
_wxSocketInternalTimer timer;
+ //wxLogMessage("Entrando a DeferRead, nbytes = %d", nbytes);
wxASSERT(m_defering == NO_DEFER);
// Set the defering mode to READ.
// If the socket is readable, call DoDefer for the first time
if (GSocket_Select(m_socket, GSOCK_INPUT_FLAG))
+ {
+ //wxLogMessage("Llamando al primer DoDefer");
DoDefer();
+ }
- // Wait for buffer completion.
+ // Wait for buffer completion.
while (m_defer_buffer != NULL)
- wxYield();
+ PROCESS_EVENTS();
timer.Stop();
m_defer_timer = NULL;
// Return the number of bytes read from the socket.
+ //wxLogMessage("Saliendo de DeferRead: total: %d bytes", nbytes - m_defer_nbytes);
return nbytes-m_defer_nbytes;
}
wxSocketBase& wxSocketBase::Read(char* buffer, wxUint32 nbytes)
{
- int ret = 1;
+ // Mask read events
+ m_reading = TRUE;
- m_error = FALSE;
+ m_lcount = _Read(buffer, nbytes);
- // we try this even if the connection has already been closed.
- m_lcount = GetPushback(buffer, nbytes, FALSE);
- nbytes -= m_lcount;
- buffer += m_lcount;
+ // If in WAITALL mode, all bytes should have been read.
+ if (m_flags & WAITALL)
+ m_error = (m_lcount != nbytes);
+ else
+ m_error = (m_lcount == 0);
- if (!m_connected || !m_socket)
- {
- // if no data retrieved AND not connected, it is an error.
- if (!m_lcount)
- m_error = TRUE;
+ // Trigger another read event if there is still data available.
+ m_reading = FALSE;
+ // TODO: TriggerRead
- return *this;
- }
+ return *this;
+}
+
+wxUint32 wxSocketBase::_Read(char* buffer, wxUint32 nbytes)
+{
+ int total;
+ int ret = 1;
- // If we have got the whole needed buffer, return immediately
- if (!nbytes)
+ // we try this even if the connection has already been closed.
+ total = GetPushback(buffer, nbytes, FALSE);
+ nbytes -= total;
+ buffer += total;
+
+ // If the socket is not connected, or we have got the whole
+ // needed buffer, return immedately
+ if (!m_connected || !m_socket || !nbytes)
+ return total;
+
+ // Possible combinations (they are checked in this order)
+ // NOWAIT
+ // SPEED | WAITALL
+ // SPEED
+ // WAITALL
+ // NONE
+ //
+ if (m_flags & NOWAIT) // NOWAIT
{
- return *this;
- }
+ GSocket_SetNonBlocking(m_socket, TRUE);
+ ret = GSocket_Read(m_socket, buffer, nbytes);
+ GSocket_SetNonBlocking(m_socket, FALSE);
- if (m_flags & SPEED & WAITALL) // SPEED && WAITALL
+ if (ret > 0)
+ total += ret;
+ }
+ else if ((m_flags & SPEED) && (m_flags & WAITALL)) // SPEED, WAITALL
{
while (ret > 0 && nbytes > 0)
{
ret = GSocket_Read(m_socket, buffer, nbytes);
- m_lcount += ret;
+ total += ret;
buffer += ret;
nbytes -= ret;
}
// In case the last call was an error ...
if (ret < 0)
- m_lcount ++;
+ total++;
}
- else if (m_flags & SPEED) // SPEED && !WAITALL
+ else if (m_flags & SPEED) // SPEED, !WAITALL
{
ret = GSocket_Read(m_socket, buffer, nbytes);
if (ret > 0)
- m_lcount += ret;
+ total += ret;
}
- else // !SPEED
+ else // NONE or WAITALL
{
ret = DeferRead(buffer, nbytes);
if (ret > 0)
- m_lcount += ret;
+ total += ret;
}
- // If we have read some data, then it is not an error, even
- // when in WAITALL mode, the last low-level IO call might
- // have failed.
- if (!m_lcount)
- m_error = TRUE;
-
- return *this;
+ return total;
}
wxSocketBase& wxSocketBase::ReadMsg(char* buffer, wxUint32 nbytes)
{
-#define MAX_BUFSIZE (10 * 1024)
+#define MAX_DISCARD_SIZE (10 * 1024)
+ wxUint32 len, len2, sig, total;
+ bool error;
int old_flags;
- unsigned long len, len2, sig;
struct
{
- char sig[4];
- char len[4];
+ unsigned char sig[4];
+ unsigned char len[4];
} msg;
- // sig should be an explicit 32-bit unsigned integer; I've seen
- // compilers in which wxUint32 was actually a 16-bit unsigned integer
+ // Mask read events
+ m_reading = TRUE;
+ total = 0;
+ error = TRUE;
old_flags = m_flags;
- SetFlags(WAITALL | SPEED);
+ SetFlags((m_flags & SPEED) | WAITALL);
- Read((char *)&msg, sizeof(msg));
- if (m_lcount != sizeof(msg))
- {
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
- }
+ if (_Read((char *)&msg, sizeof(msg)) != sizeof(msg))
+ goto exit;
- sig = msg.sig[0] & 0xff;
- sig |= (wxUint32)(msg.sig[1] & 0xff) << 8;
- sig |= (wxUint32)(msg.sig[2] & 0xff) << 16;
- sig |= (wxUint32)(msg.sig[3] & 0xff) << 24;
+ sig = (wxUint32)msg.sig[0];
+ sig |= (wxUint32)(msg.sig[1] << 8);
+ sig |= (wxUint32)(msg.sig[2] << 16);
+ sig |= (wxUint32)(msg.sig[3] << 24);
if (sig != 0xfeeddead)
{
- wxLogDebug(_T("Warning: invalid signature returned to ReadMsg\n"));
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
+ wxLogMessage(wxT("Warning: invalid signature returned to ReadMsg"));
+ goto exit;
}
- len = msg.len[0] & 0xff;
- len |= (wxUint32)(msg.len[1] & 0xff) << 8;
- len |= (wxUint32)(msg.len[2] & 0xff) << 16;
- len |= (wxUint32)(msg.len[3] & 0xff) << 24;
+ len = (wxUint32)msg.len[0];
+ len |= (wxUint32)(msg.len[1] << 8);
+ len |= (wxUint32)(msg.len[2] << 16);
+ len |= (wxUint32)(msg.len[3] << 24);
+
+ //wxLogMessage("Readmsg: %d %d %d %d -> len == %d",
+ // msg.len[0], msg.len[1], msg.len[2], msg.len[3], len);
if (len > nbytes)
{
else
len2 = 0;
- // The "len &&" in the following statements is necessary so
- // that we don't attempt to read (and possibly hang the system)
- // if the message was zero bytes long
- if (len && Read(buffer, len).LastCount() != len)
+ // This check is necessary so that we don't attemp to read if
+ // the msg was zero bytes long.
+ if (len)
{
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
+ total = _Read(buffer, len);
+
+ if (total != len)
+ goto exit;
}
if (len2)
{
- char *discard_buffer = new char[MAX_BUFSIZE];
+ char *discard_buffer = new char[MAX_DISCARD_SIZE];
long discard_len;
+ // NOTE: discarded bytes don't add to m_lcount.
do
{
- discard_len = ((len2 > MAX_BUFSIZE)? MAX_BUFSIZE : len2);
- discard_len = Read(discard_buffer, discard_len).LastCount();
+ discard_len = ((len2 > MAX_DISCARD_SIZE)? MAX_DISCARD_SIZE : len2);
+ discard_len = _Read(discard_buffer, discard_len);
len2 -= discard_len;
}
while ((discard_len > 0) && len2);
delete [] discard_buffer;
if (len2 != 0)
- {
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
- }
- }
- if (Read((char *)&msg, sizeof(msg)).LastCount() != sizeof(msg))
- {
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
+ goto exit;
}
+ if (_Read((char *)&msg, sizeof(msg)) != sizeof(msg))
+ goto exit;
- sig = msg.sig[0] & 0xff;
- sig |= (wxUint32)(msg.sig[1] & 0xff) << 8;
- sig |= (wxUint32)(msg.sig[2] & 0xff) << 16;
- sig |= (wxUint32)(msg.sig[3] & 0xff) << 24;
+ sig = (wxUint32)msg.sig[0];
+ sig |= (wxUint32)(msg.sig[1] << 8);
+ sig |= (wxUint32)(msg.sig[2] << 16);
+ sig |= (wxUint32)(msg.sig[3] << 24);
if (sig != 0xdeadfeed)
{
- m_error = TRUE;
- wxLogDebug(_T("Warning: invalid signature returned to ReadMsg\n"));
+ //wxLogMessage(wxT("Warning: invalid signature returned to ReadMsg"));
+ goto exit;
}
+ // everything was OK
+ error = FALSE;
+
+exit:
+ m_error = error;
+ m_lcount = total;
+ m_reading = FALSE;
SetFlags(old_flags);
+
+ // TODO: TriggerRead
return *this;
-#undef MAX_BUFSIZE
+#undef MAX_DISCARD_SIZE
}
wxSocketBase& wxSocketBase::Peek(char* buffer, wxUint32 nbytes)
{
- Read(buffer, nbytes);
- CreatePushbackAfter(buffer, nbytes);
+ // Mask read events
+ m_reading = TRUE;
+ m_lcount = _Read(buffer, nbytes);
+ Pushback(buffer, nbytes);
+
+ // If in WAITALL mode, all bytes should have been read.
+ if (m_flags & WAITALL)
+ m_error = (m_lcount != nbytes);
+ else
+ m_error = (m_lcount == 0);
+
+ // Trigger another read event if there is still data available.
+ m_reading = FALSE;
+
+ // TODO: TriggerRead
return *this;
}
-int wxSocketBase::DeferWrite(const char *buffer, wxUint32 nbytes)
+wxUint32 wxSocketBase::DeferWrite(const char *buffer, wxUint32 nbytes)
{
// Timer for timeout
_wxSocketInternalTimer timer;
wxASSERT(m_defering == NO_DEFER);
+ //wxLogMessage("Entrando a DeferWrite");
m_defering = DEFER_WRITE;
// If the socket is writable, call DoDefer for the first time
if (GSocket_Select(m_socket, GSOCK_OUTPUT_FLAG))
+ {
+ //wxLogMessage("Llamando al primer DoDefer");
DoDefer();
+ }
- // Wait for buffer completion.
+ // Wait for buffer completion.
while (m_defer_buffer != NULL)
- wxYield();
+ PROCESS_EVENTS();
- // Stop timer
- m_defer_timer = NULL;
timer.Stop();
+ // Disable defering mode
+ m_defer_timer = NULL;
m_defering = NO_DEFER;
+ wxString s;
+ s.Printf("Saliendo de DeferWrite: total %d bytes", nbytes-m_defer_nbytes);
+ //wxLogMessage(s);
+
return nbytes-m_defer_nbytes;
}
wxSocketBase& wxSocketBase::Write(const char *buffer, wxUint32 nbytes)
{
- int ret = 1;
+ // Mask write events
+ m_writing = TRUE;
- m_lcount = 0;
- m_error = FALSE;
+ m_lcount = _Write(buffer, nbytes);
+
+ // If in WAITALL mode, all bytes should have been written.
+ if (m_flags & WAITALL)
+ m_error = (m_lcount != nbytes);
+ else
+ m_error = (m_lcount == 0);
+
+ // Trigger another write event if the socket is still writable
+ m_writing = FALSE;
+
+ // TODO: TriggerWrite
+ return *this;
+}
+
+wxUint32 wxSocketBase::_Write(const char *buffer, wxUint32 nbytes)
+{
+ wxUint32 total = 0;
+ int ret = 1;
if (!m_connected || !m_socket)
+ return 0;
+
+ // Possible combinations (they are checked in this order)
+ // NOWAIT
+ // SPEED | WAITALL
+ // SPEED
+ // WAITALL
+ // NONE
+ //
+ if (m_flags & NOWAIT) // NOWAIT
{
- m_error = TRUE;
- return *this;
- }
+ GSocket_SetNonBlocking(m_socket, TRUE);
+ ret = GSocket_Write(m_socket, buffer, nbytes);
+ GSocket_SetNonBlocking(m_socket, FALSE);
- if (m_flags & SPEED & WAITALL) // SPEED && WAITALL
+ if (ret > 0)
+ total = ret;
+ }
+ else if ((m_flags & SPEED) && (m_flags & WAITALL)) // SPEED, WAITALL
{
while (ret > 0 && nbytes > 0)
{
ret = GSocket_Write(m_socket, buffer, nbytes);
- m_lcount += ret;
+ total += ret;
buffer += ret;
nbytes -= ret;
}
// In case the last call was an error ...
if (ret < 0)
- m_lcount ++;
+ total ++;
}
- else if (m_flags & SPEED) // SPEED && !WAITALL
+ else if (m_flags & SPEED) // SPEED, !WAITALL
{
ret = GSocket_Write(m_socket, buffer, nbytes);
if (ret > 0)
- m_lcount += ret;
+ total = ret;
}
- else // !SPEED
+ else // NONE or WAITALL
{
ret = DeferWrite(buffer, nbytes);
if (ret > 0)
- m_lcount += ret;
+ total = ret;
}
- // If we have written some data, then it is not an error,
- // even when in WAITALL mode, the last low-level IO call
- // might have failed.
- if (!m_lcount)
- m_error = TRUE;
-
- return *this;
+ return total;
}
wxSocketBase& wxSocketBase::WriteMsg(const char *buffer, wxUint32 nbytes)
{
+ wxUint32 total;
+ bool error;
int old_flags;
struct {
- char sig[4];
- char len[4];
+ unsigned char sig[4];
+ unsigned char len[4];
} msg;
+ // Mask write events
+ m_writing = TRUE;
+
+ error = TRUE;
+ total = 0;
+ old_flags = m_flags;
+ SetFlags((m_flags & SPEED) | WAITALL);
+
// warning about 'cast truncates constant value'
#ifdef __VISUALC__
- #pragma warning(disable: 4310)
+# pragma warning(disable: 4310)
#endif // __VISUALC__
- msg.sig[0] = (char) 0xad;
- msg.sig[1] = (char) 0xde;
- msg.sig[2] = (char) 0xed;
- msg.sig[3] = (char) 0xfe;
+ msg.sig[0] = (unsigned char) 0xad;
+ msg.sig[1] = (unsigned char) 0xde;
+ msg.sig[2] = (unsigned char) 0xed;
+ msg.sig[3] = (unsigned char) 0xfe;
- msg.len[0] = (char) nbytes & 0xff;
- msg.len[1] = (char) (nbytes >> 8) & 0xff;
- msg.len[2] = (char) (nbytes >> 16) & 0xff;
- msg.len[3] = (char) (nbytes >> 24) & 0xff;
+ msg.len[0] = (unsigned char) nbytes & 0xff;
+ msg.len[1] = (unsigned char) (nbytes >> 8) & 0xff;
+ msg.len[2] = (unsigned char) (nbytes >> 16) & 0xff;
+ msg.len[3] = (unsigned char) (nbytes >> 24) & 0xff;
- // GRG: We need WAITALL | SPEED
- old_flags = m_flags;
- SetFlags(WAITALL | SPEED);
+ //wxLogMessage("Writemsg: %d %d %d %d -> %d",
+ // nbytes & 0xff,
+ // (nbytes >> 8) & 0xff,
+ // (nbytes >> 16) & 0xff,
+ // (nbytes >> 24) & 0xff,
+ // nbytes
+ // );
- if (Write((char *)&msg, sizeof(msg)).LastCount() < sizeof(msg))
- {
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
- }
- if (Write(buffer, nbytes).LastCount() < nbytes)
- {
- SetFlags(old_flags);
- m_error = TRUE;
- return *this;
- }
+ if (_Write((char *)&msg, sizeof(msg)) < sizeof(msg))
+ goto exit;
+
+ total = _Write(buffer, nbytes);
- msg.sig[0] = (char) 0xed;
- msg.sig[1] = (char) 0xfe;
- msg.sig[2] = (char) 0xad;
- msg.sig[3] = (char) 0xde;
+ if (total < nbytes)
+ goto exit;
+
+ msg.sig[0] = (unsigned char) 0xed;
+ msg.sig[1] = (unsigned char) 0xfe;
+ msg.sig[2] = (unsigned char) 0xad;
+ msg.sig[3] = (unsigned char) 0xde;
msg.len[0] = msg.len[1] = msg.len[2] = msg.len[3] = (char) 0;
- if (Write((char *)&msg, sizeof(msg)).LastCount() < sizeof(msg))
- m_error = TRUE;
+ if ((_Write((char *)&msg, sizeof(msg))) < sizeof(msg))
+ goto exit;
- SetFlags(old_flags);
+ // everything was OK
+ error = FALSE;
+
+exit:
+ m_error = error;
+ m_lcount = total;
+ m_writing = FALSE;
+
+ // TODO: TriggerWrite
return *this;
#ifdef __VISUALC__
- #pragma warning(default: 4310)
+# pragma warning(default: 4310)
#endif // __VISUALC__
}
wxSocketBase& wxSocketBase::Unread(const char *buffer, wxUint32 nbytes)
{
- m_error = FALSE;
- m_lcount = 0;
-
if (nbytes != 0)
- {
- CreatePushbackAfter(buffer, nbytes);
- m_lcount = nbytes;
- }
- return *this;
-}
+ Pushback(buffer, nbytes);
-bool wxSocketBase::IsData() const
-{
- if (!m_socket)
- return FALSE;
+ m_error = FALSE;
+ m_lcount = nbytes;
- return (GSocket_Select(m_socket, GSOCK_INPUT_FLAG));
+ return *this;
}
-// GRG: DoDefer() no longer needs to know which event occured,
-// because this was only used to catch LOST events and set
-// m_defer_buffer = NULL; this is done in OnRequest() now.
-
void wxSocketBase::DoDefer()
{
int ret;
if (!m_defer_buffer)
return;
- switch (m_defering)
+ switch(m_defering)
{
case DEFER_READ:
ret = GSocket_Read(m_socket, m_defer_buffer, m_defer_nbytes);
break;
default:
ret = -1;
- break;
}
if (ret >= 0)
// If we are waiting for all bytes to be acquired, keep the defering
// mode enabled.
- if ((m_flags & WAITALL) == 0 || m_defer_nbytes == 0 || ret < 0)
+ if (!(m_flags & WAITALL) || !m_defer_nbytes || ret < 0)
{
m_defer_buffer = NULL;
}
m_defer_buffer += ret;
m_defer_timer->Start(m_timeout * 1000, FALSE);
}
+
+ //wxLogMessage("DoDefer ha transferido %d bytes", ret);
}
-void wxSocketBase::Discard()
+wxSocketBase& wxSocketBase::Discard()
{
#define MAX_BUFSIZE (10*1024)
+ int old_flags;
char *my_data = new char[MAX_BUFSIZE];
wxUint32 recv_size = MAX_BUFSIZE;
wxUint32 total = 0;
- SaveState();
- SetFlags(NOWAIT); // GRG: SPEED was not needed here!
+ // Mask read events
+ m_reading = TRUE;
+
+ old_flags = m_flags;
+ SetFlags(NOWAIT);
while (recv_size == MAX_BUFSIZE)
{
- recv_size = Read(my_data, MAX_BUFSIZE).LastCount();
+ recv_size = _Read(my_data, MAX_BUFSIZE);
total += recv_size;
}
- RestoreState();
delete [] my_data;
-
m_lcount = total;
m_error = FALSE;
+ // Trigger another read event if there is still data available.
+ m_reading = FALSE;
+
+ // TODO: TriggerRead
+ return *this;
+
#undef MAX_BUFSIZE
}
// they poll the socket, using GSocket_Select(), to check for
// the specified combination of event flags, until an event
// occurs or until the timeout ellapses. The polling loop
-// calls wxYield(), so this won't block the GUI.
+// calls PROCESS_EVENTS(), so this won't block the GUI.
bool wxSocketBase::_Wait(long seconds, long milliseconds, wxSocketEventFlags flags)
{
int state = -1;
// Check for valid socket
- if ((!m_connected && !m_establishing) || !m_socket)
+ if (!m_socket)
+ return FALSE;
+
+ // If it is not a server, it must be connected or establishing connection
+ if ((m_type != SOCK_SERVER) && (!m_connected && !m_establishing))
return FALSE;
// Check for valid timeout value
timeout = m_timeout * 1000;
// Activate timer
- timer.m_state = &state;
- timer.m_new_val = 0;
- timer.Start(timeout, TRUE);
+ if (timeout)
+ {
+ timer.m_state = &state;
+ timer.m_new_val = 0;
+ timer.Start(timeout, TRUE);
+ }
// Active polling (without using events)
- result = GSocket_Select(m_socket, flags);
-
- while ((result == 0) && (state == -1))
+ //
+ // NOTE: this duplicates some of the code in OnRequest (lost
+ // connection and connection establishment handling) but this
+ // doesn't hurt. It has to be here because the event might
+ // be a bit delayed, and it has to be in OnRequest as well
+ // because maybe the WaitXXX functions are not being used.
+ //
+ // Do this at least once (important if timeout == 0, when
+ // we are just polling)
+ do
{
- wxYield();
- result = GSocket_Select(m_socket, flags);
+ result = GSocket_Select(m_socket, flags | GSOCK_LOST_FLAG);
+
+ // Connection lost
+ if (result & GSOCK_LOST_FLAG)
+ {
+ timer.Stop();
+ m_defer_buffer = NULL;
+ Close();
+ return TRUE;
+ }
+
+ // Incoming connection (server) or connection established (client)
+ if (result & GSOCK_CONNECTION_FLAG)
+ {
+ timer.Stop();
+ m_connected = TRUE;
+ m_establishing = FALSE;
+ return TRUE;
+ }
+
+ // If we are in the middle of a R/W operation, do not propagate
+ // to users.
+ if ( ((result & GSOCK_INPUT_FLAG) || (result & GSOCK_OUTPUT_FLAG))
+ && (!m_writing) && (!m_reading) )
+ {
+/* TODO: remove this
+ if (m_defer_buffer == NULL)
+*/
+ {
+ timer.Stop();
+ return TRUE;
+ }
+ }
+
+ if (timeout != 0)
+ PROCESS_EVENTS();
}
+ while ((state == -1) && timeout);
timer.Stop();
-
- return (result != 0);
+ return FALSE;
}
bool wxSocketBase::Wait(long seconds, long milliseconds)
bool wxSocketBase::WaitForRead(long seconds, long milliseconds)
{
- return _Wait(seconds, milliseconds, GSOCK_INPUT_FLAG | GSOCK_LOST_FLAG);
+ // Check pushback buffer
+ if (m_unread)
+ return TRUE;
+
+ return _Wait(seconds, milliseconds, GSOCK_INPUT_FLAG);
}
bool wxSocketBase::WaitForWrite(long seconds, long milliseconds)
{
- return _Wait(seconds, milliseconds, GSOCK_OUTPUT_FLAG | GSOCK_LOST_FLAG);
+ return _Wait(seconds, milliseconds, GSOCK_OUTPUT_FLAG);
}
bool wxSocketBase::WaitForLost(long seconds, long milliseconds)
m_flags = _flags;
}
-wxSocketBase::wxSockFlags wxSocketBase::GetFlags() const
-{
- return m_flags;
-}
-
// --------------------------------------------------------------
// wxSocketBase callback management
// --------------------------------------------------------------
// wxSocketBase automatic notifier
// --------------------------------------------------------------
-static void wx_socket_callback(GSocket *socket, GSocketEvent event, char *cdata)
+// All events (INPUT, OUTPUT, CONNECTION, LOST) are now always
+// internally watched; but users will only be notified of those
+// events they are interested in.
+
+static void LINKAGEMODE wx_socket_callback(GSocket * WXUNUSED(socket),
+ GSocketEvent event, char *cdata)
{
wxSocketBase *sckobj = (wxSocketBase *)cdata;
wxSocketEvent event(m_id);
wxSocketEventFlags flag = EventToNotify(req_evt);
+ // fprintf(stderr, "%s: Entering OnRequest (evt %d)\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt);
+
+ // NOTE: this duplicates some of the code in _Wait (lost
+ // connection and connection establishment handling) but
+ // this doesn't hurt. It has to be here because maybe the
+ // WaitXXX are not being used, and it has to be in _Wait
+ // as well because the event might be a bit delayed.
+ //
switch(req_evt)
{
case wxSOCKET_CONNECTION:
case wxSOCKET_OUTPUT:
if (m_defer_buffer)
{
- // GRG: DoDefer() no longer needs to know which
- // event occured, because this was only used to
- // catch LOST events and set m_defer_buffer to
- // NULL, and this is done in OnRequest() now.
+ // fprintf(stderr, "%s: Habia buffer, evt %d skipped\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt);
DoDefer();
// Do not notify to user
return;
break;
}
+ // If we are in the middle of a R/W operation, do not
+ // propagate events to users.
+ if (((req_evt == wxSOCKET_INPUT) && m_reading) ||
+ ((req_evt == wxSOCKET_OUTPUT) && m_writing))
+ {
+ // fprintf(stderr, "%s: Swallowed evt %d\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt);
+ return;
+ }
+
if (((m_neededreq & flag) == flag) && m_notify_state)
{
+ // fprintf(stderr, "%s: Evt %d delivered\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt);
event.m_socket = this;
event.m_skevt = req_evt;
ProcessEvent(event);
if (m_cbk)
m_cbk(*this, req_evt, m_cdata);
}
+
+ // fprintf(stderr, "%s: Exiting OnRequest (evt %d)\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt);
}
-void wxSocketBase::OldOnNotify(wxSocketNotify evt)
+void wxSocketBase::OldOnNotify(wxSocketNotify WXUNUSED(evt))
{
}
}
// --------------------------------------------------------------
-// wxSocketBase pushback library
+// wxSocketBase pushback
// --------------------------------------------------------------
-void wxSocketBase::CreatePushbackAfter(const char *buffer, wxUint32 size)
-{
- char *curr_pos;
-
- if (m_unread != NULL)
- m_unread = (char *) realloc(m_unread, m_unrd_size+size);
- else
- m_unread = (char *) malloc(size);
-
- curr_pos = m_unread + m_unrd_size;
-
- memcpy(curr_pos, buffer, size);
- m_unrd_size += size;
-}
-
-void wxSocketBase::CreatePushbackBefore(const char *buffer, wxUint32 size)
+void wxSocketBase::Pushback(const char *buffer, wxUint32 size)
{
if (m_unread == NULL)
m_unread = (char *)malloc(size);
if (size > (m_unrd_size-m_unrd_cur))
size = m_unrd_size-m_unrd_cur;
+
memcpy(buffer, (m_unread+m_unrd_cur), size);
if (!peek) {
bool wxSocketServer::WaitForAccept(long seconds, long milliseconds)
{
- return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG | GSOCK_LOST_FLAG);
+ return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG);
}
// --------------------------------------------------------------
return FALSE;
GSocket_SetTimeout(m_socket, m_timeout);
+ GSocket_SetCallback(m_socket, GSOCK_INPUT_FLAG | GSOCK_OUTPUT_FLAG |
+ GSOCK_LOST_FLAG | GSOCK_CONNECTION_FLAG,
+ wx_socket_callback, (char *)this);
// GRG: If wait == FALSE, then the call should be nonblocking.
// When we are finished, we put the socket to blocking mode
GSocket_SetPeer(m_socket, addr_man.GetAddress());
err = GSocket_Connect(m_socket, GSOCK_STREAMED);
- GSocket_SetCallback(m_socket, GSOCK_INPUT_FLAG | GSOCK_OUTPUT_FLAG |
- GSOCK_LOST_FLAG | GSOCK_CONNECTION_FLAG,
- wx_socket_callback, (char *)this);
if (!wait)
GSocket_SetNonBlocking(m_socket, FALSE);
bool wxSocketClient::WaitOnConnect(long seconds, long milliseconds)
{
- bool ret;
-
- if (m_connected) // Already connected
+ if (m_connected) // Already connected
return TRUE;
- if (!m_establishing || !m_socket) // No connection in progress
+ if (!m_establishing || !m_socket) // No connection in progress
return FALSE;
- ret = _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG | GSOCK_LOST_FLAG);
-
- // GRG: m_connected and m_establishing will be updated in
- // OnRequest(), but we do it here anyway because sometimes
- // the event might be a bit delayed, and if that happens,
- // when WaitOnConnect() returns, m_connected will still be
- // FALSE. We have to do it as well in OnRequest because
- // maybe WaitOnConnect() is not being used...
-
- if (ret)
- {
- m_connected = GSocket_Select(m_socket, GSOCK_CONNECTION_FLAG);
- m_establishing = FALSE;
- }
-
- return m_connected;
+ return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG);
}
-
// --------------------------------------------------------------
// wxSocketEvent
// --------------------------------------------------------------