-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comby reports wrong range is the line ending is \r\n
(CRLF) instead of \n
(LF)
#183
Comments
Hi there! I did the following (though I am not on a windows machine, but I think this will help us get to the issue):
The output of the above is:
|
as you can see with the above output there are 2 lines in the file, but the comby thinks it's just 1 huge line and reports the range as such. EDIT: the value appears to be wrong as well |
The matched part for
The current convention is to put the end of the range at the position after the match. Thus, if the last character of the range is a newline, the position after is the next line, at the first column position (line 3, column 1). It's possible to change the convention of what range means, but as it is this is currently consistent and doesn't seem incorrect to me. That said, the value for
indeed looks wrong to me, since the |
Describe the bug
Comby reports wrong range is the line ending is
\r\n
(CRLF) instead of\n
(LF)Reproducing
\r\n
as line endingcomby -json-lines
to get output in jsonExpected behavior
Correct range is reported
The text was updated successfully, but these errors were encountered: