KittyPatch/kitty
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remote control: Improve create-marker error messages

Browse Source 
No more missing color errors when text/regex are missing.
Use `mark group` in error messages instead of `color`.
This commit is contained in:
pagedown 2023-01-27 23:31:06 +08:00
parent 8638e42135
commit aacb4db2dc
No known key found for this signature in database
GPG Key ID: E921CF18AC8FF6EB
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
kitty/options/utils.py
View File

@ -329,13 +329,13 @@ def parse_marker_spec(ftype: str, parts: Sequence[str]) -> Tuple[str, Union[str,
if ftype.startswith('i'): if ftype.startswith('i'):
flags |= re.IGNORECASE flags |= re.IGNORECASE
if not parts or len(parts) % 2 != 0: if not parts or len(parts) % 2 != 0:
raise ValueError('No color specified in marker: {}'.format(' '.join(parts))) raise ValueError('Mark group number and text/regex are not specified in pairs: {}'.format(' '.join(parts)))
ans = [] ans = []
for i in range(0, len(parts), 2): for i in range(0, len(parts), 2):
try: try:
color = max(1, min(int(parts[i]), 3)) color = max(1, min(int(parts[i]), 3))
except Exception: except Exception:
raise ValueError(f'color {parts[i]} in marker specification is not an integer') raise ValueError(f'Mark group in marker specification is not an integer: {parts[i]}')
sspec = parts[i + 1] sspec = parts[i + 1]
if 'regex' not in ftype: if 'regex' not in ftype:
sspec = re.escape(sspec) sspec = re.escape(sspec)