The diff text widget is read-only, so there's zero point in
building an undo stack. This change reduces memory consumption of
this widget by about 95%.

Memory usage of the whole program for viewing a reference commit
before; 579'692'744 bytes, after: 32'724'446 bytes.

Test procedure:

 - Choose a largish commit and check it out. In this case one with
   90'802 lines, 5'006'902 bytes.

 - Have a Tcl version with memory debugging enabled. This is,
   build one with --enable-symbols=mem passed to configure.

 - Instrument Gitk to regularly show a memory dump. E.g. by adding
   these code lines at the very bottom:

     proc memDump {} {
         catch {
             set output [memory info]
             puts $output
         }

         after 3000 memDump
     }

     memDump

 - Start Gitk, it'll load this largish commit into the diff text
   field automatically (because it's the current commit).

 - Wait until memory consumption levels out and note the numbers.

Note that the numbers reported by [memory info] are much smaller
than the ones reported in 'top' (1.75 GB vs. 105 MB in this case),
likely due to all the instrumentation coming with the debug
version of Tcl.
Signed-off-by: NMarkus Hitter <mah@jump-ing.de>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Signed-off-by: NSatoshi Yasushima <s.yasushima@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

When -S or -G is used as a filter option, the resulting commit list
rarely contains all matching commits. Only a certain number of commits
are displayed and the rest are missing.

"git log --boundary -S" does not return as many boundary commits as you
might expect. gitk makes up for this in closevarcs() by adding missing
parent (boundary) commits. However, it does not change $numcommits,
which limits how many commits are shown. In the end, some commits at the
end of the commit list are simply not shown.

Change $numcommits whenever a missing parent is added to the current
view.
Signed-off-by: NStefan Dotterweich <stefandotterweich@gmx.de>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Some systems don't recognize "lime" as a color, leading to errors when
gitk is run.  What we want is a bright green, so use "#00ff00" instead.
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Signed-off-by: NVasco Almeida <vascomalmeida@sapo.pt>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Force creation of destination bin directory.  Without this, gitk
would fail to install if this directory didn't already exist.
Signed-off-by: NVasco Almeida <vascomalmeida@sapo.pt>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Hi,

I made another branch dialog related change, included in this message.
It applies on top of my other two patches.

Rogier.

------- 8< ------------------- 8< --------------

Only the SHA1 was included. It's convenient to have the title
mentioned as well.
Signed-off-by: NRogier Goossens <goossens.rogier@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Git allows checking out remote branches, creating a local tracking
branch in the process. Allow gitk to do this as well, provided a
local branch of the same name does not yet exist.
Signed-off-by: NRogier Goossens <goossens.rogier@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Signed-off-by: NRogier Goossens <goossens.rogier@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@ozlabs.org>

Make Help > About & Key bindings dialogs readable if theme
has changed font color to something incompatible with white.
Signed-off-by: NGuillermo S. Romero <gsromero@infernal-iceberg.com>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NJean-Noel Avila <jn.avila@free.fr>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NJean-Noel Avila <jn.avila@free.fr>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NRalf Thielow <ralf.thielow@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NAlexander Shopov <ash@kambanaria.org>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NPeter Krefting <peter@softwolves.pp.se>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

The widgets on top of the diff window are very tightly packed. Make
them breathe a little by adding an 'i'-spaced padding between them.
Signed-off-by: NGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

The fonts set in setoptions aren't consistently picked up by ttk, which
uses its own predefined fonts. This is noticeable when switching
between using and not using ttk with custom fonts or in HiDPI settings
(where the default TTK fonts do _not_ respect tk sclaing).

Fix by mapping the ttk fontset to the one used by gitk internally.
Signed-off-by: NGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

msgfmt(1) wants this header.
Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

Signed-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

 Color name "green" was darken since Tcl/Tk 7.6.
 Because color name scheme was changed from "X11 colors" to "Web colors".

 Use "lime" to keep colors.

See also:
http://www.tcl.tk/cgi-bin/tct/tip/403.htmlSigned-off-by: NYOKOTA Hiroshi <yokota@netlab.cs.tsukuba.ac.jp>

This brings in an update to the Vietnamese translation.
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Signed-off-by: NDimitriy Ryazantcev <dimitriy.ryazantcev@gmail.com>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Assigned either to the first letter or some unique letter.  At least
there are no conflicts, as far as I see...
Signed-off-by: NTakashi Iwai <tiwai@suse.de>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

Just follow the English accelerator keys.
Signed-off-by: NTakashi Iwai <tiwai@suse.de>
Signed-off-by: NPaul Mackerras <paulus@samba.org>

The commit d99b4b0d ("gitk: Accelerators for the main menu")
modified the menu item strings with the accelerator, but the
translations didn't follow, thus the menus are shown without
translations.

This patch systematically update the msgid keys just to follow this
change.  The contents aren't changed, so the accelerator won't work in
these locales for now.  Each locale translator needs to add proper
acceleration keys appropriately.
Signed-off-by: NTakashi Iwai <tiwai@suse.de>
Signed-off-by: NPaul Mackerras <paulus@samba.org>