From cc358a60c41e60d0204659d3593cb263cfddae8e Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Sat, 6 Jul 2024 20:07:14 -0400 Subject: [PATCH] refactor(org): remove unused hooks These hooks had the potential to be too expensive (in larger Org files) to be enabled by default. --- modules/lang/org/autoload/org-tables.el | 9 --------- modules/lang/org/autoload/org.el | 7 ------- modules/lang/org/config.el | 5 +---- 3 files changed, 1 insertion(+), 20 deletions(-) diff --git a/modules/lang/org/autoload/org-tables.el b/modules/lang/org/autoload/org-tables.el index f231562cd..8ac85d087 100644 --- a/modules/lang/org/autoload/org-tables.el +++ b/modules/lang/org/autoload/org-tables.el @@ -53,15 +53,6 @@ re-align the table if necessary. (Necessary because org-mode has a (if org-table-may-need-update (org-table-align)) (goto-char pt)))) -;;;###autoload -(defun +org-enable-auto-reformat-tables-h () - "Realign tables & update formulas when exiting insert mode (`evil-mode'). -Meant for `org-mode-hook'." - (when (featurep 'evil) - (add-hook 'evil-insert-state-exit-hook #'+org-realign-table-maybe-h nil t) - (add-hook 'evil-replace-state-exit-hook #'+org-realign-table-maybe-h nil t) - (advice-add #'evil-replace :after #'+org-realign-table-maybe-a))) - ;;;###autoload (defun +org-delete-backward-char-and-realign-table-maybe-h () "Ensure deleting characters with backspace doesn't deform the table cell." diff --git a/modules/lang/org/autoload/org.el b/modules/lang/org/autoload/org.el index 87085b434..499b1dddb 100644 --- a/modules/lang/org/autoload/org.el +++ b/modules/lang/org/autoload/org.el @@ -538,10 +538,3 @@ All my (performant) foldings needs are met between this and `org-show-subtree' (when org-occur-highlights (org-remove-occur-highlights) t)) - -;;;###autoload -(defun +org-enable-auto-update-cookies-h () - "Update statistics cookies when saving or exiting insert mode (`evil-mode')." - (when (bound-and-true-p evil-local-mode) - (add-hook 'evil-insert-state-exit-hook #'org-update-parent-todo-statistics nil t)) - (add-hook 'before-save-hook #'org-update-parent-todo-statistics nil t)) diff --git a/modules/lang/org/config.el b/modules/lang/org/config.el index d344d57b1..1c9bb3f06 100644 --- a/modules/lang/org/config.el +++ b/modules/lang/org/config.el @@ -1305,10 +1305,7 @@ between the two." ;; `org-indent-mode', so we turn off show-paren-mode altogether #'doom-disable-show-paren-mode-h ;; disable `show-trailing-whitespace'; shows a lot of false positives - #'doom-disable-show-trailing-whitespace-h - ;; #'+org-enable-auto-reformat-tables-h - ;; #'+org-enable-auto-update-cookies-h - ) + #'doom-disable-show-trailing-whitespace-h) (add-hook! 'org-load-hook #'+org-init-org-directory-h