Skip Menu |

This queue is for tickets about the Perl-Metrics-Simple CPAN distribution.

Report information
The Basics
Id: 56441
Status: resolved
Worked: 4 hours (240 min)
Priority: 0/
Queue: Perl-Metrics-Simple

People
Owner: matisse [...] spamcop.net
Requestors: MAGGIEXYZ [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.15



Subject: custom @Perl::Metrics::Simple::Analysis::File::LOGIC_KEYWORDS and OPERATORS
Thanks much for the module! It would be nice to able to set @Perl::Metrics::Simple::Analysis::File::LOGIC_KEYWORDS and OPERATORS for specific situations. For example, some code relies heavily on map transformations. It will be useful to add map and/or grep to the keywords since they loop over array. Thanks, Maggie
Subject: patch_custom_ko
Download patch_custom_ko
application/octet-stream 1.8k

Message body not shown because it is not plain text.

Thanks for the patch. I'll have a deeper look this weekend.
My apologies - got caught up in other $work stuff this pas weekend. Will try and work on this this week.
Oy. I apologize again. I have been extra busy at $work and only prepared the changes today. I have a new version ready for release (0.15) and am waiting on approval from the legal folks at work (required for code I release publicly.) The changes I have ready are: Changes for https://rt.cpan.org/Public/Bug/Display.html?id=56441 Thanks to Maggie J. Xiong for the inquiry, inspiration and patch. You can now set: @Perl::Metrics::Simple::Analysis::File::LOGIC_KEYWORDS and OPERATORS to custom values before creating a new Perl::Metrics::Simple::Analysis::File object. Added to the default @LOGIC_KEYWORDS used in calculating complexity: map grep Added to the default @LOGIC_OPERATORS used in calculating complexity: !~ // < <=> == =~ Show quoted text
>
cmp eq gt lt ne ~~ Added two accessors on Perl::Metrics::Simple::Analysis::File to obtain the values in use: logic_keywords() logic_operators()
Great! Thank you! Maggie
I've uploaded version 0.15 to PAUSE to it should show up in CPAN within a day or so, and then I'll marked this resolved.