[ 
https://issues.apache.org/jira/browse/BCEL-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14706721#comment-14706721
 ] 

Mark Roberts commented on BCEL-202:
-----------------------------------

The len=2 is accounting for the 'number_of_entries' field prior to the array of 
stack maps.

I suppose you could put the number_of_locals and number_of_stack_items 
arguments back in the constructor with a comment that these items are now 
ignored and the lengths are derived from the matching array arguments.



> StackMap[Table]Entry.copy() needs to be deep; Improved support for StackMaps
> ----------------------------------------------------------------------------
>
>                 Key: BCEL-202
>                 URL: https://issues.apache.org/jira/browse/BCEL-202
>             Project: Commons BCEL
>          Issue Type: Bug
>            Reporter: Mark Roberts
>         Attachments: stack-map.diff
>
>
> There are several ways a user can modify a Java class file that should cause 
> BCEL to update the StackMaps automatically.  Unfortunately, it does not.  
> These additional methods at least allow users to take care of these issues 
> for themselves.
> The patch also fixes a bug - StackMapTableEntry.copy() needs to be a deep 
> copy to prevent StackMapTypes from being reused.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to