Changeset 12060 for source/ariba/utility/transport/messages/shared_buffer.hpp

Timestamp:

Jun 19, 2013, 11:05:49 AM (11 years ago)

Author:

hock@…

Message:

Reintegrate branch: 20130111-hock-message_classes

improvements:

• new message classes (reboost, zero-copy)
• "fast path" for direct links (skip overlay layer)
• link-properties accessible from the application
• SystemQueue can call boost::bind functions
• protlib compatibility removed (32bit overhead saved in every message)
• addressing2
• AddressDiscovery discoveres only addresses on which we're actually listening
• ariba serialization usage reduced (sill used in OverlayMsg)
• Node::connect, easier and cleaner interface to start-up ariba from the application
• ariba configs via JSON, XML, etc (boost::property_tree)
• keep-alive overhead greatly reduced
• (relayed) overlay links can actually be closed now
• lost messages are detected in most cases
• notification to the application when link is transformed into direct-link
• overlay routing: send message to second best hop if it would be dropped otherwise
• SequenceNumbers (only mechanisms, so for: upward compatibility)
• various small fixes

regressions:

• bluetooth is not yet working again
• bootstrap modules deactivated
• liblog4xx is not working (use cout-logging)

This patch brings great performance and stability improvements at cost of backward compatibility.
Also bluetooth and the bootstrap modules have not been ported to the new interfaces, yet.

Location:

source/ariba/utility/transport

Files:

• . (modified) (1 prop)
• messages/shared_buffer.hpp (modified) (4 diffs)

source/ariba/utility/transport/messages/shared_buffer.hpp

@@ -9 +9 @@
 
 #include <cstring>
+#include <string>
 #include <boost/shared_ptr.hpp>
 
@@ -18 +19 @@
 #include "buffer.hpp"
 
+#include <stdexcept> 
+
 namespace reboost {
+
+class illegal_sub_buffer: public std::runtime_error
+{
+public:
+    /** Takes a character string describing the error.  */
+    explicit illegal_sub_buffer(const std::string& __arg)  :
+        std::runtime_error(__arg)
+    {
+    }
+    
+    virtual ~illegal_sub_buffer() throw() {}
+};
 
 /**
@@ -104 +119 @@
                 parent(new deleteable_buffer(buffer, size))
         {
+        }
+
+//    /// XXX debug... copy!
+//      /// create shared buffer from buffer
+//      inline shared_buffer_t(const char* buffer, bsize_t size) :
+//              buffer_t(), parent(new deleteable_buffer(size)) {
 //              memcpy(parent->mutable_data(), buffer, parent->size());
-//              data(parent->mutable_data());
-//              this->size(parent->size());
-        }
+//              data(parent->mutable_data()); this->size(parent->size());
+//      }
 
         /// clone data from a normal buffer
@@ -129 +149 @@
 
         /// return sub-buffer.
-        inline self operator()(bsize_t index, bsize_t size = 0) const {
+        inline self operator()(bsize_t index, bsize_t size = 0) const
+        {
+            // special cases
+            if ( index + size > size_ )
+            {
+                // empty sub-buffer
+            if ( index == size_ )
+            {
+                self n;
+                return n;
+            }
+         
+            // ERROR: index out of bounds
+            throw illegal_sub_buffer("Index or size out of bounds in shared_buffer"); 
+            }
+
+            // regular case
                 self n(*this);
                 n.data_ += index;