added specific copy constructor implementatin to ensure string buffer are detached during copy (therefore providing thread-safe copy constructor for non thread-safe std::string copy constructor implementation).

2 files changed, 10 insertions, 1 deletions

diff --git a/include/cppunit/Message.h b/include/cppunit/Message.h
index 9b12431..1ae51cc 100644
--- a/include/cppunit/Message.h
+++ b/include/cppunit/Message.h
@@ -40,6 +40,9 @@ class CPPUNIT_API Message
 public:
   Message();
 
+  // Ensure thread-safe copy by detaching the string.
+  Message( const Message &other );
+
   explicit Message( const std::string &shortDescription );
 
   Message( const std::string &shortDescription,
@@ -54,6 +57,8 @@ public:
            const std::string &detail2,
            const std::string &detail3 );
 
+  Message &operator =( const Message &other );
+
   /*! \brief Returns the short description.
    * \return Short description.
    */
@@ -127,7 +132,6 @@ public:
    */
   bool operator ==( const Message &other ) const;
 
-
   /*! \brief Tests if a message is different from another one.
    * \param other Message this message is compared to.
    * \return \c true if the two message are not identical, \c false otherwise.
diff --git a/include/cppunit/SourceLine.h b/include/cppunit/SourceLine.h
index 71eeaed..f7a85df 100644
--- a/include/cppunit/SourceLine.h
+++ b/include/cppunit/SourceLine.h
@@ -32,9 +32,14 @@ class CPPUNIT_API SourceLine
 public:
   SourceLine();
 
+  // Ensure thread-safe copy by detaching the string buffer.
+  SourceLine( const SourceLine &other );
+
   SourceLine( const std::string &fileName,
               int lineNumber );
 
+  SourceLine &operator =( const SourceLine &other );
+
   /// Destructor.
   virtual ~SourceLine();