Changeset 5624 for source/ariba/communication

Timestamp:

Aug 3, 2009, 2:29:12 PM (15 years ago)

Author:

Christoph Mayer

Message:

maemo segfault wegen static object ctors workaround

Location:

source/ariba/communication

Files:

• BaseCommunication.cpp (modified) (3 diffs)
• BaseCommunication.h (modified) (2 diffs)
• EndpointDescriptor.cpp (modified) (1 diff)
• EndpointDescriptor.h (modified) (2 diffs)

source/ariba/communication/BaseCommunication.cpp

@@ -244 +244 @@
         } else {
                 LinkDescriptor& linkDesc = queryLocalLink(link);
-                if (linkDesc.isUnspecified()) return EndpointDescriptor::UNSPECIFIED;
+                if (linkDesc.isUnspecified()) return EndpointDescriptor::UNSPECIFIED();
                 return linkDesc.remoteEndpoint;
         }
@@ -534 +534 @@
                 if (linkSet[i]->localLink == link) return (LinkDescriptor&)*linkSet[i];
 
-        LinkDescriptor ret;
-        ret.unspecified = true;
-        return ret;
+        return LinkDescriptor::UNSPECIFIED();
 }
 
@@ -544 +542 @@
                 if (linkSet[i]->remoteLink == link) return (LinkDescriptor&)*linkSet[i];
 
-        LinkDescriptor ret;
-        ret.unspecified = true;
-        return ret;
+        return LinkDescriptor::UNSPECIFIED();
 }
 

source/ariba/communication/BaseCommunication.h

@@ -234 +234 @@
                         localLink(LinkID::UNSPECIFIED), localLocator(NULL),
                         remoteLink(LinkID::UNSPECIFIED), remoteLocator(NULL),
-                        remoteEndpoint(EndpointDescriptor::UNSPECIFIED), up(false), unspecified(false) {
+                        up(false) {
                 }
 
@@ -243 +243 @@
 
                 bool isUnspecified() const {
-                        return unspecified;
+                        return (this == &UNSPECIFIED());
+                }
+
+                static LinkDescriptor& UNSPECIFIED(){
+                        static LinkDescriptor* unspec = NULL;
+                        if(unspec == NULL) unspec = new LinkDescriptor();
+                        return *unspec;
                 }
 

source/ariba/communication/EndpointDescriptor.cpp

@@ -42 +42 @@
 namespace communication {
 
-const EndpointDescriptor EndpointDescriptor::UNSPECIFIED;
 vsznDefault(EndpointDescriptor);
 
-EndpointDescriptor::EndpointDescriptor() {
+EndpointDescriptor::EndpointDescriptor(){
 }
 
-EndpointDescriptor::~EndpointDescriptor() {
+EndpointDescriptor::~EndpointDescriptor(){
 
 }
 
 EndpointDescriptor::EndpointDescriptor(const EndpointDescriptor& rh) :
-        endpoints(rh.endpoints) {
+        endpoints(rh.endpoints){
 }
 
 EndpointDescriptor::EndpointDescriptor(const endpoint_set& endpoints ) :
-        endpoints(endpoints) {
+        endpoints(endpoints){
 }
 
-EndpointDescriptor::EndpointDescriptor(const string& str) : endpoints(str) {
+EndpointDescriptor::EndpointDescriptor(const string& str) : endpoints(str){
 }
 

source/ariba/communication/EndpointDescriptor.h

@@ -56 +56 @@
 
 public:
-        /// the unspecified endpoint descriptor
-        static const EndpointDescriptor UNSPECIFIED;
-
         /// creates an empty endpoint descriptor with zero endpoints
         EndpointDescriptor();
@@ -79 +76 @@
         }
 
+        static EndpointDescriptor& UNSPECIFIED() {
+                static EndpointDescriptor* unspec = NULL;
+                if(unspec == NULL) unspec = new EndpointDescriptor();
+
+                return *unspec;
+        }
+
         /// returns true, if this object is the unspecified object
         bool isUnspecified() const {
-                return this == &UNSPECIFIED;
+                return (this == &UNSPECIFIED());
         }