search

fstirlitz / luaparse

A Lua parser written in JavaScript
https://fstirlitz.github.io/luaparse/
MIT License
459 stars 91 forks source link

Remove empty statements in Lua 5.1 #42

Closed ghost closed 7 years ago

ghost commented 7 years ago

In Lua 5.1 empty statements can only appear at the end of a statement. Check this: https://www.lua.org/manual/5.1/manual.html#2.4.1

I made two pull requests, but both got closed. You guys are bad.

ghost commented 7 years ago

@fstirlitz Stop being ignorant, you look a super kid!!! I didn't change the name of the repeatStatement, it was my editor's ReplaceAll tool.

ghost commented 7 years ago

You're stupid enough.

ghost commented 7 years ago

I adviced you, but continue ignoring my advices.

ghost commented 7 years ago

I know how to use GitHub better than you. AND again, the make test blah command ISN'T VALID in my command prompt. I ran npm install , in luaparse's root, but NOTHING.

ghost commented 7 years ago

I adviced you this many times, but continue with this such stupid and demonic ignorance.

ghost commented 7 years ago

There's no reason to continue fixing bugs in this shit of luaparse.

oxyc commented 7 years ago

make is a *nix build tool that the project uses. It's unrelated to node, but definitely worth looking into as it's widely used.

Sorry you feel that way. I do agree with @fstirlitz, and the PR was not mergeable. Thanks for your work though, and we might pick it up if we have the time later on.

fstirlitz commented 7 years ago

Fixed in 1a144cea96fce1c658eebe24dad9fb44b9de851c. Go see how your commit should have looked like.

You're stupid enough.

I adviced you this many times, but continue with this such stupid and demonic ignorance.

There's no reason to continue fixing bugs in this shit of luaparse.

I likewise hope you also have a lovely day!!!!!!!1111💖💖💖💖💖💖💖😻😻😻