Re: [Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-05-05 Thread Kevin Wolf
Am 01.05.2014 um 10:56 hat Matthew Booth geschrieben: > On 30/04/14 16:16, Kevin Wolf wrote: > > Am 30.04.2014 um 16:20 hat Matthew Booth geschrieben: > >> curl_parse_filename wasn't removing the option string from the url, > >> resulting in a 404. > >> > >> This change is a essentially a rewrite o

Re: [Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-05-01 Thread Eric Blake
On 05/01/2014 02:56 AM, Matthew Booth wrote: > On 30/04/14 16:16, Kevin Wolf wrote: >> Am 30.04.2014 um 16:20 hat Matthew Booth geschrieben: >>> curl_parse_filename wasn't removing the option string from the url, >>> resulting in a 404. >>> > > Alternatively I could completely change the syntax.

Re: [Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-05-01 Thread Matthew Booth
On 30/04/14 16:16, Kevin Wolf wrote: > Am 30.04.2014 um 16:20 hat Matthew Booth geschrieben: >> curl_parse_filename wasn't removing the option string from the url, >> resulting in a 404. >> >> This change is a essentially a rewrite of that function as I also need >> to extend it to handle more opti

Re: [Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-04-30 Thread Kevin Wolf
Am 30.04.2014 um 16:20 hat Matthew Booth geschrieben: > curl_parse_filename wasn't removing the option string from the url, > resulting in a 404. > > This change is a essentially a rewrite of that function as I also need > to extend it to handle more options. The rewrite is also much easier > to r

[Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-04-30 Thread Matthew Booth
curl_parse_filename wasn't removing the option string from the url, resulting in a 404. This change is a essentially a rewrite of that function as I also need to extend it to handle more options. The rewrite is also much easier to read. Signed-off-by: Matthew Booth --- block/curl.c | 81 +++

[Qemu-devel] [PATCH 1/3] curl: Fix parsing of readahead option from filename

2014-04-30 Thread Matthew Booth
curl_parse_filename wasn't removing the option string from the url, resulting in a 404. This change is a essentially a rewrite of that function as I also need to extend it to handle more options. The rewrite is also much easier to read. Signed-off-by: Matthew Booth --- block/curl.c | 59 +++