Landscape.io was complaining about a "del" acting on loop variables that were "possibly undefined" because it didn't recognize that the loop was iterating over a hard-coded non-empty list.
(Make monitor-next and monitor-prev actually do what they say)
(using "return" in place of "raise" in EnumSafeDict)
(Since the internal API isn't frozen yet, there's no downside and it helps to make it more clear which operations aren't intended to have side-effects ...which will help come test suite time.)
- It is now possible to specify additional modifiers in the [keys] section to allow things like Ctrl+Alt+KP_4 being tiling and Ctrl+Alt+Shift+KP_4 being moving to another workspace. - Should remove the need to manually import keysym groups outside latin1 and miscellany - Supports more familiar aliases like Alt instead of Mod1 and Ctrl instead of Control. Also, in the process, I made the validity checks more granular and helpful.