Re: [PATCH v4 02/14] add a hashtable implementation that supports O(1) removal

2013-11-08 Thread Karsten Blees
Am 07.11.2013 22:40, schrieb Junio C Hamano: > Karsten Blees writes: > >> +`void hashmap_add(struct hashmap *map, void *entry)`:: >> + >> +Adds a hashmap entry. This allows to add duplicate entries (i.e. >> +separate values with the same key according to hashmap_cmp_fn). >> ++ >> +`map` i

Re: [PATCH v4 13/14] builtin/update-index.c: cleanup update_one

2013-11-08 Thread Karsten Blees
Am 07.11.2013 22:40, schrieb Junio C Hamano: > Karsten Blees writes: > >> -if (p < path || p > path + strlen(path)) >> -free((char *)p); >> +free(p); > > The non-const cast was there for a reason. I'll locally fix it up > (ther

Re: [PATCH v4 12/14] fix 'git update-index --verbose --again' output

2013-11-08 Thread Karsten Blees
Am 07.11.2013 23:12, schrieb Junio C Hamano: > Karsten Blees writes: > >> 'git update-index --verbose' consistently reports paths relative to the >> work-tree root. The only exception is the '--again' option, which reports >> paths relative to the current working directory. >> >> Change do_reupda

Selectively commit/publish files to GIT

2013-11-08 Thread Thomas Manson
Hi, I've converting my Bazaar repository to GIT. I've successfully done this conversion and I now want to publish my source code to github. The problem is that in Bazaar, I've commited some big files (63MB & 173MB), but this files are no longer in my project, only in the revisions files of

Re: [PATCH 1/3] for-each-ref: introduce %C(...) for color

2013-11-08 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > If %(authordate) is "I want to see the author date here", and > %(authordate:short) is "I want to see the author date here in the > short form", you would expect "I want colored output in green" to be > spelled as %(color:green), or something, perhaps? Last time, we almost

Re: Selectively commit/publish files to GIT

2013-11-08 Thread Magnus Bäck
On Friday, November 08, 2013 at 05:52 EST, Thomas Manson wrote: [...] > How can I search git history for big files and remove them, or mark > them to be not published ? > > I've tryed this solution found on the link in the error: > > git filter-branch --force --index-filter 'git rm --

Re: [PATCH v4 02/14] add a hashtable implementation that supports O(1) removal

2013-11-08 Thread Philip Oakley
From: "Karsten Blees" However, defining the constants inversely is a bit unintuitive (i.e. 1.25 instead of 0.8, 6 instead of 0.166). Perhaps the thresholds should also be calculated once on resize, not on every add / remove. What about this: #define HASHMAP_GROW_AT 80 #define HASHMAP_SHRINK

Re: [PATCH v4 02/14] add a hashtable implementation that supports O(1) removal

2013-11-08 Thread Junio C Hamano
Karsten Blees writes: > What about this: > > #define HASHMAP_GROW_AT 80 > #define HASHMAP_SHRINK_AT 16 I am not too enthused for three reasons. The fact that these are 100-based numbers is not written down anywhere other than the place they are used, the places that use these need to consistentl

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-08 Thread Torsten Bögershausen
On 2013-11-07 23.19, Jeff King wrote: > On Thu, Nov 07, 2013 at 09:58:02PM +, Ramsay Jones wrote: > >> These patches fix various errors/warnings on the cygwin, MinGW and >> msvc builds, provoked by the jk/pack-bitmap branch. > > Thanks. Your timing is impeccable, as I was just sitting down to

Re: [PATCH 1/3] for-each-ref: introduce %C(...) for color

2013-11-08 Thread Junio C Hamano
Ramkumar Ramachandra writes: > ... %C(...) is > different in that it doesn't actually output anything, but changes the > color of tokens following it. While I'm not opposed to %(color:...), I > would prefer a color syntax that is different from other-token syntax, > like in pretty-formats. You m

[PATCH] git-fetch-pack uses URLs like git-fetch

