GNU bug report logs - #24694
Document url--allow-chars for external use?

Previous Next

Full log

Message #28 received at 24694 <at> debbugs.gnu.org (full text, mbox):

From: Hong Xu <hong <at> topbug.net>
To: Robert Pluim <rpluim <at> gmail.com>, Eli Zaretskii <eliz <at> gnu.org>
Cc: 24694 <at> debbugs.gnu.org
Subject: Re: bug#24694: Document url--allow-chars for external use?
Date: Fri, 11 Oct 2019 19:21:34 -0700

[Message part 1 (text/plain, inline)]

On 10/11/19 7:32 AM, Robert Pluim wrote:
>>>>>> On Fri, 11 Oct 2019 15:38:03 +0300, Eli Zaretskii <eliz <at> gnu.org> said:
> 
>      >> From: Hong Xu <hong <at> topbug.net>
>      >> Cc: 24694 <at> debbugs.gnu.org
>      >> Date: Fri, 4 Oct 2019 12:31:27 -0700
>      >>
>      >> > Make url-hexify-string accept a list of allowed chars.
>      >> >
>      >> > 	* url-util.el (url-hexify-string): Accept a list of allowed chars.
>      >> > 	* url.texi (URI Encoding): Update url-hexify-string doc and index improvements.
>      >> >
>      >>
>      >> It's been a few years... I'm wondering whether this patch is still interesting?
> 
>      Eli> I think it is, yes.  Can you send a patch relative to the current
>      Eli> master branch?
> 
> Api nit: if the first argument is going to be a list or a vector, you
> could just keep one argument and check 'vectorp'.
> 

Thanks, I've updated the patch as attached.

[0001-Make-url-hexify-string-accept-a-list-of-allowed-char.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.