On Sun, May 8, 2011 at 19:03, Sverre Rabbelier <srabbelier@xxxxxxxxx> wrote: > Heya, > > On Sun, May 8, 2011 at 14:10, Ãvar ArnfjÃrà Bjarmason <avarab@xxxxxxxxx> wrote: >> This adds skeleton no-op functions to git-sh-i18n.sh analogous to the >> gettext.c skeleton functions for C, adds *.sh scripts to the "pot" >> target for message extraction, and updates the git-sh-i18n--envsubst >> tests to use the new test_i18ncmp function. > > I seem to remember there were some concerns about performance with a > previous version of this series. Have you done any before/after > timings on, say, the git test suite? Or am I remembering incorrectly? When I benchmark this on my (Linux) system it runs at 0% difference between the current test suite and the one with this series. The concern was about Windows, where forks are more expensive, so e.g. every time we shell out to sed/awk/perl/grep or git-sh-i18n--envsubst we incur a larger speed hit than on Unix systems. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html