tokkee.org

Add missing NULL in blame_grep

Commit 436674f introduced a grep_text utility function which
takes a NULL-terminated list; the callsite in blame_grep
forgot to put in a NULL, leading to segmentation faults.

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

Fix a potential problem with reading tokens larger then BUFSIZ

If io_get() is not able to find the end marker it will go into
an infinite loop. To fix this make it possible to reallocate and
increase the buffer if necessary.

To avoid problems with callers reading into a stack allocated buffer,
change these to no longer use the "zero-copy" strategy. This in turns
cleans the callers up a bit.

Update the current branch information when reloading all references

Fixes problem of improper highlighting of current branch after switching
branch and reloading.

Define an allocator for run requests

Comparing the size of the binary before and after this change, shows
that the size does not increase when -02 is used.

Remove the need for alloc variables

Make the granular allocators more customizable using macros

Add a DEFINE_ALLOCATOR macro which can be used to define a custom
allocator which is more type safe than the current realloc_items and
which removed the need for using temporary variables for holding the
newly allocated chunk. The custom allocator also allows the allocation
granularity to be tweaked to the need of the user instead of being
hardcoded to 256 chunks for all.

Define a tree_grep and fixing searching

It was broken by the move to the annotated tree view.

Simplify searching in view lines by defining grep_text utility

Reduce memory and startup time by storing author times as time_t

Which means that drawing will have to do some more formatting, however,
the startup time is shorter since gmtime_r() is no longer called. Same
repository as last commit.

18980 fonseca   20   0 17776  15m  872 S    0  0.8   0:01.26 tig
18630 fonseca   20   0 23224  20m  868 S    0  1.0   0:01.26 tig-0.14.1

Add small cache for author names to reduce memory foot-print

Numbers from an old linux kernel repository with 2740 unique authors
shows that it takes a little longer time to start up but the gain is
quite good. Furthermore, it lifts the name length restriction.

18191 fonseca   20   0 19712  17m  872 S    3  0.8   0:01.34 tig
18180 fonseca   20   0 23224  20m  868 S    0  1.0   0:01.26 tig-0.14.1

tig-0.14.1

Remove unneeded doupdate from view_driver

Keep the cursor fixed while initial stage progress is reported

Fix draw_lineno to always set the static format buffer

Fix bug in draw_space to not access memory outside the space buffer

Refactor draw_lineno to use draw_graphic

Fix horizontal scrolling

... to not be limited to the view width and to account for
view->yoffset.

Remove limitation of horizontal scrolling

Tig now behaves similar to less in that the user can scroll without
limit to the right.

tigrc(5): suggest git aliases for external commands requiring shell ops

Fix the view clearing to only be effective for displayed views

The feature was introduced in ec9aa4f62ae2977d598dda865e6a50b9fc799428.

Make behavior of horizontal scrolling configurable

Requested by bill lam.

Fix handling of quoted strings in the config file

parse_string() adapts the string length to automatically remove quotation
marks when copying the string. However, when calling string_ncopy_do()
strlen(arg) used to be called again instead of using the adapted value.

This e.g. led to wrong locale settings when using
  set commit-encoding = "UTF-8"
and thus a slightly messed up display.

Thanks to Gerfried Fuchs for reporting this.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

tigrc(5): fix error in examples for the set command

Fixes Debian bug #514990 reported by Gerfried Fuchs.

Tree view: draw submodule entry modes as "m---------"

Inspired by gitweb.

Status view: report failures to update a file

Also, wait showing update progress until 5% has been reached.

Ignore broken pipe signals

Can be caused if a forked git process exits with an error. Tig should
detect this by checking the exit code of the child using the return
value done_io().

Use putenv instead of setenv

Not all platforms (e.g. Solaris) provide setenv. Based on patch by
SungHyun Nam.

manual: Correct the keys to move the cursor one line up/down

Signed-off-by: Carsten Brockmann <cabro@gmx.net>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

Status view: upon failure to open a file show error message

Refactor and share view maximization code from view-close handling

Fix io_strerror return type

Add support for handling core.worktree

If this option is found set up GIT_DIR and GIT_WORK_TREE. Also, pass
the known git directory path to git ls-remote.

Requested-by: bill lam

Optimize read_repo_config_option to reduce string comparisons

Move setting of remote branch from repo config to separate function

Mark enum_maps and other data tables as const

Use check_blame_commit when handling REQ_ENTER

NEWS: Mention Jeff's uninitialized variable bug fix

From commit 2677b744313c939890a8f6b3316f9d5452581ab8.

Handle blaming beyond the creation of file more gracefully

Currently when you ask to re-blame from the parent of a
commit that created the file, blame_read_file will complain
that it cannot get the file contents ("No blame exist").

At the time we try to read the file, it is too late to abort
the operation, as we have already changed to the new blame
view. However, we can detect this situation early by
limiting the selection of the parent revision to the
particular path of interest: if it returns a parent even
with path-limiting, then we know the path exists; if not,
then we know it doesn't.

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

Blame view: add guesstimation of line number when blaming parent commit

... by looking through the commit's diff for the hunk which introduced
the line. Also, try to set the filename properly.

Use file and line number information when loading blame for commit

This was developed in parallel and is very similar to patch posted by
Jeff King, however, with different goals in mind.

Message-Id: <20090207112613.GA18079@coredump.intra.peff.net>

Make the blame view expand tabs at drawing time

Introduce common view position update helper

So jumping to search results and line numbers will work more uniform.

Improve restoring of the view position to bound the offset

Jump to line when a number is entered in the prompt

Warn users about integers in ~/.tigrc being out of bound

Fix uninitialized variable in string_expand_length

This led to totally unpredictable results from the function. The style
matches the loop in string_expand.

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

Add support for 256 colors by allowing "colorN" names similar to Mutt

Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

Improve on branch information in the status view

The file checks was inspired by the prompt code from the git bash
completion script.

Simplify setting the default for the system configuration file

Refactor and unify timezone parsing

Remove the need for the OPEN_NOMAXIMIZE flag

It was only used for the status view together with the OPEN_RELOAD flag.
However, since reloading and refreshing is the same for the status use
OPEN_REFRESH instead.

Remove unused OPEN_BACKGROUNDED flag

Use enum_map for handling obsolete color names

Use enum_map for handling obsolete request names

Refactor the int_map interface into new enum_map interface

Renames int_map to enum_map and set_from_int_map to map_enum, which
now uses string_enum_compare instead of strncasecmp.

Cleanup and simplify option file parsing

tig-0.14

Read tigrc(5) options from git configuration files

tig(1): correct info on tree view related environment variables

Was not updated during the introduction of %(directory) and move to the
IO API.

Add release script documenting the release procedure

Spell check all text files and add dictionary to speed it up

tigrc(5): document the tree view colors

Also renames the tree-parent line to tree-head.

Refactor file mode drawing; rename tree-mode color to mode

Replace usage of the main-author color with the author color

It is used in draw_author() which is used by the main, blame and tree
views.

Move usage string near parse_options

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.