Re: Convert::ASN1 loading times

[John B. <jj...@be...>]

OK - here is the second cut of the Convert::ASN1 save/load. I've reworked
it in the light of Graham's comments. (Including those on recursion :-)

Attached is a gzipped tar with the patch for Convert/ASN1.pm, an
'ldap.asn' file and an 'Net/LDAP/ASN.pm.new' which you need to use the
->load method when you start Net::LDAP. Patch is against version 0.07 of
ASN1.pm (i.e. the current CPAN rather than the current CVS).

The fastest load time for this I've seen is 0.31s, against a fastest load
time of 0.55s for the current code. (This is looking at 'time
-MNet::LDAP::ASN -e exit').

Using '-d:DProf' and looking at the CumulS column: The new ->load function
takes ~0.07s and the old ->prepare function takes ~0.31s. These
measurements seem to agree with the 'time' method and suggest a nice
speedup.

The format of the ldap.asn file should now be more obvious to humans.

The empty tag bug should be fixed.

It tests out OK with my test app here. YMMV.

regards,

jb