Re: Patch: New test for when to show inline diffs
On 14.05.2011 12:41, Morten Kloster wrote:
> The test for whether to show color-coded inline differences in
> SVNLineDiff::ShowInlineDiff had some problems. In particular, it would
> always fail if both ends of the line had changed, no matter how similar the
> old and new lines were otherwise (since diffcounts = origcounts - 1 + #ends
> that have changed). The new test also incorporates how fragmented the
> differences are. The new test contains two double-valued parameters that may
> be subject to individual preferences; the chosen values keep the behaviour
> reasonably close to the old test (except for the end-changed problem, of
> course): Single-token deletions or insertions will tend to be shown less
> than before, single-token replacements or two-token deletions/insertions
> will be unchanged, and multi-token changes will be shown more often than
> Old test failed if both ends were changed, no matter how similar the lines
> were otherwise (almost certainly unintentional; see r8518). New test
> incorporates how fragmented the changes are.
Thanks for the patch and the detailed explanation!
Committed your patch in r21365.
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-05-14 15:57:22 CEST
This is an archived mail posted to the TortoiseSVN Dev