CXTPGraphicBitmapPng

Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic

   Hi,

could you split CXTPGraphicBitmapPng::LoadFromResource into two methods, LoadFromResource and LoadFromBuffer?

LPBYTE lpBuffer = (LPBYTE)LockResource(hMem);
if (!lpBuffer || nBufferSize < 10)
   return FALSE;

CMemFile file(lpBuffer, nBufferSize);
return LoadFromBuffer  (...);

<--- Split function here --->

BOOL CXTPGraphicBitmapPng::LoadFromBuffer (...)
{
   CMemFile file(lpBuffer, nBufferSize)
   ....
}

That doesn't add any extra code but makes the class more flexible.

Author	Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic
ABuenger Members Profile Send Private Message Find Members Posts Add to Buddy List Newbie Joined: 02 February 2006 Status: Offline Points: 1075	Post Options Post Reply Quote ABuenger Report Post Thanks(0) Quote Reply Topic: CXTPGraphicBitmapPng Posted: 29 November 2006 at 9:09am
	Hi, could you split CXTPGraphicBitmapPng::LoadFromResource into two methods, LoadFromResource and LoadFromBuffer? LPBYTE lpBuffer = (LPBYTE)LockResource(hMem); if (!lpBuffer \|\| nBufferSize < 10) return FALSE; ~~CMemFile file(lpBuffer, nBufferSize);~~ return LoadFromBuffer (...); <--- Split function here ---> BOOL CXTPGraphicBitmapPng::LoadFromBuffer (...) { CMemFile file(lpBuffer, nBufferSize) .... } That doesn't add any extra code but makes the class more flexible.
	Codejock support

Oleg Members Profile Send Private Message Find Members Posts Add to Buddy List Admin Group Joined: 21 May 2003 Location: United States Status: Offline Points: 11234	Post Options Post Reply Quote Oleg Report Post Thanks(0) Quote Reply Posted: 30 November 2006 at 1:41am
	ok
	Oleg, Support Team CODEJOCK SOFTWARE SOLUTIONS