Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | group external functions by where they come from | dholland | 2014-03-29 | 1 | -6/+13 | |
| | ||||||
* | be consistent about whether functions are static | dholland | 2014-03-29 | 1 | -5/+5 | |
| | ||||||
* | handle compiled-in pathnames the customary way | dholland | 2014-03-29 | 4 | -13/+56 | |
| | ||||||
* | make some globals static | dholland | 2014-03-29 | 2 | -14/+9 | |
| | ||||||
* | Additional minor cleanup and remove a straggler data declaration | dholland | 2014-03-29 | 4 | -19/+23 | |
| | | | from hunt_common.h. | |||||
* | Group function declarations by which source file they come from, and | dholland | 2014-03-29 | 1 | -28/+36 | |
| | | | delete several that don't actually exist. | |||||
* | be consistent about whether functions are static | dholland | 2014-03-29 | 2 | -7/+7 | |
| | ||||||
* | sprinkle static | dholland | 2014-03-29 | 4 | -32/+21 | |
| | ||||||
* | Flush unwanted file. | dholland | 2014-03-29 | 2 | -42/+2 | |
| | ||||||
* | Move definitions to the file they're used in. | dholland | 2014-03-29 | 2 | -11/+11 | |
| | ||||||
* | move a comment that should have gone with its symbol | dholland | 2014-03-29 | 2 | -9/+9 | |
| | ||||||
* | Remove tautological #if DEBUG. | dholland | 2014-03-29 | 1 | -15/+2 | |
| | ||||||
* | Split hunt and huntd's includes, instead of incestuously sharing a pile | dholland | 2014-03-29 | 10 | -257/+381 | |
| | | | | | | | | | | | | of external data and function declarations between the two programs. Common constants and defines now go in hunt_common.h. Stuff that belongs only to hunt is in hunt/hunt_private.h. Stuff that belongs only to huntd is in huntd/hunt.h. Copy some declarations that are used in both programs under the same names (but are not actually the same objects) from huntd/hunt.h to hunt/hunt_private.h. Move others that are only used in hunt. Remove some entirely unused material, and tidy up standard includes. | |||||
* | Remove unsafe and unnecessary pointer casts. | dholland | 2014-03-29 | 1 | -7/+6 | |
| | ||||||
* | catch up to huntd changes. | dholland | 2014-03-29 | 3 | -37/+38 | |
| | | | XXX: this needs to stop promiscuously sharing .h files. | |||||
* | tsort contents of file | dholland | 2014-03-29 | 1 | -49/+43 | |
| | ||||||
* | const in unused code | dholland | 2014-03-29 | 1 | -7/+7 | |
| | ||||||
* | remove stubbed-out function | dholland | 2014-03-29 | 3 | -53/+7 | |
| | ||||||
* | Rip out the code that broadcasts talk requests to the local | dholland | 2014-03-29 | 7 | -707/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | hunt-players mailing list. In this day and age it doesn't even begin to work: even if you were to have such a mailing list locally, sendmail wouldn't expand it for you, the addresses you got back wouldn't be talk-requestable addresses, talk requests don't cross NAT or firewalls safely, talk doesn't support ipv6, and nobody runs talkd anyway. It doesn't make sense to try to replace it with some other kind of notify scheme either; there might still be a multiuser machine somewhere with enough simultaneous users that broadcasting something akin to talk requests locally might make sense, and where a nontrivial number of the users actually play hunt, but probably not, and if so those users are likely to be better off using wall(1) anyway. Nowadays games will be set up by looking for people in chat or by emailing or texting friends. Theoretically someone could set up an internet hunt metaserver for finding hunt games, but that would be something entirely different anyway and I doubt there's demand. | |||||
* | Remove empty header file. | dholland | 2014-03-29 | 6 | -50/+9 | |
| | ||||||
* | Use setproctitle() instead of handrolled version. | dholland | 2014-03-29 | 2 | -26/+7 | |
| | ||||||
* | Make the code for issuing talk requests to find players actually build. | dholland | 2014-03-29 | 6 | -45/+55 | |
| | ||||||
* | don't declare own boolean type | dholland | 2014-03-29 | 9 | -128/+120 | |
| | ||||||
* | use random(), not a handrolled 16-bit generator | dholland | 2014-03-29 | 1 | -12/+4 | |
| | ||||||
* | Use getaddrinfo() instead of gethostbyname(). | dholland | 2014-03-29 | 1 | -7/+23 | |
| | ||||||
* | typo in error message | dholland | 2014-03-29 | 1 | -3/+3 | |
| | ||||||
* | const | dholland | 2014-03-29 | 1 | -4/+4 | |
| | ||||||
* | simplify silly logic | dholland | 2014-03-29 | 1 | -14/+11 | |
| | ||||||
* | temp is a pointer, and tempmax, not sizeof(temp), is the length of | apb | 2014-03-28 | 1 | -3/+3 | |
| | | | the buffer that it points to. Adjust a strlcat() call to suit. | |||||
* | If you are going to use memcpy instead of struct assignment, then at | apb | 2014-03-28 | 1 | -3/+3 | |
| | | | | | least use sizeof(the correct variable). The use of memcmp to test two pointers for equality, one line earlier, is also weird but may be correct. | |||||
* | Add note cautioning against bothering with WARNS=6 until gcc improves | dholland | 2014-03-23 | 1 | -1/+4 | |
| | | | | | | | | (or -Wconversion is removed from WARNS=6) as it produces loads of false positives. The most entertaining of these that I've seen this afternoon: games/hack/hack.apply.c:143:22: error: conversion to 'unsigned char:1' from 'int' may alter its value [-Werror=conversion] flags.move = multi = 0; ^ | |||||
* | fix signedness issue | dholland | 2014-03-23 | 1 | -4/+4 | |
| | ||||||
* | gratuitous (but not harmful) changes to pass gcc48's -Wconversion | dholland | 2014-03-23 | 1 | -10/+12 | |
| | ||||||
* | Use ssize_t for read() results. Don't use int for lseek() results. | dholland | 2014-03-22 | 1 | -6/+5 | |
| | | | Call srandom() with time(), not getpid(). | |||||
* | Use ssize_t for read() results. Don't use int for lseek() results. | dholland | 2014-03-22 | 1 | -6/+6 | |
| | ||||||
* | Use uid_t to hold getuid() results, and don't check getuid for failure | dholland | 2014-03-22 | 1 | -5/+3 | |
| | | | as it isn't allowed to fail. | |||||
* | use size_t instead of int for a length | dholland | 2014-03-22 | 1 | -3/+3 | |
| | ||||||
* | Use random(), not rand(). | dholland | 2014-03-22 | 2 | -5/+5 | |
| | ||||||
* | avoid unnecessary floating point ops | dholland | 2014-03-22 | 1 | -4/+7 | |
| | ||||||
* | avoid signed left-shift | dholland | 2014-03-22 | 1 | -5/+5 | |
| | ||||||
* | Use CHAR_BIT, not 8. | dholland | 2014-03-22 | 1 | -2/+3 | |
| | ||||||
* | don't declare ospeed; it's in <termcap.h> | dholland | 2014-03-22 | 1 | -3/+2 | |
| | ||||||
* | Make the list of place types an enumeration. Use unsigned values for | dholland | 2014-03-22 | 6 | -29/+30 | |
| | | | place numbers. | |||||
* | use a more suitable type | dholland | 2014-03-22 | 1 | -3/+4 | |
| | ||||||
* | use more suitable types | dholland | 2014-03-22 | 3 | -11/+13 | |
| | ||||||
* | woops, more of previous. | dholland | 2014-03-22 | 2 | -7/+7 | |
| | ||||||
* | Make some things unsigned that should be (why would we need e.g. a | dholland | 2014-03-22 | 4 | -45/+52 | |
| | | | negative number of airports?) and remove some related bogus casts. | |||||
* | Don't cast the return value of malloc. | dholland | 2014-03-22 | 2 | -13/+13 | |
| | ||||||
* | Minor tidyup. | dholland | 2014-03-22 | 1 | -6/+6 | |
| | ||||||
* | tag decl() __noinline, as inlining it would defeat its purpose | dholland | 2014-03-22 | 1 | -3/+3 | |
| | | | | (which is to not have interesting string literals appear in the compiled binary) |