2013-11-08 Thread Torsten Bögershausen
"git fetch-pack" allows [:] to point out the source repository. Use the term , which is already used in "git fetch" or "git pull" to describe URLs supported by Git. --- Documentation/git-fetch-pack.txt | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/Documentatio

Re: [PATCH v3] push: Enhance unspecified push default warning

2013-11-08 Thread Junio C Hamano
Matthieu Moy writes: > Jonathan Nieder writes: > >> When push.default is set to 'matching', git will push local branches >> to remote branches that already exist with the same (matching) name. > > Yes, that's better than the original patch (and remains two lines). > ... > >> In Gi

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-08 Thread Ramsay Jones
On 08/11/13 17:10, Torsten Bögershausen wrote: > On 2013-11-07 23.19, Jeff King wrote: >> On Thu, Nov 07, 2013 at 09:58:02PM +, Ramsay Jones wrote: >> >>> These patches fix various errors/warnings on the cygwin, MinGW and >>> msvc builds, provoked by the jk/pack-bitmap branch. >> >> Thanks. You

Re: [PATCH v3] push: Enhance unspecified push default warning

2013-11-08 Thread Junio C Hamano
Junio C Hamano writes: > OK, I'll tentatively update the draft to read like this, redo the > endgame patch on top and requeue. ... and the corresponding part of the endgame patch now reads like this. I suspect that we may want a bigger change to unstress 'simple' at that phase of the transition

[PATCH] l10n: de.po: translate 68 new messages

2013-11-08 Thread Ralf Thielow
Translate 68 new messages came from git.pot update in 727b957 (l10n: git.pot: v1.8.5 round 1 (68 new, 9 removed)). Signed-off-by: Ralf Thielow --- po/de.po | 191 --- 1 file changed, 86 insertions(+), 105 deletions(-) diff --git a/po/d

[PATCH] po/TEAMS: update Thomas Rast's email address

2013-11-08 Thread Ralf Thielow
Signed-off-by: Ralf Thielow --- po/TEAMS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/po/TEAMS b/po/TEAMS index 107aa59..f9a99ed 100644 --- a/po/TEAMS +++ b/po/TEAMS @@ -8,7 +8,7 @@ Leader: Byrial Jensen Language: de (German) Repository:https://github.co

[PATCH] l10n: de.po: improve error message when pushing to unknown upstream

2013-11-08 Thread Ralf Thielow
Signed-off-by: Ralf Thielow --- po/de.po | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/po/de.po b/po/de.po index a005dcc..8b824cc 100644 --- a/po/de.po +++ b/po/de.po @@ -7923,7 +7923,7 @@ msgid "" "to update which remote branch." msgstr "" "Sie versenden nach '%s', welch

[PATCHv2] l10n: de.po: translate 68 new messages

2013-11-08 Thread Ralf Thielow
Translate 68 new messages came from git.pot update in 727b957 (l10n: git.pot: v1.8.5 round 1 (68 new, 9 removed)). Signed-off-by: Ralf Thielow --- v2 fixed a typo in the translation of the message from "builtin/repack.c:161". s/Fenster/Fensters po/de.po | 191 ---

[ANNOUNCE] Git v1.8.4.3

2013-11-08 Thread Junio C Hamano
The latest maintenance release Git v1.8.4.3 is now available at the usual places. The release tarballs are found at: http://code.google.com/p/git-core/downloads/list and their SHA-1 checksums are: 43b1edc95b3ab77f9739d789b906ded0585fe7a2 git-1.8.4.3.tar.gz eb4eb4991464f44deda19d1435d972114

Re: [PATCH 1/5] compat/bswap.h: Fix build on cygwin, MinGW and msvc

