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

simplify got_gotweb_openfile/flushfile Return only the FILE *, and require only that to close subsequently. Having to manage the file descriptor too makes using these functions unnecessary complex. millert@ agrees

simplify previous; avoid temporary string blind ok stsp@

show reference labels next to commit messages in tog log view requested by mpi@ ok op@

fix wrong assumption about author timestamp in test_blame_lines_shifted_skip Not all commits will share the same author timestamp. This test would fail when run at the wrong time of day as found out by op@'s regress runner. ok op@