[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/substitute 673c22fe54 35/39: Rename private function fo
From: |
ELPA Syncer |
Subject: |
[elpa] externals/substitute 673c22fe54 35/39: Rename private function for clarity and move it further up |
Date: |
Mon, 16 Jan 2023 11:58:52 -0500 (EST) |
branch: externals/substitute
commit 673c22fe5401229a861cf814327897c3d585ae69
Author: Protesilaos Stavrou <info@protesilaos.com>
Commit: Protesilaos Stavrou <info@protesilaos.com>
Rename private function for clarity and move it further up
---
substitute.el | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/substitute.el b/substitute.el
index e85adb2bba..23be368ce9 100644
--- a/substitute.el
+++ b/substitute.el
@@ -82,6 +82,13 @@ Use this to produce a more readable version of TARGET for
prompts
and related."
(replace-regexp-in-string "\\\\_<\\(?1:.*?\\)\\\\_>" "\\1" target))
+(defun substitute--determine-face ()
+ "Return face to highlight target of substitute."
+ (if-let* ((face 'lazy-highlight)
+ (facep face))
+ face
+ 'secondary-selection))
+
(defun substitute--prompt-without-highlight (target scope)
"Prompt for string while referencing TARGET and SCOPE."
(let ((pretty-target (substitute--pretty-target target)))
@@ -93,13 +100,6 @@ and related."
'substitute--history
pretty-target)))
-(defun substitute--highlight-face ()
- "Return face to highlight target of substitute."
- (if-let* ((face 'lazy-highlight)
- (facep face))
- face
- 'secondary-selection))
-
(defun substitute--prompt-with-highlight (target scope)
"Prompt for string while referencing TARGET and SCOPE.
Highlight the TARGET's matching occurences per the user option
@@ -107,7 +107,7 @@ Highlight the TARGET's matching occurences per the user
option
(let ((pretty-target (substitute--pretty-target target)))
(unwind-protect
(progn
- (highlight-regexp target (substitute--highlight-face))
+ (highlight-regexp target (substitute--determine-face))
(substitute--prompt-without-highlight pretty-target scope))
(unhighlight-regexp target))))
- [elpa] externals/substitute b665f84d4b 14/39: Reuse code in substitute--prompt-with-highlight, (continued)
- [elpa] externals/substitute b665f84d4b 14/39: Reuse code in substitute--prompt-with-highlight, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 89c3389e1c 15/39: Rename local variable for clarity, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 45096c23d2 16/39: Update symbol of command for clarity, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute f45d238078 21/39: Add missing word in doc string, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 8ac3904158 23/39: Simplify substitute--setup-scope, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 73a1904ea4 24/39: Do not check for fn in 'while' loop, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute cd757fd209 27/39: Remove needless lambdas, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 2fa0edaa5a 29/39: Fix below/above way of identifying current target, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute af3472dc34 33/39: Do not operate on nil thing, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 9ddf421c55 34/39: Rename private function for clarity, ELPA Syncer, 2023/01/16
- [elpa] externals/substitute 673c22fe54 35/39: Rename private function for clarity and move it further up,
ELPA Syncer <=
- [elpa] externals/substitute 18c9830a5c 36/39: Rename private function for clarity, ELPA Syncer, 2023/01/16