typo of copy-seq?

typo of copy-seq?

[It's me FKtPp ;)]

hi all,

there's a typo in the broswe-url.el of today's CVS HEAD.

patch following:

--8<---------------cut here---------------start------------->8---
*** browse-url.el	10  9 2007 12:29:36 +0800	1.60
--- browse-url.el	10  9 2007 16:54:25 +0800	
***************
*** 621,627 ****
  
  (defun browse-url-encode-url (url)
    "Encode all `confusing' characters in URL."
!   (let ((encoded-url (copy-seq url)))
      (while (string-match "%" encoded-url)
        (setq encoded-url (replace-match "%25" t t encoded-url)))
      (while (string-match "[*\"()',=;? ]" encoded-url)
--- 621,627 ----
  
  (defun browse-url-encode-url (url)
    "Encode all `confusing' characters in URL."
!   (let ((encoded-url (copy-sequence url)))
      (while (string-match "%" encoded-url)
        (setq encoded-url (replace-match "%25" t t encoded-url)))
      (while (string-match "[*\"()',=;? ]" encoded-url)
--8<---------------cut here---------------end--------------->8---

Re: typo of copy-seq?

[Johannes Weiner]

[-- Attachment #1.1: Type: text/plain, Size: 326 bytes --]

Hi FKtPp (what a Realname... ;),

On Mon, Sep 10, 2007 at 05:01:53PM +0800, It's me FKtPp ;) wrote:
> there's a typo in the broswe-url.el of today's CVS HEAD.

I used `copy-seq' from cl.el intuitively.  This bug has already been
reported and it is changed to `copy-sequence' in HEAD now.  Please update
your sources.

	Hannes

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel