Commit Briefs
plug memory leak in got_pack_dump_delta_chain_to_file()
found by smatch. ok stsp@
pass the argument to calloc in the right order; no-op in practice
reported by smatch.
fix logging during gotwebd shutdown
During `rcctl stop gotwebd' the sockets process gets a SIGTERM and dies, and this makes the parent process assume it exited abnormally. Instead, catch SIGINT and SIGTERM and exit gracefully. Issue reported by xs on IRC. ok jamsek
gotwebd: fix colour of target lines in dark mode
reported by xs on IRC, thanks!
Test log -b <path> traversing a merge commit.
This tests for the bug fixed by c8255edc. ok stsp@
handle merge commits that are unrelated to requested changed path history
Problem found and fix tested by James Cook.
plug some fd leaks in the fdopen{,dir} error paths
There's also a memory leak fixed. ok stsp@
style: no-op change in practice
change so it matches the style used in the rest of the tree. ok stsp
remove unneded wbuf->fd = -1
There's no need to set the fd to -1 on ibufs created with imsg_create(3), and it was probably never needed.
convert to use imsg_get_fd()
While here also fix a fd leak in got-read-pack. We were dup'ing imsg.fd without closing imsg.fd later; instead just use imsg_get_fd() to extract the file descriptor. Tested by falsifian and Kyle Ackerman, thanks! 'go ahead' stsp@
gotwebd: try to improve copy-paste from BLOB pages
This moves the space inside the line anchor and makes it non-selectable, so that when copy-pasting, only the content of line lines is actually copied. At the moment, this seems to behave slightly incorrectly in firefox which inserts a double line break due to the presence of unselectable elements inside the selection. Hopefully it will be fixed, chromium behaves fine. It also still works decently on text-browsers. ok stsp@
allow remotes without urls in git config in gotd
This is b624328edd but for the code used by gotd. ok stsp@