On 8/27/21 5:04 AM, William F Pokorny wrote:
> On 8/26/21 3:19 PM, William F Pokorny wrote:
>> The code there has a hard LL literal assuming long long int is 64 bits.
> 
> Blast it, I meant to write:
> 
> "The code there has a hard LL literal assuming POV_LONG is 64 bits."
> 
> With C++11 'long long int' and 'unsigned long long int' are always at 
> least 64 bits.
> 

In working on documentation for my povr branch changes I noticed the 
issue here is essentially github issue #384.

"vfesession.cpp:342 overflow in conversion from long long int to
long int on m68k #384"

https://github.com/POV-Ray/povray/issues/384

I've for a month or so in those handful of configuration files simply 
had the lines:

// C++11 required. We can set following macros directly.
#define POV_LONG  std::int_least64_t
#define POV_ULONG std::uint_least64_t

over the conditional logic which was there.

This looks to be what clipka recommended in responding to #384.

Bill P.

Post a reply to this message