diff mbox series

ui/gtk-egl.c: fix build break

Message ID 20240515115005.3553790-1-anushree.mathur@linux.vnet.ibm.com (mailing list archive)
State New, archived
Headers show
Series ui/gtk-egl.c: fix build break | expand

Commit Message

Anushree Mathur May 15, 2024, 11:50 a.m. UTC
Qemu build is failing due to an unused variable.
Removing it to fix the build break.

Signed-off-by: Anushree Mathur <anushree.mathur@linux.vnet.ibm.com>
---
 ui/gtk-egl.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Philippe Mathieu-Daudé May 15, 2024, 12:06 p.m. UTC | #1
Hi Anushree,

On 15/5/24 13:50, Anushree Mathur wrote:
> Qemu build is failing due to an unused variable.
> Removing it to fix the build break.
> 
> Signed-off-by: Anushree Mathur <anushree.mathur@linux.vnet.ibm.com>
> ---
>   ui/gtk-egl.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
> index 0473f689c9..3177992b91 100644
> --- a/ui/gtk-egl.c
> +++ b/ui/gtk-egl.c
> @@ -70,7 +70,6 @@ void gd_egl_draw(VirtualConsole *vc)
>       QemuDmaBuf *dmabuf = vc->gfx.guest_fb.dmabuf;
>   #endif
>       int ww, wh, ws;
> -    int fence_fd;
>   
>       if (!vc->gfx.gls) {
>           return;

Well, it is used few lines later when CONFIG_GBM
is defined ;)

Cédric sent a complete patch:
https://lore.kernel.org/qemu-devel/20240515100520.574383-1-clg@redhat.com/
diff mbox series

Patch

diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c
index 0473f689c9..3177992b91 100644
--- a/ui/gtk-egl.c
+++ b/ui/gtk-egl.c
@@ -70,7 +70,6 @@  void gd_egl_draw(VirtualConsole *vc)
     QemuDmaBuf *dmabuf = vc->gfx.guest_fb.dmabuf;
 #endif
     int ww, wh, ws;
-    int fence_fd;
 
     if (!vc->gfx.gls) {
         return;