Re: [PATCH 3/3 v2] use cache for function names in hunk headers

From: Clemens Buchacher <drizzd@aon.at>
Date: Sep 26, 2010, 20:43 UTC
Message-ID: <20100926204318.GA19588@localhost>
In-Reply-To: <4C9F7450.9060208@lsrfire.ath.cx>

On Sun, Sep 26, 2010 at 06:26:56PM +0200, René Scharfe wrote:

> 
> Is xf needed?  Does xdl_emit_diff() handle multiple files in one go?

Right now it does not.

> If you inline xdl_find_func() the struct isn't needed anymore.

[...]

>
> How about something like this?

Yes, that's better. Thanks.

Show 7 quoted lines

> -	/*
> -	 * Be quite stupid about this for now.  Find a line in the old file
> -	 * before the start of the hunk (and context) which starts with a
> -	 * plausible character.
> -	 */
>
> It also removes an outdated comment.

Actually, in the default case, the comment is still correct and helpful IMO.

> The inlining part should probably split out in its own patch..

I am not sure what you mean here. Do you want to add the parameter funclineprev to the function and remove the function in the next page, or do you want to refactor the below into an inline function?

Clemens