The (unused) parameter of thread entry points is named `argv', but the

argument is actually the value of global variable `player'.  Rename to
`unused'.

src/server/idle.c

@@ -42,7 +42,7 @@
 
 /*ARGSUSED*/
 void
-player_kill_idle(void *argv)
+player_kill_idle(void *unused)
 {
     struct player *p;
     time_t now;