cas value not being converted to perl correctly on 32bit system

[JRaspass]

Migrated from rt.cpan.org #65381 (status was 'open')

Requestors:

• JGOLDBERG@cpan.org

Attachments:

• Fast.xs.patch

From jgoldberg@cpan.org on 2011-02-02 20:05:27
:

When retrieving the cas value on a 32 bit machine, the value is being incorrectly converted to a
perl uv. Attached is a patch to convert it to a perl string after a gets() and then back to a long
long when calling cas(). Thank you for your work on this module!

[JRaspass]

From kroki@cpan.org on 2011-02-02 20:53:20
:

On Wed Feb 02 15:05:27 2011, JGOLDBERG wrote:

Attached is a patch to convert it to a perl string after a
gets() and then back to a long long when calling cas().

Thanks for the report, looks valid. Patch is also good (would be
perfect if test case is also included). However, since the original
conversion to SvUV is wrong, this is kinda strange now that we get CAS
value over text protocol, then convert it to an integer, then again to
string. Shouldn't the fix be just to pass CAS as a string to Perl,
without intermediate ull representation? This would also have an
advantage that CAS would become completely opaque. I really appreciate
if you work on such version of the fix :).