serjant / daisydiff

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

Patch for alternative output formats #16

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Attached you will find a patch for current SVN (12 November 2009) that does 
the following.

1. Conversion of some classes to interfaces (e.g HTMLDiffer and 
HTMLOutput) so that developers can plugin alternative output formats 
(e.g. PDF) 

2. Some unit tests (and addition of Junit 4 instead of Junit 3). Coverage
is now almost 45%

3. An extra field in the internal structures of daisyDiff so that in 
the case of layout removals (e.g. removal 
of an italics tag around a sentence) the output can also "remember" 
what was the old layout. Currently only a textual representation is 
kept (the one shown on the tooltips) but this is not enough for 
further processing. 

All changes are essentially refactorings so that diff functionality 
has not changed at all. 

Please review and apply if you are happy with it.

I would be glad to change it further if there are any problems (coding 
style, comments, naming e.t.c.)

The patch was created with Eclipse. I tested it by checking out from SVN 
the daisy-diff source and applied it again via Eclipse.

Thank you

Original issue reported on code.google.com by kkape...@gmail.com on 12 Nov 2009 at 9:27

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks
I'm looking for a volunteer to apply and test this patch.

Original comment by guy...@gmail.com on 18 Feb 2010 at 1:42

GoogleCodeExporter commented 9 years ago
Applied in r143

Original comment by kkape...@gmail.com on 22 Feb 2010 at 9:28