Codejock Developer Community : CXTPImageManagerIconHandle bug! http://forum.codejock.com/ Copyright (c) 2006-2013 Web Wiz Forums - All Rights Reserved. Mon, 13 Apr 2026 02:11:00 +0000 Tue, 09 Sep 2008 14:56:31 +0000 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 12.04 360 forum.codejock.com/RSS_post_feed.asp?TID=12041 <![CDATA[Codejock Developer Community]]> http://forum.codejock.com/forum_images/codejock-logo.gif http://forum.codejock.com/ <![CDATA[CXTPImageManagerIconHandle bug! : Hi, you can change it but you...]]> http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40861&title=cxtpimagemanagericonhandle-bug#40861 Author: Oleg
Subject: 12041
Posted: 09 September 2008 at 2:56pm

Hi,
you can change it but you will get a lot of resource leaks.
 
better change your method to get reference
 
bool foo(CXTPImageManagerIconHandle& out)
]]> Tue, 09 Sep 2008 14:56:31 +0000 http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40861&title=cxtpimagemanagericonhandle-bug#40861 <![CDATA[CXTPImageManagerIconHandle bug! : I disagree. The class is broken...]]> http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40860&title=cxtpimagemanagericonhandle-bug#40860 Author: znakeeye
Subject: 12041
Posted: 09 September 2008 at 2:43pm

I disagree. The class is broken since it cannot be returned without destroying the icon.]]> Tue, 09 Sep 2008 14:43:49 +0000 http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40860&title=cxtpimagemanagericonhandle-bug#40860 <![CDATA[CXTPImageManagerIconHandle bug! : Hi, Its&#039;wrong. Don&#039;t...]]> http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40759&title=cxtpimagemanagericonhandle-bug#40759 Author: Oleg
Subject: 12041
Posted: 08 September 2008 at 3:27am

Hi,
 
Its'wrong. Don't change it.
]]> Mon, 08 Sep 2008 03:27:40 +0000 http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40759&title=cxtpimagemanagericonhandle-bug#40759 <![CDATA[CXTPImageManagerIconHandle bug! : CXTPImageManagerIconHandle...]]> http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40649&title=cxtpimagemanagericonhandle-bug#40649 Author: znakeeye
Subject: 12041
Posted: 02 September 2008 at 4:14am

CXTPImageManagerIconHandle foo()
{
    CXTPImageManagerIconHandle hIcon;
    hIcon = ...
   
    return hIcon; // Destructor called for hIcon
}
 
hIcon.m_bClearHandles == TRUE... so the returned object will be corrupted. I believe you should use auto_ptr-thinking here!
 
In copy-constructor and assignment-operator the right-hand-side variable m_bClearHandles should be set to FALSE!
 
h1 = h2; // h2.m_bClearHandles = FALSE
 
My solution:
mutable BOOL m_bClearHandles;
 
CXTPImageManagerIconHandle::CXTPImageManagerIconHandle(const CXTPImageManagerIconHandle& hHandle)
{
    ...
    hHandle.m_bClearHandles = FALSE;
}
]]> Tue, 02 Sep 2008 04:14:20 +0000 http://forum.codejock.com/forum_posts.asp?TID=12041&PID=40649&title=cxtpimagemanagericonhandle-bug#40649