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