weak place in VariantAsPyObject()

[Alexey-T]

    varInteger:  Result := PyLong_FromLong( DeRefV );
    varInt64:    Result := PyLong_FromLongLong( DeRefV );   

FPC shows error here-- DeRefV is Variant, but it wants NativeInt (ie C_Long on linux x64). I added typecast to LongInt and now its ok.

    varInteger:  Result := PyLong_FromLong( LongInt(DeRefV) );

[Alexey-T]

and next line -- FPC eats it but better to add typecast Int64(). to make sure we don't loose bits of number.

[pyscripter]

This is an fpc issue. DeRefV is a Variant. The compiler is supposed to auto convert it to the appropriate type. PyLong_fromLong expects a c_long. The compiler should convert it to a c_long whatever that is (Integer in Windows and NativeInt on Linux).

Same about PyLong_FromLongLong( DeRefV ) The function expects an Int64. The Variant should be converted to Int64.
You don't normally need to typecast Variants.

What is the exact error message you get?

To work around this fpc bug does this typecast work in fpc?
varInteger: Result := PyLong_FromLong( c_long(DeRefV) );

[Alexey-T]

OK, I agree

https://gitlab.com/freepascal.org/fpc/source/-/issues/39417

[pyscripter]

Don't we need a workaround until this is fixed?

Does this typecast work in fpc?
varInteger: Result := PyLong_FromLong( c_long(DeRefV) );

[Alexey-T]

No, this gives the same error, with C_Long().

[pyscripter]

Then an ($IFDEF} is needed as a workaround.

[Alexey-T]

I don't ask you to add IFDEF there! it's less good code. I ask for LongInt()

[pyscripter]

But this will lose accuracy when the c_long is 64 bits, so it is not strictly speaking correct.

[Alexey-T]

Ok then. you may add IFDEF or not touch it, i will add my fix here