On 1/18/07 5:43 AM, "Yaakov Belch" <[EMAIL PROTECTED]> wrote:
> (Sorry for the late reply ---- I had to finish some other work before
> continuing on this issue.)
>
> I have implemented a bug fix and would appreciate your advice on how to
> include it correctly into the official version.
> The new Inline.pm file can be downloaded here:
> http://yaakovnet.net/Inline-0.45.pm
>
> Here is a discussion of the changes:
>
> 1) I guess I have to increment the version number from 0.44 to 0.45:
>
> $Inline::VERSION = '0.45';
>
> 2) I removed the old checksum computation:
>
> require Digest::MD5;
> $o->{INLINE}{md5} = Digest::MD5::md5_hex($o->{API}{code});
>
> 3) I replaced it with a call to a new computation:
>
> $o->{INLINE}{md5} = $o->md5_checksum;
>
> 4) Here is the new md5_checksum method:
Personally I don't think an MD5 checksum is really necessary, it doesn't
need cryptographic strength. Seems like a checksum like what unpack() can
provide would be fine.
-Ken
