Odd logic for deciding whether to display inline diffs in TMerge

Hi folks,

I was looking through the TMerge code to try to work out when inline
diffs are shown and I found this line in BaseView.cpp (line 1289 in r8347)

if ((diffcounts < maxcommon)||
     (smalldiffs==0)||((diffcounts/smalldiffs) > 1))

The last test divides 2 integers and compares with 1, and the previous
one avoids division by zero. Isn't it easier simply to compare the 2
integers, or am I missing something subtle?

if ((diffcounts < maxcommon)||(diffcounts > smalldiffs))

Simon

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org