spotted by + ok op@

`type' and `file' are strings so use `const char *' for both.

different yacc implementations (e.g. GNU bison) may put the various parts in a different order, and so the definition of the function may not be visible in the generated code.

Allow administrators to tweak the default authentication and request timeouts if needed, and to tweak the limit of concurrent connections for specific user accounts. with several tweaks from and ok op@

This header isn't needed for direct compilation, but should be included as part of the main tarball.

unbreaks `make dist'

it's only needed at build time to generate a source file.

was added there mistakenly. it's not part of the "compiler", it's part of the "user library" and provides useful functions for the consumers. see also 1dec34dfda7ccb3caadab65aca907251ab8fd72e

People coming from CVS might not expect names of patch contributers to show up in commit history listings, so make things easy for them. discussed with op@

Because gotd expects some parameters to be present they must not be optional at the syntax level.

gotwebd defaults to listening on a unix socket, and this man page aims to provide an appropriate configuration snippet in EXAMPLES. suggested by Paul Wrankin via gameoftrees IRC

otherwise if binaryfilename fails to match the line, `name' may be left pointing to free'd memory.

Diffs that remove binary files don't have hunks so got patch would skip over them, treating that part of the diff as "noise". Different programs outputs a slightly different diff for this kind of patches, but the "Binary files ... and /dev/null differ" is usually shown, so try to match it. The adedd regress test covers got diff, git diff, and OpenBSD' /usr/bin/diff. CVS diffs will currently fail because the guessed file name will be wrong. CVS prints the file name in the Index and RCS lines which got patch currently ignores, and shows an useless path in the "Binary files /tmp/cvs... and /dev/null differ" line. Discussed with and ok stsp@

Regression from 156a1144 spotted by stsp, thanks!

For now the limit is set at compile-time. It will become configurable via gotd.conf soon. ok op@

qs is guaranteed to be not NULL reached that point.

should be placed further down, after BLOB and RSS are handled, but the call to `gotweb_render_header' is successfully clear on its own.