Mario Frasca writes:
> On 28/06/2020 01:32, Kyle Meyer wrote:
>> Thanks for reporting. Looks like org-todo's call to org-map-entries has
>> been broken since its introduction in 7.8.05. Should be fixed with
>> 8a2eb7d1b.
>
> 3ed035ce3
>
> with this patch, you don't need to locally redefine the
> org-loop-over-headlines-in-active-region variable, do you?
What have you tried on your end to answer that? Here are some
approaches:
* Read the commit message in 3ed035ce3 (org.el: Clarify argument in
org-todo's call to org-map-entries). Does its explanation fit with
your understanding? Perhaps it's wrong or not written clearly, but
it seems like a good starting point.
* Drop the let-binding of org-loop-over-headlines-in-active-region in
org-todo, and try it out. Does it work as before? Better yet, step
through the call of the function, comparing the behavior before and
after.
* Drop the let-binding and run the test I added in 8a2eb7d1b (org.el:
Fix org-todo argument handling when looping over region).
