>>>>> Glenn Morris wrote:
> Katsumi Yamaoka wrote:

>> Today I tried synch'ing the Gnus trunk with the Emacs trunk and
>> found a problem concerning `float-time'.

> Sorry, I did my best to think about these issues.

>> In time-date.el, `define-obsolete-function-alias' is not available
>> in Emacs 21 that Gnus still supports.  It causes an error when
>> loading gnus-load.el.

> I'm not suggesting this as a solution to this issue, but does the very
> latest Gnus really need to run on 3 major versions of Emacs? When are
> you going to drop support for Emacs 21?

Basically I'm positive to drop old Emacsen.  It seems enough to
support the latest major version and one predecessor.  Those who
use Emacs 21 may perhaps not want to work on upgrading Gnus.
But in another thought, we don't have so much difficulty in
supporting Emacs 21 for the present.  Therefore I think it's not
time to drop Emacs 21.

>> In ecomplete.el, gnus-util.el and time-date.el, (featurep 'xemacs)
>> is used for checking if `float-time' is available like this:
>>
>>   (if (featurep 'xemacs)
>>       (time-to-seconds time)
>>     (float-time time))
>>
>> However, XEmacs may implement `float-time' in the future.

> I used featurep in preference to fboundp to avoid compilation
> warnings, and so that the compiler could optimise the test away.

Yes, I knew it.  (featurep 'xemacs) is useful in Emacs 22, 23
and XEmacs 21.5, so I hesitated to replace it with `fboundp'.
But it looks easy to provide `float-time' in XEmacs.

>> Therefore I made changes in those files as attached below.  WDYT?

> Fine. I would then stick a with-no-warnings around every use of
> time-to-seconds, and an eval-and-compile around the revised definition
> of gnus-float-time.

Unfortunately `with-no-warnings' is not available in Emacs 21
and XEmacs.  How about using macros like the following?  (For
warnings that old Emacsen and XEmacs issue, we can use lpath.el
and dgnushack.el.)