From 9d7622798296ec8ce1427c87ffdc0dddec048294 Mon Sep 17 00:00:00 2001
From: Jules Aguillon <jules@j3s.fr>
Date: Fri, 26 Jul 2024 00:10:54 +0200
Subject: [PATCH] check_layout.py: Handle compass-point attributes

---
 check_layout.output | 6 ++----
 check_layout.py     | 8 +++++++-
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/check_layout.output b/check_layout.output
index 2aeb76842..4545118d0 100644
--- a/check_layout.output
+++ b/check_layout.output
@@ -30,8 +30,7 @@ Layout includes some ASCII punctuation but not all, missing: $, &, *, ., /, <, >
 # cyrl_jcuken_uk
 0 warnings
 # cyrl_lynyertz_sr
-Layout doesn't define some important keys, missing: delete, loc esc, loc tab
-1 warnings
+0 warnings
 # cyrl_ueishsht
 0 warnings
 # cyrl_yaverti
@@ -102,8 +101,7 @@ Duplicate keys: !, ', ,, -, ., ?
 # latn_qwerty_sk
 0 warnings
 # latn_qwerty_sr
-Layout doesn't define some important keys, missing: delete, loc esc, loc tab
-1 warnings
+0 warnings
 # latn_qwerty_tly
 Duplicate keys: a, c, j, q
 Layout doesn't define some important keys, missing: loc esc, loc tab
diff --git a/check_layout.py b/check_layout.py
index 47a17b627..9224f79af 100644
--- a/check_layout.py
+++ b/check_layout.py
@@ -9,6 +9,12 @@
     "greekmath", "numeric", "emoji_bottom_row",
     "clipboard_bottom_row" ])
 
+KEY_ATTRIBUTES = set([
+    "key0",
+    "key1", "key2", "key3", "key4", "key5", "key6", "key7", "key8",
+    "nw", "ne", "sw", "se", "w", "e", "n", "s"
+    ])
+
 def warn(msg):
     global warning_count
     print(msg)
@@ -39,7 +45,7 @@ def unexpected_keys(keys, symbols, msg):
 def parse_row_from_et(row, keys, dup):
     for key in row:
         for attr in key.keys():
-            if attr.startswith("key"):
+            if attr in KEY_ATTRIBUTES:
                 k = key.get(attr).removeprefix("\\")
                 if k in keys: dup.add(k)
                 keys.add(k)