Bug: warning "The method hookContextMenu()|fillContextMenu(IMenuManager) from the type J2SConsoleView is never used locally"

abego · abego · commit 76a452e88254 · 2017-06-22T18:42:12.000+02:00
diff --git a/sources/net.sf.j2s.ui/src/net/sf/j2s/ui/console/J2SConsoleView.java b/sources/net.sf.j2s.ui/src/net/sf/j2s/ui/console/J2SConsoleView.java
@@ -181,19 +181,6 @@ public void changed(LocationEvent event) {
 		});
 	}
 
-	private void hookContextMenu() {
-		MenuManager menuMgr = new MenuManager("#PopupMenu");
-		menuMgr.setRemoveAllWhenShown(true);
-		menuMgr.addMenuListener(new IMenuListener() {
-			public void menuAboutToShow(IMenuManager manager) {
-				J2SConsoleView.this.fillContextMenu(manager);
-			}
-		});
-		Menu menu = menuMgr.createContextMenu(browser);
-		browser.setMenu(menu);
-		//getSite().registerContextMenu(menuMgr, viewer);
-	}
-
 	private void contributeToActionBars() {
 		IActionBars bars = getViewSite().getActionBars();
 		fillLocalPullDown(bars.getMenuManager());
@@ -204,16 +191,10 @@ private void fillLocalPullDown(IMenuManager manager) {
 		manager.add(actionReload);
 		manager.add(new Separator());
 		manager.add(showAddressBarAction);
+// FIXME: commented-out code. To be removed after St. Olaf merge.
 //		manager.add(actionStop);
 	}
 
-	private void fillContextMenu(IMenuManager manager) {
-		manager.add(actionReload);
-//		manager.add(actionStop);
-		// Other plug-ins can contribute there actions here
-		manager.add(new Separator(IWorkbenchActionConstants.MB_ADDITIONS));
-	}
-	
 	private void fillLocalToolBar(IToolBarManager manager) {
 		manager.add(actionBack);
 		manager.add(actionForward);
