tokkee.org

Tree view: improve handling of empty trees

Fix reverting of unmerged status entries

... by using update-index --cacheinfo.

Add support for horizontal scrolling

It is still basically a big hack, and might need some tuning.

Expand tabs in displayed lines to not rely on ncurses to expand them

Treat empty '/' as "find next"

When the user requests a forward or backwards search, if
they provide no search string then we just abort the search.
However, many other programs (such as less and vi) treat
this as "repeat the last search", so users may have their
fingers trained to do this rather than use the explicit
"find next" feature.

This patch causes tig, if there was a previous search, to
find the next (or previous, if "?" was used) match in such a
case.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

TODO: elaborate and regroup into 'before tig-1.0' and 'long term goals'

Move initial view setup into parse_options

Move parse_option so it has access to view definitions

Remove parsing of deprecated option -S and subcommands log and diff

Rename & move read_properties and git_properties

They should be part of the IO API and are now called io_load and
run_io_load.

Use warn() for warnings emitted during config file loading

Fix scrolling bugs in gnome-terminal and (u)xterm

Introduces workarounds to fix glitches appearing after the recent screen
drawing optimizations.

BUGS: document problem with scrolling in (u)xterm

Tune the view clearing to wait until 2 seconds has passed

The code introduced in ec9aa4f62ae2977d598dda865e6a50b9fc799428 broke
the smoothness of updates in some cases.

Handle all cursor positioning in get_input

Also fixes use of the C++ true keyword which crept in in commit
babaa6f7a97cbe3948588e5181de0801ce32b792 (Refactor user input handling
into separate function).

Optimize drawing by updating the screen in one go

Switch to use wnoutrefresh everywhere and call doupdate in the get_input
loop. This allows multiple updates to be made while only one burst of
IO occurs.

Abbreviate author names to initials when author-width < 6

Refactor author drawing into draw_author

Fix tokenizing when parsing ~/.tigrc

When moving to use argv_from_string() in set_option() a regression was
introduced causing problems with parsing of multiple spaces between
arguments. For example:

color date              yellow  default

Reported by Clifford Caoile.

Workaround bug exposed by the redrawwin removal in do_scroll_view

The bug means that the message from scrolling up one line when
impossible followed by scrolling down one line is not removed by the
next action.

Workaround this by inserting an extra call to report("") before the call
to wrefresh().

Make cursor updating when resizing the display and loading a view

Change scrollok strategy to leave it off unless when calling wscrl

Should lead to fewer calls.

Initialize status_empty flag to FALSE

Fixes cursor restoring when loading the status view at startup.

Eliminate unneeded calls to redrawwin

This optimize the view drawing especially when scrolling/moving down the
main view with the diff view open. Should make browsing more bareable
over slow links.

TODO: a small commit cache should be employed at some point

Help view: show the action name similar as in Mutt's help view

Remove preallocation of view lines in help_open

corrected doubly used ID view-manipulation in manual.txt

Signed-off-by: Stefan Naewe <stefan.naewe@atlas-elektronik.com>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

tigrc(5) & manual: move view specific actions out of the misc group

Add support for loading blame for parent commits

Requested by Jeff King in <20080410040213.GA29618@sigill.intra.peff.net>

Refactor prompting for user input

Rename tree-parent action to parent

Minor cleanup of the tree view code; fix enter on the first line

TODO: remove resolved item

Tree view: improve to look less like plumbing

It is now more like `ls` by annotating the output of ls-tree with commit
information.

Refactor main_read to create parse_author_line

update_view: remove an unneeded goto

Fix floating point bug in the blame reporting

Also call end_update for views with custom open function

Broken by 49dc4030694e22b7f141bfb13565783985c48b77

Add -O0 to the debug flags

Rectify arg indexes changes in blame --no-color fix

Introduced in 78685441f4effd81082a53abb4c1246ebd1e0e49

Fix status loading of unmerged entries to avoid access to freed memory

Fix kill_io() to only call kill(2) when pid is non-zero

Blame view: fix reloading of the diff view for changes not yet committed

The regression was introduced in e00ea5d (Blame: do to not reload the
diff view for same commits).

Oops, always specify --no-color when using git diff

