QDiff decorates differences incorrectly under certain circumstances

Bug #531304 reported by Craig Hewetson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QBzr
Confirmed
Low
Unassigned

Bug Description

Delete line X, run qdiff and you will see that X-1 shows up as deleted. See screenshot.

Tags: qdiff
Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote :
Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote :

Same difference except I used p4merge via the external diff mechanism. You will notice that it is correct.

Revision history for this message
Alexander Belchenko (bialix) wrote : Re: [Bug 531304] [NEW] QDiff decorates differences incorrectly under certain circumstances

Craig Hewetson пишет:
> Public bug reported:
>
> Delete line X, run qdiff and you will see that X-1 shows up as deleted.
> See screenshot.

Smells like the encodings problem, again.

Please check if your file has non-ascii characters inside.
Can you try to run
bzr qdiff --encoding latin-1
and see if it makes any difference?

Revision history for this message
Gary van der Merwe (garyvdm) wrote :

Please upload the file, or a file that reproduces the bug.

Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote : Re: [Bug 531304] [NEW] QDiff decorates differences incorrectly under certain circumstances

Nope. See attachment.

Do you think it might be line endings? In p4merge I've set it to ignore line
ending differences.

Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote : Re: [Bug 531304] Re: QDiff decorates differences incorrectly under certain circumstances
  • XEngine.java Edit (245 bytes, text/x-java; charset=US-ASCII; name="XEngine.java")

here goes, look, i had to remove content, but i left a few lines.
This file use to live in VSS and its encoding is ISO-8859-1.
I've also tested qdiff with this encoding.

Revision history for this message
Gordon Tyler (doxxx) wrote : Re: [Bug 531304] Re: QDiff decorates differences incorrectly under certain circumstances

On 03/03/2010 12:27 PM, Craig Hewetson wrote:
> ** Attachment added: "XEngine.java"
> http://launchpadlibrarian.net/40094170/XEngine.java

Unless this is an artifact of your attachment or removal of content, the
last line ends in a CR but all the other lines end in CRLF. Would this
confuse qdiff?

Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote : Re: [Bug 531304] Re: QDiff decorates differences incorrectly under certain circumstances

I wish i could give the original file... I'm sure it would make it easier.

But if i look at the original file in a hex editor at the end of each line I
get 0x0D 0x0A
When trying to reproduce this problem with a new file i can only get 0x0A at
the end of each line.

I think this file was originally create on Windows with line endings 0x0D
0x0A and all new files I've created since then on linux have line endings
0x0A.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.