[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/vc-fossil 9e52264 028/111: emacs style (cont), some logic
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/vc-fossil 9e52264 028/111: emacs style (cont), some logic issues seem apparent... |
Date: |
Wed, 29 Sep 2021 08:59:15 -0400 (EDT) |
branch: elpa/vc-fossil
commit 9e522645dcd3f5b269408893d051e07c700a7dd1
Author: barak <barak>
Commit: barak <barak>
emacs style (cont), some logic issues seem apparent...
---
vc/el/vc-fossil.el | 66 +++++++++++++++++++++++++++---------------------------
1 file changed, 33 insertions(+), 33 deletions(-)
diff --git a/vc/el/vc-fossil.el b/vc/el/vc-fossil.el
index 4c14dda..c51ada5 100644
--- a/vc/el/vc-fossil.el
+++ b/vc/el/vc-fossil.el
@@ -129,34 +129,36 @@
"Get Fossil status for all files in a directory"
;; (message dir)
(insert (vc-fossil--run "update" "-n" "-v" "current" dir))
- (let* ((result)
- (done nil)
- (root (vc-fossil-root dir))
- line status-word file)
+ (let ((result '())
+ (done nil)
+ (root (vc-fossil-root dir)))
(goto-char (point-min))
(while (and (not (eobp)) (not done))
- (setq line (buffer-substring-no-properties (point) (line-end-position)))
- (setq status-word (car (split-string line)))
- (setq done (string-match "-----" status-word))
- (setq file (if done "." (substring line (+ (length status-word) 1))))
- (setq file (expand-file-name file root))
- (setq file (file-relative-name file dir))
- (setq result (if done result
- (cons (list file (vc-fossil-state-code status-word))
result)))
+ (let* ((line (buffer-substring-no-properties (point)
(line-end-position)))
+ (status-word (car (split-string line))))
+ (setq done (string-match "-----" status-word))
+ (let ((file (if done "." (substring line (+ (length status-word) 1)))))
+ (let ((file (expand-file-name file root)))
+ (let ((file (file-relative-name file dir)))
+ (unless done
+ (setq result
+ (cons (list file (vc-fossil-state-code status-word))
+ result)))))))
(forward-line))
(funcall update-function result nil)))
(defun vc-fossil-after-dir-status (callback)
"Function to call after the status process has finished"
(message "after-dir-status called %s" (buffer-string))
- (let (result line state status-word file)
+ (let ((result '()))
(goto-char (point-min))
(while (not (eobp))
- (setq line (buffer-substring-no-properties (point) (line-end-position)))
- (message line)
- (setq state (vc-fossil-state-code (car (split-string line))))
- (setq file (expand-file-name (substring line (+ (length status-word)
1))))
- (setq result (cons (list file state) result))
+ (let ((line (buffer-substring-no-properties (point) (line-end-position)))
+ (status-word '()))
+ (message line)
+ (let* ((state (vc-fossil-state-code (car (split-string line))))
+ (file (expand-file-name (substring line (+ (length status-word)
1)))))
+ (setq result (cons (list file state) result))))
(forward-line))
(funcall callback result t)))
@@ -261,18 +263,17 @@
"Fossil specific version of the `vc-previous-revision'."
(when file
(with-temp-buffer
- (let* ((found (not rev))
- (newver nil)
- line version)
+ (let ((found (not rev))
+ (newver nil))
(insert (vc-fossil--run "finfo" "-l" "-b" file))
;; (vc-fossil--call "fossil" "finfo" "-l" "-b" file)
(goto-char (point-min))
(while (not (eobp))
- (setq line (buffer-substring-no-properties (point)
(line-end-position)))
- ;; (message line)
- (setq version (car (split-string line)))
- (setq newver (or newver (and found version)))
- (setq found (string= version rev))
+ (let* ((line (buffer-substring-no-properties (point)
(line-end-position)))
+ (version (car (split-string line))))
+ ;; (message line)
+ (setq newver (or newver (and found version)))
+ (setq found (string= version rev)))
(forward-line))
newver))))
@@ -280,17 +281,16 @@
"Fossil specific version of the `vc-previous-revision'."
(when file
(with-temp-buffer
- (let* ((found (not rev))
- (oldver nil)
- line version)
+ (let ((found (not rev))
+ (oldver nil))
(insert (vc-fossil--run "finfo" "-l" "-b" file))
;; (vc-fossil--call "fossil" "finfo" "-l" "-b" file)
(goto-char (point-min))
(while (not (eobp))
- (setq line (buffer-substring-no-properties (point)
(line-end-position)))
- (setq version (car (split-string line)))
- (setq found (string= version rev))
- (setq oldver (or oldver found version))
+ (let* ((line (buffer-substring-no-properties (point)
(line-end-position)))
+ (version (car (split-string line))))
+ (setq found (string= version rev))
+ (setq oldver (or oldver found version)))
(forward-line))
oldver))))
- [nongnu] elpa/vc-fossil 9f90307 097/111: Add changes from pdo, (continued)
- [nongnu] elpa/vc-fossil 9f90307 097/111: Add changes from pdo, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 7e84860 091/111: Attempted pushes. Sigh, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 73c5145 088/111: Tweaked :synchro and :checkout header line formats., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil d9190c0 100/111: Fossil at the end of the VC backends list., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 5d66231 107/111: Changes from upstream, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 4254ef5 104/111: Get changes from upstream upto 2020-09-20, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil f07cd00 108/111: Upstream updates, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil acb773e 098/111: Merge Changes, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil abdb548 099/111: Update to pdo's latest, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil f365e43 110/111: vc-fossil.el: Update from fossil., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 9e52264 028/111: emacs style (cont), some logic issues seem apparent...,
ELPA Syncer <=
- [nongnu] elpa/vc-fossil 7c5af95 102/111: enable Fossil VC backend at load / autoload time, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 93c2104 101/111: Merge pull request #1 from barak/trunk, ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil bf1969f 033/111: Undo file-truename in log because emacs-23.3.1 gives relative names which confuse fossil., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil c058f3d 039/111: Remove debug message from `vc-fossil-after-dir-status`., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil f4795ba 044/111: Pass `vc-checkout-switches` on checkout., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 6619ffb 048/111: Improve log command., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil b632993 050/111: Handle switches in diff command., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 560d708 051/111: Implement annotate commanad., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil 169cff6 053/111: Suppress echo message when enabling word wrap in log view., ELPA Syncer, 2021/09/29
- [nongnu] elpa/vc-fossil e2ec578 060/111: Use `vc-fossil-command` if the output is inserted in a buffer., ELPA Syncer, 2021/09/29