[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Patch - prohibited operations should not have implementations

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Tue, 21 Dec 2010 20:38:19 +0100

On 21.12.2010 09:21, Dmitry wrote:
> Hey.
>
> Many classes in TSVN are non-copyable by design. They have copy constructor and assignment operator private. Yeat some of them have those operations with empty implementation:
>
> class Class:
> private:
> Class(const Class&){}
> void operator=(const Class&){}
> };
>
> this is dangerous - those implementations might be accidentially called from within the same class member functions and make the program malfunction.
>
> Attached is an untested patch that addresses this. If it breaks the build it will mean that it is actually effective.

It didn't break the build.
Committed in r20630.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2692487
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2010-12-21 20:38:39 CET

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.