You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was poking around Compute.ConcreteNew.hs trying to fix #52, and I noticed that the indices for bound variables (VGen Int [Value]) are not correct. You can even see this in #52: the offending piece of code is this
inerror. render $
ppL loc (hang ("unsupported token gluing: "++"("++show v1 ++", "++show v2 ++")\n"++show (local env) ++"\n") 4
(Glue (vt v1) (vt v2)))
With this change, I can cause an error (unsupported token gluing in my case, both appropriate and inappropriate), and then I can see the variables I have in the local environment, and how vt prints out the wrong variable name for the index i in the VGen i xs.
I tried to remove the call to reverse in value2term but the indices were just differently wrong.
I don't dare touch this further, because this seems like a fundamental thing in the GF compiler. Originally I just wanted to add another case in the glue function to match the tokens that were erroneously covered by "unsupported token gluing".
The text was updated successfully, but these errors were encountered:
The problem
I was poking around
Compute.ConcreteNew.hs
trying to fix #52, and I noticed that the indices for bound variables (VGen Int [Value]
) are not correct. You can even see this in #52: the offending piece of code is thisand the error is this.
My attempts
I added more printouts for the error message for unsupported token gluing, as follows:
With this change, I can cause an error (unsupported token gluing in my case, both appropriate and inappropriate), and then I can see the variables I have in the local environment, and how
vt
prints out the wrong variable name for the indexi
in theVGen i xs
.I tried to remove the call to
reverse
invalue2term
but the indices were just differently wrong.I don't dare touch this further, because this seems like a fundamental thing in the GF compiler. Originally I just wanted to add another case in the
glue
function to match the tokens that were erroneously covered by "unsupported token gluing".The text was updated successfully, but these errors were encountered: