mickey committed e65a529

* Greatly improved the region shifting code.

Comments (0)

Files changed (1)

 ;;  * Add basic syntax highlighting to inferior python.
+;;  * Shift regions of code around and reindents according to the
+;;    indentation depth of that block
 ;;; How to use
   (error "NYI")
-(defun python-mp-shift-region (arg)
+(defun python-mp-indentation-at-point (pt)
+  "Determines the indentation at PT. This approach does not use
+\\[python-mode]'s internal data structures as we're not
+interested in the *possible* indentation levels but merely what
+PT currently has.
+If the line contains nothing but whitespace (as determined by the
+syntax table) then and `indent-count' is 0 we recursively
+backtrack one line at a time until that condition is no longer
+  (save-excursion
+    (unless (bobp)
+      (goto-char pt)
+      (beginning-of-line)
+      (setq indent-count (- (progn
+           ;; `line-end-position' seems like the best way to limit the
+           ;; search; but is it enough?
+           (skip-syntax-forward " " (line-end-position))
+           (point))
+         (point-at-bol)))
+      ;; can `indent-count' ever be less than 0?  make sure we're eolp
+      ;; also or we run into the nasty situation where `indent-count'
+      ;; is 0 but without an empty line.
+      (if (and (= indent-count 0) (eolp))
+          (progn
+            (forward-line -1)
+            (python-mp-indentation-at-point (point)))
+        indent-count))))
+(defun python-mp-shift-region (arg subr)
   "Shifts the active region ARG times up (backward if ARG is
 negative) and reindents the code according to the indentation
 depth in that block. "
   ;;; `'.
   (if (region-active-p)
+        ;; (if (or (and (bobp) (< arg 0))
+        ;;         (and (eobp) (< arg 0)))
+        ;;     (error "Cannot shift region the further."))
         (if (> (point) (mark))
         (let ((column (current-column))
           (move-to-column column t)
           (set-mark (point))
           (insert text)
+          ;; post-move stuff here.
+          (if (eq subr 'smart)
+              (indent-region (point) (mark) (python-mp-indentation-at-point (mark))))          
           (setq deactivate-mark nil)))
     (error "Region shifting only works when transient-mark-mode is enabled.")))
   "If the region is active and \\[transient-mark-mode] is enabled
 the region will be shifted down ARG times and reindented."
   (interactive "*p")
-  (python-mp-shift-region arg))
+  (python-mp-shift-region arg 'smart))
 (defun python-mp-shift-region-up (arg)
   "If the region is active and \\[transient-mark-mode] is enabled
 the region will be shifted up ARG times and reindented."
   (interactive "*p")
-  (python-mp-shift-region (- arg)))
+  (python-mp-shift-region (- arg) 'smart))
 ;;; inferior-python-mode