gdi32: Don't create a default color table for pattern brushes, use the DC colors instead.

This commit is contained in:
Alexandre Julliard 2011-12-12 12:23:12 +01:00
parent d5fe87e2e8
commit 65a2c885f1
2 changed files with 26 additions and 7 deletions

View File

@ -79,8 +79,6 @@ static BOOL store_bitmap_bits( BRUSHOBJ *brush, BITMAPOBJ *bmp )
return FALSE; return FALSE;
} }
if (info->bmiHeader.biBitCount <= 8 && !info->bmiHeader.biClrUsed) fill_default_color_table( info );
/* release the unneeded space */ /* release the unneeded space */
HeapReAlloc( GetProcessHeap(), HEAP_REALLOC_IN_PLACE_ONLY, info, HeapReAlloc( GetProcessHeap(), HEAP_REALLOC_IN_PLACE_ONLY, info,
get_dib_info_size( info, DIB_RGB_COLORS )); get_dib_info_size( info, DIB_RGB_COLORS ));
@ -162,6 +160,8 @@ BOOL get_brush_bitmap_info( HBRUSH handle, BITMAPINFO *info, void **bits, UINT *
if (brush->info) if (brush->info)
{ {
memcpy( info, brush->info, get_dib_info_size( brush->info, brush->usage )); memcpy( info, brush->info, get_dib_info_size( brush->info, brush->usage ));
if (info->bmiHeader.biBitCount <= 8 && !info->bmiHeader.biClrUsed)
fill_default_color_table( info );
*bits = brush->bits.ptr; *bits = brush->bits.ptr;
*usage = brush->usage; *usage = brush->usage;
ret = TRUE; ret = TRUE;

View File

@ -1468,10 +1468,11 @@ static BOOL matching_pattern_format( dib_info *dib, dib_info *pattern )
return TRUE; return TRUE;
} }
static BOOL select_pattern_brush( dibdrv_physdev *pdev ) static BOOL select_pattern_brush( dibdrv_physdev *pdev, BOOL *needs_reselect )
{ {
char buffer[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )]; char buffer[FIELD_OFFSET( BITMAPINFO, bmiColors[256] )];
BITMAPINFO *info = (BITMAPINFO *)buffer; BITMAPINFO *info = (BITMAPINFO *)buffer;
RGBQUAD color_table[2];
RECT rect; RECT rect;
dib_info pattern; dib_info pattern;
@ -1490,12 +1491,31 @@ static BOOL select_pattern_brush( dibdrv_physdev *pdev )
copy_bitmapinfo( info, pdev->brush_pattern_info ); copy_bitmapinfo( info, pdev->brush_pattern_info );
fill_color_table_from_pal_colors( info, pdev->dev.hdc ); fill_color_table_from_pal_colors( info, pdev->dev.hdc );
init_dib_info_from_bitmapinfo( &pattern, info, pdev->brush_pattern_bits, 0 ); init_dib_info_from_bitmapinfo( &pattern, info, pdev->brush_pattern_bits, 0 );
*needs_reselect = TRUE;
} }
else else
{ {
init_dib_info_from_bitmapinfo( &pattern, pdev->brush_pattern_info, pdev->brush_pattern_bits, 0 ); init_dib_info_from_bitmapinfo( &pattern, pdev->brush_pattern_info, pdev->brush_pattern_bits, 0 );
} }
if (pattern.bit_count == 1 && !pattern.color_table)
{
/* monochrome DDB pattern uses DC colors */
COLORREF color = GetTextColor( pdev->dev.hdc );
color_table[0].rgbRed = GetRValue( color );
color_table[0].rgbGreen = GetGValue( color );
color_table[0].rgbBlue = GetBValue( color );
color_table[0].rgbReserved = 0;
color = GetBkColor( pdev->dev.hdc );
color_table[1].rgbRed = GetRValue( color );
color_table[1].rgbGreen = GetGValue( color );
color_table[1].rgbBlue = GetBValue( color );
color_table[1].rgbReserved = 0;
pattern.color_table = color_table;
pattern.color_table_size = 2;
*needs_reselect = TRUE;
}
copy_dib_color_info(&pdev->brush_dib, &pdev->dib); copy_dib_color_info(&pdev->brush_dib, &pdev->dib);
pdev->brush_dib.height = pattern.height; pdev->brush_dib.height = pattern.height;
@ -1537,13 +1557,14 @@ static BOOL pattern_brush(dibdrv_physdev *pdev, dib_info *dib, int num, const RE
int i, j; int i, j;
const WINEREGION *clip; const WINEREGION *clip;
POINT origin; POINT origin;
BOOL needs_reselect = FALSE;
if(pdev->brush_and_bits == NULL) if(pdev->brush_and_bits == NULL)
{ {
switch(pdev->brush_style) switch(pdev->brush_style)
{ {
case BS_DIBPATTERN: case BS_DIBPATTERN:
if (!pdev->brush_dib.bits.ptr && !select_pattern_brush( pdev )) if (!pdev->brush_dib.bits.ptr && !select_pattern_brush( pdev, &needs_reselect ))
return FALSE; return FALSE;
if(!create_pattern_brush_bits(pdev)) if(!create_pattern_brush_bits(pdev))
return FALSE; return FALSE;
@ -1600,9 +1621,7 @@ static BOOL pattern_brush(dibdrv_physdev *pdev, dib_info *dib, int num, const RE
} }
release_wine_region( region ); release_wine_region( region );
/* we need to recompute the bits each time for DIB_PAL_COLORS */ if (needs_reselect) free_pattern_brush( pdev );
if (pdev->brush_style == BS_DIBPATTERN && pdev->brush_pattern_usage == DIB_PAL_COLORS)
free_pattern_brush( pdev );
return TRUE; return TRUE;
} }