Thread (4 messages) flat view 4 messages, 3 authors, 2016-06-15

Re: [PATCH v2 4/5] Make boundary characters for --color-words configurable

From: Jakub Narebski <hidden>
Date: 2016-06-15 22:44:36

"Ping Yin" [off-list ref] writes:
On Mon, May 12, 2008 at 12:27 AM, Junio C Hamano [off-list ref] wrote:
quoted
"Ping Yin" [off-list ref] writes:
quoted
With following patch, the diff output becomes (i don't know which
one is better)

OpenOffice.org has {+a }user setting for defining the minimum length for
words to be hyphenated. By default the word length is counted from the
whole word - even for compound words. For example the {compound +}word
'elokuvalippu' is {+considered }12 characters long. The word will be hyphenated like
 'elo-ku-va-lip-pu' in all cases when the minimum word length is set to
 12 or less. If the minimum length is set to 13 or more the word is not
 hyphenated at all.
 Yeah, after playing with it a bit, I realize that my original
 stated goal of not playing games with "newline suppression" goes
 very against what color-words, which is a word oriented diff,
 tries to achieve.  It appears that it is necessary to reintroduce
 suppressed_newline.
No matter how well we play with suppressed_newline, we still can't
achieve the best result by doing word diff between multiple minus
lines and multiple plus lines.

 ( i think the result of vimdiff can be considered as the best).
Is the vimdiff algorithm described anywhere? What about wdiff output?
 
To achieve the best, we have to find the pairs of lines (one minus and
one plus for each pair) which most match each other, and then do the
word diff for each pair.
Wouldn't be enough to treat run of plus/minus lines as a single block,
tokenize, do token-based (as opposed to line-based) diff, then show it
using linebreaks of the destination file (pluses line)?

-- 
Jakub Narebski
Poland
ShadeHawk on #git
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help