Community
Participate
Working Groups
"Ignore white spaces" options should work in two ways: 1. Ignore white spaces only 2. Ignore All (whitespaces + CR/LF) This is very useful feature, which is abstent in Eclipse, but exists in IntelliJ IDEA 4
This would be very useful for me too.
What for would be an "Ignore white spaces only"? I expect "Ignore white space" to always ignore spaces and tabs as well as line feeds and carriage returns. It is a pain that eclipse's compare is not able to ignore the latter.
Bug #45857 (which I think is getting closed in favor of this bug) under my comment describes a use case that describes why ignore white spaces is broken. The big problem (according to Marko Schulz in Bug #45857) is that Eclipse's compare is line based. Another use case around here occurs when there is a piece of legacy code that is not formatted to the team coding standards. Simultaneously you and a fellow developer edit the same piece of code. He runs the formatter but you don't and suddenly Eclipse reports conflicts on almost every line of the class when you go to synchronize with your repository. Very annoying.
*** Bug 45857 has been marked as a duplicate of this bug. ***
If you compare a file where the only difference is CR+LF vs. LF, then you get a change in every line but the change itself isn't visible. In this case, it would be great if the compare view would show some strange character like "CR" at the end of the line to make it obvious where the change is. Also, all compare views should have toggle buttons to switch various ignore options (ignore whitespace, ignore CRLF, ignore empty lines, ignore whitespace before text, ignore whitespace after text).
(In reply to comment #5) > (...) it would be great if the compare view would show some strange character like "CR" at the > end of the line to make it obvious where the change is. IMO we could highlight whitespace chars (when shown) that differ the same way we highlight individual changes (when the option is on).
(In reply to comment #6) > IMO we could highlight whitespace chars (when shown) that differ the same way > we highlight individual changes (when the option is on). That would work but please add buttons to toggle these options in the view! It's very annoying to have to close the dialog, open the preferences or switch to a perspective where the toggle button is available.
Are you referring to the proposed "Highlight whitespace chars diff" toggle? This should be a pref imo. "Ignore white space" and "Show Whitespace Characters" actions are both available in context menu.
After several years with Eclipse, I completely missed the context menu in the diff viewer =8*O My apologies. As to my original request, I see this on the screen: о о <tr class="error even">о <tr class="error even">о <td class="Date">...</td>о <td class="Date">...</td>о The diff is that the "о" after 'even">' on the left side is made of "CR+LF" while in the right view, it's just "LF". The dialog (which get from assertEquals() in a test case) doesn't show *any* difference but the test failed. This is very anoying. Can you show the little left arrow or maybe "оо" to make this visible? Test case: assertEquals("a\r\nb", "a\nb");
After debugging I realized that the case with Compare Results dialog is a little bit different. The problem here is not with the merge viewer displaying the difference in a wrong way, but with content of the left and the right side. They are equal[1] and both are: "a\nb"! This is caused by AppendingProcessingState[2] class with produces "actual" and "expected" strings to be displayed in the dialog. This particular case looks more like a JDT/JUnit bug, so you should open a bug against them. Of course, the original request on this bug is still valid. [1] org.eclipse.jdt.internal.junit.ui.CompareResultDialog.setFailedTest(TestElement) [2] org.eclipse.jdt.internal.junit.model.RemoteTestRunnerClient.AppendingProcessingState.readMessage(String)
