jawj / OverlappingMarkerSpiderfier

Deals with overlapping markers in Google Maps JS API v3, Google Earth-style
http://blog.mackerron.com
836 stars 238 forks source link

code is not valid [current] coffeescript #153

Open bdirito opened 6 years ago

bdirito commented 6 years ago

build script just calls $ coffee [...]

However OMS is not valid coffeescript. I assume this is due to some coffee language update. The code either needs to be updated to current coffeescript standards and/or include a coffee compiler version specification.

    (return i if o is obj) for o, i in arr
[stdin]:3:6: error: cannot use a pure statement in an expression
    (return i if o is obj) for o, i in arr
     ^^^^^^^^

https://coffeescript.org/#try:%20%20p.arrIndexOf%20%3D%20(arr%2C%20obj)%20-%3E%20%0A%20%20%20%20return%20arr.indexOf(obj)%20if%20arr.indexOf%3F%0A%20%20%20%20(return%20i%20if%20o%20is%20obj)%20for%20o%2C%20i%20in%20arr%0A%20%20%20%20-1

jstobaughRCH commented 4 years ago

It appears this library has not been ported to support coffeescript 2.x. It compiles properly with coffeescript 1.12.7. Install with: npm install --global coffeescript@1.12.7