shr-copy-url: arg misnamed?

["T.V Raman" <raman@google.com>]

Should the arg in this function be renamed?

The name "url" is confusing, and is likely being shadowed 
(defun shr-copy-url (url)
  "Copy the URL under point to the kill ring.
If IMAGE-URL (the prefix) is non-nil, or there is no link under
point, but there is an image under point then copy the URL of the
image under point instead."
  (interactive (list (shr-url-at-point current-prefix-arg)))
  (if (not url)
      (message "No URL under point")
    (setq url (url-encode-url url))
    (kill-new url)
    (message "Copied %s" url)))
-- 

--