You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jakarta.apache.org by bu...@apache.org on 2010/05/24 17:09:06 UTC
DO NOT REPLY [Bug 49332] New: Type.getReturnTypeSize() doesn't
decode Type.getTypeSize() output
https://issues.apache.org/bugzilla/show_bug.cgi?id=49332
Summary: Type.getReturnTypeSize() doesn't decode
Type.getTypeSize() output
Product: BCEL
Version: unspecified
Platform: All
OS/Version: All
Status: NEW
Severity: blocker
Priority: P2
Component: Main
AssignedTo: notifications@jakarta.apache.org
ReportedBy: pugh@cs.umd.edu
Created an attachment (id=25475)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=25475)
Fix for this bug
The Type.getReturnTypeSize() method doesn't decode Type.getTypeSize() output.
The same bug was present in FieldInstruction.getFieldSize(), filed and fixed as
bug 48280 https://issues.apache.org/bugzilla/show_bug.cgi?id=48280
As a result of this bug, the BCEL calculates the wrong size of the return type
for methods, leading to FindBugs going off the rails.
The fix is the same as to fix FieldInstruction.getFieldSize(): use Type.size()
to decode the coded value.
I've attached a patch with the fix, and I've also looked at the other uses of
getTypeSize(), and all the other invocations seem OK.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@jakarta.apache.org
For additional commands, e-mail: notifications-help@jakarta.apache.org
DO NOT REPLY [Bug 49332] Type.getReturnTypeSize() doesn't decode
Type.getTypeSize() output
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=49332
Dave Brosius <db...@mebigfatguy.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #1 from Dave Brosius <db...@mebigfatguy.com> 2010-05-24 22:06:53 EDT ---
Applied to head, thanks!
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@jakarta.apache.org
For additional commands, e-mail: notifications-help@jakarta.apache.org