Bastien writes: > The real problem is (random t) in previous versions of org-id.el.
Yes. > Do you mean this should be (random) instead of (random t) in those > versions? Why? If so, we can simply get rid of the compatibility > macro and use (random). That would be the correct thing to do. >> No package should ever use `(random t)´ at all and especially not >> repeatedly. > > I thought (random t) was okay especially because related functions > (e.g. org-id-new) are *never* repeated at regular intervals. That unfortunate misconception is built into many Emacs packages, hence the effort to finally make this operation a no-op and move the seeding of the PRNG to the Emacs init code. > Besides, (random t) is documented in Emacs <24.3 and XEmacs. Sure, but you still aren't supposed to re-seed a PRNG each time you want a random number from it. Especially not with the seeding method that Emacs used to use. Each Emacs session should call `(random t)´ exactly once, in the user init file. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Factory and User Sound Singles for Waldorf rackAttack: http://Synth.Stromeko.net/Downloads.html#WaldorfSounds