Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-05 Thread Junio C Hamano
Christian Couder writes: > Tell me if I can do something. I think the tip of 'pu' with cc/starts-n-ends-with-endgame is in fairly a good shape. The evil merge there may need to be updated over time, but I think I can manage. -- To unsubscribe from this list: send the line "unsubscribe git" in t

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-04 Thread Christian Couder
On Wed, Dec 4, 2013 at 11:02 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Christian Couder writes: >> >>> Ok, the commit is in the use_starts_ends_with branch on this github repo: >>> >>> https://github.com/chriscool/git.git >> >> I looked at the patches, and they looked alright. The

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-04 Thread Junio C Hamano
Junio C Hamano writes: > Christian Couder writes: > >> Ok, the commit is in the use_starts_ends_with branch on this github repo: >> >> https://github.com/chriscool/git.git > > I looked at the patches, and they looked alright. The endgame needs > to be on a separate topic to be held until a dist

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-04 Thread Junio C Hamano
Christian Couder writes: > Ok, the commit is in the use_starts_ends_with branch on this github repo: > > https://github.com/chriscool/git.git I looked at the patches, and they looked alright. The endgame needs to be on a separate topic to be held until a distant future, though. Will queue. Th

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-04 Thread Jeff King
On Tue, Dec 03, 2013 at 09:24:40PM +0100, Christian Couder wrote: > > I skimmed the patch and didn't notice any problems. It's > > whitespace-damaged by gmail, of course. The simplest thing is probably > > to just point us at a public repo with the commit, if you have one. > > Ok, the commit is i

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-03 Thread Christian Couder
On Tue, Dec 3, 2013 at 1:46 PM, Jeff King wrote: > > This made it through to me, but not the list. There are at least two > things going on: > > 1. The message is 172K, which is over the 100K limit imposed by vger; > the list is silently dropping it. > > 2. In my case, the direct mail was

Re: Fwd: [PATCH 4/5] Replace {pre,suf}fixcmp() with {starts,ends}_with()

2013-12-03 Thread Jeff King
On Tue, Dec 03, 2013 at 08:03:39AM +0100, Christian Couder wrote: > I am trying to forward this patch from my gmail address, as it doesn't > seem to go through when I send it from my home machine. This made it through to me, but not the list. There are at least two things going on: 1. The mess