plugin/emacs: Don't use region intersection in Flymake while reporting

This is causing diagnostics to not be cleared sometimes (see quick-lint#398), it was first introduced for developement, because restarting Flymake without changes in buffers would not delete old diagnostics, I think it's fine if we not specify this option. However I suspect this is a bug in Flymake. Signed-off-by: wagner riffel <[email protected]>
wgrr · Jul 24, 2021 · 6a223f8 · 6a223f8
1 parent 664a06e
commit 6a223f8
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/plugin/emacs/flymake-quicklintjs.el b/plugin/emacs/flymake-quicklintjs.el
@@ -87,8 +87,7 @@ REPORT-FN is Flymake's callback."
                                         (buffer-substring-no-properties
                                          (point-min) (point-max)))))))
                        (if (or diags (zerop (process-exit-status p)))
-                           (funcall report-fn diags
-                                    :region (cons (point-min) (point-max)))
+                           (funcall report-fn diags)
                          (funcall report-fn
                                   :panic :explanation
                                   (buffer-substring