>>>>> "Michael" == Michael Paquier <michael@paquier.xyz> writes:
> On Sat, Mar 03, 2018 at 07:55:41PM +0000, Andrew Gierth wrote:
>> sequence_options thinks it can put all the sequence parameters
>> through makeInteger and have them come out intact.
Michael> Yeah, on 32b machines "long" is 4 bytes...
And on 64-bit Windows, too.
Michael> Perhaps it would be the occasion to introduce a T_Integer64
Michael> type for Value which gets stored as a string? And as far as I
Michael> can see defGetInt64 is only used by sequences.
The slightly misnamed T_Float is what's currently used for Value nodes
which contain numeric values as strings. So there'd be no point in a new
type tag if you're still going to store the value as a string.
--
Andrew (irc:RhodiumToad)