[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 3/5] ui/cocoa: Fix switched_to_fullscreen warning
From: |
Philippe Mathieu-Daudé |
Subject: |
[PULL 3/5] ui/cocoa: Fix switched_to_fullscreen warning |
Date: |
Wed, 13 Jul 2022 00:16:05 +0200 |
From: Peter Delevoryas <peter@pjd.dev>
I noticed this error while building QEMU on Mac OS X:
[1040/1660] Compiling Objective-C object libcommon.fa.p/ui_cocoa.m.o
../ui/cocoa.m:803:17: warning: variable 'switched_to_fullscreen' set but
not used [-Wunused-but-set-variable]
static bool switched_to_fullscreen = false;
^
1 warning generated.
I think the behavior is fine if you remove "switched_to_fullscreen", I can
still switch in and out of mouse grabbed mode and fullscreen mode with this
change, and Command keycodes will only be passed to the guest if the mouse
is grabbed, which I think is the right behavior. I'm not sure why a static
piece of state was needed to handle that in the first place. Perhaps the
refactoring of the flags-state-change fixed that by toggling the Command
keycode on.
I tested this with an Ubuntu core image on macOS 12.4
wget
https://cdimage.ubuntu.com/ubuntu-core/18/stable/current/ubuntu-core-18-i386.img.xz
xz -d ubuntu-core-18-i386.img.xz
qemu-system-x86_64 -drive file=ubuntu-core-18.i386.img,format=raw
Fixes: 6d73bb643aa7 ("ui/cocoa: Clear modifiers whenever possible")
Signed-off-by: Peter Delevoryas <peter@pjd.dev>
Reviewed-by: Akihiko Odaki <akihiko.odaki@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220702044304.90553-1-peter@pjd.dev>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
ui/cocoa.m | 8 --------
1 file changed, 8 deletions(-)
diff --git a/ui/cocoa.m b/ui/cocoa.m
index 6a4dccff7f..e883c7466e 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -800,7 +800,6 @@ - (bool) handleEventLocked:(NSEvent *)event
int buttons = 0;
int keycode = 0;
bool mouse_event = false;
- static bool switched_to_fullscreen = false;
// Location of event in virtual screen coordinates
NSPoint p = [self screenLocationOfEvent:event];
NSUInteger modifiers = [event modifierFlags];
@@ -952,13 +951,6 @@ - (bool) handleEventLocked:(NSEvent *)event
// forward command key combos to the host UI unless the mouse is
grabbed
if (!isMouseGrabbed && ([event modifierFlags] &
NSEventModifierFlagCommand)) {
- /*
- * Prevent the command key from being stuck down in the guest
- * when using Command-F to switch to full screen mode.
- */
- if (keycode == Q_KEY_CODE_F) {
- switched_to_fullscreen = true;
- }
return false;
}
--
2.36.1
- [PULL 0/5] Darwin patches for 2022-07-12, Philippe Mathieu-Daudé, 2022/07/12
- [PULL 1/5] hvf: Enable RDTSCP support, Philippe Mathieu-Daudé, 2022/07/12
- [PULL 2/5] configure: Restrict TCG to emulation, Philippe Mathieu-Daudé, 2022/07/12
- [PULL 3/5] ui/cocoa: Fix switched_to_fullscreen warning,
Philippe Mathieu-Daudé <=
- [PULL 4/5] ui/cocoa: Take refresh rate into account, Philippe Mathieu-Daudé, 2022/07/12
- [PULL 5/5] avocado: Fix BUILD_DIR if it's equal to SOURCE_DIR, Philippe Mathieu-Daudé, 2022/07/12
- Re: [PULL 0/5] Darwin patches for 2022-07-12, Peter Maydell, 2022/07/14