Changeset 2437 for source/ariba/Name.h

Timestamp:

Feb 17, 2009, 12:05:25 PM (15 years ago)

Author:

Christoph Mayer

Message:

-memory leaks und corruption gefixt

File:

• source/ariba/Name.h (modified) (6 diffs)

source/ariba/Name.h

@@ -42 +42 @@
 #include <iostream>
 #include <memory.h>
+#include <string>
+
+using std::string;
 
 // forward declaration
@@ -60 +63 @@
 class Name {
         friend std::ostream& operator<<( std::ostream&, const ::ariba::Name& );
-
-private:
-        bool _hreadable;
-        bool _copy;
-        int _length;
-        uint8_t* _bytes;
-
-        inline void init(const char* name, int len, bool copy, bool hreadable) {
-                if (len == -1)
-                        len = strlen(name);
-
-                if (copy) {
-                        _bytes = new uint8_t[len];
-                        memcpy(_bytes, name, len);
-                } else {
-                        _bytes = (uint8_t*)name;
-                }
-
-                _copy = copy;
-                _length = len;
-                _hreadable = hreadable;
-        }
-
 public:
         static const Name UNSPECIFIED;
 
-public:
         /**
          * Constructs a new, yet unspecified name.
          */
-        inline Name() {
-                _bytes = NULL;
-                _length = 0;
-                _copy = false;
-                _hreadable = false;
-        }
+        Name();
 
         /**
@@ -105 +79 @@
          * @param copy A flag, whether the name's memory needs to be copied
          */
-        inline Name(const char* name, int len = -1, bool copy = false) {
-                init(name, len, copy, len == -1);
-        }
+        Name(const char* name, int len = -1, bool copy = false);
 
         /**
@@ -114 +86 @@
          * @param name A human readable name
          */
-        inline Name(std::string name) {
-                init(name.c_str(), -1, true, true);
-        }
+        Name(string name);
 
         /**
          * The copy constructor.
          */
-        inline Name(const Name& name) {
-                init((const char*)name.bytes(), name.length(), true, name._hreadable);
-        }
+        Name(const Name& name);
 
         /**
          * Destroys the name and releases underlying memory if this name is a copy.
          */
-        inline ~Name() {
-                if (_copy) delete _bytes;
-        }
+        virtual ~Name();
 
         /**
@@ -137 +103 @@
          * @return The binary data
          */
-        inline const uint8_t* bytes() const {
-                return _bytes;
-        }
+        const uint8_t* bytes() const;
 
         /**
@@ -146 +110 @@
          * @return The length of the name
          */
-        inline const size_t length() const {
-                return _length;
-        }
+        const size_t length() const;
 
         /**
          * The common implementation of the "equal" operator.
          */
-        inline bool operator==(const Name& name) const {
-                if (_bytes == NULL || name._bytes == NULL) return false;
-                if (name.length() != length()) return false;
-                return (memcmp(name.bytes(), bytes(), length()) == 0);
-        }
+        bool operator==(const Name& name) const;
 
         /**
          * The common implementation of the "unequal" operator.
          */
-        inline bool operator!=(const Name& name) const {
-                return !(*this == name);
-        }
+        bool operator!=(const Name& name) const;
 
         /**
          * Returns true, if the name is yet unspecified
          */
-        inline bool isUnspecified() {
-                return *this == UNSPECIFIED;
-        }
+        bool isUnspecified() const;
 
         /**
          * Returns a random name.
          */
-        inline static Name random() {
-                char name[17];
-                for (int i=0;i<16; i++)
-                        name[i] = ("abcdefghijklmnopqrstuvwxyz")[((unsigned)rand())%26];
-                name[16] = 0;
-                return Name(name);
-        }
+        static Name random();
 
         /**
          * Returns a human-readable representation of this name
          */
-        inline std::string toString() const {
-                if (_hreadable) {
-                        char str[256];
-                        for (int i=0; i<length(); i++) str[i] = bytes()[i];
-                        str[length()] = 0;
-                        return std::string(str);
-                } else {
-                        return std::string("<not readable>");
-                }
-        }
+        string toString() const;
 
         // hack: to be changed!
-        inline NodeID toNodeId() const {
-                return NodeID::sha1( bytes(), length() );
-        }
+        NodeID toNodeId() const;
 
         // hack: to be changed!
-        inline SpoVNetID toSpoVNetId() const {
-                return SpoVNetID::sha1(bytes(), length() );
-        }
+        SpoVNetID toSpoVNetId() const;
+
+private:
+        bool _hreadable;
+        bool _copy;
+        int _length;
+        uint8_t* _bytes;
+
+        void init(const char* name, int len, bool copy, bool hreadable);
 };