summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNathan Moreau <nathan.moreau@m4x.org>2020-06-18 15:14:48 +0200
committerGitHub <noreply@github.com>2020-06-18 15:14:48 +0200
commit00177f542976601d7f114fed82caaa3daad7b177 (patch)
tree81159ee736e56ce508f528b80f91e7769b9e3cef
parent9eb2c632e32e9979a1694c1e62961d7ad66e8f4a (diff)
downloadrust-mode-00177f542976601d7f114fed82caaa3daad7b177.tar.gz
compile-mode integration: add link to code references. (#391)
This allows to jump to code references such as lines 82 and 132 in the following snippet error message. error[E0061]: this function takes 1 parameter but 2 parameters were supplied --> file6.rs:132:34 | 82 | fn duration_ms_since(time: &Option<SystemTime>) -> u128 { | ------------------------------------------------------- defined here ... 132 | self.total_time_ms = duration_ms_since(&self.program_start, 2); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-rw-r--r--rust-mode-tests.el41
-rw-r--r--rust-mode.el9
2 files changed, 38 insertions, 12 deletions
diff --git a/rust-mode-tests.el b/rust-mode-tests.el
index da874ec..c0e88fa 100644
--- a/rust-mode-tests.el
+++ b/rust-mode-tests.el
@@ -3346,14 +3346,20 @@ impl Two<'a> {
(mapcar (lambda (r)
(let ((match-pos
(nth (cdr r) spec)))
- (if (eq :type (car r))
- (cond ((consp match-pos)
- (compilation-face match-pos))
- (t
- (cdr (assoc match-pos '((1 . compilation-warning)
- (0 . compilation-info)
- (2 . compilation-error))))))
- (match-string match-pos))))
+ (cond ((and (eq :type (car r)) (consp match-pos))
+ (compilation-face match-pos))
+ ((eq :type (car r))
+ (cdr (assoc match-pos '((1 . compilation-warning)
+ (0 . compilation-info)
+ (2 . compilation-error)))))
+ ((and (null match-pos) (eq :column (car r)))
+ 'back-to-indentation)
+ ((and (null match-pos) (eq :file (car r)))
+ 'like-previous-one)
+ ((null match-pos)
+ (error (format "%S" (car r))))
+ (t
+ (match-string match-pos)))))
;; see compilation-error-regexp-alist
'((:file . 1)
(:line . 2)
@@ -3373,16 +3379,27 @@ impl Two<'a> {
(insert " ::: file5.rs:12:34\n\n")
;; should not match
(insert "werror found a -> b\n --> no_match.rs:12:34\n\n")
-
+ (insert "error[E0061]: this function takes 1 parameter but 2 parameters were supplied\n --> file6.rs:132:34
+ |
+82 | fn duration_ms_since(time: &Option<SystemTime>) -> u128 {
+ | ------------------------------------------------------- defined here
+...
+132 | self.total_time_ms = duration_ms_since(&self.program_start, 2);
+ | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+")
(should (equal
'((("file1.rs" "12" "34" compilation-error "file1.rs:12:34")
("file2.rs" "12" "34" compilation-error "file2.rs:12:34")
("file3.rs" "12" "34" compilation-warning "file3.rs:12:34")
- ("file4.rs" "12" "34" compilation-info "file4.rs:12:34"))
- (("file5.rs" "12" "34" compilation-info "file5.rs:12:34")))
+ ("file4.rs" "12" "34" compilation-info "file4.rs:12:34")
+ ("file6.rs" "132" "34" compilation-error "file6.rs:132:34"))
+ (("file5.rs" "12" "34" compilation-info "file5.rs:12:34"))
+ ((like-previous-one "82" back-to-indentation compilation-info "82")
+ (like-previous-one "132" back-to-indentation compilation-info "132")))
(mapcar #'rust-collect-matches
(list rustc-compilation-regexps
- rustc-colon-compilation-regexps))))))
+ rustc-colon-compilation-regexps
+ rustc-refs-compilation-regexps))))))
;; If electric-pair-mode is available, load it and run the tests that use it. If not,
;; no error--the tests will be skipped.
diff --git a/rust-mode.el b/rust-mode.el
index d5e0b93..9711a79 100644
--- a/rust-mode.el
+++ b/rust-mode.el
@@ -1947,6 +1947,12 @@ See `compilation-error-regexp-alist' for help on their format.")
"Specifications for matching `:::` hints in rustc invocations.
See `compilation-error-regexp-alist' for help on their format.")
+(defvar rustc-refs-compilation-regexps
+ (let ((re "^\\([0-9]+\\)[[:space:]]*|"))
+ (cons re '(nil 1 nil 0 1)))
+ "Specifications for matching code references in rustc invocations.
+See `compilation-error-regexp-alist' for help on their format.")
+
;; Match test run failures and panics during compilation as
;; compilation warnings
(defvar cargo-compilation-regexps
@@ -1978,6 +1984,9 @@ the compilation window until the top of the error is visible."
(eval-after-load 'compile
'(progn
(add-to-list 'compilation-error-regexp-alist-alist
+ (cons 'rustc-refs rustc-refs-compilation-regexps))
+ (add-to-list 'compilation-error-regexp-alist 'rustc-refs)
+ (add-to-list 'compilation-error-regexp-alist-alist
(cons 'rustc rustc-compilation-regexps))
(add-to-list 'compilation-error-regexp-alist 'rustc)
(add-to-list 'compilation-error-regexp-alist-alist