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

Bug 423835

Summary: Wrong declaration marked in occurrences
Product: [ECD] Orion Reporter: Michael Rennie <Michael_Rennie>
Component: JS ToolsAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: mamacdon
Version: unspecified   
Target Milestone: 5.0 M2   
Hardware: All   
OS: All   
Whiteboard:

Description Michael Rennie CLA 2013-12-11 11:42:28 EST 
Consider the following snippet:

function myfunc(p1) {
	function myinner(p1) {
		var foo = p1; //carat on p1
	}
};

place carat where indicated. Notice the p1 from myfunc is marked (WRONG).

We need to make sure we are only marking the smallest enclosing scope for an occurrence.
Comment 1 Michael Rennie CLA 2013-12-31 12:20:11 EST 
Pushed fix + tests to: https://git.eclipse.org/r/20196
Comment 2 Michael Rennie CLA 2013-12-31 12:48:59 EST 
(In reply to Michael Rennie from comment #1)
> Pushed fix + tests to: https://git.eclipse.org/r/20196

I pushed an update that adds a few finder tests