Module Name: src Committed By: dholland Date: Sat Jul 4 01:01:19 UTC 2009
Modified Files: src/games/hunt/huntd: answer.c ctl_transact.c driver.c execute.c expl.c faketalk.c get_names.c shots.c Log Message: Remove a pile of bogus casts. One object file diff, crosschecked and found harmless. To generate a diff of this commit: cvs rdiff -u -r1.11 -r1.12 src/games/hunt/huntd/answer.c cvs rdiff -u -r1.6 -r1.7 src/games/hunt/huntd/ctl_transact.c \ src/games/hunt/huntd/execute.c cvs rdiff -u -r1.14 -r1.15 src/games/hunt/huntd/driver.c cvs rdiff -u -r1.4 -r1.5 src/games/hunt/huntd/expl.c cvs rdiff -u -r1.12 -r1.13 src/games/hunt/huntd/faketalk.c cvs rdiff -u -r1.7 -r1.8 src/games/hunt/huntd/get_names.c \ src/games/hunt/huntd/shots.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/games/hunt/huntd/answer.c diff -u src/games/hunt/huntd/answer.c:1.11 src/games/hunt/huntd/answer.c:1.12 --- src/games/hunt/huntd/answer.c:1.11 Sun Jun 28 21:12:35 2009 +++ src/games/hunt/huntd/answer.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: answer.c,v 1.11 2009/06/28 21:12:35 dholland Exp $ */ +/* $NetBSD: answer.c,v 1.12 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: answer.c,v 1.11 2009/06/28 21:12:35 dholland Exp $"); +__RCSID("$NetBSD: answer.c,v 1.12 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ # include <ctype.h> @@ -90,15 +90,15 @@ machine = gethostid(); # endif version = htonl((u_int32_t) HUNT_VERSION); - (void) write(newsock, (char *) &version, LONGLEN); - (void) read(newsock, (char *) &uid, LONGLEN); + (void) write(newsock, &version, LONGLEN); + (void) read(newsock, &uid, LONGLEN); uid = ntohl(uid); (void) read(newsock, name, NAMELEN); (void) read(newsock, &team, 1); - (void) read(newsock, (char *) &enter_status, LONGLEN); + (void) read(newsock, &enter_status, LONGLEN); enter_status = ntohl((unsigned long) enter_status); (void) read(newsock, Ttyname, NAMELEN); - (void) read(newsock, (char *) &mode, sizeof mode); + (void) read(newsock, &mode, sizeof mode); mode = ntohl(mode); /* @@ -161,7 +161,7 @@ i = pp - Monitor + MAXPL + 3; } else { socklen = 0; - (void) write(newsock, (char *) &socklen, + (void) write(newsock, &socklen, sizeof socklen); (void) close(newsock); return FALSE; @@ -173,7 +173,7 @@ i = pp - Player + 3; } else { socklen = 0; - (void) write(newsock, (char *) &socklen, + (void) write(newsock, &socklen, sizeof socklen); (void) close(newsock); return FALSE; @@ -409,7 +409,7 @@ ip->i_score = ip->i_kills / (double) ip->i_entries; } else { - ip = (IDENT *) malloc(sizeof (IDENT)); + ip = malloc(sizeof(*ip)); if (ip == NULL) { /* Fourth down, time to punt */ ip = &punt; Index: src/games/hunt/huntd/ctl_transact.c diff -u src/games/hunt/huntd/ctl_transact.c:1.6 src/games/hunt/huntd/ctl_transact.c:1.7 --- src/games/hunt/huntd/ctl_transact.c:1.6 Wed Jun 11 12:00:22 2003 +++ src/games/hunt/huntd/ctl_transact.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: ctl_transact.c,v 1.6 2003/06/11 12:00:22 wiz Exp $ */ +/* $NetBSD: ctl_transact.c,v 1.7 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -39,7 +39,7 @@ #if 0 static char sccsid[] = "@(#)ctl_transact.c 5.2 (Berkeley) 3/13/86"; #else -__RCSID("$NetBSD: ctl_transact.c,v 1.6 2003/06/11 12:00:22 wiz Exp $"); +__RCSID("$NetBSD: ctl_transact.c,v 1.7 2009/07/04 01:01:18 dholland Exp $"); #endif #endif /* not lint */ @@ -80,7 +80,7 @@ do { /* resend message until a response is obtained */ for (retries = MAX_RETRY; retries > 0; retries -= 1) { - cc = sendto(ctl_sockt, (char *)&msg, sizeof (msg), 0, + cc = sendto(ctl_sockt, &msg, sizeof (msg), 0, &daemon_addr, sizeof (daemon_addr)); if (cc != sizeof (msg)) { if (errno == EINTR) @@ -104,7 +104,7 @@ * request/acknowledgements being sent) */ do { - cc = recv(ctl_sockt, (char *)rp, sizeof (*rp), 0); + cc = recv(ctl_sockt, rp, sizeof (*rp), 0); if (cc < 0) { if (errno == EINTR) continue; Index: src/games/hunt/huntd/execute.c diff -u src/games/hunt/huntd/execute.c:1.6 src/games/hunt/huntd/execute.c:1.7 --- src/games/hunt/huntd/execute.c:1.6 Sun Jun 28 21:12:35 2009 +++ src/games/hunt/huntd/execute.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: execute.c,v 1.6 2009/06/28 21:12:35 dholland Exp $ */ +/* $NetBSD: execute.c,v 1.7 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: execute.c,v 1.6 2009/06/28 21:12:35 dholland Exp $"); +__RCSID("$NetBSD: execute.c,v 1.7 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ # include <stdlib.h> @@ -496,7 +496,7 @@ { BULLET *bp; - bp = (BULLET *) malloc(sizeof (BULLET)); /* NOSTRICT */ + bp = malloc(sizeof(*bp)); if (bp == NULL) { if (owner != NULL) message(owner, "Out of memory"); Index: src/games/hunt/huntd/driver.c diff -u src/games/hunt/huntd/driver.c:1.14 src/games/hunt/huntd/driver.c:1.15 --- src/games/hunt/huntd/driver.c:1.14 Sun Jun 28 21:12:35 2009 +++ src/games/hunt/huntd/driver.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: driver.c,v 1.14 2009/06/28 21:12:35 dholland Exp $ */ +/* $NetBSD: driver.c,v 1.15 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: driver.c,v 1.14 2009/06/28 21:12:35 dholland Exp $"); +__RCSID("$NetBSD: driver.c,v 1.15 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ # include <sys/ioctl.h> @@ -145,20 +145,20 @@ if (fdset[2].revents & POLLIN) { namelen = DAEMON_SIZE; port_num = htons(sock_port); - (void) recvfrom(Test_socket, (char *) &msg, sizeof msg, + (void) recvfrom(Test_socket, &msg, sizeof msg, 0, (struct sockaddr *) &test, &namelen); switch (ntohs(msg)) { case C_MESSAGE: if (Nplayer <= 0) break; reply = htons((u_short) Nplayer); - (void) sendto(Test_socket, (char *) &reply, + (void) sendto(Test_socket, &reply, sizeof reply, 0, (struct sockaddr *) &test, DAEMON_SIZE); break; case C_SCORES: reply = htons(stat_port); - (void) sendto(Test_socket, (char *) &reply, + (void) sendto(Test_socket, &reply, sizeof reply, 0, (struct sockaddr *) &test, DAEMON_SIZE); break; @@ -167,7 +167,7 @@ if (msg == C_MONITOR && Nplayer <= 0) break; reply = htons(sock_port); - (void) sendto(Test_socket, (char *) &reply, + (void) sendto(Test_socket, &reply, sizeof reply, 0, (struct sockaddr *) &test, DAEMON_SIZE); break; @@ -977,7 +977,7 @@ for (ip = Scores; ip != NULL; ip = nextip) { nextip = ip->i_next; - (void) free((char *) ip); + (void) free(ip); } Scores = NULL; } Index: src/games/hunt/huntd/expl.c diff -u src/games/hunt/huntd/expl.c:1.4 src/games/hunt/huntd/expl.c:1.5 --- src/games/hunt/huntd/expl.c:1.4 Tue Jan 27 20:30:29 2004 +++ src/games/hunt/huntd/expl.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: expl.c,v 1.4 2004/01/27 20:30:29 jsm Exp $ */ +/* $NetBSD: expl.c,v 1.5 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: expl.c,v 1.4 2004/01/27 20:30:29 jsm Exp $"); +__RCSID("$NetBSD: expl.c,v 1.5 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ # include <stdlib.h> @@ -57,7 +57,7 @@ return; if (x < 0 || x >= WIDTH) return; - ep = (EXPL *) malloc(sizeof (EXPL)); /* NOSTRICT */ + ep = malloc(sizeof(*ep)); ep->e_y = y; ep->e_x = x; ep->e_char = type; @@ -132,7 +132,7 @@ for (pp = Monitor; pp < End_monitor; pp++) check(pp, y, x); # endif - free((char *) ep); + free(ep); } for (x = EXPLEN - 1; x > 0; x--) Expl[x] = Expl[x - 1]; Index: src/games/hunt/huntd/faketalk.c diff -u src/games/hunt/huntd/faketalk.c:1.12 src/games/hunt/huntd/faketalk.c:1.13 --- src/games/hunt/huntd/faketalk.c:1.12 Sun Jun 28 21:12:35 2009 +++ src/games/hunt/huntd/faketalk.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: faketalk.c,v 1.12 2009/06/28 21:12:35 dholland Exp $ */ +/* $NetBSD: faketalk.c,v 1.13 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: faketalk.c,v 1.12 2009/06/28 21:12:35 dholland Exp $"); +__RCSID("$NetBSD: faketalk.c,v 1.13 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ #include "bsd.h" @@ -123,7 +123,7 @@ * start fetching addresses */ - if ((sp = getservbyname("smtp", (char *) NULL)) == NULL) { + if ((sp = getservbyname("smtp", NULL)) == NULL) { # ifdef LOG syslog(LOG_ERR, "faketalk: smtp protocol not supported\n"); # else @@ -239,7 +239,7 @@ msg.type = DELETE; msg.id_num = (int) htonl(response.id_num); daemon_addr.sin_addr = his_machine_addr; - if (sendto(ctl_sockt, (char *) &msg, sizeof (msg), 0, + if (sendto(ctl_sockt, &msg, sizeof (msg), 0, (struct sockaddr *) &daemon_addr, sizeof(daemon_addr)) != sizeof(msg)) p_error("send delete remote"); Index: src/games/hunt/huntd/get_names.c diff -u src/games/hunt/huntd/get_names.c:1.7 src/games/hunt/huntd/get_names.c:1.8 --- src/games/hunt/huntd/get_names.c:1.7 Wed Jun 11 12:00:22 2003 +++ src/games/hunt/huntd/get_names.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: get_names.c,v 1.7 2003/06/11 12:00:22 wiz Exp $ */ +/* $NetBSD: get_names.c,v 1.8 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: get_names.c,v 1.7 2003/06/11 12:00:22 wiz Exp $"); +__RCSID("$NetBSD: get_names.c,v 1.8 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ #include "bsd.h" @@ -146,7 +146,7 @@ msg.r_name[NAME_SIZE - 1] = '\0'; /* if he is on the same machine, then simply copy */ - if (memcmp((char *) &his_machine_name, (char *) &my_machine_name, + if (memcmp(&his_machine_name, &my_machine_name, sizeof(his_machine_name)) == 0) memcpy(&his_machine_addr, &my_machine_addr, sizeof(his_machine_name)); Index: src/games/hunt/huntd/shots.c diff -u src/games/hunt/huntd/shots.c:1.7 src/games/hunt/huntd/shots.c:1.8 --- src/games/hunt/huntd/shots.c:1.7 Sun Jun 28 21:12:35 2009 +++ src/games/hunt/huntd/shots.c Sat Jul 4 01:01:18 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: shots.c,v 1.7 2009/06/28 21:12:35 dholland Exp $ */ +/* $NetBSD: shots.c,v 1.8 2009/07/04 01:01:18 dholland Exp $ */ /* * Copyright (c) 1983-2003, Regents of the University of California. * All rights reserved. @@ -32,7 +32,7 @@ #include <sys/cdefs.h> #ifndef lint -__RCSID("$NetBSD: shots.c,v 1.7 2009/06/28 21:12:35 dholland Exp $"); +__RCSID("$NetBSD: shots.c,v 1.8 2009/07/04 01:01:18 dholland Exp $"); #endif /* not lint */ # include <err.h> @@ -149,7 +149,7 @@ } chkshot(bp, next); - free((char *) bp); + free(bp); } for (pp = Player; pp < End_player; pp++) @@ -315,7 +315,7 @@ pp->p_ident->i_saved++; message(pp, "Absorbed charge (good shield!)"); pp->p_ident->i_absorbed += bp->b_charge; - free((char *) bp); + free(bp); (void) snprintf(Buf, sizeof(Buf), "%3d", pp->p_ammo); cgoto(pp, STAT_AMMO_ROW, STAT_VALUE_COL); @@ -497,7 +497,7 @@ pp = play_at(bp->b_y, bp->b_x); pp->p_ammo += bp->b_charge; message(pp, "**** Absorbed drone ****"); - free((char *) bp); + free(bp); (void) snprintf(Buf, sizeof(buf), "%3d", pp->p_ammo); cgoto(pp, STAT_AMMO_ROW, STAT_VALUE_COL); outstr(pp, Buf, 3); @@ -785,7 +785,7 @@ } break; } - nbp = (BULLET *) malloc(sizeof (BULLET)); + nbp = malloc(sizeof(*nbp)); *nbp = *bp; # ifdef VOLCANO move_slime(nbp, nbp->b_type == SLIME ? SLIMESPEED : LAVASPEED, next); @@ -811,7 +811,7 @@ if (speed == 0) { if (bp->b_charge <= 0) - free((char *) bp); + free(bp); else save_bullet(bp); return; @@ -847,7 +847,7 @@ } if (--bp->b_charge <= 0) { - free((char *) bp); + free(bp); return; } @@ -952,7 +952,7 @@ move_slime(nbp, speed - 1, next); } - free((char *) bp); + free(bp); } /*