[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emacs-diffs] master 506270f: Don't compare arguments that can be nil
From: |
Oscar Fuentes |
Subject: |
[Emacs-diffs] master 506270f: Don't compare arguments that can be nil |
Date: |
Fri, 15 Dec 2017 19:40:37 -0500 (EST) |
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)))
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Emacs-diffs] master 506270f: Don't compare arguments that can be nil,
Oscar Fuentes <=