On 2026-05-10 07:02, Ihor Radchenko wrote:
> This assumes that all the instances of filtered/invisible entry are
> really hidden. Do we really need to make such assumption?

IUUC ‘(not (invisible-p …))’ should be used instead to respect
‘buffer-invisibility-spec’. Or, are you referring to the fact that I
only check visibility of the closest overlay? In my testing I couldn’t
come up with a scenario where only some of the clocking overlays were
hidden, but I can check inside the loop:

(sort (remq nil (delete-dups
                 (mapcar (lambda (o)
                           (when-let* ((_ (eq (overlay-get o 'type)
                                              'org-agenda-clocking))
                                       (start (overlay-start o))
                                       (_ (not (invisible-p start))))
                             start))
                         (overlays-in (point-min) (point-max)))))
      :key (lambda (p) (count-lines p pt t)))

Thanks,

-- 
Jacob S. Gordon
[email protected]
Please don’t send me HTML emails or MS Office/Apple iWork documents.
https://useplaintext.email/#etiquette
https://www.fsf.org/campaigns/opendocument

Reply via email to