[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: Oto BREZINA <otik_at_printflow.eu>
Date: Sat, 26 Mar 2011 09:40:02 +0100

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
> INVALID_HANDLE_VALUE.
> Even this may be handled in operator= method
May we try to collect all requirements?

-- 
Oto BREZINA, Printflow s.r.o., EU
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2714299
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-03-26 09:46:00 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.