Bug 97187 - [rendering] Shows Single Char for primitve Types
Summary: [rendering] Shows Single Char for primitve Types
Status: CLOSED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1 RC2   Edit
Assignee: Jerome Lanneluc CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 97399 98476 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-05-30 04:59 EDT by Benjamin Pasero CLA
Modified: 2005-06-10 09:33 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Pasero CLA 2005-05-30 04:59:31 EDT
In cases where I hover over fields which are primitive types, the JavaDoc is
just displaying the single char representation of the Type, e.g. "Z Main.foo"
for a boolean.

When hovering over local types, I am seeing the type in the Hover, e.g. "boolean
Main.bar()".

I think the JavaDoc should always try to show the type, especially since it is
not quite clear to read a "boolean" out of "Z".

Ben
Comment 1 Dirk Baeumer CLA 2005-05-30 06:47:25 EDT
Martin, I quess that happens now for resolved field declarations in
JavaElementLabels. I enabled the code after JDT/Core has fixed the bug. We
should indeed render boolean here not Z.
Comment 2 Benjamin Pasero CLA 2005-05-30 06:52:23 EDT
Note that this happens for all primitive types (e.g. "D" for "double").

Possible connection: Bug #97206

Ben
Comment 3 Martin Aeschlimann CLA 2005-05-30 06:56:01 EDT
The bug is in jdt.core. The signature returned here is 'LZ;'

public class B {
	boolean fBoolean;
	{
		fBoolean= true;  // hover over 'fBoolean'
	}

}
Comment 4 Philipe Mulet CLA 2005-05-30 07:45:17 EDT
+1 for RC2
Comment 5 Martin Aeschlimann CLA 2005-05-31 05:49:31 EDT
*** Bug 97399 has been marked as a duplicate of this bug. ***
Comment 6 Jerome Lanneluc CLA 2005-05-31 06:41:26 EDT
Changed BindingKeyParser and its subclasses to handle base types separately from
other types (see consumeBaseType(char[]) ).
Added regression test BindingKeyTest#test039()
Comment 7 Dirk Baeumer CLA 2005-06-06 08:47:22 EDT
*** Bug 98476 has been marked as a duplicate of this bug. ***
Comment 8 Olivier Thomann CLA 2005-06-06 20:08:13 EDT
Verified in N20050606-0010
Comment 9 Frederic Fusier CLA 2005-06-10 09:33:11 EDT
Verified for 3.1 RC2 using build I20050610-0010