Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 350047 - org.eclipse.wst.jsdt.internal.core.util.Util#scanTypeSignature does not handle boolean signatures
Summary: org.eclipse.wst.jsdt.internal.core.util.Util#scanTypeSignature does not handl...
Status: RESOLVED INVALID
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: 3.2.5   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.2.5   Edit
Assignee: Ian Tewksbury CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-22 09:03 EDT by Ian Tewksbury CLA
Modified: 2011-06-23 14:45 EDT (History)
1 user (show)

See Also:
itewksbu: review? (cmjaun)

Attachments
Fix Patch (1.58 KB, patch)
2011-06-22 09:04 EDT, Ian Tewksbury CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Tewksbury CLA 2011-06-22 09:03:00 EDT 
Currently org.eclipse.wst.jsdt.internal.core.util.Util#scanTypeSignature  does not handel boolean signatures represented by the character 'Z'.  So if a method is found with a boolean in the signature an IllegalArgumentException is thrown.  Checking out JDT their version of the Util class and Signature class, containing the constants, uses and has a boolean character constant.

I will provide a patch so that the Util class will support a boolean in the signature.

This was found while working on Bug 306958.
Comment 1 Ian Tewksbury CLA 2011-06-22 09:04:13 EDT 
Created attachment 198396 [details]
Fix Patch

Patch which adds handling for boolean in the type signature.
Comment 2 Chris Jaun CLA 2011-06-22 09:34:12 EDT 
That's odd....I didn't think it was possible for something to have a base type boolean signature any more.

I thought all "boolean" types were treated as the object Boolean.
Comment 3 Ian Tewksbury CLA 2011-06-22 09:38:34 EDT 
would you rather go through and remove the boolean base type?
Comment 4 Chris Jaun CLA 2011-06-23 14:45:34 EDT 
There should not be any base types used in JSDT for booleans.

Whatever is causing something to be set as a base type boolean should be fixed.