On Thu, May 20, 2010 at 03:48:10PM +0100, Nicholas Clark wrote:
> r1215/r1217 make the warning "isn't a cop" mandatory:
> ------------------------------------------------------------------------
> r1215 | [email protected] | 2010-05-06 23:39:37 +0100 (Thu, 06 May 2010) | 
> 2 lines
> Added CXt_LOOP to cx_block_type and the ability to show the int value for 
> unknown types.
> 
> This warning gets hit a lot on 5.12.0 and later, for example:
> 
>         start_cop_of_context CXt_LOOP_PLAIN op 'enter' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'gvsv' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'concat' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'const' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'concat' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'entereval' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'unstack' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'iter' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'and' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'leaveloop' isn't a cop
>         start_cop_of_context CXt_LOOP_LAZYIV op 'const' isn't a cop
> 
> (and CXt_LOOP_FOR, but not CXt_LOOP_LAZYSV)

> Is this correct? I don't really understand what that part of the code is
> doing.
> 
> Certainly, something needs to change before a production release is cut.

Yeah, I'd deleted that code a while ago but hadn't checked it in yet.
Done now.

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