Re: read-regexp

[Eli Zaretskii]

> From: Juri Linkov <address@hidden>
> Cc: address@hidden
> Date: Sat, 18 Oct 2008 19:19:45 +0300
> 
> The problem is in the ambiguous name of the argument DEFAULT.

No, the problem is also with the "Additional defaults are ..." part,
which makes it sound like DEFAULT is treated like those "additional
defaults".  But in fact, it isn't.

> I thought that the name STRING-DESCRIBING-DEFAULT is too long and clumsy
> but had no better idea.  Unless you want to propose a better name, I will
> rename the argument DEFAULT of `read-regexp' to the same name
> STRING-DESCRIBING-DEFAULT as below with docstring fixes:

DEFAULT-VALUE sounds good enough to me, but please also make sure that
the doc string does not hint it is somehow related to the "additional
defaults".  In fact, I suggest to rephrase the last sentence of the
doc string such that it doesn't use the word "defaults", but instead
explains that those are values you pass to read-from-minibuffer for
making them available via M-n.

Thanks.