On Tue, Oct 2, 2012 at 8:37 AM, Tomas Doran <bobtf...@bobtfish.net> wrote:
> I entirely agree - there should never be explicit code like this needed. > > However, currently, if you remove it - then the tests fail. > Funny, my tests start passing. ;) > > This may be as the tests are utterly wrong, or it may be as there is > another bug we're not seeing / aware of. > > If you could suggest a way to resolve this issue (either by fixing > something more than just removing these lines, or by deciding the tests are > wrong and correcting them), then I'd be fully in support of fixing this. > I guess I was posting to ask why the code was added in the first place. Thought I might be missing something. Since then I looked at RJBS JSONpm and it just lets the json encoding do its thing: http://cpansearch.perl.org/src/RJBS/MooseX-Storage-Format-JSONpm-0.093092/lib/MooseX/Storage/Format/JSONpm.pm I'll look at the tests. Thanks, -- Bill Moseley mose...@hank.org
