curl-users
Re: Patch to use filename from Content-disposition header
Date: Mon, 11 Jan 2010 23:14:33 +0100
On Monday 11 of January 2010 22:03:10 Björn Stenberg wrote:
> Kamil Dudka wrote:
> > np, where can I get current version of the patch actually?
>
> Oops, here it is.
Looks good to me. Here is my follow-up making some aspects more visible.
I hope it does not cause another bug as the one Daniel mentioned. I've just
mistakenly lost the (0== ...) from that expression somehow. Anyway it's bad
idea to have the same expression at three places. Poking around I discovered
the desired comment bellow, so that I only moved it up.
The strrchr() stuff also seems to work well, I overlooked the 'r' in the
middle :-)
Kamil
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-users
FAQ: http://curl.haxx.se/docs/faq.html
Etiquette: http://curl.haxx.se/mail/etiquette.html
- text/x-diff attachment: content-disposition-v3-inc.patch