[BzrTools/MERGE] cdiff --check-style Highlight trailing white space
Marius Kruger
amanic at gmail.com
Sun Apr 20 01:07:28 BST 2008
I finally got time to look at this again,
(I got tired of merging my changes in locally every time I update.)
On Wed, Jan 17, 2007 at 6:48 PM, Aaron Bentley <aaron.bentley at utoronto.ca>
wrote:
> Marius Kruger wrote:
> > On 1/17/07, *John Arbash Meinel* <john at arbash-meinel.com
>
> > I attached a bundle to 'Highlight trailing white space'
> > its probably an ugly hack, but I like the result.
>
> Apparently, we also need "Contains leading tabs" :-)
done
I realise it was a bit of a joke, but I did it any ways as it might
obviously be useful.
It also gives a summary of new lines containing tabs (if any).
>
> I like the idea, but I don't like having two different tests for
> trailing whitespace. Don't Repeat Yourself, as Martin recently reminded
> me.
I tried to do this better,
but that forced me to send the match object to colorstring(), or pass the
components seperately.
The colorstring() is not a nice api any more, so I was thinking
to change mine to _colorstring() and bring back the original one (for in
case someone used it elsewhere).
> > We can also highlight text which is over 79 chars...
> > what do you think?
>
> David Allouche actually had a neat one for this. Instead of "trailing
> whitespace" or "under 80 chars", he did "wraps idempotently".
I'm not sure what you mean by "wraps idempotently",
but my new version highlight long lines nicely.
regards
marius
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20080420/6c1f93b7/attachment.htm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzrtools_highlight_2.patch
Type: text/x-diff
Size: 9241 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080420/6c1f93b7/attachment.bin
More information about the bazaar
mailing list