Skip Menu |

This queue is for tickets about the Keyword-TailRecurse CPAN distribution.

Report information
The Basics
Id: 130704
Status: resolved
Priority: 0/
Queue: Keyword-TailRecurse
People
Owner: Nobody in particular
Requestors: violapiratejunky [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: 0.02

History Show all quoted text
  Tue Oct 15 00:44:13 2019 violapiratejunky [...] gmail.com - Ticket created
Subject: Use snake_case instead of camelCase
This is a great module to avoid setting @_ and using goto yourself! Since this is the first version, I wonder if it would be a good idea now to switch to using the more common perl syntax of snake_case? Or perl keywords that break the snake_case rule seem to to use all lowercase, like tailrecurse (some examples of this would be localtime or wantarray): https://learn.perl.org/docs/keywords.html
  Tue Oct 15 15:51:01 2019 JLCOOPER [...] cpan.org - Correspondence added
On Tue Oct 15 00:44:13 2019, srchulo wrote: Show quoted text
> This is a great module to avoid setting @_ and using goto yourself! > Since this is the first version, I wonder if it would be a good idea > now to switch to using the more common perl syntax of snake_case? Or > perl keywords that break the snake_case rule seem to to use all > lowercase, like tailrecurse (some examples of this would be localtime > or wantarray): > > https://learn.perl.org/docs/keywords.html
Hi, Thanks for the feedback. I agree that a consistent style is important, unfortunately we all like a different style :) How about if I add in the option to choose either `tailRecurse`, `tailrecurse` or `tail_recurse` when importing the module. That would cover the majority of styles people use and will be backwards compatible, as I can default it to `tailRecurse` if nothing's been specified. Regards, Jason
  Tue Oct 15 15:51:01 2019 The RT System itself - Status changed from 'new' to 'open'
  Fri Oct 18 00:22:52 2019 violapiratejunky [...] gmail.com - Correspondence added
Hey Jason, Thanks for the response! I think that sounds like a good way to handle it :) On Tue Oct 15 15:51:01 2019, JLCOOPER wrote: Show quoted text
> On Tue Oct 15 00:44:13 2019, srchulo wrote:
> > This is a great module to avoid setting @_ and using goto yourself! > > Since this is the first version, I wonder if it would be a good idea > > now to switch to using the more common perl syntax of snake_case? Or > > perl keywords that break the snake_case rule seem to to use all > > lowercase, like tailrecurse (some examples of this would be localtime > > or wantarray): > > > > https://learn.perl.org/docs/keywords.html
> > Hi, > Thanks for the feedback. I agree that a consistent style is > important, unfortunately we all like a different style :) > > How about if I add in the option to choose either `tailRecurse`, > `tailrecurse` or `tail_recurse` when importing the module. That would > cover the majority of styles people use and will be backwards > compatible, as I can default it to `tailRecurse` if nothing's been > specified. > > Regards, > Jason
  Wed Oct 23 09:38:20 2019 JLCOOPER [...] cpan.org - Correspondence added
Hi, Version 0.02 of Keyword::TailRecurse has been released which allows you to choose to use a keyword of either 'tailRecurse', 'tail_recurse' or 'tailrecurse'. I'll resolve this case now, but if you have any other ideas or suggestions then feel free to raise a new case. Regards, Jason.
  Wed Oct 23 09:38:21 2019 JLCOOPER [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Oct 23 09:38:21 2019 JLCOOPER [...] cpan.org - Fixed in 0.02 added
  Wed Oct 23 21:53:45 2019 violapiratejunky [...] gmail.com - Correspondence added
This looks great! Thank you :) On Wed Oct 23 09:38:20 2019, JLCOOPER wrote: Show quoted text
> Hi, > Version 0.02 of Keyword::TailRecurse has been released which allows > you to choose to use a keyword of either 'tailRecurse', 'tail_recurse' > or 'tailrecurse'. > > I'll resolve this case now, but if you have any other ideas or > suggestions then feel free to raise a new case. > > Regards, > Jason.