bug#48489: 28.0.50; Incorrect Edebug instrumentation for old `when-let' form

[Philipp Stephani]

Am Di., 18. Mai 2021 um 18:27 Uhr schrieb Philipp Stephani
<p.stephani2@gmail.com>:
>
> Am Di., 18. Mai 2021 um 18:25 Uhr schrieb Lars Ingebrigtsen <larsi@gnus.org>:
> >
> > Philipp Stephani <p.stephani2@gmail.com> writes:
> >
> > > Hmm, I can't reproduce this on Debian. But I don't see how this could
> > > be OS-dependent.
> >
> > It reproduces fine here on Debian/bullseye, at least:
> >
> > 1 unexpected results:
> >    FAILED  edebug-tests-duplicate-symbol-backtrack
> >
>
> Yeah, I can also reproduce it now. Not sure what changed in the
> meantime on my system.

Ah. See the FIXME in the test:

                     ;; FIXME: There are twice as many inner
                     ;; definitions as expected due to Bug#42701.
                     ;; Once that bug is fixed, remove the duplicates.

Bug#42701 still isn't fixed, but the fix to bug#48489 has "suppressed"
its symptom in this case. We can therefore resolve this FIXME.
However, then the edebug-tests-duplicate-symbol-backtrack doesn't
really test any more what it should be testing. So I'll see that I can
change it to restore the previous behavior (which requires
backtracking and overlapping &or branches).