Re: Inconsistency between code and manual: org-lowest-priority or org-priority-lowest
Hi Kyle, Kyle Meyer writes: > Daniele Nicolodi writes: > >> On 30/10/2020 05:57, Kyle Meyer wrote: > >>> The org-X-priority -> org-priority-X rename happened in v9.4, with >>> org-X-priority names retained as aliases. So, it sounds like there are >>> some leftover bits in the code. >> >> You are absolutely right. This is what you get when you read the manual >> for the latest version but look at the code for an old one... > > Quickly grepping, a few instances of the old names remain in the code > base, if you're still interested in sending a patch. I fixed the ones I've found in 370cf49cd and ff5fd323b. Thanks! -- Bastien
Re: Inconsistency between code and manual: org-lowest-priority or org-priority-lowest
Daniele Nicolodi writes: > On 30/10/2020 05:57, Kyle Meyer wrote: >> The org-X-priority -> org-priority-X rename happened in v9.4, with >> org-X-priority names retained as aliases. So, it sounds like there are >> some leftover bits in the code. > > You are absolutely right. This is what you get when you read the manual > for the latest version but look at the code for an old one... Quickly grepping, a few instances of the old names remain in the code base, if you're still interested in sending a patch.
Re: Inconsistency between code and manual: org-lowest-priority or org-priority-lowest
On 30/10/2020 05:57, Kyle Meyer wrote: > Daniele Nicolodi writes: > >> Hello, >> >> I found an inconsistency between the manual and implementation: the >> Priorities section says that the range of valid priorities can be set >> modifying the `org-priority-highest', `org-priority-lowest' and >> `org-priority-default' variables. However, in the code the variables >> names are `org-highest-priority', `org-lowest-priority' and >> `org-default-priority'. >> >> In can submit a patch with a fix. However, in a case like this, should >> the code modified to match the documentation, or vice-versa? > > The org-X-priority -> org-priority-X rename happened in v9.4, with > org-X-priority names retained as aliases. So, it sounds like there are > some leftover bits in the code. You are absolutely right. This is what you get when you read the manual for the latest version but look at the code for an old one... Cheers, Dan
Re: Inconsistency between code and manual: org-lowest-priority or org-priority-lowest
Daniele Nicolodi writes: > Hello, > > I found an inconsistency between the manual and implementation: the > Priorities section says that the range of valid priorities can be set > modifying the `org-priority-highest', `org-priority-lowest' and > `org-priority-default' variables. However, in the code the variables > names are `org-highest-priority', `org-lowest-priority' and > `org-default-priority'. > > In can submit a patch with a fix. However, in a case like this, should > the code modified to match the documentation, or vice-versa? The org-X-priority -> org-priority-X rename happened in v9.4, with org-X-priority names retained as aliases. So, it sounds like there are some leftover bits in the code. Thanks.
Inconsistency between code and manual: org-lowest-priority or org-priority-lowest
Hello, I found an inconsistency between the manual and implementation: the Priorities section says that the range of valid priorities can be set modifying the `org-priority-highest', `org-priority-lowest' and `org-priority-default' variables. However, in the code the variables names are `org-highest-priority', `org-lowest-priority' and `org-default-priority'. In can submit a patch with a fix. However, in a case like this, should the code modified to match the documentation, or vice-versa? Thank you. Cheers, Dan