diff options
2 files changed, 9 insertions, 3 deletions
diff --git a/org.eclipse.search/new search/org/eclipse/search/ui/text/TextSearchQueryProvider.java b/org.eclipse.search/new search/org/eclipse/search/ui/text/TextSearchQueryProvider.java index 810d56e1d6d..ea4246f48b4 100644 --- a/org.eclipse.search/new search/org/eclipse/search/ui/text/TextSearchQueryProvider.java +++ b/org.eclipse.search/new search/org/eclipse/search/ui/text/TextSearchQueryProvider.java @@ -9,7 +9,6 @@ * Markus Schorn - initial API and implementation * Christian Walther (Indel AG) - Bug 399094: Add whole word option to file search *******************************************************************************/ - package org.eclipse.search.ui.text; import org.eclipse.core.runtime.CoreException; @@ -21,6 +20,7 @@ import org.eclipse.ui.IWorkingSet; import org.eclipse.search.internal.ui.SearchPlugin; import org.eclipse.search.ui.ISearchQuery; + /** * Abstract base class for text search query providers supplied via the <code>org.eclipse.search.textSearchQueryProvider</code> * extension point. The plug-in preference <code>org.eclipse.search.textSearchQueryProvider<code> defines the preferred @@ -66,7 +66,10 @@ public abstract class TextSearchQueryProvider { * * This implementation returns <code>false</code>, subclasses can override. * - * @return <code>true</code> if the pattern should match only whole words + * @return <code>true</code> if the pattern should match only whole words. <code>true</code> + * is not allowed if {@link #isRegExSearch()} returns <code>true</code>. In this + * case, clients can either ignore this option or throw an + * {@link IllegalArgumentException}. * @since 3.9 */ public boolean isWholeWordSearch() { diff --git a/org.eclipse.search/new search/org/eclipse/search2/internal/ui/text2/DefaultTextSearchQueryProvider.java b/org.eclipse.search/new search/org/eclipse/search2/internal/ui/text2/DefaultTextSearchQueryProvider.java index 4f7f859e2e2..8ea25125726 100644 --- a/org.eclipse.search/new search/org/eclipse/search2/internal/ui/text2/DefaultTextSearchQueryProvider.java +++ b/org.eclipse.search/new search/org/eclipse/search2/internal/ui/text2/DefaultTextSearchQueryProvider.java @@ -9,9 +9,10 @@ * Markus Schorn - initial API and implementation * Christian Walther (Indel AG) - Bug 399094: Add whole word option to file search *******************************************************************************/ - package org.eclipse.search2.internal.ui.text2; +import org.eclipse.core.runtime.Assert; + import org.eclipse.core.resources.IResource; import org.eclipse.ui.IWorkingSet; @@ -21,6 +22,7 @@ import org.eclipse.search.ui.ISearchQuery; import org.eclipse.search.ui.text.FileTextSearchScope; import org.eclipse.search.ui.text.TextSearchQueryProvider; + public class DefaultTextSearchQueryProvider extends TextSearchQueryProvider { /* (non-Javadoc) @@ -32,6 +34,7 @@ public class DefaultTextSearchQueryProvider extends TextSearchQueryProvider { boolean regEx= input.isRegExSearch(); boolean caseSensitive= input.isCaseSensitiveSearch(); boolean wholeWord= input.isWholeWordSearch(); + Assert.isLegal(!(wholeWord && regEx)); return new FileSearchQuery(text, regEx, caseSensitive, wholeWord, scope); } |