clang warning in wx/mousestate.h
|Reported by:||wojdyr||Owned by:|
/home/wojdyr/local/include/wx-2.9/wx/mousestate.h:106:13: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default] default: ^ /home/wojdyr/local/include/wx-2.9/wx/mousestate.h:108:24: warning: will never be executed [-Wunreachable-code] return false; ^~~~~ 2 warnings generated.
If the default: case is needed for some compilers, maybe it can be put after
case wxMOUSE_BTN_NONE: case wxMOUSE_BTN_MAX:
this would avoid the second warning.
(BTW recently I had a chance to work with github/bitbucket pull requests, it's a very convenient workflow. It would be nice if wxwidgets migrated to git in the future. According to github, svn access to their git repos also works.)