Re: [PATCH v2 00/11] sha1_name: improvements

2013-05-08 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > Will queue. Thanks. Nit: you might want to add my s-o-b on patches 73027d and b018e8 queued in pu. -- 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/majordo

Re: [PATCH v2 00/11] sha1_name: improvements

2013-05-07 Thread Junio C Hamano
Felipe Contreras writes: > When merging this series to the @ shortcut one, there will be > conflicts, this is how I propose fixing them: > > return len; /* syntax Ok, not enough switches */ > - if (0 < len && len == namelen) > + if (len > 0 && len == namelen) >

Re: [PATCH v2 00/11] sha1_name: improvements

2013-05-07 Thread Felipe Contreras
On Tue, May 7, 2013 at 4:55 PM, Felipe Contreras wrote: > While trying to add support for the @ shortcut lots of cleanups arised. Here > they are in a single series. > > Felipe Contreras (7): > tests: at-combinations: simplify setup > tests: at-combinations: check ref names directly > tests:

[PATCH v2 00/11] sha1_name: improvements

2013-05-07 Thread Felipe Contreras
Hi, While trying to add support for the @ shortcut lots of cleanups arised. Here they are in a single series. Felipe Contreras (7): tests: at-combinations: simplify setup tests: at-combinations: check ref names directly tests: at-combinations: improve nonsense() sha1_name: remove no-op