[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 1/8] ui/gtk: Remove NULL checks in gd_switch
From: |
Gerd Hoffmann |
Subject: |
[PULL 1/8] ui/gtk: Remove NULL checks in gd_switch |
Date: |
Thu, 11 Mar 2021 13:33:54 +0100 |
From: Akihiko Odaki <akihiko.odaki@gmail.com>
c821a58ee7 ("ui/console: Pass placeholder surface to display")
eliminated the possibility that NULL is passed as surface to
dpy_gfx_switch and removed some NULL checks from gd_switch, but the
removal was not thoroughly. Remaining NULL checks were confusing for
Coverity and probably also for humans. This change removes those NULL
checks.
Reported-by: Coverity (CID 1448421)
Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20210308140713.17901-1-akihiko.odaki@gmail.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
ui/gtk.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/ui/gtk.c b/ui/gtk.c
index 3edaf041defc..eb1c9658e24f 100644
--- a/ui/gtk.c
+++ b/ui/gtk.c
@@ -547,9 +547,7 @@ static void gd_switch(DisplayChangeListener *dcl,
VirtualConsole *vc = container_of(dcl, VirtualConsole, gfx.dcl);
bool resized = true;
- trace_gd_switch(vc->label,
- surface ? surface_width(surface) : 0,
- surface ? surface_height(surface) : 0);
+ trace_gd_switch(vc->label, surface_width(surface),
surface_height(surface));
if (vc->gfx.surface) {
cairo_surface_destroy(vc->gfx.surface);
@@ -560,7 +558,7 @@ static void gd_switch(DisplayChangeListener *dcl,
vc->gfx.convert = NULL;
}
- if (vc->gfx.ds && surface &&
+ if (vc->gfx.ds &&
surface_width(vc->gfx.ds) == surface_width(surface) &&
surface_height(vc->gfx.ds) == surface_height(surface)) {
resized = false;
--
2.29.2
- [PULL 0/8] Ui 20210311 patches, Gerd Hoffmann, 2021/03/11
- [PULL 2/8] ui/cocoa: Use kCGColorSpaceSRGB, Gerd Hoffmann, 2021/03/11
- [PULL 3/8] docs: Fix removal text of -show-cursor, Gerd Hoffmann, 2021/03/11
- [PULL 4/8] ui/cocoa: Show QEMU icon in the about window, Gerd Hoffmann, 2021/03/11
- [PULL 8/8] ui/cocoa: Fix mouse association state, Gerd Hoffmann, 2021/03/11
- [PULL 7/8] ui/cocoa: Mark variables static, Gerd Hoffmann, 2021/03/11
- [PULL 6/8] ui/cocoa: Clear modifiers whenever possible, Gerd Hoffmann, 2021/03/11
- [PULL 5/8] ui/cocoa: Do not rely on the first argument, Gerd Hoffmann, 2021/03/11
- [PULL 1/8] ui/gtk: Remove NULL checks in gd_switch,
Gerd Hoffmann <=
- Re: [PULL 0/8] Ui 20210311 patches, Peter Maydell, 2021/03/12