Fix redeclaring i

[Koeng101]

When running teal in gopher-lua (https://github.com/yuin/gopher-lua), types get checked by the system, and the single line i, node.decltype = i, decltype caused a complete failure to run teal. When replaced with the equivalent node.decltype = decltype, the system runs just fine.

These should be functionally equivalent. Nothing else is changed and it does not appear to change any tests.

[tooolbox]

Nice.

Refs:

• Bug: Incorrect result for multi-assignment yuin/gopher-lua#315
• Support Teal yuin/gopher-lua#314

[tooolbox]

@Koeng101 I'm curious what you ended up doing for the load vs. loadstring bit, because I saw you threw in a fix for that and then took it out. I'm using compat52.lua and its dependencies to run Teal in Gopher-Lua, but it's not ideal since it calls string.dump which blows up, so I had to modify the lib.

[Koeng101]

@tooolbox I added in a check into tl.tl that if it detected version 5.1, it would use load instead. The link is here - 4406b73

kinda dumb simple:

      if _VERSION == "Lua 5.1" then
         chunk = loadstring(code)
         err = "5.1 error"
      else
         chunk, err = load(code, "@" .. found_filename, "t")
      end

I threw it out because I thought it might not be idiomatic or well made lua code (you also need to add that check into the other occurrence of load()) . For my application, I actually want to use the lua conversion, so I just do this:

err = L.DoString(fmt.Sprintf("return tl.gen([========[%s]========])", tealCode))
        if err != nil {
                return "", err
        }
        parsedLua := L.Get(1)

It outputs a lua string, which I save, and then I just run the outputted lua. Kinda hacky, but is what I wanted to do in the first place.

I had no idea that yuin/gopher-lua#314 was there, or else I probably would have been a lot less inclined to figure out something that worked.

[tooolbox]

Gotcha, so you're loading the Lua string yourself. Okay, thanks for sharing that.