Changeset 174

Timestamp:

03/21/09 17:06:46 (15 years ago)

Author:

bhilburn

Message:

Moving the SendMessage? function over to a const char*, which (I think) should be
the proper way to do it. Still need someone to explain the '\0000' in the old
function...

Location:

vtcross/trunk/src

Files:

• include/vtcross/socketcomm.h (modified) (1 diff)
• lib/socketcomm/socketcomm.cpp (modified) (3 diffs)

vtcross/trunk/src/include/vtcross/socketcomm.h

@@ -26 +26 @@
 /* TODO
  */
-int32_t SendMessage(int32_t socketFD, char* message);
+//int32_t SendMessage(int32_t socketFD, char* message);
+int32_t SendMessage(int32_t socketFD, const char* message) ;
 
 

vtcross/trunk/src/lib/socketcomm/socketcomm.cpp

@@ -14 +14 @@
 #include <netinet/in.h>
 #include <stdint.h>
+#include <string>
 #include <sys/types.h>
 #include <sys/socket.h>
@@ -75 +76 @@
 
     
+/*
 int32_t
 SendMessage(int32_t socketFD, char* message) 
@@ -83 +85 @@
     strcat(message, "\0000");
 
+    ssize_t numSentBytes = send(socketFD, message, (strlen(message) + 1), 0);
+    if(numSentBytes < 0) {
+        ERROR(1, "Error sending to server.");
+    }
+    else if(numSentBytes == 0) {
+        LOG("socket_comm::SendMessage - Server closed the socket.\n");
+    }
+   
+    return numSentBytes;
+}
+*/
+
+// TODO this function is here to handle calls to send const char* messages. Note
+// that the std::string.c_str() function auto-appends a null character at the
+// end of the cstring, so the strcat function call in the previous function
+// isn't necessary here... I think... although I still don't really understand
+// what exactly that call is for.
+int32_t
+SendMessage(int32_t socketFD, const char* message) 
+{
     ssize_t numSentBytes = send(socketFD, message, (strlen(message) + 1), 0);
     if(numSentBytes < 0) {