Enable JSHint's unused rule.

evilstreak / markdown-js

A Markdown parser for javascript

7.7k stars 863 forks source link

Closed XhmikosR closed 10 years ago

XhmikosR commented 11 years ago

Only one warning is left but I'm not sure how to fix it... (check Travis).

This fixes uglifyjs' warnings I reported in the past too.

/CC @ashb

ashb commented 11 years ago

I seem to recall that the last warning was in fact a bug in jshint. Will take a look at some point

XhmikosR commented 11 years ago

If that's the case, we can disable the unused warning for that code only so the tests pass and report the issue to JSHint.

XhmikosR commented 11 years ago

@ashb: This should be ready too, apart from that one warning.

XhmikosR commented 11 years ago

I tried to disable this specific warning for that for loop but JSHint keeps complaining.

Maybe it's time to file a bug report upstream.

XhmikosR commented 10 years ago

bump

Rebased against the latest HEAD.