Re: [PATCH v2 4/5] sha1_name: Parse less while finding common prefix

2017-10-02 Thread Derrick Stolee
My v3 patch is incoming, but I wanted to respond directly to this message. On 9/25/2017 7:42 PM, Stefan Beller wrote: On Mon, Sep 25, 2017 at 2:54 AM, Derrick Stolee wrote: Create get_hex_char_from_oid() to parse oids one hex character at a time. This prevents

Re: [PATCH v2 4/5] sha1_name: Parse less while finding common prefix

2017-09-25 Thread Stefan Beller
On Mon, Sep 25, 2017 at 2:54 AM, Derrick Stolee wrote: > Create get_hex_char_from_oid() to parse oids one hex character at a > time. This prevents unnecessary copying of hex characters in > extend_abbrev_len() when finding the length of a common prefix. > > p0008.1:

[PATCH v2 4/5] sha1_name: Parse less while finding common prefix

2017-09-25 Thread Derrick Stolee
Create get_hex_char_from_oid() to parse oids one hex character at a time. This prevents unnecessary copying of hex characters in extend_abbrev_len() when finding the length of a common prefix. p0008.1: find_unique_abbrev() for existing objects --