From d3db7126e654ac947db7653250030c858070e746 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Philipp=20M=C3=B6ller?= Date: Tue, 28 Jan 2014 17:54:56 +0100 Subject: cmake-mode.el: Don't assign to free variable save-point Drop use of the free variable from unscreamify-cmake-buffer. The proper way to do this would be defvar or let, but it is more sensible to drop the feature completely: replace-match will shift the point and the saved-point will be invalid. More careful coding could avoid this, but seems overkill. --- Auxiliary/cmake-mode.el | 2 -- 1 file changed, 2 deletions(-) diff --git a/Auxiliary/cmake-mode.el b/Auxiliary/cmake-mode.el index 7355b7c..eb6e84e 100644 --- a/Auxiliary/cmake-mode.el +++ b/Auxiliary/cmake-mode.el @@ -186,7 +186,6 @@ the indentation. Otherwise it retains the same position on the line" (defun unscreamify-cmake-buffer () "Convert all CMake commands to lowercase in buffer." (interactive) - (setq save-point (point)) (goto-char (point-min)) (while (re-search-forward "^\\([ \t]*\\)\\(\\w+\\)\\([ \t]*(\\)" nil t) (replace-match @@ -195,7 +194,6 @@ the indentation. Otherwise it retains the same position on the line" (downcase (match-string 2)) (match-string 3)) t)) - (goto-char save-point) ) ;------------------------------------------------------------------------------ -- cgit v0.12