reword misleading comment in struct got_repository WRT struct got_pack

gotd no longer uses chroot; remove comments suggesting otherwise

gotwebd: avoid needless double fseek() got_gotweb_closefile() already takes care of rewinding before truncating and closing, so no need to replicate here. ok stsp@

gotwebd: lower log priority of unexpected disconnections no need to be so loud when a client disconnects earlier than expected, log it only in verbose mode. Don't log GOT_ERR_CANCELLED too since that is only used to communicate the callers that we need to stop. ok stsp@

relax build_refs_str to allow a NULL reflist, simplifes the callers ok stsp@

oops, one was missing

got_operations.c: use consistently a space before labels makes diff prettier and easier to read; instead of the label we have the function name in the hunk line.

fix indent; touches only `return 0;' lines

tog: fix segfault in draw_commit build_refs_str() can succeed returning a NULL string if a commit has some refs pointing to it but that were all filtered out, resulting in a NULL-deref. ok stsp@

don't zero fields of a calloc()-allocated struct

remove fsync() from got_gotweb_closefile() discussed with millert@

rename got_gotweb_flushfile -> got_gotweb_closefile suggested and ok millert@

make got_gotweb_flushfile always close the file ok millert@

log close() failure in gotweb_free_transport

rename some local fdN vars after previous change