viewer: exceeding newline on '\r\n' terminator

[slyfox]

Steps to reproduce:
$ printf "1\r\n2\r\n" > /tmp/foo.txt
$ mcview /tmp/foo.txt # ugh, stupious newline between 1 and 2
$ mcedit /tmp/foo.txt # normal mode

4.7.0-pre2-29-g5a8923e

[angel_il]

branch: 1595_newline (parent: master)
changeset: 0d6b5b48c9b01015a02c2878c8bf499a3a5574ae

[slyfox]

Fix does not seem to work on pattern, like this:

printf "1\r\n2\r\n3\r4\n\r5\r"

At least newline count mismatches between viewer/editor: 3<->4 and 4<->5 look weird.

I'm not sure what we expect to see on MacOSX.

[angel_il]

hypothesis: '\n', '\r\n' and '\r' is a <EOL>

1\r\n2\r\n3\r4\n\r5\r

will be transformed in

1 <EOL>
2 <EOL>
3 <EOL>
4 <EOL>
<EOL>
5 <EOL>

what's wrong?

[slyfox]

Replying to angel_il:

hypothesis: '\n', '\r\n' and '\r' is a <EOL>

1\r\n2\r\n3\r4\n\r5\r

will be transformed in

1 <EOL>
2 <EOL>
3 <EOL>
4 <EOL>
<EOL>
5 <EOL>

what's wrong?> hypothesis: '\n', '\r\n' and '\r' is a <EOL>

1\r\n2\r\n3\r4\n\r5\r

will be transformed in

1 <EOL>
2 <EOL>
3 <EOL>
4 <EOL>
<EOL>
5 <EOL>

what's wrong?

It's inconsistent with _our_ editor:
editor:

1^M
2^M
3^M4     # no newline!
^M5^M    #single newline

As you see '\r' is not treated as newline, i'm just curious why viewer should screw it up?

viewer from branch:

1
2
3
4        # whoops
         # whoops again
5

BTW, as we try to view/edit foreign files we mignt need choice where file did come from:
​http://en.wikipedia.org/wiki/Newline

[angel_il]

slyfox

1
2
3
4        # whoops
         # whoops again
5

because '\r', \r\n, \n is the end of line.
\n\r - not is the EOL

[angel_il]

Fixed: cd27ee24df84fc958c8e4af60a3699b8f780be4a