Hi,
On Wed, 23 Sep 2009, Linus Torvalds wrote:
> On Wed, 23 Sep 2009, Johannes Schindelin wrote:
> >
> > print_wrapped_text() will insert its own newlines. Up until now, if the
> > text passed to it contained newlines, they would not be handled properly
> > (the wrapping got confused after that).
> >
> > The strategy is to replace a single new-line with a space, but keep double
> > new-lines so that already-wrapped text with empty lines between paragraphs
> > will be handled properly.
>
> May I suggest doing this _only_ if the newline is followed by an
> alphanumeric characer?
>
> If the thing is indented ("newline + space") or quoted ("newline + ">" or
> whatever) then reflowing it is likely wrong and will result in an unholy
> mess.
It seems Junio is dead-set on ignoring my patches, so I will probably just
keep the patch as-is in my tree for the time being. The other Johannes'
patch is too limited for me to use, so I _will_ keep my patch, but I have
way too little time to fight an uphill battle to get my patch in. And
since it stays a private patch (just like my strbuf_vaddf() patch) I will
have to refrain from putting more work and time into it than I already
have.
Sorry,
Dscho
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html