From: "Richard M. Stallman" <rms@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: tiny changes to url-cookie.el
Date: Tue, 28 Jun 2005 00:17:17 -0400 [thread overview]
Message-ID: <E1Dn7XB-0000jA-4a@fencepost.gnu.org> (raw)
In-Reply-To: <m37jgfrggr.fsf@P131831.SAP.Corp> (message from Klaus Straubinger on Mon, 27 Jun 2005 17:01:56 +0200 (CEST))
In short, the PATH argument is meant to be the local part of the
internet site because sites maybe want to set and retrieve different
cookies for different parts of their site.
In that case, we shouldn't call it a "path"; GNU convention is to use
that term only for lists of directories to search. So I changed those
argument names just now. I also added a doc string which I could
figure out partly. But many functions in that file don't have doc
strings, and need them. Or the doc strings are very sketchy and not
clear.
Therefore, I think the two lines above should be replaced by
(real-fname (url-filename (if proxy-obj proxy-obj url)))
and no change to url-cookie-retrieve should be necessary. Sorry for the
confusion.
Thanks. I will install that change, hoping you're right.
(It is done more cleanly using `or' instead of `if'.)
next prev parent reply other threads:[~2005-06-28 4:17 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-06-22 15:31 tiny changes to url-cookie.el Klaus Straubinger
2005-06-23 16:54 ` Richard M. Stallman
2005-06-27 15:01 ` Klaus Straubinger
2005-06-28 4:17 ` Richard M. Stallman [this message]
[not found] ` <m3d5q7rhgz.fsf@P131831.SAP.Corp>
2005-06-28 7:25 ` Klaus Straubinger
2005-06-29 3:58 ` Richard M. Stallman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=E1Dn7XB-0000jA-4a@fencepost.gnu.org \
--to=rms@gnu.org \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).