Skip to content
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

Some whitespace character differences may not be detected #404

Open
alcosml opened this issue Oct 29, 2024 · 2 comments
Open

Some whitespace character differences may not be detected #404

alcosml opened this issue Oct 29, 2024 · 2 comments

Comments

@alcosml
Copy link

alcosml commented Oct 29, 2024

Two identical files, one w/c has CRLF and another w/c has LF as line ending can be detected as matching despite having whitespaces not ignored.
image1
image2

@pnedev
Copy link
Owner

pnedev commented Oct 29, 2024

@alcosml ,

Just as an info:
I am no longer supporting the old Compare plugin.
The newest plugin is called ComparePlus (its current version is 1.2) - it is supported.

Unfortunately both plugins are not taking into account the line endings of the files - those are always ignored as the compare process works on line-by-line basis (regardless of the Ignore settings). I am not planning to change that because line endings are not visible in Notepad++ (unless you have explicitly configured that but then you will directly see line endings differ) and then one will see that there are changes between some lines but that changes are not apparent (they are only in the invisible line endings).

I can probably implement a warning about comparing files with different line endings but that will be generic and not line-by-line (for example a case when you have one line only with LF and some other lines in the same file with both CRLF will not be detected).

@pnedev
Copy link
Owner

pnedev commented Nov 5, 2024

@alcosml ,

I added a warning message on each new compare if the line endings of the compared files are not the same and informing the user that the compare process will ignore that fact. This change will be available in the next ComparePlus plugin release. Your screenshot is about the older Compare plugin which is no longer supported (by me).

BR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants