My commit to reset committer info during rebase was too strict in some use cases. Even when simply forwarding a branch the rebase operation could now fail if GOT_AUTHOR is not set. To fix this, fall back on existing commiter information if no author is configured. And try to obtain author info from Git config in case GOT_AUTHOR is not set. Problems reported by Mikhail.

Rename GOT_PACK_NUM_TEMPFILES in gotwebd, since it clashes with a name already used in lib/repository.c. ok tracey

ok stsp@

Don't move the selection cursor due to a bogus offset after closing child views following use of 'B', C-l, and backspace log view key maps. ok stsp@

Replace view->mode == TOG_VIEW_LOG checks for requesting commits to populate the log view with calls to view->resize(). ok stsp@

ok op@

When switching a child log view from a h- to a v-split, new log lines are empty. Rather than (mis)trace height and (not) call request_log_commits(), let resize_log_view() do the math and request commits if needed to draw new lines. ok op@

got_repo_pack_fds_close will try to close all of the GOT_PACK_NUM_TEMPFILES fds, even the ones that didn't manage to get a value from got_opentempfd(), which would result in a close(0) if we do not initialize to -1 here.

With the narrow scope of bytes_sent_cur it would always be zero, and we would always update progress. Found by llvm's scan-build (Dead store). OK stsp

Found by llvm's scan-build. OK stsp

Found by llvm's scan-build. OK stsp

Found by llvm's scan-build (bad free). OK stsp

This can't happen in practice, but llvm's scan-build doesn't know this. OK stsp

Found by llvm's scan-build (dead store). OK stsp

Found by llvm's scan-build (dead store). OK stsp

Found by llvm's scan-build (dead store). OK stsp

Found by llvm's scan-build (dead store). OK stsp

OK stsp

Found by llvm's scan-build (dead store). OK stsp

Found by llvm's scan-build (dead store). OK stsp