Commit Briefs
sync usage with reality
The -h and -V/--version flags must be given before the command; with those flags the command is actually optional, but don't over-complicate the usage string with these nitpicks. ok/improvements stsp@
backout Make & command in tog's log view display "searching..."
The limit feature is more than just a search operation, tog's man page has been updated accordingly.
Make & command in tog's log view display "searching..." while loading commits.
This command acts like a search operation, and the display should reflect that.
tog: open help in fullscreen irrespective of view configuration
Prompted by a suggestion from stsp to make the help view distinct rather than follow the default behaviour of established views. Occupying the fullscreen seems to be a somewhat established convention for TUI apps. ok stsp@
drop unconventional got_object_commit_dup() routine
Replace with got_object_commit_retain() to increment commit object reference counter. suggested by and ok stsp@
tog: don't embed utf8 glyphs into tog.c source code
Reported by stsp: Embedded utf8 precludes developers running C locales from browsing the code. Fix was suggested by stsp with hints from the tmux(1) codebase. ok stsp@
tog: refactor search setup into tog_view function pointers
Suggested by stsp. While here, add declarations for new help view routines. ok stsp@
tog: implement runtime help accessible via H,F1 keymaps
Original idea inspired from discussion with tb, tobhe, and stsp at g2k22: display view-specific help, with option to toggle display of full key map reference when {H,F1} is pressed again inside the help view. Includes improvements suggested by stsp. ok stsp@
tog: add new log view limit feature to filter commits
Use the & key map to enter a pattern with which to limit the displayed commits to those matching the provided pattern; similar to less(1) and mutt's limit feature. Includes various tweaks from op. ok plus fixes from op@
make got_commit_graph_iter_next use caller-provided storage for the id
and adjust the callers. discussed with and ok stsp@
tog: make headline highlight extend the full view width
In splitscreen mode, make the active view headline highlight extend the full width of view->ncols. While here, format the tree view header like the blame view by moving the index to the path line. suggested by and ok stsp@
tog: ensure stdin is a tty
This guards against {mis,ab}use such as `tog < /dev/null`. ok plus fix from op@
plug leak in the commit graph iterator
We fail to release the memory for the nodes. To fix it however, we some consumer of the commit graph iterator need to be corrected: the returned pointer is safe to be used only up until the next iter_next call; save a copy it if it's needed afterwards too. ok stsp@
tog: remove gcc ternary if extension
ok stsp@
Group options in accordance with style(9)
patch by Josiah Frentsos
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@