Derek Chen-Becker <[email protected]> writes:
> I'm also not clear on why we upcase again when checking the bounds:
>
> ((or (< (upcase new) org-priority-highest) (> (upcase new)
> org-priority-lowest))
That's a historical code. It has been there before the earlier upcase
was introduced.
Here is the initial version of that code:
((eq action 'set)
(message "Priority A-%c, SPC to remove: " org-lowest-priority)
(setq new (read-char-exclusive))
(cond ((equal new ?\ ) (setq remove t))
((or (< (upcase new) ?A) (> (upcase new) org-lowest-priority))
(error "Priority must be between `%c' and `%c'"
?A org-lowest-priority))))
--
Ihor Radchenko // yantar92,
Org mode maintainer,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
