James E Keenan

Results 396 comments of James E Keenan

My feeling is that this pull request should be rejected because it is not needed. We added continuous integration code for travis-ci.com several years ago, but my impression is that...

Rearranging OP to better respond .... > The only meaningful difference I can see in the files produced between these versions is that `jquery-tablesorter-min.js` has been replaced by `jquery.tablesorter.min.js`. It...

This should be resolved with Devel-NYTProf version 6.11, just released to CPAN.

I went looking to close this ticket today and installed Devel::NYTProf v6.11 from CPAN, then ran the program which the OP provided. I was not very satisfied with the results....

On 9/24/21 9:13 PM, shawnlaffan wrote: > The file causing the problems is |jquery.tablesorter.min.js|, but it was > |jquery.floatThead.min.js| that was removed. > > Sorry for not spotting that when...

I'm starting to think that the "bad" output I got with Shawn Laffan's test file once I installed Devel-NYTProf-6.11 from CPAN is the result of installing this version against the...

Before proceeding further I want to make a correction to the data I provided yesterday in https://github.com/timbunce/devel-nytprof/issues/192#issuecomment-927178693. The listing of `.js` files I provided for the Devel-NYTProf-6.11 installed against my...

> ``` > # Installed 5.34.0 with NYTProf 6.11 > > $ pwd > /home/jkeenan/perl5/perlbrew/perls/perl-5.34.0/lib/site_perl/5.34.0/x86_64-linux/Devel/NYTProf > $ find ./js -type f -name '*.js' | xargs ls -l > -r--r--r-- 1...

In commit 8f4088c02b, I added debugging statements to NYTProf.xs and to t/rt70211.t. When I run the test file via `prove`, I get: ``` $ prove -vb t/rt70211.t t/rt70211.t .. XXXXXX:...

Data point: you get this spurious log warning whenever you simply include Devel::NYTProf via the command line. ``` $ perl -MDevel::NYTProf -E 'say q|hello world|;' hello world fid 1 has...