From 192beb6f39bd3116b606507984938159693ee32e Mon Sep 17 00:00:00 2001 From: Bogdan Gheorghe Date: Wed, 9 Jun 2004 14:54:28 +0000 Subject: [PATCH] Oops - forgot the change log for 66273... --- core/org.eclipse.cdt.ui/ChangeLog | 12 ++++------ .../internal/ui/search/CSearchResultPage.java | 4 +++- .../cdt/ui/CSearchResultLabelProvider.java | 23 +++++++++++++++++-- 3 files changed, 29 insertions(+), 10 deletions(-) diff --git a/core/org.eclipse.cdt.ui/ChangeLog b/core/org.eclipse.cdt.ui/ChangeLog index 3fa78c6f6b8..06854a9da92 100644 --- a/core/org.eclipse.cdt.ui/ChangeLog +++ b/core/org.eclipse.cdt.ui/ChangeLog @@ -1,12 +1,10 @@ +2004-06-09 Bogdan Gheorghe + Fix for Bug 66273 - Opening declaration causes stack overflow + + * src/org/eclipse/cdt/internal/ui/util/EditorUtility.java + ChangeLog -2004-06-08 Tanya Wolff - Fix for bug 66139 - * src/org/eclipse/cdt/internal/ui/editor/CEditorMessages.properties - Fix for bug 66136 - * src/org/eclipse/cdt/internal/ui/preferences/PreferencesMessages.properties - * src/org/eclipse/cdt/internal/ui/preferences/CFileTypesPreferencesBlock.java - 2004-06-08 Tanya Wolff Fix for Bug 63467 - choice format for displaying plural diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/search/CSearchResultPage.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/search/CSearchResultPage.java index 2dc34630ae8..aa7d15c6069 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/search/CSearchResultPage.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/search/CSearchResultPage.java @@ -154,7 +154,7 @@ public class CSearchResultPage extends AbstractTextSearchViewPage { * @see org.eclipse.search.ui.text.AbstractTextSearchViewPage#configureTableViewer(org.eclipse.jface.viewers.TableViewer) */ protected void configureTableViewer(TableViewer viewer) { - viewer.setLabelProvider(new CountLabelProvider(this, new CSearchResultLabelProvider())); + viewer.setLabelProvider(new CountLabelProvider(this, new CSearchResultLabelProvider(this))); _contentProvider=new CSearchTableContentProvider(viewer); viewer.setContentProvider(_contentProvider); setSortOrder(_currentSortOrder); @@ -248,6 +248,8 @@ public class CSearchResultPage extends AbstractTextSearchViewPage { if (getLayout() != FLAG_LAYOUT_FLAT) addGroupActions(tbm); } + + } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/CSearchResultLabelProvider.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/CSearchResultLabelProvider.java index 58e63560721..edbc220f643 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/CSearchResultLabelProvider.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/CSearchResultLabelProvider.java @@ -25,6 +25,7 @@ import org.eclipse.core.runtime.CoreException; import org.eclipse.jface.resource.ImageDescriptor; import org.eclipse.jface.viewers.LabelProvider; import org.eclipse.search.ui.ISearchResultViewEntry; +import org.eclipse.search.ui.text.Match; import org.eclipse.swt.graphics.Image; import org.eclipse.swt.graphics.Point; @@ -43,6 +44,8 @@ public class CSearchResultLabelProvider extends LabelProvider { public static final String POTENTIAL_MATCH = CSearchMessages.getString("CSearchResultLabelProvider.potentialMatch"); //$NON-NLS-1$ + private CSearchResultPage searchPage = null; + public CSearchResultLabelProvider(){ _sortOrder = SHOW_PATH; } @@ -51,8 +54,7 @@ public class CSearchResultLabelProvider extends LabelProvider { * @param page */ public CSearchResultLabelProvider(CSearchResultPage page) { - - // TODO Auto-generated constructor stub + searchPage = page; } public Image getImage( Object element ) { @@ -160,6 +162,18 @@ public class CSearchResultLabelProvider extends LabelProvider { String result = ""; //$NON-NLS-1$ String path = (resource != null ) ? resource.getFullPath().toString() : ""; //$NON-NLS-1$ + // Object key = computeGroupByKey(match); + + if (searchPage != null){ + int matchCount = searchPage.getDisplayedMatchCount(new Match(match,match.getStartOffset(),match.getEndOffset() - match.getStartOffset())); + if (matchCount < 2){ + System.out.println("Singular"); + }else{ + System.out.println("Plural"); //$NON-NLS-1$ + } + } + + switch( getOrder() ){ case SHOW_NAME_ONLY: result = match.getName(); @@ -202,6 +216,11 @@ public class CSearchResultLabelProvider extends LabelProvider { _sortOrder = orderFlag; } + public Object computeGroupByKey(IMatch match) { + + return match.getParentName() + "::" + match.getName() + " - " + match.getLocation(); //$NON-NLS-1$ //$NON-NLS-2$ + } + private int _sortOrder; private int _textFlags; private int _imageFlags;