diff options
author | Determinant <tederminant@gmail.com> | 2020-05-22 13:45:57 -0400 |
---|---|---|
committer | Determinant <tederminant@gmail.com> | 2020-05-22 13:45:57 -0400 |
commit | 9b2098bcbc93633bbdbac7402a33d503e1adb77b (patch) | |
tree | 6f753422135ac2ee11eb04a6e1b43f9b425bfc24 | |
parent | ce6e5757b048e6186b6554c323ca5ff9e8f1ab67 (diff) |
improve the error message
-rwxr-xr-x | name2color.py | 8 | ||||
-rwxr-xr-x | tmux-colortag-prompt.sh | 6 |
2 files changed, 8 insertions, 6 deletions
diff --git a/name2color.py b/name2color.py index e407a5a..2137687 100755 --- a/name2color.py +++ b/name2color.py @@ -11,7 +11,7 @@ def warn(msg): def error(msg): - print(msg + '\n') + print("ColorTag: {}\n".format(msg)) saved_state = os.path.expanduser("~/.tmux-colortag.state") @@ -67,14 +67,16 @@ if args.clear_name: changed = True if not (args.color_idx is None): - if args.color_idx < 0 and args.color_idx >= 256: + if args.color_idx < 0 or args.color_idx >= 256: error("invalid color code") + sys.exit(0) state[args.idx] = args.color_idx changed = True if not (args.color_name is None): - if args.color_name < 0 and args.color_name >= 256: + if args.color_name < 0 or args.color_name >= 256: error("invalid color code") + sys.exit(0) state[args.name] = args.color_name changed = True diff --git a/tmux-colortag-prompt.sh b/tmux-colortag-prompt.sh index 74cfdf8..8e2aeaf 100755 --- a/tmux-colortag-prompt.sh +++ b/tmux-colortag-prompt.sh @@ -6,10 +6,10 @@ case "$1" in tmux command-prompt -p '[ColorTag]:' "run-shell 'idx=#I name=#W $CURRENT_DIR/tmux-colortag-prompt.sh %1'" ;; color-idx) - "$CURRENT_DIR/name2color.py" "$idx" "$name" --color-idx "$2" || echo "invalid argument" + "$CURRENT_DIR/name2color.py" "$idx" "$name" --color-idx "$2" || echo "ColorTag: invalid argument" ;; color-name) - "$CURRENT_DIR/name2color.py" "$idx" "$name" --color-name "$2" || echo "invalid argument" + "$CURRENT_DIR/name2color.py" "$idx" "$name" --color-name "$2" || echo "ColorTag: invalid argument" ;; clear-idx) "$CURRENT_DIR/name2color.py" "$idx" "$name" --clear-idx @@ -20,5 +20,5 @@ case "$1" in clear-all) "$CURRENT_DIR/name2color.py" "$idx" "$name" --clear ;; - *) echo "invalid ColorTag command"; exit 0;; + *) echo "ColorTag: invalid command"; exit 0;; esac |