Attachment 69253 Details for Bug 189106 – Proposition of modifications 001

[patch] Proposition of modifications 001

patch_189106_20070530.txt (text/plain), 1.72 KB, created by Krzysztof Michalski

on 2007-05-30 07:52:58 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Krzysztof Michalski

Created: 2007-05-30 07:52:58 EDT

Size: 1.72 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.ui.workbench
>Index: Eclipse UI/org/eclipse/ui/dialogs/FilteredItemsSelectionDialog.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/dialogs/FilteredItemsSelectionDialog.java,v
>retrieving revision 1.50
>diff -u -r1.50 FilteredItemsSelectionDialog.java
>--- Eclipse UI/org/eclipse/ui/dialogs/FilteredItemsSelectionDialog.java	24 May 2007 18:08:35 -0000	1.50
>+++ Eclipse UI/org/eclipse/ui/dialogs/FilteredItemsSelectionDialog.java	30 May 2007 11:51:55 -0000
>@@ -1911,7 +1911,8 @@
> 
> 			contentProvider.addHistoryItems(itemsFilter);
> 
>-			contentProvider.refresh();
>+			if (contentProvider.getSortedItems().length > 0)
>+				contentProvider.refresh();
> 
> 			filterJob.schedule();
> 
>@@ -2705,8 +2706,8 @@
> 		 * 
> 		 * @return filtered items
> 		 */
>-		private Object[] getItems(boolean sort) {
>-			if (sort || lastSortedItems.size() != items.size()) {
>+		public Object[] getSortedItems() {
>+			if (lastSortedItems.size() != items.size()) {
> 				synchronized (lastSortedItems) {
> 					lastSortedItems.clear();
> 					lastSortedItems.addAll(items);
>@@ -2723,7 +2724,7 @@
> 		 */
> 		public void rememberResult(ItemsFilter itemsFilter) {
> 			List itemsList = Collections.synchronizedList(Arrays
>-					.asList(getItems(false)));
>+					.asList(getSortedItems()));
> 			// synchronization
> 			if (itemsFilter == filter) {
> 				lastCompletedFilter = itemsFilter;
>@@ -2882,7 +2883,7 @@
> 			}
> 
> 			// get already sorted array
>-			Object[] filteredElements = getItems(false);
>+			Object[] filteredElements = getSortedItems();
> 
> 			if (monitor != null) {
> 				monitor.worked(ticks);

Actions: View | Diff

Attachments on bug 189106: 69253 | 69280 | 69305 | 69340