[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 07/10] glr2.cc: fix yyresolveValue
From: |
Akim Demaille |
Subject: |
[PATCH 07/10] glr2.cc: fix yyresolveValue |
Date: |
Sun, 6 Dec 2020 14:10:34 +0100 |
When "tests: glr2.cc: run the glr-regression tests" tests are run,
before this commit the following tests used to loop endlessly:
709: Badly Collapsed GLR States: glr2.cc FAILED
(glr-regression.at:123)
715: Improper merging of GLR delayed action sets: glr2.cc FAILED
(glr-regression.at:397)
718: Duplicate representation of merged trees: glr2.cc FAILED
(glr-regression.at:495)
751: Leaked semantic values when reporting ambiguity: glr2.cc FAILED
(glr-regression.at:1632)
After this commit, no test loops and 709, 715, and 751 pass. Only 718
still fails.
* data/skeletons/glr2.cc (yyresolveValue): Add missing incrementation
of the iteration variable.
---
data/skeletons/glr2.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/data/skeletons/glr2.cc b/data/skeletons/glr2.cc
index a1194b61..6b60b38f 100644
--- a/data/skeletons/glr2.cc
+++ b/data/skeletons/glr2.cc
@@ -2439,6 +2439,7 @@ public:
{
yybest->mergeWith (*yyp);
yypPrev->setNext(yyp->next());
+ yyp = yypPrev->next();
}
else
{
--
2.29.2
- [PATCH 00/10] glr2.cc: fixes, Akim Demaille, 2020/12/06
- [PATCH 01/10] glr.c: fix line numbers in logs, Akim Demaille, 2020/12/06
- [PATCH 02/10] glr2.cc: fix when the stack is not expandable, Akim Demaille, 2020/12/06
- [PATCH 03/10] glr2.cc: use 'const' for some constant local variables, Akim Demaille, 2020/12/06
- [PATCH 04/10] glr2.cc: turn some pointers into references, Akim Demaille, 2020/12/06
- [PATCH 05/10] glr2.cc: fix memory corruption bug, Akim Demaille, 2020/12/06
- [PATCH 06/10] glr2.cc: misc cleanups, Akim Demaille, 2020/12/06
- [PATCH 07/10] glr2.cc: fix yyresolveValue,
Akim Demaille <=
- [PATCH 08/10] glr2.cc: add support for parse.assert, Akim Demaille, 2020/12/06
- [PATCH 09/10] glr2.cc: use the same format for traces as glr.c, Akim Demaille, 2020/12/06
- [PATCH 10/10] glr2.cc: run the glr-regression tests, Akim Demaille, 2020/12/06