Bug 86293 - [search] Search for method declaration with pattern "run()" reports match in binary field instead of anonymous class
Summary: [search] Search for method declaration with pattern "run()" reports match in ...
Status: VERIFIED 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.2 RC1   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 103469 131752 (view as bug list)
Depends on:
Blocks: 131750
  Show dependency tree
 
Reported: 2005-02-23 11:58 EST by Markus Keller CLA
Modified: 2006-04-13 10:54 EDT (History)
2 users (show)

See Also:


Attachments
Patch to fix this issue (7.33 KB, patch)
2006-04-09 11:10 EDT, Frederic Fusier CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2005-02-23 11:58:22 EST
I20050222-0821

- Import plugin org.eclipse.jface as binary plugin.
- Select package org.eclipse.jface.resource and then do Search > Java ... .
- Search string "run()", Method, Declarations, Scope: Selected Resources.

-> Search e.g. reports the field ColorRegistry#displayRunnable as match:
    protected Runnable displayRunnable = new Runnable() {
        public void run() {
            clearCaches();
        }
    };

-> Expected: the IMethod of run() is reported as enclosing element (the element
that appears in the Outline when you select "run" and press F3).
Comment 1 Frederic Fusier CLA 2005-03-14 15:19:03 EST
If time permit
Comment 2 Frederic Fusier CLA 2005-04-26 12:03:04 EDT
Already there in 3.0 => defer
Comment 3 Frederic Fusier CLA 2005-08-05 04:03:44 EDT
Reopen for 3.2
Comment 4 Frederic Fusier CLA 2006-03-14 13:37:28 EST
*** Bug 131752 has been marked as a duplicate of this bug. ***
Comment 5 Frederic Fusier CLA 2006-03-16 09:37:51 EST
*** Bug 103469 has been marked as a duplicate of this bug. ***
Comment 6 Frederic Fusier CLA 2006-04-09 11:10:37 EDT
Created attachment 38084 [details]
Patch to fix this issue
Comment 7 Frederic Fusier CLA 2006-04-09 11:19:41 EDT
All JDT/Core model and JDT/UI tests pass with this patch.

Philippe, please approve for RC1, thanks
Comment 8 Philipe Mulet CLA 2006-04-09 12:46:06 EDT
+1 for 3.2RC1
Comment 9 Frederic Fusier CLA 2006-04-09 12:53:34 EDT
Patch released in HEAD.
Comment 10 Jerome Lanneluc CLA 2006-04-13 10:54:40 EDT
Verified for 3.2 RC1 using build I20060413-0010