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

Re: AW: Conceptual Patch - Handle wrapper class

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Sat, 26 Mar 2011 09:47:13 +0100

On 26.03.2011 09:40, Oto BREZINA wrote:
> On 2011-03-15 22:27, Oto BREZINA wrote:
>> On 2011-03-15 22:09, Stefan Küng wrote:
>>> That's why I don't like to use wrapper classes if there's no way to
>>> make them work in all situations, or break *reliably* in situations
>>> where it must not be used.
>> I still don't think that there is no way to make it working in all
>> situations, just not sure as worthy it is.
>>> And I think that's exactly the reason why the existing CHandle class
>>> only supports NULL as the invalid value and not INVALID_HANDLE_VALUE.
>> Sounds as good start point for me :)
>> Let say:
>> - Invalid is only NULL,
>> - INVALID_HANDLE_VALUE - don't call CloseHandle
>> - You are forced to enter invalid value into constructor
>> CTsvnHandle::CTsvnHandle(HANDLE h, HANDLE hInvalid) ....
>> m_h = (h==hInvalid) ? NULL : h;
>> BIG BAD new invalid value is NULL while you code always checks for
>> Even this may be handled in operator= method
> May we try to collect all requirements?

I've got something working. Need to do some more tests before I commit
the change (which is quite big, considering that handles are used
throughout the app).


   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-03-26 09:47:27 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.