Re: Time to merge scratch/correct-warning-pos into master, perhaps?

[Brahimi Saifullah]

Hello Alan.

Flymake is not working as of the latest commit (3023e7c).
The offending function is `elisp-flymake--byte-compile-done',
I've narrowed down the issue to the following section:
[...]
         (cl-loop with data =
                  (with-current-buffer output-buffer
                    (goto-char (point-min))
                    (search-forward ":elisp-flymake-output-start")
                    (read (point-marker)))
                  for (string pos _fill level) in data
                  do (goto-char pos)
                                ^^^
[...]
With DATA being:

    (("Invalid read syntax: \")\", 8, 4" nil nil :error)
     ("reference to free variable `x'" nil t :warning))

POS here, the second element of the list, is nil, as you can see.

For comparison, this is what DATA looks like on Emacs 28

    (("Invalid read syntax: \")\", 8, 4" 196 nil :error)
     ("reference to free variable `x'" 195 t :warning))