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

[Alan Mackenzie]

Hello, Brahimi.

On Sat, Jan 15, 2022 at 21:27:04 -0300, Brahimi Saifullah wrote:
> Hello Alan.

> Flymake is not working as of the latest commit (3023e7c).

Thanks indeed for the report.

> 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))

That looks like some minor glitch interfering.

I don't actually use Flymake mode.  Could you possibly give me a recipe
to reproduce the bug, then either João or I can fix it.

Thanks again!

-- 
Alan Mackenzie (Nuremberg, Germany).