Bug #54293 for Perl-Metrics-Simple: Allow using a ref-to-SCALAR of file contents instead of a file path.

Thu Feb 04 05:00:25 2010 CHORNY [...] cpan.org - Ticket created

Subject:

existing PPI::Document

I'd like to have ability to work on existing PPI::Document. For ex., Padre IDE already has created PPI::Document, so it would be faster. -- Alexandr Ciornii, http://chorny.net

Thu Feb 04 10:34:11 2010 matisse [...] spamcop.net - TimeEstimated changed from (no value) to '180'

Thu Feb 04 10:34:11 2010 matisse [...] spamcop.net - TimeLeft changed from (no value) to '180'

Thu Feb 04 10:34:11 2010 matisse [...] spamcop.net - Status changed from 'new' to 'open'

Thu Feb 04 10:34:11 2010 matisse [...] spamcop.net - Given to MATISSE

Thu Feb 04 10:35:29 2010 matisse [...] spamcop.net - Correspondence added

Interesting idea. Thanks for the suggestion. Can you provide one or two examples of how you would actually use the new feature? I'd like to see those before I make up something that might not be as useful for you.

Thu Feb 04 16:12:08 2010 CHORNY [...] cpan.org - Correspondence added

Feb 04 10:35:29 2010, MATISSE wrote Show quoted text

> Interesting idea. Thanks for the suggestion. > > Can you provide one or two examples of how you would actually use the > new feature? > I'd like to see those before I make up something that might not be as > useful for you.

IMHO, best would be to let Perl::Metrics::Simple accept either text in form of scalar ref (as PPI::Document does) or PPI::Document object. Patch attached, with several other small changes. -- Alexandr Ciornii, http://chorny.net

Subject:

Perl-Metrics-Simple-patch.rar

Download Perl-Metrics-Simple-patch.rar
application/octetstream 7.8k

Message body not shown because it is not plain text.

Thu Feb 04 16:13:53 2010 CHORNY [...] cpan.org - Correspondence added

Also in .zip, in case you don't have RAR. -- Alexandr Ciornii, http://chorny.net

Subject:

Perl-Metrics-Simple-patch.zip

Download Perl-Metrics-Simple-patch.zip
application/zip 9k

Message body not shown because it is not plain text.

Thu Feb 04 20:39:43 2010 matisse [...] spamcop.net - Correspondence added

On Thu Feb 04 16:12:08 2010, CHORNY wrote: Show quoted text

> IMHO, best would be to let Perl::Metrics::Simple accept either text in > form of scalar ref (as PPI::Document does) or PPI::Document object.

That seems very reasonable. I looked at the patches quickly today and they seem fine. I'll look more carefully and hopefully add them this weekend. -M

Sun Feb 07 15:43:50 2010 matisse [...] spamcop.net - Subject changed from 'existing PPI::Document' to 'Allow using a ref-to-SCALAR of file contents instead of a file path.'

Sun Feb 07 15:43:50 2010 matisse [...] spamcop.net - TimeWorked changed from (no value) to '60'

Sun Feb 07 16:05:46 2010 matisse [...] spamcop.net - Correspondence added

I've applied the patches and uploaded a new release, 0.14 to CPAN. It should be available soon. Thanks again!

Sun Feb 07 16:11:46 2010 matisse [...] spamcop.net - TimeWorked changed from '60' to '90'

Sun Feb 07 16:11:46 2010 matisse [...] spamcop.net - TimeLeft changed from '180' to '0'

Mon Feb 08 10:58:58 2010 matisse [...] spamcop.net - Status changed from 'open' to 'resolved'

Mon Feb 08 10:58:59 2010 matisse [...] spamcop.net - Fixed in 0.14 added