[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Elliott Hughes
Subject: Re: [bug-grep] Re: [patch #3644] --initial-tab and 3 newly colorized items
Date: Wed, 26 Jan 2005 09:47:55 -0800

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:

Attachment: 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:


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).


reply via email to

[Prev in Thread] Current Thread [Next in Thread]