Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 347064

Summary:

NPE in JavaScriptDebugHover

Product:

[WebTools] JSDT

Reporter:

Michael Rennie <Michael_Rennie>

Component:

Debug

Assignee:

Michael Rennie <Michael_Rennie>

Status:

RESOLVED FIXED

QA Contact:

Michael Rennie <Michael_Rennie>

Severity:

normal

Priority:

CC:

thatnitind

Version:

3.3

Target Milestone:

3.4

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
fix	none

Description Michael Rennie

2011-05-24 15:18:27 EDT

Created attachment 196478 [details]
fix

java.lang.NullPointerException
	at org.eclipse.wst.jsdt.debug.internal.ui.eval.JavaScriptDebugHover.getHoverInfo2(JavaScriptDebugHover.java:78)
	at org.eclipse.wst.jsdt.internal.ui.text.java.hover.BestMatchHover.getHoverInfo2(BestMatchHover.java:147)
	at org.eclipse.wst.jsdt.internal.ui.text.java.hover.JavaEditorTextHoverProxy.getHoverInfo2(JavaEditorTextHoverProxy.java:78)
	at org.eclipse.jface.text.TextViewerHoverManager$4.run(TextViewerHoverManager.java:166)

The problem is coming from JavaScriptValue#getVariables(), the current implementation is not following the API, i.e. it should not be returning null.

Comment 1 Michael Rennie

2011-07-07 12:53:52 EDT

applied patch to HEAD