ok tracey@

While here pad line numbers with spaces instead of zeroes to match the blob view.

ok tracey@

positive feedback from Kyle Ackerman ok/tweak tracey@

ok tracey@

Diff from Kyle Ackerman. tracey@ and I likes it.

while here clean up some *_header id not used / not needed.

there's no need to override the background-color for the 'even' row. This will simplify the addition of a dark mode.

ok tracey@

The repo_write process still announces HEAD twice, once with its symref target and once with the resolved symref object hash. I cannot recall if this is deliberate but let's accept it for now to avoid test failures being flagged due to this issue, which is mostly cosmetic.

This should prevent a use-after-free crash I observed in gotwebd. ok op@

While here, remove completely the libevent handling of SIGPIPE in favour of the already existing signal(3) call to ignore it; there's no point in logging a SIGPIPE.

Diff from Kyle Ackerman, thank you! I've added a imsg_free() call before the break too.

I always find confusing if IMSG_SIZE_CHECK() takes pointers or not, and we had at least a few instances of wrong usages, so inline (with exact size checks) and remove the macro. ok stsp@

It only had 4 slots so was never quite useful, and sharing of sock->pack_fds across cached repositories seems problematic. with help from + ok op@

main_compose_sockets() has the 'feature' of implicitly dup(2)'ing the passed file descriptior. But it's not what we need for the temp fds, since those needs to be unique per-children. debugged with stsp@

allows to run gotwebd from the got checkout again.