You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2013/07/27 07:09:34 UTC

svn commit: r1507576 - in /subversion/trunk/subversion/bindings/cxxhl/src/aprwrap: array.hpp hash.hpp

Author: brane
Date: Sat Jul 27 05:09:34 2013
New Revision: 1507576

URL: http://svn.apache.org/r1507576
Log:
* subversion/bindings/cxxhl/src/aprwrap/hash.hpp (Hash):
   Remove copy constructors. The hash wrapper is trivially copyable
   and the default assignment operators and copy constructors work fine.
* subversion/bindings/cxxhl/src/aprwrap/array.hpp (Array):
   Remove copy constructor; same rationale as above.
   (Note that ConstArray does need it because it also has a conversion
    constructor from Array.)

Modified:
    subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/array.hpp
    subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/hash.hpp

Modified: subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/array.hpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/array.hpp?rev=1507576&r1=1507575&r2=1507576&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/array.hpp (original)
+++ subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/array.hpp Sat Jul 27 05:09:34 2013
@@ -58,13 +58,6 @@ public:
     {}
 
   /**
-   * Create a new proxy for the APR array wrapped by @a that.
-   */
-  Array(const Array& that) throw()
-    : m_array(that.m_array)
-    {}
-
-  /**
    * Create a new proxy for the APR array @a array.
    */
   explicit Array(apr_array_header_t* array)

Modified: subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/hash.hpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/hash.hpp?rev=1507576&r1=1507575&r2=1507576&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/hash.hpp (original)
+++ subversion/trunk/subversion/bindings/cxxhl/src/aprwrap/hash.hpp Sat Jul 27 05:09:34 2013
@@ -120,13 +120,6 @@ public:
     {}
 
   /**
-   * Create a new proxy for the APR hash table wrapped by @a that.
-   */
-  explicit Hash(const Hash& that) throw()
-    : m_hash(that.m_hash)
-    {}
-
-  /**
    * Create a proxy for the APR hash table @a hash.
    */
   explicit Hash(apr_hash_t* hash)
@@ -275,13 +268,6 @@ public:
     {}
 
   /**
-   * Create a new proxy for the APR hash table wrapped by @a that.
-   */
-  explicit Hash(const Hash& that) throw()
-    : inherited(that)
-    {}
-
-  /**
    * Create a proxy for the APR hash table @a hash.
    */
   explicit Hash(apr_hash_t* hash)