Skip Menu |

This queue is for tickets about the Clone CPAN distribution.

Report information
The Basics
Id: 50174
Status: resolved
Priority: 0/
Queue: Clone

People
Owner: Nobody in particular
Requestors: salvatore.bonaccorso [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.31
Fixed in: (no value)



Subject: works slower than Storable::dclone on hashes with big number of keys (Documentation update?)
Hi In the Debian Distribution the following Bugreport was reported: http://bugs.debian.org/516409 ---(snip)---- Package: libclone-perl Version: 0.31-1 Severity: normal The documentation says that Clone::clone works faster than Storable::dclone. However, this is not true for hashes with big number of keys. A simple attached benchmark shows the difference by two factor, and on my real-world examples this number increases to three. Please advance the documentation for Clone accordingly. ---(snap)---- the benchmark is also attached to the bugreport at the above URL. Many thanks for consideration! Bests Salvatore
On Fri Oct 02 16:06:57 2009, https://launchpad.net/~carnil wrote: Show quoted text
> Hi > > In the Debian Distribution the following Bugreport was reported: > http://bugs.debian.org/516409 > > ---(snip)---- > Package: libclone-perl > Version: 0.31-1 > Severity: normal > > > The documentation says that Clone::clone works faster than > Storable::dclone. However, this is not true for hashes with big number > of keys. A simple attached benchmark shows the difference by two factor, > and on my real-world examples this number increases to three. > > Please advance the documentation for Clone accordingly. > ---(snap)---- > > the benchmark is also attached to the bugreport at the above URL. > > Many thanks for consideration! > > Bests > Salvatore
Hi Salvatore, thanks for the report! We have updated the mention in 0.32 to note how dclone() is faster for deeper structures, while clone() is faster for shallower ones. The 0.32 update also fixes a long-standing issues with cloning Fieldhashes, so a downstream update is advised. Thanks! breno