2013-11-08 Thread Jeff King
On Fri, Nov 08, 2013 at 01:45:50AM +0100, SZEDER Gábor wrote: > Hi, > > On Thu, Nov 07, 2013 at 09:59:38PM +, Ramsay Jones wrote: > > +static inline uint64_t default_bswap64(uint64_t val) > > +{ > > + return (((val & (uint64_t)0x00ffULL) << 56) | > > + ((val & (uint64_

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-08 Thread Jeff King
On Fri, Nov 08, 2013 at 06:10:30PM +0100, Torsten Bögershausen wrote: > Side question: > Do we have enough test coverage for htonll()/ntohll(), > or do we want do the "module test" which I send a couple of days before ? The series adds tests for building and using the ewah bitmaps, which in turn

Re: [PATCH v3] push: Enhance unspecified push default warning

2013-11-08 Thread Marc Branchaud
On 13-11-08 01:02 PM, Junio C Hamano wrote: > Matthieu Moy writes: > >> Jonathan Nieder writes: >> >>> When push.default is set to 'matching', git will push local branches >>> to remote branches that already exist with the same (matching) name. >> >> Yes, that's better than the original

Re: Selectively commit/publish files to GIT

2013-11-08 Thread Thomas Manson
Hi Magnus, thanks for pointing that out. I must have overlooked the webpage, when I saw that I thought it was some kind of alias that designated files greater than a specific files (server side configurated as the 100MB limit) I'll try again ;) Thomas. On Fri, Nov 8, 2013 at 3:13 PM, Magnus

Re: Selectively commit/publish files to GIT

2013-11-08 Thread Thomas Manson
Thanks Magnus, it did solve my issue. I've noticed by the way that i've accidentally commited the whole .bzr directory (that hold all my commit in Bazaar style). I've deleted this folder as well using -r git filter-branch --force --index-filter 'git rm -r --cached --ignore-unmatch .bzr' --pr

Re: [PATCH v5 06/10] fast-export: add new --refspec option

2013-11-08 Thread Junio C Hamano
Jeff King writes: > On Wed, Nov 06, 2013 at 01:00:42PM -0800, Junio C Hamano wrote: > >> It follows that the syntax naturally support >> >> git fast-export refs/heads/master:refs/heads/foobar >> >> I would think. >> >> That approach lets you express ref mapping without a new option >> --r

Re: Selectively commit/publish files to GIT

2013-11-08 Thread Keshav Kini
Thomas Manson writes: > The problem is that in Bazaar, I've commited some big files (63MB & > 173MB), but this files are no longer in my project, only in the > revisions files of Bazaar and now Git. > > I don't need this files to be pushed on Github. > > How can I search git history for big

Re: [PATCH v5 06/10] fast-export: add new --refspec option

2013-11-08 Thread Jeff King
On Fri, Nov 08, 2013 at 03:44:43PM -0800, Junio C Hamano wrote: > > The "^origin" is not a refspec, and finding the refspec in the > > dot-expression would involve parsing it into two components. I think you > > can come up with a workable system by parsing the arguments as revision > > specifiers

[PATCH 1/4] Minor grammatical fixes in "git log" man page

2013-11-08 Thread Jason St. John
git-log.txt: grammatical fixes under --log-size option Signed-off-by: Jason St. John --- Documentation/git-log.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt index 34097ef..243a9c5 100644 --- a/Documentation/git

[PATCH 2/4] Emphasize options and force ASCIIDOC escaping of "--"

2013-11-08 Thread Jason St. John
rev-list-options.txt: replace e.g. `--foo` with '\--foo' rev-list-options.txt: emphasize, instead of quote, some option arguments (e.g. "foo-option" becomes 'foo-option') rev-list-options.txt: force ASCIIDOC escaping of "--" (e.g. '--bar' becomes '\--bar') rev-list-options.txt: add single q

[PATCH 3/4] Correct word usage of "timezone" in "Documentation" directory

2013-11-08 Thread Jason St. John
"timezone" is two words, not one (i.e. "time zone" is correct). Correct this in these files: -- date-formats.txt -- git-blame.txt -- git-cvsimport.txt -- git-fast-import.txt -- git-svn.txt -- gitweb.conf.txt -- rev-list-options.txt Signed-off-by: Jason St. John --- Sources: (1) https://en.wikipe

[PATCH 4/4] Correct word usage of "timezone" in "Documentation/RelNotes" directory

2013-11-08 Thread Jason St. John
"timezone" is two words, not one (i.e. "time zone" is correct). Signed-off-by: Jason St. John --- I broke this off into a separate patch in case the release notes are essentially "fixed" in history and typos, misspellings, etc. don't get corrected. Sources are provided below the commit message in

Re: [PATCH 1/4] Minor grammatical fixes in "git log" man page

2013-11-08 Thread Jonathan Nieder
Hi, Jason St. John wrote: > git-log.txt: grammatical fixes under --log-size option Thanks. [...] > --- a/Documentation/git-log.txt > +++ b/Documentation/git-log.txt > @@ -56,10 +56,10 @@ Note that this affects all diff-based output types, e.g. > those > produced by --stat etc. > > --log-siz

Re: [PATCH 4/4] Correct word usage of "timezone" in "Documentation/RelNotes" directory

2013-11-08 Thread Jonathan Nieder
Jason St. John wrote: > I broke this off into a separate patch in case the release notes are > essentially "fixed" in history and typos, misspellings, etc. don't get > corrected. Yeah, I'd prefer not to retroactively fix release notes for cases like this. Thanks, Jonathan -- To unsubscribe from

[PATCH] git-remote-mediawiki: do not remove installed files in "clean" target

2013-11-08 Thread Jonathan Nieder
Running "make clean" after a successful "make install" should not result in a broken mediawiki remote helper. Reported-by: Thorsten Glaser Signed-off-by: Jonathan Nieder --- Thanks for the very pleasant mediawiki remote helper. contrib/mw-to-git/Makefile | 1 - 1 file changed, 1 deletion(-) d

[PATCH 20/86] http: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- http.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/http.c b/http.c index f3e1439..f2621cd 100644 --- a/http.c +++ b/http.c @@ -422,7 +422,7 @@ void http_init(struct remote *remote, const char *url, int proactive_auth)

[PATCH 23/86] log-tree: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- log-tree.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/log-tree.c b/log-tree.c index 8534d91..25b46a9 100644 --- a/log-tree.c +++ b/log-tree.c @@ -98,7 +98,7 @@ static int add_ref_decoration(const char *refname, const unsigned c

[PATCH 24/86] merge-recursive: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- merge-recursive.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/merge-recursive.c b/merge-recursive.c index dbb7104..9043362 100644 --- a/merge-recursive.c +++ b/merge-recursive.c @@ -2063,13 +2063,13 @@ int parse_merge_opt(struct me

[PATCH 17/86] tag: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- tag.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tag.c b/tag.c index 78d272b..0b00c76 100644 --- a/tag.c +++ b/tag.c @@ -86,7 +86,7 @@ int parse_tag_buffer(struct tag *item, const void *data, unsigned long size)

[PATCH 01/86] strbuf: add has_prefix() to be used instead of prefixcmp()

2013-11-08 Thread Christian Couder
prefixcmp() cannot be really used as a comparison function as it is not antisymmetric: prefixcmp("foo", "foobar") < 0 prefixcmp("foobar", "foo") == 0 So it is not suitable as a function for passing to qsort. And in fact it is used nowhere as a comparison function. So we should re

[PATCH 18/86] sequencer: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- sequencer.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sequencer.c b/sequencer.c index 06e52b4..64cc6f0 100644 --- a/sequencer.c +++ b/sequencer.c @@ -41,7 +41,7 @@ static int is_cherry_picked_from_line(const char *buf, int len

[PATCH 14/86] parse-options: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- parse-options.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/parse-options.c b/parse-options.c index 62e9b1c..4f7fddc 100644 --- a/parse-options.c +++ b/parse-options.c @@ -273,13 +273,13 @@ is_abbreviated:

[PATCH 27/86] setup: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/setup.c b/setup.c index f08dd64..6ed292a 100644 --- a/setup.c +++ b/setup.c @@ -82,7 +82,7 @@ int check_filename(const char *prefix, const char *arg) const char *name;

[PATCH 31/86] send-pack: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- send-pack.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/send-pack.c b/send-pack.c index b228d65..c90049a 100644 --- a/send-pack.c +++ b/send-pack.c @@ -109,7 +109,7 @@ static int receive_status(int in, struct ref *refs) struc

[PATCH 00/86] replace prefixcmp() with has_prefix()

2013-11-08 Thread Christian Couder
Here is a big patch series to replace prefixcmp() with a new has_prefix() function. So the first patch of this series introduces has_prefix() and the last patch removes prefixcmp(). Except in a few cases, I used a script that does basically the following to generate the commits in between: === #

[PATCH 34/86] pkt-line: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- pkt-line.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkt-line.c b/pkt-line.c index 70f1950..d967727 100644 --- a/pkt-line.c +++ b/pkt-line.c @@ -24,8 +24,8 @@ static void packet_trace(const char *buf, unsigned int len, int write)

[PATCH 26/86] refs: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- refs.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/refs.c b/refs.c index 3710748..3dc38df 100644 --- a/refs.c +++ b/refs.c @@ -637,7 +637,7 @@ static int do_one_ref(struct ref_entry *entry, void *cb_data

[PATCH 49/86] builtin/clean: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/clean.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/builtin/clean.c b/builtin/clean.c index 615cd57..98887cd 100644 --- a/builtin/clean.c +++ b/builtin/clean.c @@ -100,7 +100,7 @@ static int parse_clean_color_slot(const char

[PATCH 25/86] notes: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- notes.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/notes.c b/notes.c index b69c0b8..e23d38a 100644 --- a/notes.c +++ b/notes.c @@ -1243,9 +1243,9 @@ static void format_note(struct notes_tree *t, const unsigned char *object_sha1

[PATCH 04/86] remote*: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- remote-curl.c| 14 +++--- remote-testsvn.c | 10 +- remote.c | 46 +++--- 3 files changed, 35 insertions(+), 35 deletions(-) diff --git a/remote-curl.c b/remote-curl.c index b5ebe01..437b863 1

[PATCH 35/86] alias: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- alias.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/alias.c b/alias.c index eb9f08b..0933c87 100644 --- a/alias.c +++ b/alias.c @@ -5,7 +5,7 @@ static char *alias_val; static int alias_lookup_cb(const char *k, const char *v, void *cb)

[PATCH 12/86] upload-pack: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- upload-pack.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/upload-pack.c b/upload-pack.c index a6c54e0..ce4ae75 100644 --- a/upload-pack.c +++ b/upload-pack.c @@ -394,7 +394,7 @@ static int get_common_commits(void)

[PATCH 07/86] revision: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- revision.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/revision.c b/revision.c index 0173e01..968320a 100644 --- a/revision.c +++ b/revision.c @@ -1576,9 +1576,9 @@ static int handle_revision_opt(

[PATCH 43/86] submodule: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- submodule.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/submodule.c b/submodule.c index 1905d75..21fe14e 100644 --- a/submodule.c +++ b/submodule.c @@ -201,7 +201,7 @@ void set_diffopt_flags_from_submodule_config(struct diff_options *di

[PATCH 47/86] builtin/branch: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/branch.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/branch.c b/builtin/branch.c index ad0f86d..c4cb81c 100644 --- a/builtin/branch.c +++ b/builtin/branch.c @@ -81,13 +81,13 @@ static int parse_branch_color_slot(con

[PATCH 44/86] test-string-list: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- test-string-list.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test-string-list.c b/test-string-list.c index 00ce6c9..e04a37c 100644 --- a/test-string-list.c +++ b/test-string-list.c @@ -38,7 +38,7 @@ static void write_list_compact(const

[PATCH 51/86] builtin/column: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/column.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/column.c b/builtin/column.c index e125a55..3664cc1 100644 --- a/builtin/column.c +++ b/builtin/column.c @@ -34,7 +34,7 @@ int cmd_column(int argc, const char **argv, co

[PATCH 50/86] builtin/clone: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/clone.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/clone.c b/builtin/clone.c index 84fb1bd..49b7a08 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -508,7 +508,7 @@ static void write_followtags(const struct r

[PATCH 33/86] notes-utils: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- notes-utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/notes-utils.c b/notes-utils.c index 9107c37..0a4c3de 100644 --- a/notes-utils.c +++ b/notes-utils.c @@ -70,7 +70,7 @@ static combine_notes_fn parse_combine_notes_fn(const char

[PATCH 32/86] http-backend: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- http-backend.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/http-backend.c b/http-backend.c index 8c464bd..d0183b1 100644 --- a/http-backend.c +++ b/http-backend.c @@ -226,7 +226,7 @@ static int http_config(const char *var, const char

[PATCH 55/86] builtin/fetch-pack: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/fetch-pack.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/fetch-pack.c b/builtin/fetch-pack.c index c8e8582..29a3b33 100644 --- a/builtin/fetch-pack.c +++ b/builtin/fetch-pack.c @@ -48,11 +48,11 @@ int cmd_fetch_pack

[PATCH 19/86] commit: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- commit.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/commit.c b/commit.c index de16a3c..35b276d 100644 --- a/commit.c +++ b/commit.c @@ -559,7 +559,7 @@ static void record_author_date(struct author_date_slab *author_date,

[PATCH 52/86] builtin/commit: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/commit.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/builtin/commit.c b/builtin/commit.c index 6ab4605..a8c46c6 100644 --- a/builtin/commit.c +++ b/builtin/commit.c @@ -733,7 +733,7 @@ static int prepare_to_commit(cons

[PATCH 02/86] diff: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- diff.c | 56 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/diff.c b/diff.c index a04a34d..b42523a 100644 --- a/diff.c +++ b/diff.c @@ -235,7 +235,7 @@ int git_diff_basic_config(const cha

[PATCH 40/86] environment: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- environment.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/environment.c b/environment.c index 378254c..827c6e8 100644 --- a/environment.c +++ b/environment.c @@ -176,7 +176,7 @@ const char *get_git_namespace(void) const char *strip_na

[PATCH 29/86] branch: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- branch.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/branch.c b/branch.c index 9e6c68e..fc3a61d 100644 --- a/branch.c +++ b/branch.c @@ -50,7 +50,7 @@ static int should_setup_rebase(const char *origin) void install_branch_config(int

[PATCH 59/86] builtin/fsck: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/fsck.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/fsck.c b/builtin/fsck.c index 97ce678..a80e51f 100644 --- a/builtin/fsck.c +++ b/builtin/fsck.c @@ -442,7 +442,7 @@ static void fsck_dir(int i, char *path)

[PATCH 10/86] sha1_name: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- sha1_name.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/sha1_name.c b/sha1_name.c index 0e5fe7f..3224a39 100644 --- a/sha1_name.c +++ b/sha1_name.c @@ -547,7 +547,7 @@ static int get_sha1_basic(const char *str, int len,

[PATCH 11/86] wt-status: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- wt-status.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/wt-status.c b/wt-status.c index b4e44ba..47806d1 100644 --- a/wt-status.c +++ b/wt-status.c @@ -803,7 +803,7 @@ static void wt_status_print_tracking(struct wt_statu

[PATCH 28/86] bisect: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- bisect.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bisect.c b/bisect.c index 1e46a4f..4819091 100644 --- a/bisect.c +++ b/bisect.c @@ -406,9 +406,9 @@ static int register_ref(const char *refname, const unsigned char *sha1,

[PATCH 48/86] builtin/checkout: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/checkout.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/builtin/checkout.c b/builtin/checkout.c index 0f57397..7ce15dd 100644 --- a/builtin/checkout.c +++ b/builtin/checkout.c @@ -781,7 +781,7 @@ static int switch_branches

[PATCH 39/86] convert: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- convert.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/convert.c b/convert.c index 11a95fc..dc5d3ed 100644 --- a/convert.c +++ b/convert.c @@ -1121,7 +1121,7 @@ static int is_foreign_ident(const char *str) { int i; - if (

[PATCH 45/86] builtin/apply: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/apply.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/builtin/apply.c b/builtin/apply.c index ef32e4f..49028da 100644 --- a/builtin/apply.c +++ b/builtin/apply.c @@ -1409,10 +1409,10 @@ static void recount_diff(const c

[PATCH 13/86] test-line-buffer: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- test-line-buffer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test-line-buffer.c b/test-line-buffer.c index ef1d7ba..ed0d681 100644 --- a/test-line-buffer.c +++ b/test-line-buffer.c @@ -19,7 +19,7 @@ static void handle_command(con

[PATCH 58/86] builtin/for-each-ref: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/for-each-ref.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c index 1d4083c..c22f200 100644 --- a/builtin/for-each-ref.c +++ b/builtin/for-each-ref.c @@ -432,7 +432,7 @@

[PATCH 05/86] daemon: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- daemon.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/daemon.c b/daemon.c index 34916c5..4b23800 100644 --- a/daemon.c +++ b/daemon.c @@ -235,7 +235,7 @@ static int service_enabled; static int

[PATCH 41/86] shell: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- shell.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/shell.c b/shell.c index 66350b2..3bbe774 100644 --- a/shell.c +++ b/shell.c @@ -15,7 +15,7 @@ static int do_generic_cmd(const char *me, char *arg) setup_path(); if (!arg

[PATCH 57/86] builtin/fmt-merge-msg: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/fmt-merge-msg.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/builtin/fmt-merge-msg.c b/builtin/fmt-merge-msg.c index 1c04070..09a55bf 100644 --- a/builtin/fmt-merge-msg.c +++ b/builtin/fmt-merge-msg.c @@ -109,7 +109,7 @

[PATCH 09/86] config: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- config.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/config.c b/config.c index e1d66a1..8d5d70c 100644 --- a/config.c +++ b/config.c @@ -969,25 +969,25 @@ static int git_default_mailmap_config(const char *var, const cha

[PATCH 56/86] builtin/fetch: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/fetch.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/builtin/fetch.c b/builtin/fetch.c index 8eb6cd0..78d4add 100644 --- a/builtin/fetch.c +++ b/builtin/fetch.c @@ -313,7 +313,7 @@ static int update_local_ref(stru

[PATCH 37/86] connect: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- connect.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/connect.c b/connect.c index 4086861..1cb4090 100644 --- a/connect.c +++ b/connect.c @@ -87,7 +87,7 @@ struct ref **get_remote_heads(int in, char *src_buf, size_t src_len,

[PATCH 06/86] pretty: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- pretty.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/pretty.c b/pretty.c index b4e32b7..961241a 100644 --- a/pretty.c +++ b/pretty.c @@ -40,7 +40,7 @@ static int git_pretty_formats_config(const char

[PATCH 36/86] attr: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- attr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/attr.c b/attr.c index 0e774c6..4c11f30 100644 --- a/attr.c +++ b/attr.c @@ -211,7 +211,7 @@ static struct match_attr *parse_attr_line(const char *line, const char *src, name = c

[PATCH 54/86] builtin/fast-export: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/fast-export.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/fast-export.c b/builtin/fast-export.c index 78250ea..8a9136b 100644 --- a/builtin/fast-export.c +++ b/builtin/fast-export.c @@ -476,7 +476,7 @@ static void handle_

[PATCH 53/86] builtin/describe: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/describe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/describe.c b/builtin/describe.c index b9d3603..102d958 100644 --- a/builtin/describe.c +++ b/builtin/describe.c @@ -141,7 +141,7 @@ static void add_to_known_names(con

[PATCH 46/86] builtin/archive: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/archive.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/builtin/archive.c b/builtin/archive.c index 49178f1..b808bb5 100644 --- a/builtin/archive.c +++ b/builtin/archive.c @@ -57,9 +57,9 @@ static int run_remote_archiver(int ar

[PATCH 21/86] imap-send: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- imap-send.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/imap-send.c b/imap-send.c index 6f5cc4f..670de9f 100644 --- a/imap-send.c +++ b/imap-send.c @@ -1263,7 +1263,7 @@ static int count_messages(struct strbuf *all_msgs)

[PATCH 08/86] transport*: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- transport-helper.c | 16 transport.c| 28 ++-- 2 files changed, 22 insertions(+), 22 deletions(-) diff --git a/transport-helper.c b/transport-helper.c index b32e2d6..bf8d7a0 100644 --- a/transport-helper.c +++ b

[PATCH 38/86] pager: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- pager.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pager.c b/pager.c index fa19765..cd7bc43 100644 --- a/pager.c +++ b/pager.c @@ -151,7 +151,7 @@ int decimal_width(int number) static int pager_command_config(const char *var, const cha

[PATCH 42/86] pathspec: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- pathspec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pathspec.c b/pathspec.c index ad1a9f5..182b162 100644 --- a/pathspec.c +++ b/pathspec.c @@ -150,7 +150,7 @@ static unsigned prefix_pathspec(struct pathspec_item *item,

[PATCH 15/86] fetch-pack: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- fetch-pack.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/fetch-pack.c b/fetch-pack.c index a0e0350..1526ed1 100644 --- a/fetch-pack.c +++ b/fetch-pack.c @@ -176,9 +176,9 @@ static void consume_shallow_list(struct fetch_pack_

[PATCH 03/86] fast-import: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- fast-import.c | 80 +-- 1 file changed, 40 insertions(+), 40 deletions(-) diff --git a/fast-import.c b/fast-import.c index f4d9969..14f69a1 100644 --- a/fast-import.c +++ b/fast-import.c @@ -1877,8 +1877,

[PATCH 68/86] builtin/name-rev: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/name-rev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/name-rev.c b/builtin/name-rev.c index 20fcf8c..2b74220 100644 --- a/builtin/name-rev.c +++ b/builtin/name-rev.c @@ -101,9 +101,9 @@ static const char *name_ref_

[PATCH 62/86] builtin/init-db: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/init-db.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/init-db.c b/builtin/init-db.c index 78aa387..7fe9d8c 100644 --- a/builtin/init-db.c +++ b/builtin/init-db.c @@ -266,7 +266,7 @@ static int create_default_files(const c

[PATCH 75/86] builtin/rev-parse: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/rev-parse.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/builtin/rev-parse.c b/builtin/rev-parse.c index c76b89d..b6c92e0 100644 --- a/builtin/rev-parse.c +++ b/builtin/rev-parse.c @@ -508,7 +508,7 @@ in

[PATCH 80/86] builtin/symbolic-ref: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/symbolic-ref.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/symbolic-ref.c b/builtin/symbolic-ref.c index f481959..868fab4 100644 --- a/builtin/symbolic-ref.c +++ b/builtin/symbolic-ref.c @@ -65,7 +65,7 @@ int cmd_symbolic

[PATCH 82/86] builtin/tar-tree: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/tar-tree.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/tar-tree.c b/builtin/tar-tree.c index ba3ffe6..7f225cc 100644 --- a/builtin/tar-tree.c +++ b/builtin/tar-tree.c @@ -34,7 +34,7 @@ int cmd_tar_tree(int argc, const cha

[PATCH 85/86] builtin/upload-archive: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/upload-archive.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/upload-archive.c b/builtin/upload-archive.c index af2da35..298bc76 100644 --- a/builtin/upload-archive.c +++ b/builtin/upload-archive.c @@ -37,7 +37,7 @@ int cm

[PATCH 84/86] builtin/update-ref: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/update-ref.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/builtin/update-ref.c b/builtin/update-ref.c index 702e90d..d81d959 100644 --- a/builtin/update-ref.c +++ b/builtin/update-ref.c @@ -229,15 +229,15 @@ static void

[PATCH 86/86] strbuf: remove prefixcmp() as it has been replaced with has_prefix()

2013-11-08 Thread Christian Couder
prefixcmp() is now useless as the previous commit replaced it everywhere with has_prefix(). So let's now remove it. Signed-off-by: Christian Couder --- git-compat-util.h | 1 - strbuf.c | 9 - 2 files changed, 10 deletions(-) diff --git a/git-compat-util.h b/git-compat-util.h i

[PATCH 77/86] builtin/shortlog: replace prefixcmd() with has_prefix()

2013-11-08 Thread Christian Couder
Signed-off-by: Christian Couder --- builtin/shortlog.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/shortlog.c b/builtin/shortlog.c index c226f76..3f35682 100644 --- a/builtin/shortlog.c +++ b/builtin/shortlog.c @@ -65,7 +65,7 @@ static void insert_one_record(

  1   2   >