Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] st/nine: Ignore window size if error

2019-02-28 Thread Axel Davy

On 28/02/2019 12:54, Emil Velikov wrote:

On Wed, 27 Feb 2019 at 22:49, Axel Davy  wrote:

Check GetWindowInfo and ignore the computed sizes
if there is an error.

Fixes the regression caused by:
commit 2318ca68bbeb4fa6e21a4d8c650cec3f64246596
"st/nine: Handle window resize when a presentation buffer is used"
when using old wine gallium nine patches

Related issues:
https://github.com/iXit/Mesa-3D/issues/331
https://github.com/iXit/Mesa-3D/issues/332

Fixes also crash at window destruction.

Cc: mesa-sta...@lists.freedesktop.org

Signed-off-by: Axel Davy 
---

Nittiest of nits: the following takes 1/3 the cognitive effort.




Nittiest of nits do matter. Thank you for the suggestion, I shall take 
replace the commit message with your suggestion !





Check GetWindowInfo and ignore the computed sizes if there is an error.

Fixes a regression caused by earlier commit when using old wine gallium
nine patches.

Should also address a crash at window destruction.

Related issues:
  https://github.com/iXit/Mesa-3D/issues/331
  https://github.com/iXit/Mesa-3D/issues/332

Cc: mesa-sta...@lists.freedesktop.org
Fixes: 2318ca68bbe ("st/nine: Handle window resize when a presentation
buffer is used")
Signed-off-by: Axel Davy 


HTH
-Emil



___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] st/nine: Ignore window size if error

2019-02-28 Thread Emil Velikov
On Wed, 27 Feb 2019 at 22:49, Axel Davy  wrote:
>
> Check GetWindowInfo and ignore the computed sizes
> if there is an error.
>
> Fixes the regression caused by:
> commit 2318ca68bbeb4fa6e21a4d8c650cec3f64246596
> "st/nine: Handle window resize when a presentation buffer is used"
> when using old wine gallium nine patches
>
> Related issues:
> https://github.com/iXit/Mesa-3D/issues/331
> https://github.com/iXit/Mesa-3D/issues/332
>
> Fixes also crash at window destruction.
>
> Cc: mesa-sta...@lists.freedesktop.org
>
> Signed-off-by: Axel Davy 
> ---

Nittiest of nits: the following takes 1/3 the cognitive effort.

Check GetWindowInfo and ignore the computed sizes if there is an error.

Fixes a regression caused by earlier commit when using old wine gallium
nine patches.

Should also address a crash at window destruction.

Related issues:
 https://github.com/iXit/Mesa-3D/issues/331
 https://github.com/iXit/Mesa-3D/issues/332

Cc: mesa-sta...@lists.freedesktop.org
Fixes: 2318ca68bbe ("st/nine: Handle window resize when a presentation
buffer is used")
Signed-off-by: Axel Davy 


HTH
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev