[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] don't use regexp in commit.c
[Thread Prev] | [Thread Next]
- Subject: Re: [PATCH] don't use regexp in commit.c
- From: Aris Adamantiadis <aris@xxxxxxxxxxxx>
- Reply-to: libssh@xxxxxxxxxx
- Date: Sat, 12 Feb 2011 18:49:25 +0100
- To: libssh@xxxxxxxxxx
Le 12/02/11 13:15, Bernhard R. Link a écrit : > * Aris Adamantiadis <aris@xxxxxxxxxxxx> [110212 12:32]: >> Thanks for you feedback. I have a few questions about your patch ; You >> enumerate a good list of bugs,but why remove the broken code when you >> can fix it ? > > Well, the functionality can be implemented in 16 lines of code and is > most likely even faster than goin with a regexp. So why use a regexp > with all those problems involved if the functionality can be implemented > natively reducing the total number of lines needed? Hi Bernhard, We discussed it and decided to include your patch. Your code makes sense and I tested it. Please look how Andreas put it on the git. Thanks, Aris
[PATCH] don't use regexp in commit.c | "Bernhard R. Link" <brlink@xxxxxxxxxx> |
Re: [PATCH] don't use regexp in commit.c | Aris Adamantiadis <aris@xxxxxxxxxxxx> |
Re: [PATCH] don't use regexp in commit.c | "Bernhard R. Link" <brlink@xxxxxxxxxx> |