Re: [patch, suggestion, notes] for Applying CAutoHandle (r21053, r21058)

On 28.03.2011 18:02, Oto BREZINA wrote:
> On 2011-03-28 16:47, Stefan Küng wrote:
>>> Hey.
>>>
>>> AutoHandle should not be used for "preudo-handles" returned by GetCurrentProcess() and possibly some other functions - those pseudo-handles need not be closed. One such use is currentl inside RemotecCacheLink, maybe there're others I haven't found yet.
>> Yes, they don't *need* to be closed, but they *can* be closed.
>> So no reason not to use the auto handle classes for them.
> Interesting. As GetCurentProcess() pseudo-handle can be closed and share
> value with INVALID_HANDLE_VALUE, thus INVALID_HANDLE_VALUE can be closed
> too?
> Correct?

Yes, correct.
As stated in
http://msdn.microsoft.com/en-us/library/ms683179%28VS.85%29.aspx

"Calling the CloseHandle function with a pseudo handle has no effect.".

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=2714794
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].