gometalinter fails on master branch of Go

nordinvannes commented 6 years ago

[X] GopherLua is a Lua5.1 implementation. You should be familiar with Lua programming language. Have you read Lua 5.1 reference manual carefully?
[X] GopherLua is a Lua5.1 implementation. In Lua, to keep it simple, it is more important to remove functionalities rather than to add functionalities unlike other languages . If you are going to introduce some new cool functionalities into the GopherLua code base and the functionalities can be implemented by existing APIs, It should be implemented as a library.

Please answer the following before submitting your issue:

What version of GopherLua are you using? : last
What version of Go are you using? : master branch
What operating system and processor architecture are you using? : Windows / Intel
What did you do? : Run gometalinter
What did you expect to see? : No error
What did you see instead? : An error on loop detection on preloadLimit

yuin commented 6 years ago

This issue seems to be a bug in go vet . see https://github.com/golang/go/issues/26124

yuin commented 6 years ago

Now we have a Go1.11 release. This issue has been fixed in Go1.11 release.

yuin / gopher-lua