Skip Menu |

This queue is for tickets about the Convert-Base32 CPAN distribution.

Report information
The Basics
Id: 53014
Status: resolved
Priority: 0/
Queue: Convert-Base32

People
Owner: IKEGAMI [...] cpan.org
Requestors: IKEGAMI [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.02
Fixed in: 0.03



Subject: Error detection failure
There's a typo in one of the error check making it fail to detect the following error: $ perl -MTest::More=no_plan -MConvert::Base32 -e' is encode_base32(decode_base32($_)), $_ for @ARGV; ' aaaaaa not ok 1 # Failed test at -e line 1. # got: 'aaaaa' # expected: 'aaaaaa' 1..1 # Looks like you failed 1 test of 1. The check my $input_check = length($str) %8; if ($input_check == 1 || $input_check == 3 || $input_check == 8) { Carp::croak('Length of data invalid'); } should be my $input_check = length($str) %8; if ($input_check == 1 || $input_check == 3 || $input_check == 6) { Carp::croak('Length of data invalid'); } and would be simpler as if (length($buffer) % 8 >= 5) { Carp::croak('Length of data invalid'); } if you don't mind checking after the <c>char2bits</c> loop.
Subject: Re: [rt.cpan.org #53014] Error detection failure
Date: Wed, 23 Dec 2009 13:43:40 -0800
To: bug-Convert-Base32 [...] rt.cpan.org
From: Tatsuhiko Miyagawa <miyagawa [...] gmail.com>
Hi, thanks for the patches. Are you interested in maintaining this module? Just if so, I made you a co-maintainer of this module. I'm not really interested in this anymore, so feel free to make changes and upload with your PAUSE ID. There is a svn repository that you can export or convert from http://svn.bulknews.net/repos/public/Convert-Base32/ On Wed, Dec 23, 2009 at 1:30 PM, ikegami via RT <bug-Convert-Base32@rt.cpan.org> wrote: Show quoted text
> Wed Dec 23 16:30:00 2009: Request 53014 was acted upon. > Transaction: Ticket created by ikegami >       Queue: Convert-Base32 >     Subject: Error detection failure >   Broken in: (no value) >    Severity: Unimportant >       Owner: Nobody >  Requestors: IKEGAMI@cpan.org >      Status: new >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=53014 > > > > There's a typo in one of the error check making it fail to detect the > following error: > > $ perl -MTest::More=no_plan -MConvert::Base32 -e' >    is encode_base32(decode_base32($_)), $_ for @ARGV; > ' aaaaaa > not ok 1 > #   Failed test at -e line 1. > #          got: 'aaaaa' > #     expected: 'aaaaaa' > 1..1 > # Looks like you failed 1 test of 1. > > The check > >    my $input_check = length($str) %8; >    if ($input_check == 1 || $input_check == 3 || $input_check == 8) { >        Carp::croak('Length of data invalid'); >    } > > should be > >    my $input_check = length($str) %8; >    if ($input_check == 1 || $input_check == 3 || $input_check == 6) { >        Carp::croak('Length of data invalid'); >    } > > and would be simpler as > >    if (length($buffer) % 8 >= 5) { >        Carp::croak('Length of data invalid'); >    } > > if you don't mind checking after the <c>char2bits</c> loop. > >
-- Tatsuhiko Miyagawa
Subject: Re: [rt.cpan.org #53014] Error detection failure
Date: Wed, 6 Jan 2010 21:21:42 -0500
To: bug-Convert-Base32 [...] rt.cpan.org
From: Eric Brine <ikegami [...] adaelis.com>
On Wed, Dec 23, 2009 at 4:44 PM, miyagawa@gmail.com via RT < bug-Convert-Base32@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=53014 > > > Hi, thanks for the patches. Are you interested in maintaining this > module? Just if so, I made you a co-maintainer of this module. I'm not > really interested in this anymore, so feel free to make changes and > upload with your PAUSE ID. There is a svn repository that you can > export or convert from > http://svn.bulknews.net/repos/public/Convert-Base32/ >
I've never used this module. I was submitting the bugs after they surfaced in a post on PerlMonks. I applied the fixes, added some tests, did lots of benchmarking, added some speed improvements, and uploaded v0.03 to CPAN. I didn't notice that you mentioned a repository until now. I can't seem to mark the bugs as resolved. Something is preventing me from getting the modify ticket screen. Not just in the Convert-Base32 queue, but in the queues for my other modules. - Eric
Subject: Re: [rt.cpan.org #53014] Error detection failure
Date: Thu, 7 Jan 2010 03:59:21 -0800
To: bug-Convert-Base32 [...] rt.cpan.org
From: Tatsuhiko Miyagawa <miyagawa [...] gmail.com>
Maybe your RT account is not linked to your PAUSE ID. I don't remember how that happens to my (and anyone else's) account though. On Wed, Jan 6, 2010 at 6:21 PM, ikegami@adaelis.com via RT <bug-Convert-Base32@rt.cpan.org> wrote: Show quoted text
>       Queue: Convert-Base32 >  Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=53014 > > > On Wed, Dec 23, 2009 at 4:44 PM, miyagawa@gmail.com via RT < > bug-Convert-Base32@rt.cpan.org> wrote: >
>> <URL: https://rt.cpan.org/Ticket/Display.html?id=53014 > >> >> Hi, thanks for the patches. Are you interested in maintaining this >> module? Just if so, I made you a co-maintainer of this module. I'm not >> really interested in this anymore, so feel free to make changes and >> upload with your PAUSE ID. There is a svn repository that you can >> export or convert from >> http://svn.bulknews.net/repos/public/Convert-Base32/ >>
> > I've never used this module. I was submitting the bugs after they surfaced > in a post on PerlMonks. > > I applied the fixes, added some tests, did lots of benchmarking, added some > speed improvements, and uploaded v0.03 to CPAN. I didn't notice that you > mentioned a repository until now. > > I can't seem to mark the bugs as resolved. Something is preventing me from > getting the modify ticket screen. Not just in the Convert-Base32 queue, but > in the queues for my other modules. > > - Eric > >
-- Tatsuhiko Miyagawa