Bug #70811 for Array-Iterator: currentIndex() before next()

Thu Sep 08 10:43:42 2011 sharyanto [...] cpan.org - Ticket created

Subject:

currentIndex() before next()

The initial currentIndex() is 0 before any next(). After the first next (), it's still at 0. Perhaps set initial currentIndex() to -1, or undef, to differentiate the above two cases? -- sh

Thu Sep 08 12:01:48 2011 stevan.little [...] gmail.com - Correspondence added

Steven, To be completely honest, I have not used this module in about 7 years and I have no intention of using it in the future. If you are interested in having co-maint on it and taking it over I am happy to give it, but I am no longer maintaining this actively. Sorry, - Stevan On Thu Sep 08 10:43:42 2011, SHARYANTO wrote: Show quoted text

> The initial currentIndex() is 0 before any next(). After the first next > (), it's still at 0. > > Perhaps set initial currentIndex() to -1, or undef, to differentiate the > above two cases? > > -- > sh

Thu Sep 08 12:01:49 2011 The RT System itself - Status changed from 'new' to 'open'

Thu Sep 08 20:39:03 2011 stevenharyanto [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #70811] currentIndex() before next()
Date:	Fri, 9 Sep 2011 07:38:54 +0700
To:	bug-Array-Iterator [...] rt.cpan.org
From:	Steven Haryanto <stevenharyanto [...] gmail.com>

Stevan, I'd be happy to co-maint it. Do you mind if I add other stuffs too? For one thing, I'd like to add lower_case name aliases, e.g. has_next() for hasNext(). I'm also considering adding prev/previous. Also, is there a repository? Regards, Steven On Thu, Sep 8, 2011 at 11:01 PM, Stevan Little via RT < bug-Array-Iterator@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=70811 > > > Steven, > > To be completely honest, I have not used this module in about 7 years and I > have no intention > of using it in the future. If you are interested in having co-maint on it > and taking it over I am > happy to give it, but I am no longer maintaining this actively. > > Sorry, > > - Stevan > > > On Thu Sep 08 10:43:42 2011, SHARYANTO wrote:

> > The initial currentIndex() is 0 before any next(). After the first next > > (), it's still at 0. > > > > Perhaps set initial currentIndex() to -1, or undef, to differentiate the > > above two cases? > > > > -- > > sh

> > >

Thu Sep 08 20:41:06 2011 sharyanto [...] cpan.org - Correspondence added

Never mind about prev/previous, I missed Array::Iterator::BiDirectional. On Thu Sep 08 20:39:03 2011, stevenharyanto@gmail.com wrote: Show quoted text

> Stevan, > > I'd be happy to co-maint it. Do you mind if I add other stuffs too? > For one > thing, I'd like to add lower_case name aliases, e.g. has_next() for > hasNext(). I'm also considering adding prev/previous. > > Also, is there a repository? > > Regards, > Steven

Thu Sep 08 20:57:09 2011 stevan.little [...] gmail.com - Correspondence added

Steven, You now have full co-maint via PAUSE. As for adding other stuff, no problem for me as I am not using it anymore and I don't think it ever got much adoption. Adding lower-case method names is totally fine with me too. As for a repository, this actually never made it out of CVS into SVN, which was a looooong time ago. Basically what you see on CPAN is all I have, but you are welcome to push it up to github or whatever you prefer. Thanks for taking this over. - Stevan On Thu Sep 08 20:39:03 2011, stevenharyanto@gmail.com wrote: Show quoted text

> Stevan, > > I'd be happy to co-maint it. Do you mind if I add other stuffs too? > For one > thing, I'd like to add lower_case name aliases, e.g. has_next() for > hasNext(). I'm also considering adding prev/previous. > > Also, is there a repository? > > Regards, > Steven > > On Thu, Sep 8, 2011 at 11:01 PM, Stevan Little via RT < > bug-Array-Iterator@rt.cpan.org> wrote: >

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=70811 > > > > > Steven, > > > > To be completely honest, I have not used this module in about 7

> years and I

> > have no intention > > of using it in the future. If you are interested in having co-maint

> on it

> > and taking it over I am > > happy to give it, but I am no longer maintaining this actively. > > > > Sorry, > > > > - Stevan > > > > > > On Thu Sep 08 10:43:42 2011, SHARYANTO wrote:

> > > The initial currentIndex() is 0 before any next(). After the first

> next

> > > (), it's still at 0. > > > > > > Perhaps set initial currentIndex() to -1, or undef, to

> differentiate the

> > > above two cases? > > > > > > -- > > > sh

> > > > > >

Thu Sep 08 22:12:15 2011 sharyanto [...] cpan.org - Correspondence added

Thanks. I just pushed a new release (0.07) to PAUSE and put the code on github (https://github.com/sharyanto/perl-Array-Iterator). After second thought, I add a new method iterated() instead of changing the behaviour of current_index() and current(). Best Regards, Steven On Thu Sep 08 20:57:09 2011, STEVAN wrote: Show quoted text

> Steven, > > You now have full co-maint via PAUSE. > > As for adding other stuff, no problem for me as I am not using it > anymore and I don't think it > ever got much adoption. Adding lower-case method names is totally fine > with me too. > > As for a repository, this actually never made it out of CVS into SVN, > which was a looooong > time ago. Basically what you see on CPAN is all I have, but you are > welcome to push it up to > github or whatever you prefer. > > Thanks for taking this over. > > - Stevan > > > On Thu Sep 08 20:39:03 2011, stevenharyanto@gmail.com wrote:

> > Stevan, > > > > I'd be happy to co-maint it. Do you mind if I add other stuffs too? > > For one > > thing, I'd like to add lower_case name aliases, e.g. has_next() for > > hasNext(). I'm also considering adding prev/previous. > > > > Also, is there a repository? > > > > Regards, > > Steven > > > > On Thu, Sep 8, 2011 at 11:01 PM, Stevan Little via RT < > > bug-Array-Iterator@rt.cpan.org> wrote: > >

> > > <URL: https://rt.cpan.org/Ticket/Display.html?id=70811 > > > > > > > Steven, > > > > > > To be completely honest, I have not used this module in about 7

> > years and I

> > > have no intention > > > of using it in the future. If you are interested in having co-

> maint

> > on it

> > > and taking it over I am > > > happy to give it, but I am no longer maintaining this actively. > > > > > > Sorry, > > > > > > - Stevan > > > > > > > > > On Thu Sep 08 10:43:42 2011, SHARYANTO wrote:

> > > > The initial currentIndex() is 0 before any next(). After the

> first

> > next

> > > > (), it's still at 0. > > > > > > > > Perhaps set initial currentIndex() to -1, or undef, to

> > differentiate the

> > > > above two cases? > > > > > > > > -- > > > > sh

> > > > > > > > >

> >

Thu Sep 08 22:12:16 2011 sharyanto [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Sep 08 22:39:11 2011 stevan.little [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #70811] currentIndex() before next()
Date:	Thu, 8 Sep 2011 22:38:58 -0400
To:	bug-Array-Iterator [...] rt.cpan.org
From:	Stevan Little <stevan.little [...] iinteractive.com>

Excellent, it is always wonderful to see old code get a new life. Thanks, - Stevan On Sep 8, 2011, at 10:12 PM, steven haryanto via RT wrote: Show quoted text

> Queue: Array-Iterator > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=70811 > > > Thanks. I just pushed a new release (0.07) to PAUSE and put the code on > github (https://github.com/sharyanto/perl-Array-Iterator). After second > thought, I add a new method iterated() instead of changing the > behaviour of current_index() and current(). > > Best Regards, > Steven > > On Thu Sep 08 20:57:09 2011, STEVAN wrote:

>> Steven, >> >> You now have full co-maint via PAUSE. >> >> As for adding other stuff, no problem for me as I am not using it >> anymore and I don't think it >> ever got much adoption. Adding lower-case method names is totally fine >> with me too. >> >> As for a repository, this actually never made it out of CVS into SVN, >> which was a looooong >> time ago. Basically what you see on CPAN is all I have, but you are >> welcome to push it up to >> github or whatever you prefer. >> >> Thanks for taking this over. >> >> - Stevan >> >> >> On Thu Sep 08 20:39:03 2011, stevenharyanto@gmail.com wrote:

>>> Stevan, >>> >>> I'd be happy to co-maint it. Do you mind if I add other stuffs too? >>> For one >>> thing, I'd like to add lower_case name aliases, e.g. has_next() for >>> hasNext(). I'm also considering adding prev/previous. >>> >>> Also, is there a repository? >>> >>> Regards, >>> Steven >>> >>> On Thu, Sep 8, 2011 at 11:01 PM, Stevan Little via RT < >>> bug-Array-Iterator@rt.cpan.org> wrote: >>>

>>>> <URL: https://rt.cpan.org/Ticket/Display.html?id=70811 > >>>> >>>> Steven, >>>> >>>> To be completely honest, I have not used this module in about 7

>>> years and I

>>>> have no intention >>>> of using it in the future. If you are interested in having co-

>> maint

>>> on it

>>>> and taking it over I am >>>> happy to give it, but I am no longer maintaining this actively. >>>> >>>> Sorry, >>>> >>>> - Stevan >>>> >>>> >>>> On Thu Sep 08 10:43:42 2011, SHARYANTO wrote:

>>>>> The initial currentIndex() is 0 before any next(). After the

>> first

>>> next

>>>>> (), it's still at 0. >>>>> >>>>> Perhaps set initial currentIndex() to -1, or undef, to

>>> differentiate the

>>>>> above two cases? >>>>> >>>>> -- >>>>> sh

>>>> >>>> >>>>

>> >>

> > >

Thu Sep 08 22:39:12 2011 The RT System itself - Status changed from 'resolved' to 'open'

Thu Sep 15 21:49:32 2011 sharyanto [...] cpan.org - Status changed from 'open' to 'resolved'