Re: [bug-grep] Re: [patch #3644] --initial-tab and 3 newly colorized items

[Elliott Hughes]

On Jan 26, 2005, at 09:35, Julian Foad wrote:

> Elliott Hughes wrote:
> > On Jan 26, 2005, at 07:46, Stepan Kasal wrote:
> > > I also think that the --show*function options should eventually go 
> > > in.
> > are you sure? diff does a really poor job of that. something like 
> > Exuberant ctags can do a much better job, for far more languages. why 
> > not pipe the output of diff/grep into a script that adds the 
> > annotations?
>
> I tend to agree.  It's undeniably useful to some people to have such a 
> feature available at the touch of a single option letter, but 
> implementing and supporting such a feature which doesn't actually work 
> very well overall is quite a high cost.
>
> > i have such a script for annotating patches, if anyone's interested.
>
> Yes please.  I currently use "diff -p" for generating patches, and 
> then have a script to extract a list of the functions touched to make 
> a log message template.  It gets it wrong quite often, so I have to 
> edit the result by hand.  I'd like to use your better solution.

here's the current version:

annotate-patch.rb
Description: Binary data

anyone reading this mail via Google or an archive should probably 
download this library, and get the latest version from the bin/ 
directory:

  http://www.jessies.org/~enh/software/salma-hayek/salma-hayek.tgz

there's brief documentation (including BUGS) in the comments at the 
start of the script.

suggestions and improvements gratefully received. two obvious 
directions for extension are support for grep-style addresses as input, 
and automatic generation of GNU-style ChangeLog messages (which is one 
of the things I use this script for).

--
http://www.jessies.org/~enh/