chore: code cleanup

[codepuncher]

Code cleanup

[Copilot]

Pull request overview

Refactors small pieces of plugin configuration parsing and input handling to reduce ad-hoc types and avoid unnecessary work, without changing the user-facing behavior of QuickMap’s trigger button handling.

Changes:

• Replaced std::pair return from ReadButton() with a small ButtonDef struct and returned map values directly.
• Removed redundant deleted move operations from InputHandler and marked SetButton as noexcept.
• Avoided computing “held duration” on button release when a long-press already triggered the map, and moved _pressTime.reset() to after dispatch.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
src/Plugin.cpp	Introduces ButtonDef for button config parsing and returns structured values directly.
src/InputHandler.h	Removes redundant move-deletes and updates SetButton signature to noexcept.
src/InputHandler.cpp	Updates SetButton definition and streamlines button-up handling to skip unused duration computation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.