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

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

[elpa] master ee6b97d: packages/javaimp/javaimp.el: replace kill-line wi


From: Filipp Gunbin
Subject: [elpa] master ee6b97d: packages/javaimp/javaimp.el: replace kill-line with delete-region
Date: Tue, 10 Mar 2015 18:16:11 +0000

branch: master
commit ee6b97de2627641862ee69c79366ec426f4de8f1
Author: Filipp Gunbin <address@hidden>
Commit: Filipp Gunbin <address@hidden>

    packages/javaimp/javaimp.el: replace kill-line with delete-region
---
 packages/javaimp/javaimp.el |   21 +++++++++++----------
 1 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/packages/javaimp/javaimp.el b/packages/javaimp/javaimp.el
index 888f5c1..822b891 100644
--- a/packages/javaimp/javaimp.el
+++ b/packages/javaimp/javaimp.el
@@ -617,8 +617,7 @@ argument is a list of additional classes to import."
   (interactive)
   (barf-if-buffer-read-only)
   (save-excursion
-    (let ((kill-whole-line t)
-         import-groups static-import-groups old-imports-start)
+    (let (import-groups static-import-groups old-imports-start)
       ;; existing imports
       (goto-char (point-min))
       (while (re-search-forward
@@ -633,9 +632,9 @@ argument is a list of additional classes to import."
                                              static-import-groups)))
        (beginning-of-line)
        (unless old-imports-start (setq old-imports-start (point)))
-       (kill-line)
-       ;; delete whatever happened to be between import statements
-       (when (not (equal (point) old-imports-start))
+       (delete-region (point) (save-excursion (forward-line 1) (point)))
+       ;; delete whatever was between import statements
+       (when (/= (point) old-imports-start)
          (delete-region old-imports-start (point))))
       ;; new imports
       (dolist (class new-classes)
@@ -645,15 +644,17 @@ argument is a list of additional classes to import."
          (progn
            ;; prepare the position
            (cond (old-imports-start
-                  ;; here we do not mangle with empty lines at all
+                  ;; when there were any imports, do not touch blank lines
+                  ;; before imports
                   (goto-char old-imports-start))
                  ((re-search-forward "^\\s-*package\\s-" nil t)
-                  ;; try to preserve all empty lines (if any) before the
-                  ;; following text
-                  (when (equal (forward-line) 1) (insert ?\n)) ;; last line?
+                  ;; when there is a package statement, insert one or two
+                  ;; blank lines after it
+                  (when (= (forward-line) 1) (insert ?\n)) ;; last line?
                   (insert ?\n))
                  (t
-                  ;; start from the bob; add one line after the insert pos
+                  ;; otherwise, start at the bob, insert one empty line
+                  ;; after point
                   (goto-char (point-min))
                   (insert ?\n)
                   (backward-char)))



reply via email to

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