unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#5184: 23.1; doc string of `count'
@ 2009-12-11 19:36 Drew Adams
  2011-07-13 15:03 ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2009-12-11 19:36 UTC (permalink / raw)
  To: bug-gnu-emacs

Please state what the default :test function is (eql?).

In GNU Emacs 23.1.1 (i386-mingw-nt5.1.2600)
 of 2009-07-29 on SOFT-MJASON
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (4.4)'
 







^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#5184: 23.1; doc string of `count'
  2009-12-11 19:36 bug#5184: 23.1; doc string of `count' Drew Adams
@ 2011-07-13 15:03 ` Lars Magne Ingebrigtsen
  2011-07-13 15:47   ` Drew Adams
  0 siblings, 1 reply; 4+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-13 15:03 UTC (permalink / raw)
  To: Drew Adams; +Cc: 5184

"Drew Adams" <drew.adams@oracle.com> writes:

> Please state what the default :test function is (eql?).

As with all the cl.el functions, where they follow the Common Lisp
standard, they do not specify this.  So I don't think this is a bug.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#5184: 23.1; doc string of `count'
  2011-07-13 15:03 ` Lars Magne Ingebrigtsen
@ 2011-07-13 15:47   ` Drew Adams
  2011-07-16 18:01     ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2011-07-13 15:47 UTC (permalink / raw)
  To: 'Lars Magne Ingebrigtsen'; +Cc: 5184

> > Please state what the default :test function is (eql?).
> 
> As with all the cl.el functions, where they follow the Common Lisp
> standard, they do not specify this.  So I don't think this is a bug.

That just means that there is a bug in the other doc strings as well.
`cl.el' does not reproduce Common Lisp.

We should explicitly say that the default value of :test for `count' is #'eql,
if this cl.el function has the same default :test as in Common Lisp.






^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#5184: 23.1; doc string of `count'
  2011-07-13 15:47   ` Drew Adams
@ 2011-07-16 18:01     ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2011-07-16 18:01 UTC (permalink / raw)
  To: Drew Adams; +Cc: 'Lars Magne Ingebrigtsen', 5184

>> > Please state what the default :test function is (eql?).
>> As with all the cl.el functions, where they follow the Common Lisp
>> standard, they do not specify this.  So I don't think this is a bug.
> That just means that there is a bug in the other doc strings as well.
> `cl.el' does not reproduce Common Lisp.

Well cl.el tried to reproduce CommonLisp behavior and indeed followed
the principle of focusing the doc on the differences w.r.t to CommonLisp.

> We should explicitly say that the default value of :test for `count'
> is #'eql, if this cl.el function has the same default :test as in
> Common Lisp.

I tend to agree that cl.el should be improved by making the docstrings
self-contained rather than referring implicitly to the
CommonLisp documentation.  Patches to improve this are welcome.


        Stefan





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-07-16 18:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-12-11 19:36 bug#5184: 23.1; doc string of `count' Drew Adams
2011-07-13 15:03 ` Lars Magne Ingebrigtsen
2011-07-13 15:47   ` Drew Adams
2011-07-16 18:01     ` Stefan Monnier

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).