Re: [Emacs-diffs] master bd22beb: Add seq-random-elt to seq.el

[Tino Calancha]

On Tue, 25 Oct 2016, Stefan Monnier wrote:

> > > > +Return nil if SEQUENCE is nil."
> > > > +  (if (seq-empty-p sequence)
> > > > +      (error "Sequence cannot be empty")
> > >
> > > Hmm... which is it?
>
> > I'm not sure I understand your question.
>
> The question was: should I obey the docstring or the code?
The code, always the code.

Indeed, pretty consistent, after all this function is about getting a 
_random_ element from a sequence; if the doc string assures something 
opposite as the code, then it's very hard to predict the result of the 
coin flipping experiment, regardless of which is the underline random 
generation engine.