more small log improvements

keymapper/dev/keycode_mapper.py

@@ -226,8 +226,8 @@ def _get_key(event, key_to_code, macros):
 
 def print_unreleased():
     """For debugging purposes."""
-    print('unreleased:')
+    logger.debug('unreleased:')
-    print('\n'.join([
+    logger.debug('\n'.join([
         f'    {key}: {str(value)}' for key, value in unreleased.items()
     ]))
 
@@ -367,4 +367,4 @@ def handle_keycode(key_to_code, macros, event, uinput, forward=True):
         Unreleased((event_tuple[:2]), event_tuple, None)
         return
 
-    logger.error(key, '%s unhandled. %s %s', unreleased, active_macros)
+    logger.error('%s unhandled', key)

keymapper/mapping.py

@@ -116,8 +116,8 @@ class Mapping(ConfigBase):
             raise ValueError('Expected `character` not to be None')
 
         logger.debug(
-            '%s will map to %s, replacing %s',
+            '%s will map to "%s"',
-            new_key, character, previous_key
+            new_key, character
         )
         self.clear(new_key)  # this also clears all equivalent keys
         self._mapping[new_key] = character
@@ -146,6 +146,8 @@ class Mapping(ConfigBase):
                 logger.debug('%s will be cleared', permutation)
                 del self._mapping[permutation]
                 self.changed = True
+                # there should be only one variation of the permutations
+                # in the mapping actually
 
     def empty(self):
         """Remove all mappings."""