Patch: Fix for Bug 26372 [Modula-2] Parsing of multi-subscript arrays
Andrew Burgess
andrew.burgess@embecosm.com
Tue Aug 25 09:36:13 GMT 2020
* Tom de Vries <tdevries@suse.de> [2020-08-25 11:29:18 +0200]:
> On 8/25/20 4:29 AM, Gaius Mulley wrote:
> >
> >
> > Hi Tom,
> >
> > here is a bugfix for Pr 26372 [Modula-2] Parsing of multi-subscript arrays.
> > Also included is a dejagnu testcase. No extra regressions are caused on
> > Debian GNU/Linux Buster amd64, is this ok to apply?
> >
>
> Hi Gaius,
>
> I ran into some trouble applying the patch with the patch program, so I
> applied the patch manually, and then diffed the result of that with the
> original patch and fixed whitespace problems in the original patch until
> I got duplicates, to ensure that the application process was sane.
>
> I verified that when building with the patch, the warning "rule useless
> in parser due to conflicts" is gone, and no new warnings are introduced.
>
> I ran the test-case and verified that it failed without and passed with
> the patch.
>
> I've also fixed the ChangeLog entry a bit (multidim.y -> multidim.exp)
> and split it up for gdb/ChangeLog and gdb/testsuite/Changelog, as well
> as added a PR number.
>
> I've gone ahead an applied it (with you as author, obviously). See
> https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=07758bdfa9e5a762f2ec0deeb51b11d6ad5fe376
> .
>
> Attached here for reference.
Given the issues I raised with this patch in [1] I reverted this
commit and pushed my suggested patch instead. Again maintaining the
original author.
I hope this does not offend anyone. Please let me know if this was
the wrong course of action.
Thanks,
Andrew
[1] https://sourceware.org/pipermail/gdb-patches/2020-August/171456.html
More information about the Gdb
mailing list