[Emacs-diffs] master 506270f: Don't compare arguments that can be nil

[Oscar Fuentes]

branch: master
commit 506270f9c80bf9bd7dad35a2f0aa6f477da6490b
Author: Oscar Fuentes <address@hidden>
Commit: Oscar Fuentes <address@hidden>

    Don't compare arguments that can be nil
    
    copy-region-as-kill can be called passing nil as `beg' and
    `end'. Magit does that, which caused an error when this advice was in
    effect.
    
    * lisp/ses.el (ses--advice-copy-region-as-kill): avoid comparison
      unless `beg' and `end' are non-nil.
---
 lisp/ses.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/ses.el b/lisp/ses.el
index 4c19c70..188992c 100644
--- a/lisp/ses.el
+++ b/lisp/ses.el
@@ -3051,7 +3051,7 @@ We'll assume copying front-sticky properties doesn't make 
sense, either.
 
 This advice also includes some SES-specific code because otherwise it's too
 hard to override how mouse-1 works."
-  (when (> beg end)
+  (when (and beg end (> beg end))
     (let ((temp beg))
       (setq beg end
            end temp)))