On Fri, May 27, 2011 at 03:00:59PM +0200, Denys Vlasenko wrote:
> diff -p option prints "what function we are in" header
> before each hunk. Example:
> 
>     @@ -440,19 +440,18 @@ static void parse_config_file(void)
> 
> strace source, however, upsets working of this option because
> we have (very-)old-style C function definitions in some places.
> Even though they are gradually replaced, I still often
> have broken "diff -p" output which shows last parameter
> declaration instead of function name.
> 
> This patch converts all remaining old-style C function definitions
> to a "modern" form. This does not change any actual code.

The change is definitely OK if it really doesn't change any actual code.
If you are sure there is no accidental mistake, please commit.


-- 
ldv

Attachment: pgp0mMrmIFqn9.pgp
Description: PGP signature

------------------------------------------------------------------------------
vRanger cuts backup time in half-while increasing security.
With the market-leading solution for virtual backup and recovery, 
you get blazing-fast, flexible, and affordable data protection.
Download your free trial now. 
http://p.sf.net/sfu/quest-d2dcopy1
_______________________________________________
Strace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to