CVS Repository/ src/ games/ cribbage/ io.c

Fix input handling wrt redrawing. The new version of curses does
not like shortcutting switching windows, especially if you're also
using libc for i/o. Reported by Don Wilburn; ok jsg@

As per style(9), remove remaining lint-style comments from games/

OK tb@

Include only needed header files per each source file

This one was also OK'ed by tb@

Cast ctype functions' arguments to unsigned char.

ok guenther@

Various cleanup in incard():
* get_line() always returns the same non-NULL pointer,
so testing the return value is useless.
* Garbage collect the unused variable *line.
* Check for end-of-buffer before parsing the next word instead of after it.
* Skip strlen() when the length has been measured three lines before anyway.
* Do not initialize local variables over and over again.
feedback and ok pjanzen@

Fix a mini-bug reported by pjanzen@:
When entering card names, you can use multiple words (like KING OF SPADES).
If you entered more than one consecutive blank character between words,
the function incard() took that as end-of-string and ignored the rest.
Fix this by dropping duplicate blanks up front, in get_line().
Patch simplified by me, ok pjanzen@.

While here, use beep(3) in an adjacent line instead of manually fiddling
with control characters, suggested by pjanzen@.

KNF: unify style and reduce indentation in get_line(), no binary change
ok cmp(1)

After entering an invalid three-letter card name, one letter card names
stopped working because the third letter remained in the buffer,
incard() skipped the NUL and used the old garbage.
Fix this bug reported by pjanzen@, but in a simpler way than he
suggested, by just clearing any trailing garbage from the buffer.
ok pjanzen@

Various cleanup:
1. Style: Don't use variables as format strings.
NetBSD rev. 1.26 (May 23, 2011) joerg@NetBSD via maintainer pjanzen@.
2. Style: Avoid needless pointer arithmetics.
NetBSD rev. 1.27 (Oct 13, 2012) dholland@NetBSD via pjanzen@,
who also applied some KNF to the indentation while here.
3. End curses before printing fatal error messages; from pjanzen@.

In preparation for getline and getdelim additions to libc, rename getline()
occurrences to get_line().
Based on a diff from Jan Klemkow <j-dot-klemkow-at-wemelug-dot-de> to tech.

Rename use of some builtins that makes gcc4 warn.

ok millert

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable).  these days, people use source.  these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

Some more ambigous -> ambiguous pointed out by mbalmer (there are
others, these are the ones from code unlikely to be re-merged with
upstream sources).

more ansi; khalek@linuxgamers.net

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999.  Proofed by myself and Theo.

strings... ok henning@, millert@, tedu@

We live in an ANSI C world.  Remove lots of gratuitous #ifdef __STDC__ cruft.

Better handling of out-of-range numbers.
Extraneous prompt and related comma splice removed.

Kill BOOLEAN, just use the curses/c++ bool type instead.

tags, formatting, ANSI-fication, prototypes, de-typos, and the occasional
initialization, removal of unused variable, or other minor fix.  Most
changes are from or inspired by NetBSD.

#if __STDC__ --> #ifdef __STDC__

Clean up play, let player see play more often, improve display timing.
Preliminary work from PR 2055 (as reverse patches); phil

Initial revision

initial import of NetBSD tree