Re: [RFC/PATCH v2] gitweb: Option to chop at beginning and in the middle in chop_str
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 25, 2008, 01:46 UTC
- Message-ID
- <7v8x19st7x.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <20080224125920.24448.2179.stgit@localhost.localdomain>
Jakub Narebski <jnareb@gmail.com> writes:
Show 13 quoted lines
> Make use of new feature of chop_str to better cut matched string and > its context in match info for searching commit messages (commit > search), as proposed by Junio C Hamano. For example, if you are > looking for "very long ... and how" in the first paragraph of this message > (if it were all on a single line), you would now see: > > ...st this with <<very long ... and how>> the actual out... > > instead of: > > Could som... <<very long search stri...>> the actual out... > > (where <<something>> denotes emphasized / colored fragment).
This part needs rewritten; the first paragraph of what message is that?
Also I think the subject is wrong. Yes, it is adding an option to an internal subroutine but who cares? The net effect the "gitweb" users see is that the way the grep result is shown differently, hopefully in a more understandable way, and that change is not _optional_ at all.
The code looks easier to read than before, but I may be partial ;-)