Simplifies the reading of the author/committer string and makes also the code shorter. ok stsp@

recv_imsg_error already checks the size of the message, and it does so more precisely than the removed code. ok stsp@

ints have the advantage that their size is more likely to be the same across the various architecture supported by OpenBSD, thus introducing less possible differences. INT_MAX is still (at least) a few order of magnitudes higher than the patches we dealt with (even abnormal ones.) suggested by stsp@

previous commit looked at some pretty zeroes returned from calloc instead of the actual numbers received.

under normal circumstances got-read-patch can't send negative line offsets; that's not an excuse not to guard the main process from those as well. This makes sure we bail out early instead of trying to apply the diff.

The problem that was found in the earlier version has been fixed. ok op@

ok op@

ok op@

ok op@

ok op@

This matches actual behaviour of less(1) for Ctrl+u/d, and leaves d/u free for potential future use. Suggested by naddy. Patch by Mark Jamsek

So that 'got log -S' can match also the author and committer name, and the commit ID too. ok stsp@

suggested by + ok op@

ok op@

-s for short. Moves log search to -S. Patch by Evan Silberman

This implementation marked commits and trees as enumerated before all trees which they depend on were enumerated. This behaviour leads to incomplete pack files when a tree is only partially packed and got-read-pack hits a missing tree entry as a result. The algorithm must be reworked such that packed leave nodes are marked enumerated first, then bubble-up. Found by op@