Re: url-encode-url: do not add a trailing slash for "bare" URLs (with no file/directory)

[Bastien]

Hi Stefan,

Stefan Monnier <address@hidden> writes:

>> The following trivial patch changes this.
>
> I'm no expert on URL conventions, but it looks OK to me, tho maybe the /
> is needed when there's a fragment.

Do you mean when there is a #name like in "http://www.gnu.org/#";
or "http://www.gnu.org/index.html#name"; ?

The patch does not the current behavior wrt fragments.

(url-encode-url "http://www.gnu.org/#";)
  => http://www.gnu.org/

in both cases.

With the patch we have

(url-encode-url "http://www.gnu.org#";)
  => http://www.gnu.org

instead of

(url-encode-url "http://www.gnu.org#";)
  => http://www.gnu.org/

(no trailing slash with the patch), but I don't think
http://www.gnu.org# is correct anyway.

Let me know,

-- 
 Bastien