From 2162f01111d21d0ce66ceb8be290b0a13653e691 Mon Sep 17 00:00:00 2001 From: Jake Petroules Date: Mon, 13 Mar 2017 14:22:06 -0700 Subject: Remove some redundant code in QAction Only 2 bits are needed to represent -1, 0, and 1 in a signed integer, and !(n != -1) implies n == -1 so the second condition was redundant. Change-Id: I200a17fe4e49ebc0eaa9a1988b4625267cef61a4 Reviewed-by: Gabriel de Dietrich --- src/widgets/kernel/qaction_p.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/widgets/kernel/qaction_p.h') diff --git a/src/widgets/kernel/qaction_p.h b/src/widgets/kernel/qaction_p.h index 5f4c9a35a3..6813ca08f6 100644 --- a/src/widgets/kernel/qaction_p.h +++ b/src/widgets/kernel/qaction_p.h @@ -105,7 +105,7 @@ public: uint separator : 1; uint fontSet : 1; - int iconVisibleInMenu : 3; // Only has values -1, 0, and 1 + int iconVisibleInMenu : 2; // Only has values -1, 0, and 1 int shortcutVisibleInContextMenu : 2; // Only has values -1, 0, and 1 QAction::MenuRole menuRole; -- cgit v1.2.3