Re: [PATCH] Make words boundary for --color-words configurable
- From
Ping Yin <pkufranky@gmail.com>
- Date
- May 2, 2008, 04:28 UTC
- Message-ID
- <46dff0320805012128l6cb15e1ekd40f84a9eac724d1@mail.gmail.com>
- In-Reply-To
- <7vprs5xsvi.fsf@gitster.siamese.dyndns.org>
On Fri, May 2, 2008 at 11:54 AM, Junio C Hamano <gitster@pobox.com> wrote:
> Ping Yin <pkufranky@gmail.com> writes: >
Show 11 quoted lines
> I suspect a more natural definition of word boundary is between a run of > word characters and a run of non-word characters. IOW, instead of saying > " " and "(these other)" characters are boundary, you would say > > if (foo(arg)) > > between f and space, open paren and f, second o and open paren, that open > paren and a, ... are boundaries. > > If you go that route, you would make the definition of "what is the set of > word characters" configurable.
I prefer nonwordcharacters to wordcharacters. With diff.wordcharacters, to maintain the same behaviour as before, i have to define a long list of characters. While with diff.nonwordcharacters, i need only define it to "".
So how about just renaming wordsboundary to nonwordcharacters?
-- Ping Yin