• Back to Project

Comments for ColorDiff

Re: colordiffrc

> Hmmm - looks like the patch got mangled
> - sorry. Email me privately at
> davee@sungate.co.uk if you want the
> patch.
>
> Dave.

T'is OK. I entered the changed lines manually at line
#70.

Regards,
Kevin

Re: colordiffrc
Hmmm - looks like the patch got mangled - sorry. Email me privately at davee@sungate.co.uk if you want the patch.

Dave.

Re: colordiffrc

> Nice program! Am nitpicking really :-).
>
> colordiffrc doesn't tolerate blank
> (empty lines) or
> whitespace before variable names.

Oh, OK, seeing as it's a quick fix ... ;-)

Attached patch for colordiff.pl will make it more tolerant of such whitespace:

Index: colordiff.pl
========================================
RCS file: /cvsroot/colordiff/colordiff/src/colordiff.pl,v
retrieving revision 1.4.2.5
retrieving revision 1.4.2.6
diff -r1.4.2.5 -r1.4.2.6
70c70,71
s/^\s+//;
> next if (/^#/ || /^$/);

Note that this patch is for the latest release, version 1.0.2 - and will be included in the next version.

Regards,

Dave - ColorDiff author.

great !
simple, lightweight and great !

colordiffrc
Nice program! Am nitpicking really :-).

colordiffrc doesn't tolerate blank (empty lines) or
whitespace before variable names.

i.e the following works:

---------------%< cut here %<-----------
# .colordiffrc
#
# Color values
dirstuff=magenta
---------------%< cut here %<-----------

but this doesn't:

---------------%< cut here %<-----------
# .colordiffrc
<--- blank line
# Color values
dirstuff=magenta
^^leading space here
---------------%< cut here %<-----------