emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] commit 7719734dd7 org-compat.el: New compatibility function `org


From: Bastien
Subject: Re: [O] commit 7719734dd7 org-compat.el: New compatibility function `org-random'
Date: Thu, 06 Sep 2012 19:26:03 +0200
User-agent: Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.2.50 (gnu/linux)

Achim Gratz <address@hidden> writes:

> Sorry, but that macro doesn't call `random┬┤ appropriately at all.  

The real problem is (random t) in previous versions of org-id.el.

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

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

Besides, (random t) is documented in Emacs <24.3 and XEmacs.

-- 
 Bastien



reply via email to

[Prev in Thread] Current Thread [Next in Thread]