On Thu, Nov 7, 2013 at 12:57 AM, Junio C Hamano gits...@pobox.com wrote:
* git replace helper no longer allows an object to be replaced with
another object of a different type to avoid confusion (you can
still manually craft such replacement using git update-ref, as an
escape
Jonathan Nieder jrnie...@gmail.com 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 Git 2.0 the new
On Thu, Nov 7, 2013 at 1:12 AM, Junio C Hamano gits...@pobox.com wrote:
Jonathan Nieder jrnie...@gmail.com writes:
The old name followed the pattern anything-cmp(), which suggests
a general comparison function suitable for e.g. sorting objects.
But this was not the case for suffixcmp().
Also here:
https://github.com/kblees/git/commits/kb/hashmap-v4
Sorry for the delay, but the promised static use-after-free analysis [1] turned
out to be much more involved than I had anticipated.
The Good News is that the static analysis independently found the four
use-after-free issues that
Commit 5fee995244e introduced the stage_updated_gitmodules() function to
add submodule configuration updates to the index. It assumed that even
after calling remove_cache_entry_at() the same cache entry would still be
valid. This was true in the old days, as cache entries could never be
freed, but
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
builtin/describe.c | 53 -
1 file changed, 24 insertions(+), 29 deletions(-)
diff --git a/builtin/describe.c b/builtin/describe.c
index
The existing hashtable implementation (in hash.[ch]) uses open addressing
(i.e. resolve hash collisions by distributing entries across the table).
Thus, removal is difficult to implement with less than O(n) complexity.
Resolving collisions of entries with identical hashes (e.g. via chaining)
is
No actual code changes, just move hash_filespec up and outdent part of
find_identical_files.
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
diffcore-rename.c | 98 +++
1 file changed, 49
The find_exact_renames function currently only uses the hash table for
grouping, i.e.:
1. add sources
2. add destinations
3. iterate all buckets, per bucket:
4. split sources from destinations
5. iterate destinations, per destination:
6. iterate sources to find best match
This can be simplified
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
diffcore-rename.c | 48 +---
1 file changed, 13 insertions(+), 35 deletions(-)
diff --git a/diffcore-rename.c b/diffcore-rename.c
index cfeb408..d996c6a
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
cache.h | 3 ++-
name-hash.c | 77 +++--
2 files changed, 20 insertions(+), 60 deletions(-)
diff --git a/cache.h b/cache.h
index
The new hashmap implementation supports remove, so remove and free
directory entries that are no longer referenced by active cache entries.
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
name-hash.c | 15 ---
1 file changed, 8
Note: the ce-next = NULL; in unpack-trees.c::do_add_entry can safely be
removed, as ce-next (now ce-ent.next) is always properly initialized in
name-hash.c::hash_index_entry.
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
cache.h| 8
The new hashmap implementation supports remove, so really remove unused
cache entries from the name hashmap instead of just marking them.
The CE_UNHASHED flag and CE_STATE_MASK are no longer needed.
Keep the CE_HASHED flag to prevent adding entries twice.
Signed-off-by: Karsten Blees
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
Documentation/technical/api-hash.txt | 52 -
Makefile | 2 -
cache.h | 1 -
hash.c | 110
Dear user,
Your mailbox has Exceeded the quota limit set by the administrator, you will
not be able to send or receive mail
until you revalidates your account.
Please click the link below or copy paste to your browser to validate your
mailbox.
http://tinylink.net/quotalimit
Failure to do
When cache_entry structs are removed from index_state.cache, they are not
properly freed. Freeing those entries wasn't possible before because we
couldn't remove them from index_state.name_hash.
Now that we _do_ remove the entries from name_hash, we can also free them.
Add 'free(cache_entry)' to
do_reupdate calls update_one with a cache_entry.name, there's no need for
the extra sanitation / normalization that happens in prefix_path.
cmd_update_index calls update_one with an already prefixed path, no need to
prefix_path twice.
Remove the extra prefix_path from update_one. Also remove the
On Wed, Nov 6, 2013 at 9:42 PM, Junio C Hamano gits...@pobox.com wrote:
Christian Couder chrisc...@tuxfamily.org writes:
From: Junio C Hamano gits...@pobox.com
But that is insufficient to emulate what we do, no? I.e. append
unless the last one is from the same person we are about to add.
On 13-11-06 07:01 PM, Junio C Hamano wrote:
There is a proposed rewording of advice message from git push
patch, which is tentatively queued near the tip of 'pu' for now; it
would be nice to get a few more sets of eyeballs. I am not sure if
we should merge it before the 1.8.5 final, yet (we
Also here:
https://github.com/kblees/git/commits/kb/gitignore-doc
These two patches were the result of a discussion mid-october [1] that I almost
forgot about...now including an example as suggested by Jeff.
[1] http://thread.gmane.org/gmane.comp.version-control.git/235753/focus=235856
Karsten
** means bold in ASCIIDOC, so we need to escape it.
Signed-off-by: Karsten Blees bl...@dcon.de
---
Documentation/gitignore.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/gitignore.txt b/Documentation/gitignore.txt
index 54e334e..f971960 100644
---
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we don't need to repeat this.
Signed-off-by: Karsten Blees bl...@dcon.de
---
Documentation/gitignore.txt | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git
When a remote server is unavailable or very slow, some git commands can
stall out indefinitely. It would be a very good thing if remote
commands -- but especially git fetch -- could be given a timeout.
-hpa
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a
Ramkumar Ramachandra artag...@gmail.com writes:
Junio C Hamano wrote:
... users of for-each-ref format will be _more_ familiar with
formats used by for-each-ref, and it would make a lot more sense to
keep the syntactic resemblance between existing features to show
magic things in
Matthieu Moy matthieu@grenoble-inp.fr writes:
Jonathan Nieder jrnie...@gmail.com 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
Hello,
I’m developping a git command in shell and I would like colorize the output.
I don’t find anything in git-sh-setup.
What is the best way for don’t reinvent the wheel?
Thanks in advance.
Best regards.
Nicolas.
signature.asc
Description: OpenPGP digital signature
On Thu, Nov 07, 2013 at 07:10:11PM +0100, Nicolas wrote:
I’m developping a git command in shell and I would like colorize the output.
I don’t find anything in git-sh-setup.
What is the best way for don’t reinvent the wheel?
I normally use git config --get-color ... either with standard
Karsten Blees karsten.bl...@gmail.com writes:
** means bold in ASCIIDOC, so we need to escape it.
Signed-off-by: Karsten Blees bl...@dcon.de
---
Documentation/gitignore.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/gitignore.txt
Junio C Hamano gits...@pobox.com writes:
Actually, to me, I found the two sentences the worst part in the
original. It made it sound as if the default will be switching to
'upstream', and all readers need to read the second sentence that
clarifies that it is not the case, in a somewhat
Karsten Blees karsten.bl...@gmail.com writes:
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we don't need to repeat this.
Very good, thanks.
Even though I have a suspicion that somebody else may be able to
come up with a better phrase that
Junio C Hamano gits...@pobox.com writes:
Karsten Blees karsten.bl...@gmail.com writes:
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we don't need to repeat this.
Very good, thanks.
Even though I have a suspicion that somebody else may be
On Thu, Nov 07, 2013 at 11:37:38AM -0800, Junio C Hamano wrote:
Junio C Hamano gits...@pobox.com writes:
Karsten Blees karsten.bl...@gmail.com writes:
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we don't need to repeat this.
Very
Am 07.11.2013 20:55, schrieb Jeff King:
On Thu, Nov 07, 2013 at 11:37:38AM -0800, Junio C Hamano wrote:
Junio C Hamano gits...@pobox.com writes:
Karsten Blees karsten.bl...@gmail.com writes:
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we
On Thu, Nov 07, 2013 at 09:27:16PM +0100, Karsten Blees wrote:
How about:
It is not possible to re-include a file if a parent directory of that
file is excluded. Once git considers a directory excluded, it does not
descend into the directory to consider its contents further.
Additionally, precedence of negated patterns is exactly as outlined in
the DESCRIPTION section, we don't need to repeat this.
Signed-off-by: Karsten Blees bl...@dcon.de
---
I also force-pushed to git://github.com/kblees/git.git kb/gitignore-doc.
Documentation/gitignore.txt | 19
Karsten Blees karsten.bl...@gmail.com 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` is the hashmap structure.
++
Karsten Blees karsten.bl...@gmail.com 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
(there is another instance of the same)
Karsten Blees karsten.bl...@gmail.com writes:
diff --git a/builtin/update-index.c b/builtin/update-index.c
index b654d27..acd992d 100644
--- a/builtin/update-index.c
+++ b/builtin/update-index.c
@@ -559,6 +559,7 @@ static int do_reupdate(int ac, const char **av,
const struct
100% обеспечите королеве удовлетворение http://goo.gl/n5lfgw
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Signed-off-by: Ramsay Jones ram...@ramsay1.demon.co.uk
---
compat/bswap.h | 97 --
1 file changed, 68 insertions(+), 29 deletions(-)
diff --git a/compat/bswap.h b/compat/bswap.h
index ea1a9ed..c18a78e 100644
--- a/compat/bswap.h
+++
Hi Jeff,
These patches fix various errors/warnings on the cygwin, MinGW and
msvc builds, provoked by the jk/pack-bitmap branch.
Note that this does not fix all problems on the msvc build; I have
a solution, but I don't like it. :-D So, I'm going to try a different
fix. I had hoped to have done
Noticed by sparse. (Using plain integer as NULL pointer)
Signed-off-by: Ramsay Jones ram...@ramsay1.demon.co.uk
---
khash.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/khash.h b/khash.h
index 0fdf39d..c4c1613 100644
--- a/khash.h
+++ b/khash.h
@@ -114,7 +114,7 @@ static
On MinGW, gcc complains as follows:
CC ewah/ewah_bitmap.o
ewah/ewah_bitmap.c: In function 'ewah_dump':
ewah/ewah_bitmap.c:389: warning: unknown conversion type \
character 'z' in format
ewah/ewah_bitmap.c:389: warning: unknown conversion type \
character 'z'
Signed-off-by: Ramsay Jones ram...@ramsay1.demon.co.uk
---
Makefile | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 07b0626..1950858 100644
--- a/Makefile
+++ b/Makefile
@@ -2484,8 +2484,9 @@ profile-clean:
$(RM) $(addsuffix
Noticed by sparse. (symbol '...' was not declared. Should it be
static?)
Signed-off-by: Ramsay Jones ram...@ramsay1.demon.co.uk
---
builtin/pack-objects.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c
index
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
finalize the next re-roll. I'll add these in.
Hi,
it looks like there is a From: Jens ... line missing on top of this patch.
Am 07.11.2013 15:33, schrieb Karsten Blees:
Commit 5fee995244e introduced the stage_updated_gitmodules() function to
add submodule configuration updates to the index. It assumed that even
after calling
Thanks; will queue.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Ramsay Jones ram...@ramsay1.demon.co.uk writes:
On MinGW, gcc complains as follows:
CC ewah/ewah_bitmap.o
ewah/ewah_bitmap.c: In function 'ewah_dump':
ewah/ewah_bitmap.c:389: warning: unknown conversion type \
character 'z' in format
ewah/ewah_bitmap.c:389:
When git checkout wants to create a path, e.g. a/b/c/d/e, after
seeing if the entire thing already exists (in which case we check if
that is up-to-date and do not bother to check it out, or we unlink
and recreate it), we validate that the leading directory path is
without funny symlinks by seeing
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_t)0xff00ULL) 40) |
+ ((val
Thank you Ramsay, all the patches look OK to me.
On Thu, Nov 7, 2013 at 11:19 PM, Jeff King p...@peff.net 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.
Обратить свою биографию помногообразнее, воспылать замечательнейшим делом.
Расширить круг общения. Применяются засекреченные экспресс-методы. Убыстренное
обучение. Экспресс-курс для делающих первые щаги а так же углублённых.
Приподнять личную цену на рынке работы. Вы сможете говорить по
54 matches
Mail list logo