On Mon, Jan 06, 2014 at 07:35:04PM -0800, Brodie Rao wrote:
On Mon, Jan 6, 2014 at 7:32 PM, Brodie Rao bro...@sf.io wrote:
This change ensures get_sha1_basic() doesn't try to resolve full hashes
as refs when ambiguous ref warnings are disabled.
This provides a substantial performance
Brodie Rao bro...@sf.io writes:
This change ensures get_sha1_basic() doesn't try to resolve full hashes
as refs when ambiguous ref warnings are disabled.
This provides a substantial performance improvement when passing many
hashes to a command (like git rev-list --stdin) when
Jeff King p...@peff.net writes:
Alternatively, I guess cat-file
--batch could just turn off warn_ambiguous_refs itself.
Sounds like a sensible way to go, perhaps on top of this change?
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to
On Tue, Jan 07, 2014 at 09:51:07AM -0800, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
Alternatively, I guess cat-file
--batch could just turn off warn_ambiguous_refs itself.
Sounds like a sensible way to go, perhaps on top of this change?
The downside is that we would not
This change ensures get_sha1_basic() doesn't try to resolve full hashes
as refs when ambiguous ref warnings are disabled.
This provides a substantial performance improvement when passing many
hashes to a command (like git rev-list --stdin) when
core.warnambiguousrefs is false. The check incurs 6
Jeff King p...@peff.net writes:
On Tue, Jan 07, 2014 at 09:51:07AM -0800, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
Alternatively, I guess cat-file
--batch could just turn off warn_ambiguous_refs itself.
Sounds like a sensible way to go, perhaps on top of this change?
On Tue, Jan 07, 2014 at 11:38:15AM -0800, Junio C Hamano wrote:
Alternatively, I guess cat-file
--batch could just turn off warn_ambiguous_refs itself.
Sounds like a sensible way to go, perhaps on top of this change?
The downside is that we would not warn about ambiguous refs
Jeff King p...@peff.net writes:
On Tue, Jan 07, 2014 at 11:38:15AM -0800, Junio C Hamano wrote:
Alternatively, I guess cat-file
--batch could just turn off warn_ambiguous_refs itself.
Sounds like a sensible way to go, perhaps on top of this change?
The downside is that we would
On Tue, Jan 07, 2014 at 12:31:57PM -0800, Junio C Hamano wrote:
c. Just leave it at Brodie's patch with nothing else on top.
My thinking in favor of (b) was basically does anybody actually care
about ambiguous refs in this situation anyway?. If they do, then I
think (c) is my
This change ensures get_sha1_basic() doesn't try to resolve full hashes
as refs when ambiguous ref warnings are disabled.
This provides a substantial performance improvement when passing many
hashes to a command (like git rev-list --stdin) when
core.warnambiguousrefs is false. The check incurs 6
On Mon, Jan 6, 2014 at 7:32 PM, Brodie Rao bro...@sf.io wrote:
This change ensures get_sha1_basic() doesn't try to resolve full hashes
as refs when ambiguous ref warnings are disabled.
This provides a substantial performance improvement when passing many
hashes to a command (like git rev-list
11 matches
Mail list logo
