From 5d7726e3f18a457b2e7cec68a308cbede6520fae Mon Sep 17 00:00:00 2001 From: Ilya Ivanov Date: Thu, 27 Apr 2017 15:06:56 +0300 Subject: [PATCH] Fix check for wxACC_STATE_SYSTEM_MARQUEED in accessibility code wxACC_STATE_SYSTEM_INVISIBLE was checked twice, by mistake, while wxACC_STATE_SYSTEM_MARQUEED was never checked at all. Thanks to PVS-Studio for finding this issue (V581: The conditional expressions of the 'if' operators situated alongside each other are identical). --- src/msw/ole/access.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/msw/ole/access.cpp b/src/msw/ole/access.cpp index fd06b3043f..7c5d3541d5 100644 --- a/src/msw/ole/access.cpp +++ b/src/msw/ole/access.cpp @@ -2037,8 +2037,8 @@ long wxConvertToWindowsState(long wxstate) if (wxstate & wxACC_STATE_SYSTEM_INVISIBLE) state |= STATE_SYSTEM_INVISIBLE; - if (wxstate & wxACC_STATE_SYSTEM_INVISIBLE) - state |= STATE_SYSTEM_INVISIBLE; + if (wxstate & wxACC_STATE_SYSTEM_MARQUEED) + state |= STATE_SYSTEM_MARQUEED; if (wxstate & wxACC_STATE_SYSTEM_MIXED) state |= STATE_SYSTEM_MIXED;