emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[nongnu] elpa/orgit e147f05577 61/76: eval-after-load: replace string fe


From: ELPA Syncer
Subject: [nongnu] elpa/orgit e147f05577 61/76: eval-after-load: replace string feature w/ symbol
Date: Thu, 13 Jan 2022 13:58:47 -0500 (EST)

branch: elpa/orgit
commit e147f055772cc934fe1f1d8619059badeb647c93
Author: Henrik Lissner <henrik@lissner.net>
Commit: Henrik Lissner <henrik@lissner.net>

    eval-after-load: replace string feature w/ symbol
    
    TL;DR Given (eval-after-load FILE FORM), eval-after-load is less
    efficient if FILE is a string than when it's a symbol.
    
    When FILE is a string, eval-after-load creates a regexp entry in
    after-load-alist that Emacs uses to scan the path of *every* file it
    loads, and continues to do so after FILE is loaded.
    
    When FILE is a symbol, you avoid this polling behavior. Instead, there
    is no such aggressive lookup behavior. FORM isn't evaluated until
    a (provide 'FILE) is evaluated in the target package.
---
 orgit.el | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/orgit.el b/orgit.el
index d843aaf0da..ad9e0f1730 100644
--- a/orgit.el
+++ b/orgit.el
@@ -217,7 +217,7 @@ then store a link to the commit itself."
 ;;; Command
 
 ;;;###autoload
-(eval-after-load "magit"
+(eval-after-load 'magit
   '(define-key magit-mode-map [remap org-store-link] 'orgit-store-link))
 
 ;;;###autoload
@@ -237,7 +237,7 @@ then store a link to the commit itself."
 ;;; Status
 
 ;;;###autoload
-(eval-after-load "org"
+(eval-after-load 'org
   '(orgit-link-set-parameters "orgit"
                               :store    'orgit-status-store
                               :follow   'orgit-status-open
@@ -272,7 +272,7 @@ In that case `orgit-rev-store' stores one or more links 
instead."
 ;;; Log
 
 ;;;###autoload
-(eval-after-load "org"
+(eval-after-load 'org
   '(orgit-link-set-parameters "orgit-log"
                               :store    'orgit-log-store
                               :follow   'orgit-log-open
@@ -327,7 +327,7 @@ In that case `orgit-rev-store' stores one or more links 
instead."
 ;;; Revision
 
 ;;;###autoload
-(eval-after-load "org"
+(eval-after-load 'org
   '(orgit-link-set-parameters "orgit-rev"
                               :store    'orgit-rev-store
                               :follow   'orgit-rev-open



reply via email to

[Prev in Thread] Current Thread [Next in Thread]