* Re: master adcf904: Add bignum support in Tramp
[not found] ` <20180828142711.B6D2E209A6@vcs0.savannah.gnu.org>
@ 2018-08-28 14:40 ` Robert Pluim
2018-08-28 15:07 ` Paul Eggert
0 siblings, 1 reply; 5+ messages in thread
From: Robert Pluim @ 2018-08-28 14:40 UTC (permalink / raw)
To: emacs-devel; +Cc: Michael Albinus
Michael.Albinus@gmx.de (Michael Albinus) writes:
> - ;; Convert inode.
> - (unless (listp (nth 10 attr))
> + ;; Convert inode. Big numbers have been added to Emacs 27.
> + (unless (or (fboundp 'bignump) (listp (nth 10 attr)))
Given that there are people who want to remove #'bignump, should we
(provide "bignum") somewhere to give people a canonical method for
checking for bignum support??
Robert
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: master adcf904: Add bignum support in Tramp
2018-08-28 14:40 ` master adcf904: Add bignum support in Tramp Robert Pluim
@ 2018-08-28 15:07 ` Paul Eggert
2018-08-28 15:19 ` Robert Pluim
0 siblings, 1 reply; 5+ messages in thread
From: Paul Eggert @ 2018-08-28 15:07 UTC (permalink / raw)
To: Robert Pluim, emacs-devel; +Cc: Michael Albinus
Robert Pluim wrote:
> Given that there are people who want to remove #'bignump, should we
> (provide "bignum") somewhere to give people a canonical method for
> checking for bignum support??
No need; just do (natnump (1+ most-positive-fixnum)).
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: master adcf904: Add bignum support in Tramp
2018-08-28 15:07 ` Paul Eggert
@ 2018-08-28 15:19 ` Robert Pluim
2018-08-28 15:24 ` Paul Eggert
0 siblings, 1 reply; 5+ messages in thread
From: Robert Pluim @ 2018-08-28 15:19 UTC (permalink / raw)
To: Paul Eggert; +Cc: Michael Albinus, emacs-devel
Paul Eggert <eggert@cs.ucla.edu> writes:
> Robert Pluim wrote:
>> Given that there are people who want to remove #'bignump, should we
>> (provide "bignum") somewhere to give people a canonical method for
>> checking for bignum support??
>
> No need; just do (natnump (1+ most-positive-fixnum)).
Then how come NEWS talks about bignump and fixnump?
Robert
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: master adcf904: Add bignum support in Tramp
2018-08-28 15:19 ` Robert Pluim
@ 2018-08-28 15:24 ` Paul Eggert
2018-08-28 16:37 ` Robert Pluim
0 siblings, 1 reply; 5+ messages in thread
From: Paul Eggert @ 2018-08-28 15:24 UTC (permalink / raw)
To: Robert Pluim; +Cc: Michael Albinus, emacs-devel
Robert Pluim wrote:
> Then how come NEWS talks about bignump and fixnump?
Because some contributors thought that bignump and fixnump might be useful
convenience functions for Lisp programmers. They're not, but I lost that argument.
If it were up to me, I wouldn't mention these functions in NEWS or document
them, as they cause more trouble and confusion than they cure. Worrying about
the distinction between bignums and fixnums unnecessarily complicates Lisp
programmers' lives, and we shouldn't encourage it.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: master adcf904: Add bignum support in Tramp
2018-08-28 15:24 ` Paul Eggert
@ 2018-08-28 16:37 ` Robert Pluim
0 siblings, 0 replies; 5+ messages in thread
From: Robert Pluim @ 2018-08-28 16:37 UTC (permalink / raw)
To: Paul Eggert; +Cc: Michael Albinus, emacs-devel
Paul Eggert <eggert@cs.ucla.edu> writes:
> Robert Pluim wrote:
>> Then how come NEWS talks about bignump and fixnump?
>
> Because some contributors thought that bignump and fixnump might be
> useful convenience functions for Lisp programmers. They're not, but I
> lost that argument.
They're currently useful to me with the calc bignum conversion, but
won't be needed once thatʼs done, the end result will just use
'integerp'.
> If it were up to me, I wouldn't mention these functions in NEWS or
> document them, as they cause more trouble and confusion than they
> cure. Worrying about the distinction between bignums and fixnums
> unnecessarily complicates Lisp programmers' lives, and we shouldn't
> encourage it.
I agree with that sentiment.
Robert
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-08-28 16:37 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20180828142710.26824.71640@vcs0.savannah.gnu.org>
[not found] ` <20180828142711.B6D2E209A6@vcs0.savannah.gnu.org>
2018-08-28 14:40 ` master adcf904: Add bignum support in Tramp Robert Pluim
2018-08-28 15:07 ` Paul Eggert
2018-08-28 15:19 ` Robert Pluim
2018-08-28 15:24 ` Paul Eggert
2018-08-28 16:37 ` Robert Pluim
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).