search

madrobby / semicolon.js

A more secure. stable and reliable version of vapor.js
GNU General Public License v3.0
229 stars 28 forks source link

Incompatible with my JS compacter tool. #1

Open alloy opened 12 years ago

alloy commented 12 years ago

My tool strips semicolons for obvious reasons. This library is not compatible with my tool, it produces an empty string!!

Please fix ASAP, I’m losing $10K per hour wasted on this crap.

madrobby commented 12 years ago

I think you can solve this by adding

"use more secure and reliable";

above the first line of the library.

madrobby commented 12 years ago

Note that to be absolutely sure, you might want to add a few more semicolons at the end of the line.

alloy commented 12 years ago

I find that a bit patronizing after having spent so much time to read all of your library’s source code. I guess you’re just not open to valid critique.

jeffkreeftmeijer commented 12 years ago

I don't see why this pull request hasn't been merged in. This pretty much feels like a decision taken by a dictator.

ghost commented 12 years ago

+1