False positive for 'final-newline' rule on windows

[davidgilbertson]

The final-newline rule is always reporting a warning for me and I believe it's because of the way Windows reports newlines.

Because they are actually \r\n, this line (last.content !== '\n') is always true.

If I test for \r\nas well the rule behaves as expected on Windows (although I don't have a mac handy to test it on).

if (last.content !== '\n' && last.content !== '\r\n' && parser.options.include) {

Or a shorter version...

if (!last.content.match(/\r?\n/) && parser.options.include) {

[davidgilbertson]

It looks like the issue for empty-line-between-blocks. This line might need to be this to play well with Windows:

space = (previous.content.indexOf('\n\n') >= 0 || previous.content.indexOf('\r\n\r\n') >= 0);

[davidgilbertson]

Or... having just learnt that os.EOL is a thing

var os = require('os');
if (last.content !== os.EOL && parser.options.include) {
    // etc.
}

Nice and descriptive...