branch: externals/org
commit 3c8809b3dfcf328c8222917cd55b72a599722c14
Author: Slawomir Grochowski <[email protected]>
Commit: Slawomir Grochowski <[email protected]>

    ; org-colview: Remove redundant row preparation time binding
    
    * lisp/org-colview.el (org-columns--prepare-rows): Remove redundant
    `org-columns--time' binding.  Keep summary time capture in
    `org-columns-compute-all'.
    
    The binding was introduced by ab4ae779f, which extracted
    `org-columns--prepare-rows' from `org-columns'.  It duplicates the
    binding in `org-columns-compute-all', where summary time is captured
    for actual summary computation.
    
    Refactoring: Remove Dead Code.
    
    No behavior change.
---
 lisp/org-colview.el | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/lisp/org-colview.el b/lisp/org-colview.el
index 22f01a2d82..c07fc5073e 100644
--- a/lisp/org-colview.el
+++ b/lisp/org-colview.el
@@ -991,14 +991,13 @@ the buffer."
       (move-marker org-columns-begin-marker (point))
     (setq org-columns-begin-marker (point-marker)))
   (org-columns-goto-top-level)
-  (let ((org-columns--time (float-time)))
-    (org-columns-get-format columns-format)
-    (unless org-columns-inhibit-recalculation (org-columns-compute-all))
-    (save-restriction
-      (when (and (not global) (org-at-heading-p))
-       (narrow-to-region (point) (org-end-of-subtree t t)))
-      (org-columns--compute-clock-summaries)
-      (org-columns--collect-rows))))
+  (org-columns-get-format columns-format)
+  (unless org-columns-inhibit-recalculation (org-columns-compute-all))
+  (save-restriction
+    (when (and (not global) (org-at-heading-p))
+      (narrow-to-region (point) (org-end-of-subtree t t)))
+    (org-columns--compute-clock-summaries)
+    (org-columns--collect-rows)))
 
 ;;;###autoload
 (defun org-columns (&optional global columns-format)

Reply via email to