Indexing string literals

[Erutuon]

Currently luamin converts ('abc'):match('a') to 'abc':match('a'). This results in a syntax error ("')' expected near ':'"): a string has to be enclosed in parentheses for it to be indexed. (Actually, the expression could be reduced to ('abc'):match'a', but perhaps that's best avoided.)

[xLarry]

Should be fixed by #63

$ npm i git+https://github.com/Ayplow/luamin.git
+ luamin@1.0.4
$ npx luamin -c "('abc'):match('a')"
('abc'):match('a')

[Erutuon]

Now I realize there was another problem related to indexing of string literals that I failed to mention: the minifier throws a syntax error for ('abc')['match'] and ('abc').match. Perhaps that should be put in a separate issue because it originates from a different part of the minifier.

[berbun]

Now I realize there was another problem related to indexing of string literals that I failed to mention: the minifier throws a syntax error for ('abc')['match'] and ('abc').match. Perhaps that should be put in a separate issue because it originates from a different part of the minifier.

This is an issue with luaparse#24.

[Erutuon]

This is getting farther from the original topic, but the string literal indexing is part of a larger issue. For all except table and function literals, Luamin doesn't recognize that any literal must be enclosed with parentheses in order to be indexed or have a method called on it:

(true):method()
(false):method()
(nil):method()
(1):method()
('abc'):method()
(function() end):method()
({}):method()

(true)[1]
(false)[1]
(nil)[1]
(1)[1]
('abc')[1]
(function() end)[1]
({})[1]

If each of these lines is supplied separately to Luamin, there is a syntax error or the parentheses around the literal are removed, except with method calls on table and function literals.