From f48caff3606c28dd5d9f4891d8b30dfe1c4b96be Mon Sep 17 00:00:00 2001 From: alexey-milovidov Date: Wed, 28 Mar 2018 05:27:25 +0300 Subject: [PATCH] Update COWPtr.h --- dbms/src/Common/COWPtr.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/dbms/src/Common/COWPtr.h b/dbms/src/Common/COWPtr.h index 39c912f8abc..525f2372c6c 100644 --- a/dbms/src/Common/COWPtr.h +++ b/dbms/src/Common/COWPtr.h @@ -68,10 +68,9 @@ * of this shared state. * * Caveats: - * - after a call to 'mutate' method, you can still have a reference to immutable ptr somewhere - * and it can still become shared. Also it would be better to make 'mutate' method rvalue-qualified. + * - after a call to 'mutate' method, you can still have a reference to immutable ptr somewhere. * - as 'mutable_ptr' should be unique, it's refcount is redundant - probably it would be better - * to use std::unique_ptr for it, but see above. + * to use std::unique_ptr for it somehow. */ template class COWPtr : public boost::intrusive_ref_counter