Bug #81156 for Hash-Merge: behavior name bug

Tue Nov 13 23:44:23 2012 matt.d.garrett [...] gmail.com - Ticket created

Subject:	behavior name bug
Date:	Tue, 13 Nov 2012 21:43:52 -0700
To:	bug-Hash-Merge [...] rt.cpan.org
From:	Matt Garrett <matt.d.garrett [...] gmail.com>

Hi, I'm using Hash::Merge 0.12. I can specify a behavior name in lowercase but I can't later set it because of the uc() call in set_behavior, i.e., "my $value = uc(shift);". I wonder if you'd consider either removing the uc() call from set_behavior, or adding one to specify_behavior to ensure that behavior names are all in uppercase. Thank you for writing this very useful, very customizable module. I'm using it to manipulate thermal model data for the U.S. space program. Matt

Thu Nov 15 18:45:09 2012 dmuey [...] cpan.org - Correspondence added

Hello Matt, Yeah, that is how its worked since before I was co-maint. Probably a good idea but will need some thought since it could break existing uses. Thanks! -- Dan

Thu Nov 15 18:45:10 2012 The RT System itself - Status changed from 'new' to 'open'

Mon Nov 19 08:49:14 2012 matt.d.garrett [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #81156] behavior name bug
Date:	Mon, 19 Nov 2012 06:49:03 -0700
To:	bug-Hash-Merge [...] rt.cpan.org
From:	Matt Garrett <matt.d.garrett [...] gmail.com>

Hi Daniel, Okay, thanks for writing back. Keep up the good work, Matt On Nov 15, 2012 4:45 PM, "Daniel Muey via RT" <bug-Hash-Merge@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=81156 > > > Hello Matt, > > Yeah, that is how its worked since before I was co-maint. > > Probably a good idea but will need some thought since it could break > existing uses. > > Thanks! > > -- > Dan >

Fri Aug 18 08:24:15 2017 HERMES [...] cpan.org - Correspondence added

This has been patched upstream with commit b57136e8c38a14ec5f4c8759ccf3426a426e0609 and 5cdcb03188115328030be2237f9ac8bbed7fa1ed. Fix for not dereferencing in a hash with depth greater 1 when using Clone or Clone::PP. Without passing the depth of the hashes (which was simply wrong anyway) the results are the same as if using Storable::dclone.

Fri Aug 18 08:24:16 2017 HERMES [...] cpan.org - Status changed from 'open' to 'patched'

Fri Aug 18 08:30:21 2017 matt.d.garrett [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #81156] behavior name bug
Date:	Fri, 18 Aug 2017 06:29:56 -0600
To:	bug-Hash-Merge [...] rt.cpan.org
From:	Matt Garrett <matt.d.garrett [...] gmail.com>

Thanks for letting me know. On Aug 18, 2017 6:24 AM, "Stefan Hermes via RT" <bug-Hash-Merge@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=81156 > > > This has been patched upstream with commit b57136e8c38a14ec5f4c8759ccf3426a426e0609 > and 5cdcb03188115328030be2237f9ac8bbed7fa1ed. > > Fix for not dereferencing in a hash with depth greater 1 when using Clone > or Clone::PP. Without passing the depth of the hashes (which was simply > wrong anyway) the results are the same as if using Storable::dclone. >