mirror of
https://gitlab.freedesktop.org/xorg/xserver.git
synced 2026-05-04 10:39:06 +02:00
Handle transform failure when computing shadow damage area.
PictureTransformBounds can fail, when this happens, damage the entire screen so that the shadow gets repainted correctly.
This commit is contained in:
parent
b93c9a1553
commit
15cd904558
3 changed files with 16 additions and 5 deletions
|
|
@ -177,7 +177,17 @@ xf86CrtcDamageShadow (xf86CrtcPtr crtc)
|
|||
damage_box.x2 = crtc->mode.HDisplay;
|
||||
damage_box.y1 = 0;
|
||||
damage_box.y2 = crtc->mode.VDisplay;
|
||||
PictureTransformBounds (&damage_box, &crtc->crtc_to_framebuffer);
|
||||
if (1 || !PictureTransformBounds (&damage_box, &crtc->crtc_to_framebuffer))
|
||||
{
|
||||
damage_box.x1 = 0;
|
||||
damage_box.y1 = 0;
|
||||
damage_box.x2 = pScreen->width;
|
||||
damage_box.y2 = pScreen->height;
|
||||
}
|
||||
if (damage_box.x1 < 0) damage_box.x1 = 0;
|
||||
if (damage_box.y1 < 0) damage_box.y1 = 0;
|
||||
if (damage_box.x2 > pScreen->width) damage_box.x2 = pScreen->width;
|
||||
if (damage_box.y2 > pScreen->height) damage_box.y2 = pScreen->height;
|
||||
REGION_INIT (pScreen, &damage_region, &damage_box, 1);
|
||||
DamageDamageRegion (&(*pScreen->GetScreenPixmap)(pScreen)->drawable,
|
||||
&damage_region);
|
||||
|
|
|
|||
|
|
@ -84,7 +84,6 @@ _X_EXPORT Bool
|
|||
PictureTransformPoint (PictTransformPtr transform,
|
||||
PictVectorPtr vector)
|
||||
{
|
||||
PictVector result;
|
||||
int i, j;
|
||||
xFixed_32_32 partial;
|
||||
xFixed_34_30 v[3];
|
||||
|
|
@ -227,7 +226,7 @@ PictureTransformTranslate (PictTransformPtr forward,
|
|||
return TRUE;
|
||||
}
|
||||
|
||||
_X_EXPORT void
|
||||
_X_EXPORT Bool
|
||||
PictureTransformBounds (BoxPtr b, PictTransformPtr matrix)
|
||||
{
|
||||
PictVector v[4];
|
||||
|
|
@ -240,7 +239,8 @@ PictureTransformBounds (BoxPtr b, PictTransformPtr matrix)
|
|||
v[3].vector[0] = F (b->x1); v[3].vector[1] = F (b->y2); v[3].vector[2] = F(1);
|
||||
for (i = 0; i < 4; i++)
|
||||
{
|
||||
PictureTransformPoint (matrix, &v[i]);
|
||||
if (!PictureTransformPoint (matrix, &v[i]))
|
||||
return FALSE;
|
||||
x1 = xFixedToInt (v[i].vector[0]);
|
||||
y1 = xFixedToInt (v[i].vector[1]);
|
||||
x2 = xFixedToInt (xFixedCeil (v[i].vector[0]));
|
||||
|
|
@ -258,6 +258,7 @@ PictureTransformBounds (BoxPtr b, PictTransformPtr matrix)
|
|||
if (y2 > b->y2) b->y2 = y2;
|
||||
}
|
||||
}
|
||||
return TRUE;
|
||||
}
|
||||
|
||||
_X_EXPORT Bool
|
||||
|
|
|
|||
|
|
@ -727,7 +727,7 @@ PictureTransformTranslate (PictTransformPtr forward,
|
|||
PictTransformPtr reverse,
|
||||
xFixed tx, xFixed ty);
|
||||
|
||||
void
|
||||
Bool
|
||||
PictureTransformBounds (BoxPtr b, PictTransformPtr matrix);
|
||||
|
||||
Bool
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue