Subject: Re: CVS commit: src/usr.bin/netstat
To: Rui Paulo <rpaulo@netbsd.org>
From: Bang Jun-Young <junyoung@netbsd.org>
List: source-changes
Date: 08/12/2005 09:45:26
Rui Paulo wrote:
> On 2005.08.12 08:01:56 +0000, matthew green wrote:
> |    
> |    On Fri, Aug 05, 2005 at 10:46:53AM +0300, Elad Efrat wrote:
> |    > Quentin Garnier wrote:
> |    > 
> |    > >Neither versions are correct.  PRIu64 should be used.
> |    > 
> |    > Right -- diff attached.
> |    
> |    Please use Unified diffs (-u) or context diffs (-c) in the future. Many of 
> |    us find these diff formats easier to read. They also more-easily cope with 
> |    other changes to the file.
> | 
> | 
> | while we're making diff option suggestions, i highly recommend that
> | everyone use "diff -p" all the time.  (put "diff -p" in .cvsrc.)  it
> | usually makes the patch tell you what function a chunk is from which
> | greatly improved the human-readability of it.
> 
> I agree. It's a shame that cvs(1) doesn't show that option on
> cvs diff help.

[junyoung@krishna junyoung]$ cvs --help diff
Usage: cvs diff [-lR] [-k kopt] [format_options]
    [[-r rev1 | -D date1] [-r rev2 | -D date2]] [files...]
...
  -c  -C NUM  --context[=NUM]  Output NUM (default 2) lines of copied context.
  -u  -U NUM  --unified[=NUM]  Output NUM (default 2) lines of unified context.
    -NUM  Use NUM context lines.
    -L LABEL  --label LABEL  Use LABEL instead of file name.
    -p  --show-c-function  Show which C function each change is in.
...

This is CVS 1.11.20.

Jun-Young