CVS commit: src/games/canfield/cfscores

2010-01-03 Thread David A. Holland
Module Name:src Committed By: dholland Date: Sun Jan 3 17:08:45 UTC 2010 Modified Files: src/games/canfield/cfscores: cfscores.c Log Message: Christos points out that usage should not use errx, but should use getprogname. To generate a diff of this commit: cvs rdiff -u

Re: CVS commit: src/games/canfield/cfscores

2010-01-03 Thread David Holland
On Fri, Jan 01, 2010 at 11:48:08PM +, Christos Zoulas wrote: imo, lower case usage would be more consistent with other commands. Some grep usage suggests that lowercase outnumbers capitalized by a good margin, but there's still a healthy minority of capitalized usage messages.

Re: CVS commit: src/games/canfield/cfscores

2010-01-01 Thread Marc Balmer
Am 01.01.2010 um 07:20 schrieb David A. Holland: Module Name: src Committed By: dholland Date: Fri Jan 1 06:20:45 UTC 2010 Modified Files: src/games/canfield/cfscores: cfscores.c Log Message: Send error messages to stderr. Use errx/warnx, not printf. To generate a

Re: CVS commit: src/games/canfield/cfscores

2010-01-01 Thread David Holland
On Fri, Jan 01, 2010 at 03:33:26PM +0100, Marc Balmer wrote: if (argc 2) { - printf(Usage: cfscores -a | cfscores [user]\n); - exit(1); + errx(1, Usage: cfscores -a | cfscores [user]); imo, lower case usage would be more consistent with other

Re: CVS commit: src/games/canfield/cfscores

2010-01-01 Thread Marc Balmer
Am 01.01.2010 um 18:35 schrieb David Holland: On Fri, Jan 01, 2010 at 03:33:26PM +0100, Marc Balmer wrote: if (argc 2) { - printf(Usage: cfscores -a | cfscores [user]\n); - exit(1); + errx(1, Usage: cfscores -a | cfscores [user]); imo, lower case usage

Re: CVS commit: src/games/canfield/cfscores

2010-01-01 Thread Christos Zoulas
In article 20100101173538.ga17...@netbsd.org, David Holland dholland-sourcechan...@netbsd.org wrote: On Fri, Jan 01, 2010 at 03:33:26PM +0100, Marc Balmer wrote: if (argc 2) { - printf(Usage: cfscores -a | cfscores [user]\n); - exit(1); + errx(1, Usage:

CVS commit: src/games/canfield/cfscores

2009-12-31 Thread David A. Holland
Module Name:src Committed By: dholland Date: Fri Jan 1 06:16:36 UTC 2010 Modified Files: src/games/canfield/cfscores: cfscores.c Log Message: Correct usage message. To generate a diff of this commit: cvs rdiff -u -r1.16 -r1.17 src/games/canfield/cfscores/cfscores.c

CVS commit: src/games/canfield/cfscores

2009-12-31 Thread David A. Holland
Module Name:src Committed By: dholland Date: Fri Jan 1 06:20:45 UTC 2010 Modified Files: src/games/canfield/cfscores: cfscores.c Log Message: Send error messages to stderr. Use errx/warnx, not printf. To generate a diff of this commit: cvs rdiff -u -r1.17 -r1.18

CVS commit: src/games/canfield/cfscores

2009-12-31 Thread David A. Holland
Module Name:src Committed By: dholland Date: Fri Jan 1 06:31:18 UTC 2010 Modified Files: src/games/canfield/cfscores: cfscores.c Log Message: Use NULL instead of 0 for pointer tests. Remove an unnecessary cast. To generate a diff of this commit: cvs rdiff -u -r1.18