We can assume that the querystring was initialized when we reach gotweb_render_header. ok tracey@

This adds a new helper function `gotweb_link' that outputs an HTML link with the parameter correctly escaped. ok stsp@

reported by the W3C validator.

due to an error in the order of the arguments to fcgi_printf that I've introduced last time I touched gotweb_render_header the title is now "/" (or SCRIPT_NAME) and one assets (the safari pinned tab svg) is a funny URL :)

keeping GOTWEBD_REPO_CACHESIZE * sizeof(struct cache_repo) inside the struct server makes it too large for imsg on some platforms (linux at least.) Instead, store a pointer and allocate the array when it's received on the child processes. ok stsp@

ok stsp@

ok stsp@

ok stsp@

if strdup fails we fail to free the querystring Found by clang' scan-build.

dead store found by clang' scan-build.

In 95a4a5a1e8d0952ee0d2523dc31df7170d80e0b3 I forgot to fix the links for the favicons and the web manifest.

problem introduced in commit 625e5896fc9ecf87ccfc92ad2a65cd3be58f73c0

got_get_repo_tags needs some tweaking to how tag_commit is allocated too. ok stsp@

all very unlikely. part of a bigger diff that's ok stsp@