fix gotwebd default fcgi listen port byte-order (port 10275 -> port 9000) ok tracey

gotwebd: listen on localhost only by default ok tracey

gotwebd: struct socket_conf can only use a single address, so don't use a list ok tracey

gotwebd: add fcgi_printf instead of fcgi_gen_response which outputs only a fixed strings provide a printf-like fcgi_printf: it greatly simplifies the generation of the HTML pages. While here also (probably) fix some HTML errors: the output was verified with the W3C validator and it's correct (in the sense that the tags are properly closed, there are still some other things the validator complains about.) ok/encouragement baseprime@, ok jamsek Thanks for reading such a boring diff!

fix overflow in blame callback spotted by noticing gotwebd crashing on some blame requests. Diff from stsp@ with a fix from tracey@, I'm committing it only because he is short on time. ok stsp@

gotwebd: initialize IPv4 and IPv6 sockets in the same way ok tracey

gotwebd: don't listen on FCGI sockets when FCGI is not enabled ok tracey

gotwebd: do not allocate server/socket list heads separately ok millert@

gotwebd: do not allocate address list heads separately, this is unnecessary ok millert@

fix snprintf error handling follow the "proper secure idiom" described in the CAVEATS section of printf(3). reminded by tb@ and millert@

convert two snprintf to strlcpy "looks good to me" millert@

check for fprintf failure

got(1): Replace 'Ar sign' with 'Cm sign' Cm sign matches the markup used in the ssh-keygen(1) man page. Patch by Josiah Frentsos

regress: adjust expected usage output in "revert_no_arguments" test

Group options in accordance with style(9) patch by Josiah Frentsos

gotwebd: fix double free in gotweb_process_request don't free the querystring, it's already done later by fcgi_celanup_request. ok tracey@

tog: add missing checks for NULL pointers in log view state ok jamsek

tog: don't block when processing log view 'G' key map As per stsp's TODO item, make tog respond to key presses while "loading..." history. Initial testing in src.git appears to work as intended. ok stsp@

tog: fix delayed tree index update If a child blame view has been opened, halfdelay() is reset to a 1 second refresh rate, so the index in the header lags behind subtree navigation as we set the index based on the selected tree entry, which isn't updated till that refresh cycle completes. Don't wait for the selected_entry to update; instead, compute the index from the first displayed entry and the currently selected line index. improved by and ok stsp@

tog: move all 'G' logic into log_move_cursor_down() Previously, we only handled 'G' in log_move_cursor_down() when all commits had not yet been loaded; move the case where all commits are already loaded into this routine too, and simplify by handling both cases the same. This change was prompted by Mikhail's observation of a redundant assignment in the previous 'G' handling in log_move_cursor_down(). ok stsp@

tog: fix log 'd' keymap when last commit is displayed When at the end of the log view such that the currently selected line is greater than nlines/2, add--don't assign--the page down value to the selected index so we don't jump up when the user expects tog to scroll down. While here, rework log page down logic for a smoother UX in horizontal split mode by moving the selection cursor down rather than scrolling when 'd' or 'f' is entered if the last commit is already on screen. ok stsp@

fix unused variable, function definition warnings for clang 15

convert gotwebd.conf "bind interface IP/iface" to "listen on IP/iface" ok kn, tracey

gotwebd: fix alternate colors in tree entries it was broken in 7ecc454 because now we're emitting tags with multiple `class' attributes (by mistake) and only the first one is used. However, instead of fixing the HTML `class' attribute, use a CSS selector to alternate the background colors. ok tracey@

gotwebd: add a some buffering This adds some buffering to fcgi_gen_binary_response so that we don't end up sending too many (very) small fastcgi records. While here also make fcgi_send_response non-void and make its errors "sticky". ok tracey@