Blame view: fix diffing of lines marked as not yet committed

Add support for restoring the view position in reloadable views

The status view still uses its own more specialized restoring code.
There are still some cases which are not completely handled. For
example, restoring will only be done when the current line is the first
in the view, thus changing the view position will cancel the restoring.
However, if you change back to the first line restoring will be enabled.

Fix status_exists to be consistent with data displayed in the stage view

Make it possible to call select_view_line for non-displayed views

Refactor find_next_line into select_view_line

Refactor user input handling into separate function

Fixes resizing while the prompt is open.

Add support for opening any blob in an editor

Prefer werase to wclear when possible to reduce flickering

Use werase instead of wclear in redraw_view and optionally make
redraw_display use wclear before calling redraw_view. After this, wclear
is only used when there is possibility that the screen could have been
corrupted.

Cleanup and improve option toggling actions to include help message

Fix stage reloading to not close when staging chunks from group diff

Do not terminate the update when switching back to a loading view

Move call to end_update() to begin_update() so checks for view content
is honored in the case where no reloading is necessary.

Fix regression of main view drawing

Hopefully last fix for dirty flag regressions.

Cleanup update_view's read loop

Fix regression in handling of data for non-UTF-8 locales

The bug was introduced in 65498141015b6070cf5c9b5704039389dba76e90 where
the the IO API was changed from using stdio to 'raw' file descriptors.
Part of the change removed the need for cutting the ending newline from
the input line in update_view(). This caused the resulting iconv output
buffer to not be NUL terminated since the input string no longer
contained an ending NULL byte.

Reported by Pavel Roskin.

Fix another regression from the dirty flag changes causing flickering

Use redraw_view_from() instead of redraw_view() to avoid excessive calls
to wclear().

Fix serious bug where a stack allocated variable was returned

Reproducable when calling tig with log arguments.

Improve messages when preparing to load blame commits

Fix regressions introduced in the last few dirty flag changes

Regressions was introduced in ddc5064c42f2adab639feeacf86729a44f06f216.

Tree view: make drawing more smooth by using the dirty flag

Cleanup redrawing of views when updating by using the dirty flag

Cleanup the tree_read sorting loop

Fix memory corruption bug in tree_read when sorting the entries

Triggered by defining ITEM_CHUNK_SIZE to 1.

Cleanup and simplify the code by introducing add_line_format

Cleanup the code by calling realloc_lines in add_line_data

tig-0.13

Launch mergetool from the project root directory

Fixes problem when in a subdirectory.

Clear the view after 1 second during updates where reading takes long

Inspired by Karl Chen.

IO API: use select(2) to check if pipe is readable when updating a view

IO API: reindent status_run main loop after the rewrite

No code change.

IO API: replace io_gets with helper for scanning buffers

Use in status_run() to simplify the code.

IO API: use fork+exec

IO API: use argv internally

IO API: use file descriptor internally

IO API: replace init_io_fd with io_open which calls fopen(3)

IO API: use in the status view

IO API: use when loading repository properties

This introduces an incompatibility regarding the format of the
TIG_LS_REMOTE environment variable.

IO API: use in the stage view

IO API: obsolete opt_pipe

IO API: obsolete opt_cmd by using prepare_update in status_enter

IO API: use prepare_update when parsing command line arguments

IO API: use in add_describe_ref

IO API: use in the blame view

IO API: unify tree view and the default path in begin_update

IO API: use for the main, log, diff, tree and blob views

This introduces an incompatibility with previous versions.

IO API: use when preparing to run commands from the prompt

IO API: convert status checkout/revert support

IO API: use when running external commands

IO API: refactor the run request command formatter

Adds support for new %(file), %(directory) and %(ref) identifiers.
Also adds infrastructure for working with argument arrays

IO API: add small library for reading files and running programs

Blame: do to not reload the diff view for same commits

Blame: update blame to selected commit when pressing 'B'

Blame: remove unused member of struct blame

Add setup_update for initialize incremental view updates

Refactors common parts of begin_update and blame_open.

Slightly reduce memory usage for keybindings

Move formatting of blame --incremental command to blame_read_file

Doing it in blame_open after starting the command makes error handling
harder and the code was indeed buggy.