Commit Briefs

Stefan Sperling

make tog block other keys except Backspace after End/G is pressed

This should avoid unexpected behaviour resulting from unrelated key presses messing with the log view's state variables. Pointed out by tracey, and also discussed with jasper. ok tracey



Stefan Sperling

make tog ignore Ctrl+L and B while loading all commits

Otherwise tog gets into a funky non-recoverable state if these keys are pressed instead of Backspace while commits are loading. bug found by + ok tracey


Stefan Sperling

remove the 'ctrl+u' shortcut in favour of just 'g' as alias for 'Home'

Our use of ctrl+u was not consistent with vi(1) after all. Discussed with naddy, tb, and jasper on gameoftrees@



Jasper Lievisse Adriaanse

tog: add support for navigating to first/last item of log and diff views

The keybindings that we settled on are Home, Ctrl-u and g to go to the first item and End, G to go to the last. This resembles those commonly found elsewhere, eg vi/less. discussed with and ok stsp


Stefan Sperling

use gmtime_r(3) instead of localtime_r(3) to display time in UTC as intended

Problem noticed by naddy due to failing regress tests at midnight, and then analyzed with additional help from millert. ok naddy


Stefan Sperling

fix a use-after-free in get_changed_paths() in got and tog

Once the parent commit is closed the tree_id1 pointer is no longer valid, but the pointer was still being used. Make a deep copy to fix this issue.





Christian Weisgerber

switch from SIMPLEQ to equivalent STAILQ macros

The singly-linked tail queue macros were added to OpenBSD 6.9 and are more widely available on other systems. ok stsp


Stefan Sperling

check for close(2) error in got_repo_close() and propagate errors up

ok tracey


Josh Rickmar

Fix strftime(3) short buffer checks

strftime(3) returns 0 if the buffer was too short to write the complete string (including NUL) and will never return more than maxsize-1. ok stsp


Stefan Sperling

tog: fix behaviour when 'n' is pressed before a search was started with '/'

reported by + ok naddy


Stefan Sperling

make fclose(3) failure checks consistent; check 'fclose() == EOF' everywhere

ok millert, naddy



Christian Weisgerber

tog log: terminate author field at '>' in case there is no '@'

ok stsp



Stefan Sperling

remove a comment that is now misleading



Christian Weisgerber

switch the "tog <path>" log view shortcut to the global reference list, too

ok stsp


Stefan Sperling

Add a refs argument to got_repo_match_object_id(), replacing 'resolve_tags'

Make use of this where possible to avoid re-reading references from disk. ok naddy


Stefan Sperling

switch tog ref view to the global reference list as well

ok naddy