flyfire / daisydiff

Automatically exported from code.google.com/p/daisydiff
0 stars 0 forks source link

IndexOutOfBoundsException if the only element was deleted #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. java -jar daisydiff.jar file:///c:/1.txt file:///c:/2.txt

What is the expected output? What do you see instead?
An IndexOutOfBoundsException was thrown.

What version of the product are you using? On what operating system?
latest version from repository (rev 107.

Stacktrace:
...java.lang.IndexOutOfBoundsException: Index: 1, Size: 0
        at java.util.ArrayList.add(ArrayList.java:367)
        at org.outerj.daisy.diff.html.dom.TagNode.addChild(TagNode.java:60)
        at org.outerj.daisy.diff.html.TextNodeComparator.markAsDeleted
(TextNodeComparator.java:304)
        at org.outerj.daisy.diff.html.HTMLDiffer.diff(HTMLDiffer.java:61)
        at org.outerj.daisy.diff.Main.main(Main.java:127)

Original issue reported on code.google.com by dereisb...@googlemail.com on 24 Oct 2008 at 9:19

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the quality report. 

This was a known bug in version 0.1 of DaisyDiff that has long been fixed in the
repository. I've taken this report as an opportunity to release the new version 
of
DaisyDiff.

Version 1.0 should fix this.

Original comment by guy...@gmail.com on 24 Oct 2008 at 3:20

GoogleCodeExporter commented 9 years ago
Version 1.0 doesn't fix the bug. As I wrote, I was using the latest version 
from the 
repository and the bug is still present.
I suggest to reopen this issue.

Original comment by dereisb...@googlemail.com on 27 Oct 2008 at 8:55

GoogleCodeExporter commented 9 years ago
Ok, my bad, will look in to it.

Original comment by guy...@gmail.com on 27 Oct 2008 at 9:01

GoogleCodeExporter commented 9 years ago
Fixed in r112. I also updated the package for the 1.0 release.
Thanks

Original comment by guy...@gmail.com on 27 Oct 2008 at 10:11