Re: Request for review: replacing cdiff with colordiff

[Matthew Miller <mattdm mattdm org>]

On Fri, May 20, 2005 at 08:03:48PM +0300, Ville Skyttä wrote:
> > Oh, I see that Ville has actually added a cdiff wrapper that acts exactly
> > like cdiff, thus removing my earlier complaint.
> Yes, that was mentioned in my initial message to this thread :)

Yeah, I just misunderstood before I actually saw it. :)

> The most important thing about the config file is that default settings
> in it doesn't make things _invisible_ with the most "usual" color
> schemes.  (Non-)ugliness is secondary, and because the default color
> scheme is black-on-white in Fedora, the default is geared towards such
> setups.  See also the samples in the package's doc dir (there's one for
> dark backgrounds), and note that one can configure the colors exactly
> like one pleases in ~/.colordiffrc

I'll play around and see if I can come up with something more clever. It's
too bad there's no way to automatically detect the background and switch on
the fly.

> I think chopping lines with -S would be very much unexpected behaviour.
[...]
> Although colorization doesn't always work for wrapped lines, it
> _usually_ does just fine here (using Konsole, FWIW).  Chopping the lines
> would avoid the colorization issue, but it would certainly _always_ hide
> parts of the information when long lines are present and cause the need
> to hit the right arrow which will certainly screw up the colorization.

Hmmm, yeah, the left and right scrolling with -S sure is br0ke. Okay, never
mind then. :)


-- 
Matthew Miller           mattdm mattdm org        <http://www.mattdm.org/>
Boston University Linux      ------>                <http://linux.bu.edu/>
Current office temperature: 78 degrees Fahrenheit.