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

Bug 358991

Summary: Reference should be removed
Product: z_Archived Reporter: Lu Yao <luyaoLy>
Component: EDTAssignee: Yun Feng Ma <mayunf>
Status: CLOSED INVALID QA Contact:
Severity: normal    
Priority: P3 CC: broy2, chenzhh, jqian, jspadea, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch
none
New Patch
none
Patch none

Description Lu Yao CLA 2011-09-26 22:19:07 EDT 
Build Identifier: 0926

reference, menu, source editor

Reproducible: Always

Steps to Reproduce:
1. In source editor, right click, the item Reference should be removed.
2.
3.
Comment 1 Yun Feng Ma CLA 2011-09-29 04:14:30 EDT 
Created attachment 204263 [details]
Patch

Here is a patch.
Comment 2 Yun Feng Ma CLA 2011-09-29 04:28:21 EDT 
Created attachment 204266 [details]
New Patch

A new patch.
Comment 3 Yun Feng Ma CLA 2011-09-29 05:32:18 EDT 
Resolved.
Comment 4 Brian Svihovec CLA 2011-09-29 14:16:05 EDT 
Can you provide some details regarding the reason that these menu options were removed?  

In RBD, the References and Declarations menu options were used as a quick way to invoke an EGL Search on the selected item.  Were these removed because the underlying support for this type of search not implemented yet in EDT?  If that is the case, is there a work item to re-enable these menu options?
Comment 5 Justin Spadea CLA 2011-09-29 14:29:17 EDT 
Also, if we decide to keep this change, I'm seeing the following in the console:

!MESSAGE Cannot bind to an undefined command: plug-in='org.eclipse.edt.ide.ui', id='org.eclipse.edt.ide.ui.search.findWorkspaceReferences'
!MESSAGE Cannot bind to an undefined command: plug-in='org.eclipse.edt.ide.ui', id='org.eclipse.edt.ide.ui.search.findWorkspaceDeclarations'
Comment 6 Brian Svihovec CLA 2011-10-07 11:07:22 EDT 
Reopening due to outstanding comments.
Comment 7 Yun Feng Ma CLA 2011-10-10 05:05:16 EDT 
Created attachment 204860 [details]
Patch

Hi Brian and Justin,

I talked this with Yao and Xiao Bin, looks like these functions don't work for variables in EGL editor, maybe that's an ehancement. I will bring back these menus now. Thanks.
Comment 8 Yun Feng Ma CLA 2011-10-10 05:12:53 EDT 
Resolve this bug as invalid. Thanks.
Comment 9 Justin Spadea CLA 2011-10-10 08:51:21 EDT 
I don't think it has ever supported variables, but that would be a good enhancement for the future.
Comment 10 Lu Yao CLA 2011-10-13 02:54:25 EDT 
it works on b 1011