Problem with quoted semicolons about parinfer.js

[shaunlebron]

Thanks for reporting this. This uncovers another case where Parinfer can corrupt strings.

The problem is that inserting a quote before ";" gives "";", which is still a valid clojure expression, and Parinfer will operate on anything valid.

I'm not yet sure if this problem is solvable. But I have no doubt that the information fundamental to and necessary for solving it is related to quote-directionality. If we can capture the intended direction that these quotes are pointed, we may be able to come up with some rules for handling these intermediate states created by inserting or deleting quotes.

[shaunlebron]

There are two things that I think can help this case without introducing much complexity:

• a trapdoor to disable parinfer once odd number of quotes are found
• auto-closing strings to prevent trapdoor encounters

details here: https://github.com/shaunlebron/parinfer/wiki/The-Problem-with-Quotes

[shaunlebron]

this is fixed in the quotefix branch. see test cases here

I'm going to finish #63 before cutting a release

[shaunlebron]

@snoe: sorry, forgot to follow-up on this, but this has been fixed for the past few releases (at 1.4.0 now). Here's the demo of your case in atom-parinfer: