Attachment 211946 Details for Bug 350991 – Patch for jface.text to enable reordering

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

Patch for jface.text to enable reordering

bug350991_jface.text.patch (text/plain), 4.98 KB, created by Marcel Bruch

on 2012-03-02 05:14:21 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel Bruch

Created: 2012-03-02 05:14:21 EST

Size: 4.98 KB

patch

obsolete

>From fac0bca11ae075397dfa1d80a373b7b5cce82eaa Mon Sep 17 00:00:00 2001
>From: Marcel Bruch <marcel.bruch@gmail.com>
>Date: Fri, 2 Mar 2012 10:55:11 +0100
>Subject: [PATCH] [patch] https://bugs.eclipse.org/bugs/show_bug.cgi?id=350991
>
>---
> .../contentassist/CompletionProposalPopup.java     |   27 ++++++++++++++++++++
> .../jface/text/contentassist/ContentAssistant.java |   14 ++++++++++
> .../contentassist/ICompletionProposalSorter.java   |    7 +++++
> 3 files changed, 48 insertions(+), 0 deletions(-)
> create mode 100644 org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ICompletionProposalSorter.java
>
>diff --git a/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/CompletionProposalPopup.java b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/CompletionProposalPopup.java
>index 8644363..db5a55b 100644
>--- a/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/CompletionProposalPopup.java
>+++ b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/CompletionProposalPopup.java
>@@ -12,6 +12,8 @@
> package org.eclipse.jface.text.contentassist;
> 
> import java.util.ArrayList;
>+import java.util.Arrays;
>+import java.util.Comparator;
> import java.util.List;
> 
> import org.eclipse.osgi.util.TextProcessor;
>@@ -429,6 +431,8 @@ class CompletionProposalPopup implements IContentAssistListener {
> 	 */
> 	private boolean fIsColoredLabelsSupportEnabled= false;
> 
>+	private ICompletionProposalSorter fSorter;
>+
> 
> 	/**
> 	 * Creates a new completion proposal popup for the given elements.
>@@ -1096,6 +1100,8 @@ class CompletionProposalPopup implements IContentAssistListener {
> 		if (oldProposals != fFilteredProposals) // reentrant call was first - abort
> 			return;
> 
>+		sortProposals(proposals);
>+
> 		if (Helper.okToUse(fProposalTable)) {
> 			if (oldProposal instanceof ICompletionProposalExtension2 && fViewer != null)
> 				((ICompletionProposalExtension2) oldProposal).unselected(fViewer);
>@@ -1831,4 +1837,25 @@ class CompletionProposalPopup implements IContentAssistListener {
> 		return new ProposalSelectionHandler(operationCode);
> 	}
> 
>+	/**
>+	 * Sets the sorter to use when reordering is required by one of the completion engines.
>+	 * 
>+	 * @param sorter the sorter new sorter to be used. May be <code>null</code>.
>+	 * 
>+	 * @since 3.8
>+	 */
>+	public void setSorter(ICompletionProposalSorter sorter) {
>+		fSorter= sorter;
>+	}
>+
>+	private void sortProposals(final ICompletionProposal[] proposals) {
>+		if (fSorter != null) {
>+			System.out.println("sorting...");
>+			Arrays.sort(proposals, new Comparator() {
>+				public int compare(Object o1, Object o2) {
>+					return fSorter.compare((ICompletionProposal)o1, (ICompletionProposal)o2);
>+				}
>+			});
>+		}
>+	}
> }
>diff --git a/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ContentAssistant.java b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ContentAssistant.java
>index e69d019..130c754 100644
>--- a/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ContentAssistant.java
>+++ b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ContentAssistant.java
>@@ -985,6 +985,8 @@ public class ContentAssistant implements IContentAssistant, IContentAssistantExt
> 	 */
> 	private boolean fIsColoredLabelsSupportEnabled= false;
> 
>+	private ICompletionProposalSorter fSorter;
>+
> 
> 	/**
> 	 * Creates a new content assistant. The content assistant is not automatically activated,
>@@ -1375,6 +1377,8 @@ public class ContentAssistant implements IContentAssistant, IContentAssistantExt
> 		fContextInfoPopup= fContentAssistSubjectControlAdapter.createContextInfoPopup(this);
> 		fProposalPopup= fContentAssistSubjectControlAdapter.createCompletionProposalPopup(this, controller);
> 
>+		registerSorterWithProposalPopup();
>+
> 		registerHandler(SELECT_NEXT_PROPOSAL_COMMAND_ID, fProposalPopup.createProposalSelectionHandler(CompletionProposalPopup.ProposalSelectionHandler.SELECT_NEXT));
> 		registerHandler(SELECT_PREVIOUS_PROPOSAL_COMMAND_ID, fProposalPopup.createProposalSelectionHandler(CompletionProposalPopup.ProposalSelectionHandler.SELECT_PREVIOUS));
> 
>@@ -2462,4 +2466,14 @@ public class ContentAssistant implements IContentAssistant, IContentAssistantExt
> 		fIsColoredLabelsSupportEnabled= isEnabled;
> 	}
> 
>+	public void setSorter(ICompletionProposalSorter sorter) {
>+		fSorter= sorter;
>+		registerSorterWithProposalPopup();
>+	}
>+
>+	private void registerSorterWithProposalPopup() {
>+		if (fProposalPopup != null) {
>+			fProposalPopup.setSorter(fSorter);
>+		}
>+	}
> }
>diff --git a/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ICompletionProposalSorter.java b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ICompletionProposalSorter.java
>new file mode 100644
>index 0000000..859a9b5
>--- /dev/null
>+++ b/org.eclipse.jface.text/src/org/eclipse/jface/text/contentassist/ICompletionProposalSorter.java
>@@ -0,0 +1,7 @@
>+package org.eclipse.jface.text.contentassist;
>+
>+public interface ICompletionProposalSorter {
>+
>+	public int compare(ICompletionProposal p1, ICompletionProposal p2);
>+
>+}
>-- 
>1.7.5.4
>

Actions: View

Attachments on bug 350991: 211946 | 211947 | 211966 | 211967 | 211968 | 212115 | 212116 | 212117 | 212506 | 212507 | 212525 | 212588 | 212589