1 Commits (dfd9bff7bddce6ced0852964855e0f6f8852f170)

Author	SHA1	Message	Date
Michael Richters	b3bf1ae421	MouseKeys "janitorial" work (#1130) ... * Move MouseWrapper class into the `plugin::mousekeys` namespace This helper class is meant to be internal to MouseKeys, so it would be best to sequester it from the shared `kaleidoscope::plugin` namespace. Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Remove unnecessary explicit MouseWrapper constructor Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Remove trailing underscore from MouseWrapper class name Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Rename Mousewrapper methods to comply with coding style guide Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Make MouseWrapper warping utility methods private MouseKeys doesn't call these functions directly, and making them private helps make the API for MouseWrapper clearer. Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Make MouseWrapper public variables comply with code style guide Variables should be in `snake_case`. Since MouseKeys is the only client of `MouseWrapper`, I don't feel it's necessary to go through a deprecation process for the sake of this rename. Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Remove unnecessary explicit MouseKeys constructor Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Remove trailing underscore from MouseKeys class name Because the `MouseKeys_` class is now contained in the `kaleidoscope::plugin` namespace, but the `MouseKeys` object is in the global namespace, we can safely drop the trailing underscore from the class name, like most other plugins. Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Add some namespace comments Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Make `MouseWrapper::subpixels_per_pixel` constexpr Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Change MouseKeys variables from `static` to member variables Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Change name of MouseWrapper object Instead of `kaleidoscope::plugin::MouseWrapper`, it is now named like a variable: `kaleidoscope::plugin::mousekeys::wrapper`. Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Downcase directory name to match namespace Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com> * Adjust header includes to match new dirname Signed-off-by: Michael Richters <gedankenexperimenter@gmail.com>	3 years ago