On Thu, Dec 11, 2008 at 07:27:10PM +0000, Nicholas Clark wrote:
> On Thu, Dec 11, 2008 at 02:34:14PM +0000, [email protected] wrote:
> > Author: tim.bunce
> > Log:
> > Start work on normalizing eval sequence numbers in file and sub names.
> > Any volunteers to implement normalize_eval_seqn()?
> 
> Well, if someone implements the TODO tests, I can write the C.

It's a little tricky given the 'dump and diff' approach the tests take.
Might be easiest to add a perl API for it and write some tests using that
but that seems a little over the top for a function that would probably
never be used from perl.

> But, right now, tests are failing:

Yeah, it's in a state of flux while I'm redoing the dumping.

> -fid_fileinfo   2       sub     
> AutoLoader::__ANON__[/usr/local/perl589/lib/5.8.9/AutoLoader.pm:31]     31-31
> +fid_fileinfo   2       sub     
> AutoLoader::__ANON__[/Users/nick/Sandpit/snap5.10.x-34928/lib/perl5/5.8.9/AutoLoader.pm:31]
>      31-31

Generally I'm trying to wrestle with the effects of string evals.
That's led me to move away from editing or discarding the data when
running tests, because that makes it harder to add features that depend
on the data.  I'm shifting the 'normalization for test' to happen
during the dump.

The 'internal details' of AutoLoader.pm shouldn't be dumped at all as
it's a library module so will vary between perl versions.
The internal details used to be deleted when the profile was read but
they're not now.  I'm still implementing the new logic to achieve the
same effect on the dumping side.

> [that's with a 5.8.9-RC2 or thereabouts. I think more fail with stock 5.8.8]
> 
> I assume that /usr/local/perl589 is on your machine. What's the correct fix?

Non trivial. I'm working on it.

Tim.

--~--~---------~--~----~------------~-------~--~----~
You've received this message because you are subscribed to
the Devel::NYTProf Development User group.

Group hosted at:  http://groups.google.com/group/develnytprof-dev
Project hosted at:  http://perl-devel-nytprof.googlecode.com
CPAN distribution:  http://search.cpan.org/dist/Devel-NYTProf

To post, email:  [email protected]
To unsubscribe, email:  [email protected]
-~----------~----~----~----~------~----~------~--~---

Reply via email to