From 346c16b4e2ea26f47e0e370a490b7794492a9ebb Mon Sep 17 00:00:00 2001
From: Akshay Nair <phenax5@gmail.com>
Date: Sun, 23 Mar 2025 19:48:53 +0530
Subject: Apply clang-tidy suggestions

---
 src/keymap/KeySeqParser.hpp | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

(limited to 'src/keymap/KeySeqParser.hpp')

diff --git a/src/keymap/KeySeqParser.hpp b/src/keymap/KeySeqParser.hpp
index 2dad0c7..eb878ab 100644
--- a/src/keymap/KeySeqParser.hpp
+++ b/src/keymap/KeySeqParser.hpp
@@ -1,18 +1,17 @@
 #pragma once
 
-#include <QtCore/qnamespace.h>
 #include <QtCore>
-#include <cmath>
+#include <cstdint>
 
 struct KeyChord {
   Qt::KeyboardModifiers mod;
   Qt::Key key;
 };
-bool operator==(const KeyChord a, const KeyChord b);
+bool operator==(KeyChord chord1, KeyChord chord2);
 
-typedef QList<KeyChord> KeySequence;
+using KeySequence = QList<KeyChord>;
 
-enum KeyMatchType {
+enum KeyMatchType : uint8_t {
   NoMatch,
   Match,
   Pending,
@@ -20,8 +19,8 @@ enum KeyMatchType {
 
 class KeySeqParser {
 public:
-  static KeyMatchType keySequenceMatch(const KeySequence target,
-                                       const KeySequence current) {
+  static KeyMatchType key_sequence_match(const KeySequence &target,
+                                         const KeySequence &current) {
     for (int i = 0; i < target.length(); i++) {
       if (current.length() <= i)
         return KeyMatchType::Pending;
@@ -34,11 +33,9 @@ public:
     return KeyMatchType::Match;
   }
 
-public:
-  KeySeqParser();
-
-  KeySequence parse(QString keySequence);
+  KeySeqParser() = default;
+  KeySequence parse(QString key_sequence);
 
 private:
-  Qt::Key parseKey(QString keyName);
+  Qt::Key parse_key(const QString &key_name);
 };
-- 
cgit v1.3.1