Возможная утечка памяти при использовании GetHBITMAP () и MFC CStatic :: SetBitmap () - PullRequest
Новая
       17

Возможная утечка памяти при использовании GetHBITMAP () и MFC CStatic :: SetBitmap ()

2 голосов
/ 13 декабря 2011

Я вызываю следующую функцию каждые 100 мс. Его цель - взять изображение из renderBuffer , изменить его размер и отобразить его в элементе управления CStatic диалога с помощью SetBitmap () . Проблема в том, что я наблюдаю довольно большой всплеск использования памяти каждую секунду при выполнении этой функции. Он отображает изображение с измененным размером в элементе управления CStatis без проблем, но я вижу в диспетчере задач, что каждый второй процесс выделяет дополнительные 4 мегабайта памяти и никогда не останавливается, пока у процесса не заканчивается память.

Вот код, дайте мне знать, если у вас есть идеи, в чем может быть проблема. 

void CAppDlg::UpdatePreview( const RenderBuffer* renderBuffer )
{
HBITMAP hbmReturn = NULL; 
Gdiplus::Bitmap  *bmPhoto  = NULL;
Gdiplus::Bitmap bmPhoto( THUMBNAIL_WIDTH, THUMBNAIL_HEIGHT );
CBitmap Bmp1;

Gdiplus::Bitmap image( 780, 780, 4*780, PixelFormat32bppARGB, renderBuffer->buffer );

int sourceWidth  = image.GetWidth();
int sourceHeight = image.GetHeight();

int destX = 0,
    destY = 0; 

float nPercent  = 0;
float nPercentW = ((float)THUMBNAIL_WIDTH/(float)sourceWidth);;
float nPercentH = ((float)THUMBNAIL_HEIGHT/(float)sourceHeight);

if(nPercentH < nPercentW)
{
    nPercent = nPercentH;
    destX    = (int)((THUMBNAIL_WIDTH - (sourceWidth * nPercent))/2);
}
else
{
    nPercent = nPercentW;
    destY    = (int)((THUMBNAIL_HEIGHT - (sourceHeight * nPercent))/2);
}

int destWidth  = (int)(sourceWidth * nPercent);
int destHeight = (int)(sourceHeight * nPercent);

bmPhoto.SetResolution( image.GetHorizontalResolution(), image.GetVerticalResolution() );

Gdiplus::Graphics *grPhoto = Gdiplus::Graphics::FromImage( &bmPhoto );
Gdiplus::Color colorW(255, 255, 255, 255);
grPhoto->Clear( colorW );
grPhoto->SetInterpolationMode( Gdiplus::InterpolationModeHighQualityBicubic );
grPhoto->DrawImage( &image, Gdiplus::Rect(destX, destY, destWidth, destHeight) );

bmPhoto.GetHBITMAP( colorW, &hbmReturn );

m_BitmapPreview.SetBitmap( hbmReturn ); // ---- without this line memory usage doesn't go up rapidly every second.

DeleteObject( hbmReturn ); // ---- returns non-zero which would point out that it was freed properly.
delete grPhoto;
}

Спасибо за любую помощь!

Привет.

1 Ответ

1 голос
/ 13 декабря 2011

Полагаю, вам следует использовать DeleteObject .

Вот как должен выглядеть код, на мой взгляд: 

// ..............
bmPhoto.GetHBITMAP( colorW, &hbmReturn );

HBITMAP prev = m_BitmapPreview.SetBitmap( hbmReturn ); // ---- without this line memory usage doesn't go up rapidly every second.
if (NULL != prev)
{
   DeleteObject(prev); // *** do not forget to delete the previously associated bitmap
}

DeleteObject( hbmReturn ); // ---- returns non-zero which would point out that it was freed properly.
// .........
...