From 29abe81d9e741de721c56d097820082d8e97ec21 Mon Sep 17 00:00:00 2001 From: Markus Schorn Date: Mon, 14 Apr 2008 14:08:14 +0000 Subject: [PATCH] Fix warnings. --- .../internal/ui/text/AbstractCScanner.java | 8 +- .../ui/text/AbstractInformationControl.java | 18 ++-- ...bstractSourceViewerInformationControl.java | 6 +- .../internal/ui/text/CAutoIndentStrategy.java | 3 +- .../cdt/internal/ui/text/CBreakIterator.java | 6 +- .../internal/ui/text/CFormattingStrategy.java | 24 +++-- .../ui/text/CHelpProviderDescriptor.java | 18 ++-- .../cdt/internal/ui/text/CHelpSettings.java | 12 +-- .../internal/ui/text/CHeuristicScanner.java | 11 +-- .../ui/text/CParameterListValidator.java | 4 +- .../ui/text/CSourceViewerConfiguration.java | 16 ++-- .../ui/text/CStringAutoIndentStrategy.java | 9 +- .../cdt/internal/ui/text/CWordFinder.java | 5 +- .../ui/text/DocumentCharacterIterator.java | 9 +- .../cdt/internal/ui/text/HTML2TextReader.java | 10 +- .../internal/ui/text/HTMLAnnotationHover.java | 5 +- .../internal/ui/text/HTMLTextPresenter.java | 13 +-- .../ui/text/SequenceCharacterIterator.java | 9 +- .../SimpleCSourceViewerConfiguration.java | 5 +- .../text/c/hover/AbstractAnnotationHover.java | 14 +-- .../ui/text/c/hover/BestMatchHover.java | 20 ++-- .../internal/ui/text/c/hover/CDocHover.java | 16 ++-- .../c/hover/CEditorTextHoverDescriptor.java | 33 +++---- .../ui/text/c/hover/CSourceHover.java | 19 ++-- .../contentassist/CCompletionProposal.java | 15 ++- .../CContentAssistProcessor.java | 19 ++-- .../ContentAssistComputerParameter.java | 10 +- .../contentassist/ContentAssistProcessor.java | 93 +++++++++---------- .../DOMCompletionProposalComputer.java | 53 +++++------ .../HelpCompletionProposalComputer.java | 12 +-- .../contentassist/HippieProposalComputer.java | 6 +- .../InclusionProposalComputer.java | 31 +++---- .../KeywordCompletionProposalComputer.java | 8 +- .../ParsingBasedProposalComputer.java | 33 ++++--- .../TemplateCompletionProposalComputer.java | 26 +++--- .../text/correction/CCorrectionAssistant.java | 43 ++++----- .../text/correction/CCorrectionProcessor.java | 36 +++---- .../CSelectAnnotationRulerAction.java | 29 +++--- .../ContributedProcessorDescriptor.java | 21 ++--- .../correction/CorrectionCommandHandler.java | 9 +- .../CorrectionCommandInstaller.java | 12 +-- .../QuickAssistLightBulbUpdater.java | 2 +- .../AbstractDocCommentProposalComputer.java | 6 +- .../text/doctools/DocCommentOwnerManager.java | 24 +++-- .../internal/ui/text/doctools/ProjectMap.java | 8 +- .../CFoldingStructureProviderRegistry.java | 8 +- .../DefaultCFoldingPreferenceBlock.java | 14 +-- .../DefaultCFoldingStructureProvider.java | 51 +++++----- .../ui/text/spelling/CSpellingProblem.java | 11 ++- .../ui/text/spelling/SpellCheckIterator.java | 14 +-- .../spelling/SpellingEngineDispatcher.java | 4 +- .../WordCompletionProposalComputer.java | 19 ++-- .../text/spelling/WordQuickFixProcessor.java | 6 +- .../engine/AbstractSpellDictionary.java | 57 +++++++----- .../engine/DefaultPhoneticHashProvider.java | 30 +++--- .../spelling/engine/DefaultSpellChecker.java | 59 ++++++------ .../spelling/engine/ISpellCheckIterator.java | 2 +- .../text/spelling/engine/ISpellChecker.java | 2 +- .../spelling/engine/ISpellDictionary.java | 2 +- .../ui/text/spelling/engine/ISpellEvent.java | 2 +- .../engine/PersistentSpellDictionary.java | 9 +- .../spelling/engine/RankedWordProposal.java | 5 +- .../ui/text/spelling/engine/SpellEvent.java | 2 +- .../ui/text/template/TemplateEngine.java | 16 ++-- .../template/TemplateVariableProcessor.java | 19 ++-- .../internal/ui/text/util/CColorManager.java | 23 ++--- .../ui/typehierarchy/THHistoryListAction.java | 12 +-- .../dialogfields/CheckedListDialogField.java | 26 +++--- 68 files changed, 581 insertions(+), 601 deletions(-) diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractCScanner.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractCScanner.java index 212baa9d44b..81d8f41657b 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractCScanner.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractCScanner.java @@ -86,8 +86,8 @@ public abstract class AbstractCScanner extends BufferedRuleBasedScanner implemen if(fTokenStore.affectsBehavior(event)) { fTokenStore.adaptToPreferenceChange(event); } - for(Iterator i= pcps.iterator(); i.hasNext(); ) { - ((IPropertyChangeParticipant)i.next()).adaptToPreferenceChange(event); + for (IPropertyChangeParticipant propertyChangeParticipant : pcps) { + propertyChangeParticipant.adaptToPreferenceChange(event); } } @@ -96,8 +96,8 @@ public abstract class AbstractCScanner extends BufferedRuleBasedScanner implemen */ public boolean affectsBehavior(PropertyChangeEvent event) { boolean result= fTokenStore.affectsBehavior(event); - for(Iterator i= pcps.iterator(); !result && i.hasNext(); ) { - result |= ((IPropertyChangeParticipant)i.next()).affectsBehavior(event); + for(Iterator i= pcps.iterator(); !result && i.hasNext(); ) { + result |= (i.next()).affectsBehavior(event); } return result; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractInformationControl.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractInformationControl.java index eefb7925f51..b7678b67f69 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractInformationControl.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractInformationControl.java @@ -96,8 +96,8 @@ public abstract class AbstractInformationControl extends PopupDialog implements private boolean hasUnfilteredChild(TreeViewer viewer, Object element) { if (element instanceof IParent) { Object[] children= ((ITreeContentProvider) viewer.getContentProvider()).getChildren(element); - for (int i= 0; i < children.length; i++) - if (select(viewer, element, children[i])) + for (Object element2 : children) + if (select(viewer, element, element2)) return true; } return false; @@ -413,21 +413,19 @@ public abstract class AbstractInformationControl extends PopupDialog implements private ICElement findElement(TreeItem[] items) { ILabelProvider labelProvider= (ILabelProvider)fTreeViewer.getLabelProvider(); - for (int i= 0; i < items.length; i++) { - Object item= items[i].getData(); + for (TreeItem item2 : items) { + Object item= item2.getData(); ICElement element= null; if (item instanceof ICElement) { element= (ICElement)item; if (fStringMatcher == null) return element; - if (element != null) { - String label= labelProvider.getText(element); - if (fStringMatcher.match(label)) - return element; - } + String label= labelProvider.getText(element); + if (fStringMatcher.match(label)) + return element; } - element= findElement(items[i].getItems()); + element= findElement(item2.getItems()); if (element != null) return element; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractSourceViewerInformationControl.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractSourceViewerInformationControl.java index 9b65f64309c..3303dc347a4 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractSourceViewerInformationControl.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/AbstractSourceViewerInformationControl.java @@ -11,7 +11,6 @@ package org.eclipse.cdt.internal.ui.text; import java.util.ArrayList; -import java.util.Iterator; import java.util.List; import org.eclipse.jface.action.IMenuManager; @@ -128,8 +127,7 @@ public abstract class AbstractSourceViewerInformationControl extends PopupDialog @Override protected Control createContents(Composite parent) { Control contents= super.createContents(parent); - for (Iterator it= fColorExclusionControls.iterator(); it.hasNext(); ) { - Control ctrl = it.next(); + for (Control ctrl : fColorExclusionControls) { ctrl.setBackground(fBackgroundColor); } return contents; @@ -207,6 +205,7 @@ public abstract class AbstractSourceViewerInformationControl extends PopupDialog */ @Override protected List getBackgroundColorExclusions() { + @SuppressWarnings("unchecked") List exclusions= super.getBackgroundColorExclusions(); exclusions.addAll(fColorExclusionControls); return exclusions; @@ -217,6 +216,7 @@ public abstract class AbstractSourceViewerInformationControl extends PopupDialog */ @Override protected List getForegroundColorExclusions() { + @SuppressWarnings("unchecked") List exclusions= super.getForegroundColorExclusions(); exclusions.addAll(fColorExclusionControls); return exclusions; diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CAutoIndentStrategy.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CAutoIndentStrategy.java index 4c5c7f27bf0..ec6fe666f22 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CAutoIndentStrategy.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CAutoIndentStrategy.java @@ -765,8 +765,7 @@ public class CAutoIndentStrategy extends DefaultIndentLineAutoEditStrategy { private int computeVisualLength(char ch) { if (ch == '\t') return getVisualTabLengthPreference(); - else - return 1; + return 1; } /** diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CBreakIterator.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CBreakIterator.java index 7e6831410e0..7b95949ddec 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CBreakIterator.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CBreakIterator.java @@ -11,11 +11,12 @@ *******************************************************************************/ package org.eclipse.cdt.internal.ui.text; -import com.ibm.icu.text.BreakIterator; import java.text.CharacterIterator; import org.eclipse.core.runtime.Assert; +import com.ibm.icu.text.BreakIterator; + /** * A C break iterator. It returns all breaks, including before and after @@ -355,8 +356,7 @@ public class CBreakIterator extends BreakIterator { public boolean isBoundary(int offset) { if (offset == getText().getBeginIndex()) return true; - else - return following(offset - 1) == offset; + return following(offset - 1) == offset; } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CFormattingStrategy.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CFormattingStrategy.java index 8ecb186fca6..d5c7a6ab2d8 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CFormattingStrategy.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CFormattingStrategy.java @@ -12,6 +12,7 @@ package org.eclipse.cdt.internal.ui.text; import java.util.LinkedList; +import java.util.Map; import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.IDocument; @@ -35,9 +36,9 @@ public class CFormattingStrategy extends ContextBasedFormattingStrategy { /** Documents to be formatted by this strategy */ - private final LinkedList fDocuments= new LinkedList(); + private final LinkedList fDocuments= new LinkedList(); /** Partitions to be formatted by this strategy */ - private final LinkedList fPartitions= new LinkedList(); + private final LinkedList fPartitions= new LinkedList(); /** * Creates a new java formatting strategy. @@ -53,17 +54,18 @@ public class CFormattingStrategy extends ContextBasedFormattingStrategy { public void format() { super.format(); - final IDocument document= (IDocument)fDocuments.removeFirst(); - final TypedPosition partition= (TypedPosition)fPartitions.removeFirst(); + final IDocument document= fDocuments.removeFirst(); + final TypedPosition partition= fPartitions.removeFirst(); if (document != null && partition != null) { try { - + @SuppressWarnings("unchecked") + final Map preferences = getPreferences(); final TextEdit edit = CodeFormatterUtil.format( CodeFormatter.K_TRANSLATION_UNIT, document.get(), partition.getOffset(), partition.getLength(), 0, TextUtilities.getDefaultLineDelimiter(document), - getPreferences()); + preferences); if (edit != null) edit.apply(document); @@ -85,8 +87,14 @@ public class CFormattingStrategy extends ContextBasedFormattingStrategy { public void formatterStarts(final IFormattingContext context) { super.formatterStarts(context); - fPartitions.addLast(context.getProperty(FormattingContextProperties.CONTEXT_PARTITION)); - fDocuments.addLast(context.getProperty(FormattingContextProperties.CONTEXT_MEDIUM)); + Object property = context.getProperty(FormattingContextProperties.CONTEXT_PARTITION); + if (property instanceof TypedPosition) { + fPartitions.addLast((TypedPosition) property); + } + property= context.getProperty(FormattingContextProperties.CONTEXT_MEDIUM); + if (property instanceof IDocument) { + fDocuments.addLast((IDocument) property); + } } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpProviderDescriptor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpProviderDescriptor.java index 3309bc2c0ef..c10b9f7676c 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpProviderDescriptor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpProviderDescriptor.java @@ -38,7 +38,7 @@ public class CHelpProviderDescriptor { final private static String ELEMENT_PROVIDER = "provider"; //$NON-NLS-1$ final private static String ATTRIBUTE_ID = "id"; //$NON-NLS-1$ - private static Map fProvidersMap = null; + private static Map fProvidersMap = null; private ICHelpProvider fHelpProvider = null; private IConfigurationElement fConfigElement; @@ -79,9 +79,9 @@ public class CHelpProviderDescriptor { return null; } - private static Map getProvidersMap(){ + private static Map getProvidersMap(){ if(fProvidersMap == null){ - fProvidersMap = new HashMap(); + fProvidersMap = new HashMap(); } return fProvidersMap; } @@ -91,9 +91,9 @@ public class CHelpProviderDescriptor { if(id == null || "".equals(id)) //$NON-NLS-1$ return null; - Map providersMap = getProvidersMap(); + Map providersMap = getProvidersMap(); try{ - ICHelpProvider provider = (ICHelpProvider)providersMap.get(id); + ICHelpProvider provider = providersMap.get(id); if(provider == null){ provider = (ICHelpProvider)element.createExecutableExtension(CLASS); providersMap.put(id,provider); @@ -133,13 +133,13 @@ public class CHelpProviderDescriptor { if (provider != null && fProject != null) { ICHelpBook books[] = provider.getCHelpBooks(); if(books != null){ - List descriptorList = new ArrayList(); + List descriptorList = new ArrayList(); for(int i = 0; i < books.length; i++){ CHelpBookDescriptor des = new CHelpBookDescriptor(books[i],projectElement); if(des.matches(fProject)) descriptorList.add(des); } - fHelpBookDescriptors = (CHelpBookDescriptor[])descriptorList.toArray(new CHelpBookDescriptor[descriptorList.size()]); + fHelpBookDescriptors = descriptorList.toArray(new CHelpBookDescriptor[descriptorList.size()]); } } if(fHelpBookDescriptors == null) @@ -156,12 +156,12 @@ public class CHelpProviderDescriptor { CHelpBookDescriptor bookDescriptors[] = getCHelpBookDescriptors(); if(bookDescriptors.length == 0) return null; - List bookList = new ArrayList(); + List bookList = new ArrayList(); for(int i = 0; i < bookDescriptors.length; i++){ if(bookDescriptors[i].isEnabled() && bookDescriptors[i].matches(context)) bookList.add(bookDescriptors[i].getCHelpBook()); } - return (ICHelpBook[])bookList.toArray(new ICHelpBook[bookList.size()]); + return bookList.toArray(new ICHelpBook[bookList.size()]); } public void serialize(Document doc, Element parentElement){ diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpSettings.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpSettings.java index b109d885de4..cd76d5e341b 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpSettings.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHelpSettings.java @@ -93,13 +93,13 @@ public class CHelpSettings { if(providerDescriptors.length == 0) return new CHelpBookDescriptor[0]; - List bookList = new ArrayList(); + List bookList = new ArrayList(); for(int i = 0; i < providerDescriptors.length; i++){ CHelpBookDescriptor bookDescriptors[] = providerDescriptors[i].getCHelpBookDescriptors(); if(bookDescriptors.length != 0) bookList.addAll(Arrays.asList(bookDescriptors)); } - return (CHelpBookDescriptor[])bookList.toArray(new CHelpBookDescriptor[bookList.size()]); + return bookList.toArray(new CHelpBookDescriptor[bookList.size()]); } private static IConfigurationElement[] getConfigElements(){ @@ -129,7 +129,7 @@ public class CHelpSettings { public IFunctionSummary[] getMatchingFunctions(ICHelpInvocationContext context, String frag){ CHelpProviderDescriptor providerDescriptors[] = getCHelpProviderDescriptors(); - List sumaryList = new ArrayList(); + List sumaryList = new ArrayList(); for(int i = 0; i < providerDescriptors.length; i++){ ICHelpBook books[] = providerDescriptors[i].getEnabledMatchedCHelpBooks(context); if(books != null && books.length != 0){ @@ -144,12 +144,12 @@ public class CHelpSettings { if(sumaryList.size() == 0) return null; - return (IFunctionSummary[])sumaryList.toArray(new IFunctionSummary[sumaryList.size()]); + return sumaryList.toArray(new IFunctionSummary[sumaryList.size()]); } public ICHelpResourceDescriptor[] getHelpResources(ICHelpInvocationContext context, String name){ CHelpProviderDescriptor providerDescriptors[] = getCHelpProviderDescriptors(); - List resourcesList = new ArrayList(); + List resourcesList = new ArrayList(); for(int i = 0; i < providerDescriptors.length; i++){ ICHelpBook books[] = providerDescriptors[i].getEnabledMatchedCHelpBooks(context); if(books != null && books.length != 0){ @@ -164,7 +164,7 @@ public class CHelpSettings { if(resourcesList.size() == 0) return null; - return (ICHelpResourceDescriptor[])resourcesList.toArray(new ICHelpResourceDescriptor[resourcesList.size()]); + return resourcesList.toArray(new ICHelpResourceDescriptor[resourcesList.size()]); } public void serialize(Document doc, Element parentElement){ diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java index a9f5fdd3d18..2d632d723ea 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CHeuristicScanner.java @@ -409,10 +409,9 @@ public final class CHeuristicScanner implements Symbols { return getToken(identOrKeyword); - } else { - // operators, number literals etc - return TokenOTHER; } + // operators, number literals etc + return TokenOTHER; } /** @@ -511,10 +510,9 @@ public final class CHeuristicScanner implements Symbols { return getToken(identOrKeyword); - } else { - // operators, number literals etc - return TokenOTHER; } + // operators, number literals etc + return TokenOTHER; } @@ -1084,6 +1082,7 @@ public final class CHeuristicScanner implements Symbols { * bound < start, or UNBOUND * @return true if the current position looks like a composite type definition */ + @SuppressWarnings("fallthrough") public boolean looksLikeCompositeTypeDefinitionBackward(int start, int bound) { int token= previousToken(start - 1, bound); switch (token) { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CParameterListValidator.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CParameterListValidator.java index 283f59896c6..1125c11b783 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CParameterListValidator.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CParameterListValidator.java @@ -217,7 +217,7 @@ public class CParameterListValidator implements IContextInformationValidator, IC private int[] computeCommaPositions(String code) { final int length= code.length(); int pos= 0; - List positions= new ArrayList(); + List positions= new ArrayList(); positions.add(new Integer(-1)); while (pos < length && pos != -1) { char ch= code.charAt(pos); @@ -244,7 +244,7 @@ public class CParameterListValidator implements IContextInformationValidator, IC int[] fields= new int[positions.size()]; for (int i= 0; i < fields.length; i++) - fields[i]= ((Integer) positions.get(i)).intValue(); + fields[i]= positions.get(i).intValue(); return fields; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CSourceViewerConfiguration.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CSourceViewerConfiguration.java index 5a019090b86..ea2ac434ef7 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CSourceViewerConfiguration.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CSourceViewerConfiguration.java @@ -21,6 +21,7 @@ import org.eclipse.core.resources.IFile; import org.eclipse.core.resources.IProject; import org.eclipse.core.resources.IResource; import org.eclipse.core.runtime.CoreException; +import org.eclipse.core.runtime.IAdaptable; import org.eclipse.core.runtime.IPath; import org.eclipse.core.runtime.NullProgressMonitor; import org.eclipse.core.runtime.content.IContentType; @@ -642,10 +643,10 @@ public class CSourceViewerConfiguration extends TextSourceViewerConfiguration { CEditorTextHoverDescriptor[] hoverDescs= CUIPlugin.getDefault().getCEditorTextHoverDescriptors(); int stateMasks[]= new int[hoverDescs.length]; int stateMasksLength= 0; - for (int i= 0; i < hoverDescs.length; i++) { - if (hoverDescs[i].isEnabled()) { + for (CEditorTextHoverDescriptor hoverDesc : hoverDescs) { + if (hoverDesc.isEnabled()) { int j= 0; - int stateMask= hoverDescs[i].getStateMask(); + int stateMask= hoverDesc.getStateMask(); while (j < stateMasksLength) { if (stateMasks[j] == stateMask) break; @@ -784,8 +785,8 @@ public class CSourceViewerConfiguration extends TextSourceViewerConfiguration { // Register information provider IInformationProvider provider= new CInformationProvider(getEditor()); String[] contentTypes= getConfiguredContentTypes(sourceViewer); - for (int i= 0; i < contentTypes.length; i++) - presenter.setInformationProvider(provider, contentTypes[i]); + for (String contentType : contentTypes) + presenter.setInformationProvider(provider, contentType); presenter.setSizeConstraints(60, 10, true, true); return presenter; @@ -975,8 +976,9 @@ public class CSourceViewerConfiguration extends TextSourceViewerConfiguration { * @see org.eclipse.jface.text.source.SourceViewerConfiguration#getHyperlinkDetectorTargets(org.eclipse.jface.text.source.ISourceViewer) */ @Override - protected Map getHyperlinkDetectorTargets(ISourceViewer sourceViewer) { - Map targets= super.getHyperlinkDetectorTargets(sourceViewer); + protected Map getHyperlinkDetectorTargets(ISourceViewer sourceViewer) { + @SuppressWarnings("unchecked") + Map targets= super.getHyperlinkDetectorTargets(sourceViewer); targets.put("org.eclipse.cdt.ui.cCode", fTextEditor); //$NON-NLS-1$ return targets; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CStringAutoIndentStrategy.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CStringAutoIndentStrategy.java index aeb50bf5c1d..c0fa0532ea3 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CStringAutoIndentStrategy.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CStringAutoIndentStrategy.java @@ -13,14 +13,12 @@ package org.eclipse.cdt.internal.ui.text; import org.eclipse.jface.preference.IPreferenceStore; - import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.DefaultIndentLineAutoEditStrategy; import org.eclipse.jface.text.DocumentCommand; import org.eclipse.jface.text.IDocument; import org.eclipse.jface.text.ITypedRegion; import org.eclipse.jface.text.TextUtilities; - import org.eclipse.ui.IEditorPart; import org.eclipse.ui.IWorkbenchPage; import org.eclipse.ui.part.MultiPageEditorPart; @@ -60,11 +58,10 @@ public class CStringAutoIndentStrategy extends DefaultIndentLineAutoEditStrategy buffer.append(indentation); buffer.append("\""); //$NON-NLS-1$ continue; - } else { - buffer.append("\"" + delimiter); //$NON-NLS-1$ - buffer.append(indentation); - buffer.append("\""); //$NON-NLS-1$ } + buffer.append("\"" + delimiter); //$NON-NLS-1$ + buffer.append(indentation); + buffer.append("\""); //$NON-NLS-1$ } else { continue; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CWordFinder.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CWordFinder.java index a0dace624e2..8f4033d0bdd 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CWordFinder.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/CWordFinder.java @@ -278,8 +278,9 @@ public class CWordFinder { * @return true if curled brace found. */ public static boolean hasCBraces (String s) { - if (s.indexOf(CBRACE_L) > -1 || s.indexOf(CBRACE_R) > -1) return true; - else return false; + if (s.indexOf(CBRACE_L) > -1 || s.indexOf(CBRACE_R) > -1) + return true; + return false; } } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/DocumentCharacterIterator.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/DocumentCharacterIterator.java index 1208eaf5763..395dd6fe9b0 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/DocumentCharacterIterator.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/DocumentCharacterIterator.java @@ -97,8 +97,7 @@ public class DocumentCharacterIterator implements CharacterIterator, CharSequenc public char last() { if (fFirst == fLast) return setIndex(getEndIndex()); - else - return setIndex(getEndIndex() - 1); + return setIndex(getEndIndex() - 1); } /* @@ -127,9 +126,8 @@ public class DocumentCharacterIterator implements CharacterIterator, CharSequenc public char previous() { if (fIndex > getBeginIndex()) { return setIndex(fIndex - 1); - } else { - return DONE; } + return DONE; } /* @@ -204,8 +202,7 @@ public class DocumentCharacterIterator implements CharacterIterator, CharSequenc // ignore and return DONE return DONE; } - else - throw new IndexOutOfBoundsException(); + throw new IndexOutOfBoundsException(); } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTML2TextReader.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTML2TextReader.java index 644df51c336..0f918c98d6c 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTML2TextReader.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTML2TextReader.java @@ -31,12 +31,12 @@ import org.eclipse.swt.custom.StyleRange; * the tags or cut them out. */ public class HTML2TextReader extends SubstitutionTextReader { - private static final Map fgEntityLookup; - private static final Set fgTags; + private static final Map fgEntityLookup; + private static final Set fgTags; static { - fgTags= new HashSet(); + fgTags= new HashSet(); fgTags.add("b"); //$NON-NLS-1$ fgTags.add("br"); //$NON-NLS-1$ fgTags.add("h5"); //$NON-NLS-1$ @@ -47,7 +47,7 @@ public class HTML2TextReader extends SubstitutionTextReader { fgTags.add("li"); //$NON-NLS-1$ fgTags.add("ul"); //$NON-NLS-1$ - fgEntityLookup= new HashMap(7); + fgEntityLookup= new HashMap(7); fgEntityLookup.put("lt", "<"); //$NON-NLS-1$ //$NON-NLS-2$ fgEntityLookup.put("gt", ">"); //$NON-NLS-1$ //$NON-NLS-2$ fgEntityLookup.put("nbsp", " "); //$NON-NLS-1$ //$NON-NLS-2$ @@ -230,7 +230,7 @@ public class HTML2TextReader extends SubstitutionTextReader { } catch (NumberFormatException e) { } } else { - String str= (String) fgEntityLookup.get(symbol); + String str= fgEntityLookup.get(symbol); if (str != null) { return str; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLAnnotationHover.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLAnnotationHover.java index b3d30e7e888..59521dd800b 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLAnnotationHover.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLAnnotationHover.java @@ -40,6 +40,7 @@ public class HTMLAnnotationHover extends DefaultAnnotationHover { /* * Formats several message as HTML text. */ + @SuppressWarnings("unchecked") @Override protected String formatMultipleMessages(List messages) { StringBuffer buffer= new StringBuffer(); @@ -47,9 +48,9 @@ public class HTMLAnnotationHover extends DefaultAnnotationHover { HTMLPrinter.addParagraph(buffer, HTMLPrinter.convertToHTMLContent(CUIMessages.getString("CAnnotationHover.multipleMarkers"))); //$NON-NLS-1$ HTMLPrinter.startBulletList(buffer); - Iterator e= messages.iterator(); + Iterator e= messages.iterator(); while (e.hasNext()) - HTMLPrinter.addBullet(buffer, HTMLPrinter.convertToHTMLContent((String) e.next())); + HTMLPrinter.addBullet(buffer, HTMLPrinter.convertToHTMLContent(e.next())); HTMLPrinter.endBulletList(buffer); HTMLPrinter.addPageEpilog(buffer); diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLTextPresenter.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLTextPresenter.java index be0164c4af5..7f1e33b7752 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLTextPresenter.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/HTMLTextPresenter.java @@ -14,16 +14,16 @@ package org.eclipse.cdt.internal.ui.text; import java.io.IOException; import java.io.Reader; import java.io.StringReader; - import java.util.Iterator; + +import org.eclipse.jface.text.DefaultInformationControl; +import org.eclipse.jface.text.Region; +import org.eclipse.jface.text.TextPresentation; import org.eclipse.swt.custom.StyleRange; import org.eclipse.swt.graphics.GC; import org.eclipse.swt.widgets.Display; import org.eclipse.cdt.ui.CUIPlugin; -import org.eclipse.jface.text.DefaultInformationControl; -import org.eclipse.jface.text.Region; -import org.eclipse.jface.text.TextPresentation; public class HTMLTextPresenter implements DefaultInformationControl.IInformationPresenter { @@ -52,10 +52,11 @@ public class HTMLTextPresenter implements DefaultInformationControl.IInformation int yoursEnd= offset + insertLength -1; yoursEnd= Math.max(yoursStart, yoursEnd); - Iterator e= presentation.getAllStyleRangeIterator(); + @SuppressWarnings("unchecked") + Iterator e= presentation.getAllStyleRangeIterator(); while (e.hasNext()) { - StyleRange range= (StyleRange) e.next(); + StyleRange range= e.next(); int myStart= range.start; int myEnd= range.start + range.length -1; diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SequenceCharacterIterator.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SequenceCharacterIterator.java index f395e8a16b1..204abbcf416 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SequenceCharacterIterator.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SequenceCharacterIterator.java @@ -87,8 +87,7 @@ public class SequenceCharacterIterator implements CharacterIterator { public char last() { if (fFirst == fLast) return setIndex(getEndIndex()); - else - return setIndex(getEndIndex() - 1); + return setIndex(getEndIndex() - 1); } /* @@ -97,8 +96,7 @@ public class SequenceCharacterIterator implements CharacterIterator { public char current() { if (fIndex >= fFirst && fIndex < fLast) return fSequence.charAt(fIndex); - else - return DONE; + return DONE; } /* @@ -114,9 +112,8 @@ public class SequenceCharacterIterator implements CharacterIterator { public char previous() { if (fIndex > getBeginIndex()) { return setIndex(fIndex - 1); - } else { - return DONE; } + return DONE; } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SimpleCSourceViewerConfiguration.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SimpleCSourceViewerConfiguration.java index 4a9e78214f2..7cd309d0568 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SimpleCSourceViewerConfiguration.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/SimpleCSourceViewerConfiguration.java @@ -13,7 +13,6 @@ package org.eclipse.cdt.internal.ui.text; import org.eclipse.jface.preference.IPreferenceStore; - import org.eclipse.jface.text.IAutoEditStrategy; import org.eclipse.jface.text.IInformationControlCreator; import org.eclipse.jface.text.ITextHover; @@ -22,7 +21,6 @@ import org.eclipse.jface.text.hyperlink.IHyperlinkDetector; import org.eclipse.jface.text.information.IInformationPresenter; import org.eclipse.jface.text.source.IAnnotationHover; import org.eclipse.jface.text.source.ISourceViewer; - import org.eclipse.ui.texteditor.ITextEditor; @@ -108,8 +106,7 @@ public class SimpleCSourceViewerConfiguration extends CSourceViewerConfiguration public IContentFormatter getContentFormatter(ISourceViewer sourceViewer) { if (fConfigureFormatter) return super.getContentFormatter(sourceViewer); - else - return null; + return null; } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/AbstractAnnotationHover.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/AbstractAnnotationHover.java index 7ac5daf6cc2..5e0aec27ebb 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/AbstractAnnotationHover.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/AbstractAnnotationHover.java @@ -13,10 +13,6 @@ package org.eclipse.cdt.internal.ui.text.c.hover; import java.util.Iterator; -import org.eclipse.cdt.internal.ui.editor.CAnnotationIterator; -import org.eclipse.cdt.internal.ui.editor.CEditor; -import org.eclipse.cdt.internal.ui.text.HTMLPrinter; -import org.eclipse.cdt.ui.CUIPlugin; import org.eclipse.jface.preference.IPreferenceStore; import org.eclipse.jface.text.IRegion; import org.eclipse.jface.text.ITextViewer; @@ -29,6 +25,12 @@ import org.eclipse.ui.texteditor.AnnotationPreference; import org.eclipse.ui.texteditor.DefaultMarkerAnnotationAccess; import org.eclipse.ui.texteditor.IDocumentProvider; +import org.eclipse.cdt.ui.CUIPlugin; + +import org.eclipse.cdt.internal.ui.editor.CAnnotationIterator; +import org.eclipse.cdt.internal.ui.editor.CEditor; +import org.eclipse.cdt.internal.ui.text.HTMLPrinter; + /** * AbstractAnnotationHover * Abstract super class for annotation hovers. @@ -68,11 +70,11 @@ public class AbstractAnnotationHover extends AbstractCEditorTextHover { IAnnotationModel model= provider.getAnnotationModel(getEditor().getEditorInput()); if (model != null) { - Iterator e= new CAnnotationIterator(model, true, fAllAnnotations); + Iterator e= new CAnnotationIterator(model, true, fAllAnnotations); int layer= -1; String message= null; while (e.hasNext()) { - Annotation a= (Annotation) e.next(); + Annotation a= e.next(); AnnotationPreference preference= getAnnotationPreference(a); if (preference == null || !(preference.getTextPreferenceKey() != null && fStore.getBoolean(preference.getTextPreferenceKey()) || (preference.getHighlightPreferenceKey() != null && fStore.getBoolean(preference.getHighlightPreferenceKey())))) diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/BestMatchHover.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/BestMatchHover.java index f713d927440..f6e7e88bfda 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/BestMatchHover.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/BestMatchHover.java @@ -33,8 +33,8 @@ import org.eclipse.ui.IEditorPart; */ public class BestMatchHover extends AbstractCEditorTextHover { - private List fTextHoverSpecifications; - private List fInstantiatedTextHovers; + private List fTextHoverSpecifications; + private List fInstantiatedTextHovers; private ITextHover fBestHover; public BestMatchHover() { @@ -52,8 +52,8 @@ public class BestMatchHover extends AbstractCEditorTextHover { private void installTextHovers() { // initialize lists - indicates that the initialization happened - fTextHoverSpecifications= new ArrayList(2); - fInstantiatedTextHovers= new ArrayList(2); + fTextHoverSpecifications= new ArrayList(2); + fInstantiatedTextHovers= new ArrayList(2); // populate list CEditorTextHoverDescriptor[] hoverDescs= CUIPlugin.getDefault().getCEditorTextHoverDescriptors(); @@ -68,8 +68,8 @@ public class BestMatchHover extends AbstractCEditorTextHover { if (fTextHoverSpecifications.size() == 0) return; - for (Iterator iterator= new ArrayList(fTextHoverSpecifications).iterator(); iterator.hasNext(); ) { - CEditorTextHoverDescriptor spec= (CEditorTextHoverDescriptor) iterator.next(); + for (Iterator iterator= new ArrayList(fTextHoverSpecifications).iterator(); iterator.hasNext(); ) { + CEditorTextHoverDescriptor spec= iterator.next(); ICEditorTextHover hover= spec.createTextHover(); if (hover != null) { @@ -97,8 +97,8 @@ public class BestMatchHover extends AbstractCEditorTextHover { if (fInstantiatedTextHovers == null) return null; - for (Iterator iterator= fInstantiatedTextHovers.iterator(); iterator.hasNext(); ) { - ITextHover hover= (ITextHover)iterator.next(); + for (Iterator iterator= fInstantiatedTextHovers.iterator(); iterator.hasNext(); ) { + ITextHover hover= iterator.next(); String s= hover.getHoverInfo(textViewer, hoverRegion); if (s != null && s.trim().length() > 0) { @@ -122,8 +122,8 @@ public class BestMatchHover extends AbstractCEditorTextHover { if (fInstantiatedTextHovers == null) return null; - for (Iterator iterator= fInstantiatedTextHovers.iterator(); iterator.hasNext(); ) { - ITextHover hover= (ITextHover)iterator.next(); + for (Iterator iterator= fInstantiatedTextHovers.iterator(); iterator.hasNext(); ) { + ITextHover hover= iterator.next(); if (hover instanceof ITextHoverExtension2) { Object info= ((ITextHoverExtension2) hover).getHoverInfo2(textViewer, hoverRegion); diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CDocHover.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CDocHover.java index c95d3a49b88..8293978eb00 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CDocHover.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CDocHover.java @@ -119,14 +119,16 @@ public class CDocHover extends AbstractCEditorTextHover { */ @Override public IRegion getHoverRegion(ITextViewer viewer, int offset) { - Point selectedRange = viewer.getSelectedRange(); - if (selectedRange.x >= 0 && - selectedRange.y > 0 && - offset >= selectedRange.x && - offset <= selectedRange.x + selectedRange.y) - return new Region( selectedRange.x, selectedRange.y ); - if (viewer != null) + if (viewer != null) { + Point selectedRange = viewer.getSelectedRange(); + if (selectedRange.x >= 0 && + selectedRange.y > 0 && + offset >= selectedRange.x && + offset <= selectedRange.x + selectedRange.y) + return new Region( selectedRange.x, selectedRange.y ); + return CWordFinder.findWord(viewer.getDocument(), offset); + } return null; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CEditorTextHoverDescriptor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CEditorTextHoverDescriptor.java index 0dae58e800e..66725ca8ad2 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CEditorTextHoverDescriptor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CEditorTextHoverDescriptor.java @@ -18,24 +18,26 @@ import java.util.HashMap; import java.util.List; import java.util.StringTokenizer; -import org.eclipse.cdt.internal.ui.util.EditorUtility; -import org.eclipse.cdt.ui.CUIPlugin; -import org.eclipse.cdt.ui.PreferenceConstants; -import org.eclipse.cdt.ui.text.c.hover.ICEditorTextHover; +import org.eclipse.core.runtime.Assert; import org.eclipse.core.runtime.CoreException; import org.eclipse.core.runtime.IConfigurationElement; import org.eclipse.core.runtime.IExtensionRegistry; import org.eclipse.core.runtime.IStatus; import org.eclipse.core.runtime.Platform; import org.eclipse.core.runtime.Status; -import org.eclipse.core.runtime.Assert; import org.eclipse.swt.SWT; import org.osgi.framework.Bundle; +import org.eclipse.cdt.ui.CUIPlugin; +import org.eclipse.cdt.ui.PreferenceConstants; +import org.eclipse.cdt.ui.text.c.hover.ICEditorTextHover; + +import org.eclipse.cdt.internal.ui.util.EditorUtility; + /** * CEditorTexHoverDescriptor */ -public class CEditorTextHoverDescriptor implements Comparable { +public class CEditorTextHoverDescriptor implements Comparable { private static final String C_EDITOR_TEXT_HOVER_EXTENSION_POINT= "org.eclipse.cdt.ui.textHovers"; //$NON-NLS-1$ private static final String HOVER_TAG= "hover"; //$NON-NLS-1$ @@ -182,28 +184,27 @@ public class CEditorTextHoverDescriptor implements Comparable { /** * {@inheritDoc} */ - public int compareTo(Object o) { - return Collator.getInstance().compare(getLabel(), ((CEditorTextHoverDescriptor)o).getLabel()); + public int compareTo(CEditorTextHoverDescriptor o) { + return Collator.getInstance().compare(getLabel(), o.getLabel()); } private static CEditorTextHoverDescriptor[] createDescriptors(IConfigurationElement[] elements) { - List result= new ArrayList(elements.length); - for (int i= 0; i < elements.length; i++) { - IConfigurationElement element= elements[i]; + List result= new ArrayList(elements.length); + for (IConfigurationElement element : elements) { if (HOVER_TAG.equals(element.getName())) { CEditorTextHoverDescriptor desc= new CEditorTextHoverDescriptor(element); result.add(desc); } } Collections.sort(result); - return (CEditorTextHoverDescriptor[])result.toArray(new CEditorTextHoverDescriptor[result.size()]); + return result.toArray(new CEditorTextHoverDescriptor[result.size()]); } private static void initializeFromPreferences(CEditorTextHoverDescriptor[] hovers) { String compiledTextHoverModifiers= CUIPlugin.getDefault().getPreferenceStore().getString(PreferenceConstants.EDITOR_TEXT_HOVER_MODIFIERS); StringTokenizer tokenizer= new StringTokenizer(compiledTextHoverModifiers, VALUE_SEPARATOR); - HashMap idToModifier= new HashMap(tokenizer.countTokens() / 2); + HashMap idToModifier= new HashMap(tokenizer.countTokens() / 2); while (tokenizer.hasMoreTokens()) { String id= tokenizer.nextToken(); @@ -214,7 +215,7 @@ public class CEditorTextHoverDescriptor implements Comparable { String compiledTextHoverModifierMasks= CUIPlugin.getDefault().getPreferenceStore().getString(PreferenceConstants.EDITOR_TEXT_HOVER_MODIFIER_MASKS); tokenizer= new StringTokenizer(compiledTextHoverModifierMasks, VALUE_SEPARATOR); - HashMap idToModifierMask= new HashMap(tokenizer.countTokens() / 2); + HashMap idToModifierMask= new HashMap(tokenizer.countTokens() / 2); while (tokenizer.hasMoreTokens()) { String id= tokenizer.nextToken(); @@ -223,7 +224,7 @@ public class CEditorTextHoverDescriptor implements Comparable { } for (int i= 0; i < hovers.length; i++) { - String modifierString= (String)idToModifier.get(hovers[i].getId()); + String modifierString= idToModifier.get(hovers[i].getId()); boolean enabled= true; if (modifierString == null) modifierString= DISABLED_TAG; @@ -242,7 +243,7 @@ public class CEditorTextHoverDescriptor implements Comparable { if (hovers[i].fStateMask == -1) { // Fallback: use stored modifier masks try { - hovers[i].fStateMask= Integer.parseInt((String)idToModifierMask.get(hovers[i].getId())); + hovers[i].fStateMask= Integer.parseInt(idToModifierMask.get(hovers[i].getId())); } catch (NumberFormatException ex) { hovers[i].fStateMask= -1; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CSourceHover.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CSourceHover.java index 3126f445618..cc8e31cfbc3 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CSourceHover.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/c/hover/CSourceHover.java @@ -188,9 +188,9 @@ public class CSourceHover extends AbstractCEditorTextHover { IASTPreprocessorMacroDefinition[] macroDefs; final IASTPreprocessorMacroDefinition[] localMacroDefs= ast.getMacroDefinitions(); for (macroDefs= localMacroDefs; macroDefs != null; macroDefs= (macroDefs == localMacroDefs) ? ast.getBuiltinMacroDefinitions() : null) { - for (int i = 0; i < macroDefs.length; i++) { - if (Arrays.equals(macroDefs[i].getName().toCharArray(), macroName)) { - macroDef= macroDefs[i]; + for (IASTPreprocessorMacroDefinition macroDef2 : macroDefs) { + if (Arrays.equals(macroDef2.getName().toCharArray(), macroName)) { + macroDef= macroDef2; break; } } @@ -244,8 +244,8 @@ public class CSourceHover extends AbstractCEditorTextHover { names= findDeclarations(ast, binding); } if (names.length > 0) { - for (int i = 0; i < names.length; i++) { - String source= computeSourceForName(names[0], binding); + for (IName name : names) { + String source= computeSourceForName(name, binding); if (source != null) { return source; } @@ -417,9 +417,7 @@ public class CSourceHover extends AbstractCEditorTextHover { type= ((ITypedef)binding).getType(); } else if (binding instanceof IVariable) { type= ((IVariable)binding).getType(); - } else { - type= null; - } + } } catch (DOMException exc) { } expectClosingBrace= type instanceof ICompositeType || type instanceof IEnumeration; @@ -786,8 +784,7 @@ public class CSourceHover extends AbstractCEditorTextHover { i= 0; } finally { try { - if (reader != null) - reader.close(); + reader.close(); } catch (IOException ex) { CUIPlugin.log(ex); } @@ -831,7 +828,7 @@ public class CSourceHover extends AbstractCEditorTextHover { * name is not considered a keyword */ private boolean selectionIsKeyword(String name) { - Set keywords= ParserFactory.getKeywordSet(KeywordSetKey.KEYWORDS, ParserLanguage.CPP); + Set keywords= ParserFactory.getKeywordSet(KeywordSetKey.KEYWORDS, ParserLanguage.CPP); return keywords.contains(name); } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CCompletionProposal.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CCompletionProposal.java index 7dc16f6c915..d32de99487e 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CCompletionProposal.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CCompletionProposal.java @@ -12,12 +12,9 @@ package org.eclipse.cdt.internal.ui.text.contentassist; -import org.eclipse.cdt.internal.ui.text.CTextTools; -import org.eclipse.cdt.ui.CUIPlugin; -import org.eclipse.cdt.ui.text.ICCompletionProposal; +import org.eclipse.core.runtime.Assert; import org.eclipse.jface.preference.IPreferenceStore; import org.eclipse.jface.preference.PreferenceConverter; -import org.eclipse.core.runtime.Assert; import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.BadPositionCategoryException; import org.eclipse.jface.text.DefaultPositionUpdater; @@ -52,6 +49,11 @@ import org.eclipse.swt.graphics.Point; import org.eclipse.swt.graphics.RGB; import org.eclipse.ui.texteditor.link.EditorLinkedModeUI; +import org.eclipse.cdt.ui.CUIPlugin; +import org.eclipse.cdt.ui.text.ICCompletionProposal; + +import org.eclipse.cdt.internal.ui.text.CTextTools; + public class CCompletionProposal implements ICCompletionProposal, ICompletionProposalExtension, ICompletionProposalExtension2, ICompletionProposalExtension3 { private String fDisplayString; @@ -510,7 +512,10 @@ public class CCompletionProposal implements ICCompletionProposal, ICompletionPro * false otherwise. */ protected boolean startsWith(IDocument document, int offset, String word) { - int wordLength= word == null ? 0 : word.length(); + if (word == null) + return false; + + final int wordLength= word.length(); if (offset >= fReplacementOffset + wordLength) return false; diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CContentAssistProcessor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CContentAssistProcessor.java index e346031556e..1cc1f0a3c11 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CContentAssistProcessor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/CContentAssistProcessor.java @@ -15,7 +15,6 @@ package org.eclipse.cdt.internal.ui.text.contentassist; import java.util.ArrayList; import java.util.Arrays; -import java.util.Iterator; import java.util.List; import org.eclipse.core.runtime.CoreException; @@ -148,14 +147,14 @@ public class CContentAssistProcessor extends ContentAssistProcessor { * @see org.eclipse.cdt.internal.ui.text.contentassist.ContentAssistProcessor#filterAndSort(java.util.List, org.eclipse.core.runtime.IProgressMonitor) */ @Override - protected List filterAndSortProposals(List proposals, IProgressMonitor monitor, ContentAssistInvocationContext context) { + protected List filterAndSortProposals(List proposals, IProgressMonitor monitor, ContentAssistInvocationContext context) { IProposalFilter filter = getCompletionFilter(); ICCompletionProposal[] proposalsInput= new ICCompletionProposal[proposals.size()]; // wrap proposals which are no ICCompletionProposals boolean wrapped= false; int i=0; - for (Iterator iterator = proposals.iterator(); iterator.hasNext(); ) { - ICompletionProposal proposal= (ICompletionProposal) iterator.next(); + for (Object element : proposals) { + ICompletionProposal proposal= (ICompletionProposal) element; if (proposal instanceof ICCompletionProposal) { proposalsInput[i++]= (ICCompletionProposal)proposal; } else { @@ -177,12 +176,11 @@ public class CContentAssistProcessor extends ContentAssistProcessor { propsComp.setOrderAlphabetically(sortByAlphabet); Arrays.sort(proposalsFiltered, propsComp); } - List filteredList; + List filteredList; if (wrapped) { // unwrap again - filteredList= new ArrayList(proposalsFiltered.length); - for (int j= 0; j < proposalsFiltered.length; j++) { - ICCompletionProposal proposal= proposalsFiltered[j]; + filteredList= new ArrayList(proposalsFiltered.length); + for (ICCompletionProposal proposal : proposalsFiltered) { if (proposal instanceof CCompletionProposalWrapper) { filteredList.add(((CCompletionProposalWrapper)proposal).unwrap()); } else { @@ -190,7 +188,8 @@ public class CContentAssistProcessor extends ContentAssistProcessor { } } } else { - filteredList= Arrays.asList(proposalsFiltered); + final ICompletionProposal[] tmp= proposalsFiltered; + filteredList= Arrays.asList(tmp); } return filteredList; } @@ -222,7 +221,7 @@ public class CContentAssistProcessor extends ContentAssistProcessor { } @Override - protected List filterAndSortContextInformation(List contexts, + protected List filterAndSortContextInformation(List contexts, IProgressMonitor monitor) { return contexts; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistComputerParameter.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistComputerParameter.java index 5a016ab9c22..4df267c1f0a 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistComputerParameter.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistComputerParameter.java @@ -13,7 +13,6 @@ package org.eclipse.cdt.internal.ui.text.contentassist; import java.util.Collection; import java.util.HashMap; -import java.util.Iterator; import java.util.Map; import org.eclipse.core.commands.IParameterValues; @@ -27,11 +26,10 @@ public final class ContentAssistComputerParameter implements IParameterValues { /* * @see org.eclipse.core.commands.IParameterValues#getParameterValues() */ - public Map getParameterValues() { - Collection descriptors= CompletionProposalComputerRegistry.getDefault().getProposalCategories(); - Map map= new HashMap(descriptors.size()); - for (Iterator it= descriptors.iterator(); it.hasNext();) { - CompletionProposalCategory category= (CompletionProposalCategory) it.next(); + public Map getParameterValues() { + Collection descriptors= CompletionProposalComputerRegistry.getDefault().getProposalCategories(); + Map map= new HashMap(descriptors.size()); + for (CompletionProposalCategory category : descriptors) { map.put(category.getDisplayName(), category.getId()); } return map; diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistProcessor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistProcessor.java index 409bbb4df7d..1f497309c34 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistProcessor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ContentAssistProcessor.java @@ -16,7 +16,6 @@ package org.eclipse.cdt.internal.ui.text.contentassist; import java.util.ArrayList; import java.util.Collections; import java.util.Comparator; -import java.util.Iterator; import java.util.List; import org.eclipse.core.runtime.Assert; @@ -91,12 +90,9 @@ public class ContentAssistProcessor implements IContentAssistProcessor { */ private static final String PREF_WARN_ABOUT_EMPTY_ASSIST_CATEGORY= "EmptyDefaultAssistCategory"; //$NON-NLS-1$ - private static final Comparator ORDER_COMPARATOR= new Comparator() { + private static final Comparator ORDER_COMPARATOR= new Comparator() { - public int compare(Object o1, Object o2) { - CompletionProposalCategory d1= (CompletionProposalCategory) o1; - CompletionProposalCategory d2= (CompletionProposalCategory) o2; - + public int compare(CompletionProposalCategory d1, CompletionProposalCategory d2) { return d1.getSortOrder() - d2.getSortOrder(); } @@ -104,7 +100,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { private static final ICompletionProposal[] NO_PROPOSALS= {}; - private final List fCategories; + private final List fCategories; private final String fPartition; private final ContentAssistant fAssistant; @@ -112,7 +108,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { /* cycling stuff */ private int fRepetition= -1; - private List/*>*/ fCategoryIteration= null; + private List> fCategoryIteration= null; private String fIterationGesture= null; private int fNumberOfComputedResults= 0; private String fErrorMessage; @@ -139,8 +135,8 @@ public class ContentAssistProcessor implements IContentAssistProcessor { // this may show the warning dialog if all categories are disabled fCategoryIteration= getCategoryIteration(); - for (Iterator it= fCategories.iterator(); it.hasNext();) { - CompletionProposalCategory cat= (CompletionProposalCategory) it.next(); + for (Object element : fCategories) { + CompletionProposalCategory cat= (CompletionProposalCategory) element; cat.sessionStarted(); } @@ -172,8 +168,8 @@ public class ContentAssistProcessor implements IContentAssistProcessor { if (event.processor != ContentAssistProcessor.this) return; - for (Iterator it= fCategories.iterator(); it.hasNext();) { - CompletionProposalCategory cat= (CompletionProposalCategory) it.next(); + for (Object element : fCategories) { + CompletionProposalCategory cat= (CompletionProposalCategory) element; cat.sessionEnded(); } @@ -221,15 +217,15 @@ public class ContentAssistProcessor implements IContentAssistProcessor { long setup= DEBUG ? System.currentTimeMillis() : 0; monitor.subTask(ContentAssistMessages.ContentAssistProcessor_collecting_proposals); - List proposals= collectProposals(viewer, offset, monitor, context); + List proposals= collectProposals(viewer, offset, monitor, context); long collect= DEBUG ? System.currentTimeMillis() : 0; monitor.subTask(ContentAssistMessages.ContentAssistProcessor_sorting_proposals); - List filtered= filterAndSortProposals(proposals, monitor, context); + List filtered= filterAndSortProposals(proposals, monitor, context); fNumberOfComputedResults= filtered.size(); long filter= DEBUG ? System.currentTimeMillis() : 0; - ICompletionProposal[] result= (ICompletionProposal[]) filtered.toArray(new ICompletionProposal[filtered.size()]); + ICompletionProposal[] result= filtered.toArray(new ICompletionProposal[filtered.size()]); monitor.done(); if (DEBUG) { @@ -264,12 +260,11 @@ public class ContentAssistProcessor implements IContentAssistProcessor { fNumberOfComputedResults= 0; } - private List collectProposals(ITextViewer viewer, int offset, IProgressMonitor monitor, ContentAssistInvocationContext context) { - List proposals= new ArrayList(); - List providers= getCategories(); - for (Iterator it= providers.iterator(); it.hasNext();) { - CompletionProposalCategory cat= (CompletionProposalCategory) it.next(); - List computed= cat.computeCompletionProposals(context, fPartition, new SubProgressMonitor(monitor, 1)); + private List collectProposals(ITextViewer viewer, int offset, IProgressMonitor monitor, ContentAssistInvocationContext context) { + List proposals= new ArrayList(); + List providers= getCategories(); + for (CompletionProposalCategory cat : providers) { + List computed= cat.computeCompletionProposals(context, fPartition, new SubProgressMonitor(monitor, 1)); proposals.addAll(computed); if (fErrorMessage == null) fErrorMessage= cat.getErrorMessage(); @@ -289,7 +284,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { * @return the list of filtered and sorted proposals, ready for * display (element type: {@link ICompletionProposal}) */ - protected List filterAndSortProposals(List proposals, IProgressMonitor monitor, ContentAssistInvocationContext context) { + protected List filterAndSortProposals(List proposals, IProgressMonitor monitor, ContentAssistInvocationContext context) { return proposals; } @@ -303,26 +298,25 @@ public class ContentAssistProcessor implements IContentAssistProcessor { monitor.beginTask(ContentAssistMessages.ContentAssistProcessor_computing_contexts, fCategories.size() + 1); monitor.subTask(ContentAssistMessages.ContentAssistProcessor_collecting_contexts); - List proposals= collectContextInformation(viewer, offset, monitor); - + List proposals= collectContextInformation(viewer, offset, monitor); + monitor.subTask(ContentAssistMessages.ContentAssistProcessor_sorting_contexts); - List filtered= filterAndSortContextInformation(proposals, monitor); + List filtered= filterAndSortContextInformation(proposals, monitor); fNumberOfComputedResults= filtered.size(); - IContextInformation[] result= (IContextInformation[]) filtered.toArray(new IContextInformation[filtered.size()]); + IContextInformation[] result= filtered.toArray(new IContextInformation[filtered.size()]); monitor.done(); return result; } - private List collectContextInformation(ITextViewer viewer, int offset, IProgressMonitor monitor) { - List proposals= new ArrayList(); + private List collectContextInformation(ITextViewer viewer, int offset, IProgressMonitor monitor) { + List proposals= new ArrayList(); ContentAssistInvocationContext context= createContext(viewer, offset, false); try { - List providers= getCategories(); - for (Iterator it= providers.iterator(); it.hasNext();) { - CompletionProposalCategory cat= (CompletionProposalCategory) it.next(); - List computed= cat.computeContextInformation(context, fPartition, new SubProgressMonitor(monitor, 1)); + List providers= getCategories(); + for (CompletionProposalCategory cat : providers) { + List computed= cat.computeContextInformation(context, fPartition, new SubProgressMonitor(monitor, 1)); proposals.addAll(computed); if (fErrorMessage == null) fErrorMessage= cat.getErrorMessage(); @@ -345,7 +339,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { * @return the list of filtered and sorted proposals, ready for * display (element type: {@link IContextInformation}) */ - protected List filterAndSortContextInformation(List contexts, IProgressMonitor monitor) { + protected List filterAndSortContextInformation(List contexts, IProgressMonitor monitor) { return contexts; } @@ -426,7 +420,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { return fIsAutoActivated; } - private List getCategories() { + private List getCategories() { if (fCategoryIteration == null) return fCategories; @@ -437,22 +431,21 @@ public class ContentAssistProcessor implements IContentAssistProcessor { // fAssistant.setShowMessage(fRepetition % 2 != 0); // - return (List) fCategoryIteration.get(iteration); + return fCategoryIteration.get(iteration); } - private List getCategoryIteration() { - List sequence= new ArrayList(); + private List> getCategoryIteration() { + List> sequence= new ArrayList>(); sequence.add(getDefaultCategories()); - for (Iterator it= getSeparateCategories().iterator(); it.hasNext();) { - CompletionProposalCategory cat= (CompletionProposalCategory) it.next(); + for (CompletionProposalCategory cat : getSeparateCategories()) { sequence.add(Collections.singletonList(cat)); } return sequence; } - private List getDefaultCategories() { + private List getDefaultCategories() { // default mix - enable all included computers - List included= getDefaultCategoriesUnchecked(); + List included= getDefaultCategoriesUnchecked(); if (IDocument.DEFAULT_CONTENT_TYPE.equals(fPartition) && included.isEmpty() && !fCategories.isEmpty()) if (informUserAboutEmptyDefaultCategory()) @@ -462,10 +455,10 @@ public class ContentAssistProcessor implements IContentAssistProcessor { return included; } - private List getDefaultCategoriesUnchecked() { - List included= new ArrayList(); - for (Iterator it= fCategories.iterator(); it.hasNext();) { - CompletionProposalCategory category= (CompletionProposalCategory) it.next(); + private List getDefaultCategoriesUnchecked() { + List included= new ArrayList(); + for (Object element : fCategories) { + CompletionProposalCategory category= (CompletionProposalCategory) element; if (category.isIncluded() && category.hasComputers(fPartition)) included.add(category); } @@ -554,10 +547,10 @@ public class ContentAssistProcessor implements IContentAssistProcessor { return false; } - private List getSeparateCategories() { - ArrayList sorted= new ArrayList(); - for (Iterator it= fCategories.iterator(); it.hasNext();) { - CompletionProposalCategory category= (CompletionProposalCategory) it.next(); + private List getSeparateCategories() { + ArrayList sorted= new ArrayList(); + for (Object element : fCategories) { + CompletionProposalCategory category= (CompletionProposalCategory) element; if (category.isSeparateCommand() && category.hasComputers(fPartition)) sorted.add(category); } @@ -577,7 +570,7 @@ public class ContentAssistProcessor implements IContentAssistProcessor { int iteration= repetition % fCategoryIteration.size(); if (iteration == 0) return ContentAssistMessages.ContentAssistProcessor_defaultProposalCategory; - return toString((CompletionProposalCategory) ((List) fCategoryIteration.get(iteration)).get(0)); + return toString(fCategoryIteration.get(iteration).get(0)); } private String toString(CompletionProposalCategory category) { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/DOMCompletionProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/DOMCompletionProposalComputer.java index aded064fce6..b46dc3d156d 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/DOMCompletionProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/DOMCompletionProposalComputer.java @@ -20,6 +20,7 @@ import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.IDocument; import org.eclipse.jface.text.ITypedRegion; import org.eclipse.jface.text.TextUtilities; +import org.eclipse.jface.text.contentassist.ICompletionProposal; import org.eclipse.swt.graphics.Image; import org.eclipse.cdt.core.dom.ast.ASTTypeUtil; @@ -86,11 +87,11 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer } @Override - protected List computeCompletionProposals( + protected List computeCompletionProposals( CContentAssistInvocationContext context, IASTCompletionNode completionNode, String prefix) { - List proposals = new ArrayList(); + List proposals = new ArrayList(); if(inPreprocessorDirective(context)) { if (!inPreprocessorKeyword(context)) { @@ -183,7 +184,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer return false; } - private void addMacroProposals(CContentAssistInvocationContext context, String prefix, List proposals) { + private void addMacroProposals(CContentAssistInvocationContext context, String prefix, List proposals) { char[] prefixChars= prefix.toCharArray(); final boolean matchPrefix= !context.isContextInformationStyle(); IASTCompletionNode completionNode = context.getCompletionNode(); @@ -203,7 +204,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer } } - private void handleMacro(IASTPreprocessorMacroDefinition macro, CContentAssistInvocationContext context, String prefix, List proposals) { + private void handleMacro(IASTPreprocessorMacroDefinition macro, CContentAssistInvocationContext context, String prefix, List proposals) { final String macroName = macro.getName().toString(); final int baseRelevance= computeBaseRelevance(prefix, macroName); @@ -255,7 +256,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer CContentAssistInvocationContext cContext, String prefix, IASTCompletionContext astContext, - List proposals) { + List proposals) { if ((binding instanceof CPPImplicitFunction || binding instanceof CPPImplicitFunctionTemplate || binding instanceof CPPImplicitTypedef) @@ -293,12 +294,12 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer return name.length == 0 || name[0] == '{'; } - private void handleClass(ICPPClassType classType, IASTCompletionContext astContext, CContentAssistInvocationContext context, int baseRelevance, List proposals) { + private void handleClass(ICPPClassType classType, IASTCompletionContext astContext, CContentAssistInvocationContext context, int baseRelevance, List proposals) { if (context.isContextInformationStyle()) { try { ICPPConstructor[] constructors = classType.getConstructors(); - for (int i = 0; i < constructors.length; i++) { - handleFunction(constructors[i], context, baseRelevance, proposals); + for (ICPPConstructor constructor : constructors) { + handleFunction(constructor, context, baseRelevance, proposals); } } catch (DOMException e) { } @@ -309,10 +310,10 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer case ICPPClassType.k_class: relevance= RelevanceConstants.CLASS_TYPE_RELEVANCE; break; - case ICPPClassType.k_struct: + case ICompositeType.k_struct: relevance= RelevanceConstants.STRUCT_TYPE_RELEVANCE; break; - case ICPPClassType.k_union: + case ICompositeType.k_union: relevance= RelevanceConstants.UNION_TYPE_RELEVANCE; break; } @@ -328,7 +329,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer } } - private void handleFunction(IFunction function, CContentAssistInvocationContext context, int baseRelevance, List proposals) { + private void handleFunction(IFunction function, CContentAssistInvocationContext context, int baseRelevance, List proposals) { Image image = getImage(function); StringBuilder repStringBuff = new StringBuilder(); @@ -340,7 +341,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer String returnTypeStr = null; try { IParameter[] params = function.getParameters(); - if (params != null) + if (params != null) { for (int i = 0; i < params.length; ++i) { IType paramType = params[i].getType(); if (i > 0) { @@ -357,18 +358,18 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer } } - if (function.takesVarArgs()) { - if (params.length > 0) { - dispargs.append(','); - idargs.append(','); - } - dispargs.append("..."); //$NON-NLS-1$ - idargs.append("..."); //$NON-NLS-1$ - } else if (params.length == 0) { // force the void in - dispargs.append("void"); //$NON-NLS-1$ - idargs.append("void"); //$NON-NLS-1$ - } - + if (function.takesVarArgs()) { + if (params.length > 0) { + dispargs.append(','); + idargs.append(','); + } + dispargs.append("..."); //$NON-NLS-1$ + idargs.append("..."); //$NON-NLS-1$ + } else if (params.length == 0) { // force the void in + dispargs.append("void"); //$NON-NLS-1$ + idargs.append("void"); //$NON-NLS-1$ + } + } IFunctionType functionType = function.getType(); if (functionType != null) { IType returnType = functionType.getReturnType(); @@ -413,7 +414,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer proposals.add(proposal); } - private void handleVariable(IVariable variable, CContentAssistInvocationContext context, int baseRelevance, List proposals) { + private void handleVariable(IVariable variable, CContentAssistInvocationContext context, int baseRelevance, List proposals) { StringBuilder repStringBuff = new StringBuilder(); repStringBuff.append(variable.getName()); @@ -479,7 +480,7 @@ public class DOMCompletionProposalComputer extends ParsingBasedProposalComputer IASTCompletionContext astContext, CContentAssistInvocationContext cContext, int baseRelevance, - List proposals) { + List proposals) { if (astContext instanceof ICPPASTQualifiedName) { IASTCompletionContext parent = ((ICPPASTQualifiedName) astContext) diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HelpCompletionProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HelpCompletionProposalComputer.java index dc9f8d0e1f3..d31d4f61cfe 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HelpCompletionProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HelpCompletionProposalComputer.java @@ -17,6 +17,7 @@ import java.util.List; import org.eclipse.core.resources.IProject; import org.eclipse.core.runtime.CoreException; +import org.eclipse.jface.text.contentassist.ICompletionProposal; import org.eclipse.swt.graphics.Image; import org.eclipse.cdt.core.dom.ast.IASTCompletionNode; @@ -33,7 +34,7 @@ import org.eclipse.cdt.internal.ui.viewsupport.CElementImageProvider; public class HelpCompletionProposalComputer extends ParsingBasedProposalComputer { @Override - protected List computeCompletionProposals( + protected List computeCompletionProposals( CContentAssistInvocationContext cContext, IASTCompletionNode completionNode, String prefix) throws CoreException { @@ -58,7 +59,7 @@ public class HelpCompletionProposalComputer extends ParsingBasedProposalComputer } if (!handleHelp) { - return Collections.EMPTY_LIST; + return Collections.emptyList(); } final ITranslationUnit tu = cContext.getTranslationUnit(); @@ -77,17 +78,16 @@ public class HelpCompletionProposalComputer extends ParsingBasedProposalComputer IFunctionSummary[] summaries = CHelpProviderManager.getDefault() .getMatchingFunctions(helpContext, prefix); if (summaries == null) - return Collections.EMPTY_LIST; + return Collections.emptyList(); int repOffset = cContext.getInvocationOffset() - prefix.length(); int repLength = prefix.length(); Image image = CUIPlugin.getImageDescriptorRegistry().get( CElementImageProvider.getFunctionImageDescriptor()); - List proposals = new ArrayList(); + List proposals = new ArrayList(); - for (int j = 0; j < summaries.length; j++) { - IFunctionSummary summary = summaries[j]; + for (IFunctionSummary summary : summaries) { String fname = summary.getName() + "()"; //$NON-NLS-1$ String fdesc = summary.getDescription(); IFunctionSummary.IFunctionPrototypeSummary fproto = summary diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HippieProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HippieProposalComputer.java index 1144b0c7b4e..6c2aa73385e 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HippieProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/HippieProposalComputer.java @@ -15,6 +15,8 @@ import java.util.Arrays; import java.util.List; import org.eclipse.core.runtime.IProgressMonitor; +import org.eclipse.jface.text.contentassist.ICompletionProposal; +import org.eclipse.jface.text.contentassist.IContextInformation; import org.eclipse.ui.texteditor.HippieProposalProcessor; import org.eclipse.cdt.ui.text.contentassist.ContentAssistInvocationContext; @@ -39,14 +41,14 @@ public final class HippieProposalComputer implements ICompletionProposalComputer /* * @see org.eclipse.cdt.ui.text.contentassist.ICompletionProposalComputer#computeCompletionProposals(org.eclipse.cdt.ui.text.contentassist.ContentAssistInvocationContext, org.eclipse.core.runtime.IProgressMonitor) */ - public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { + public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { return Arrays.asList(fProcessor.computeCompletionProposals(context.getViewer(), context.getInvocationOffset())); } /* * @see org.eclipse.cdt.ui.text.contentassist.ICompletionProposalComputer#computeContextInformation(org.eclipse.cdt.ui.text.contentassist.ContentAssistInvocationContext, org.eclipse.core.runtime.IProgressMonitor) */ - public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { + public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { return Arrays.asList(fProcessor.computeContextInformation(context.getViewer(), context.getInvocationOffset())); } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/InclusionProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/InclusionProposalComputer.java index 5f6f597839e..db1ec7b27da 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/InclusionProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/InclusionProposalComputer.java @@ -31,6 +31,8 @@ import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.IDocument; import org.eclipse.jface.text.ITypedRegion; import org.eclipse.jface.text.TextUtilities; +import org.eclipse.jface.text.contentassist.ICompletionProposal; +import org.eclipse.jface.text.contentassist.IContextInformation; import org.eclipse.swt.graphics.Image; import org.eclipse.cdt.core.model.CoreModel; @@ -54,15 +56,15 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { private String fErrorMessage; - public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { - List proposals= Collections.emptyList(); + public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { + List proposals= Collections.emptyList(); fErrorMessage= null; if (context instanceof CContentAssistInvocationContext) { CContentAssistInvocationContext cContext= (CContentAssistInvocationContext) context; if (inIncludeDirective(cContext)) { // add include file proposals - proposals= new ArrayList(); + proposals= new ArrayList(); try { addInclusionProposals(cContext, proposals); } catch (Exception exc) { @@ -74,7 +76,7 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { return proposals; } - public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { + public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { return null; } @@ -113,7 +115,7 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { return false; } - private void addInclusionProposals(CContentAssistInvocationContext context, List proposals) throws Exception { + private void addInclusionProposals(CContentAssistInvocationContext context, List proposals) throws Exception { if (context.isContextInformationStyle()) { return; } @@ -130,13 +132,11 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { if (potentialIncludes.length > 0) { IInclude[] includes= context.getTranslationUnit().getIncludes(); Set alreadyIncluded= new HashSet(); - for (int i = 0; i < includes.length; i++) { - IInclude includeDirective= includes[i]; + for (IInclude includeDirective : includes) { alreadyIncluded.add(includeDirective.getElementName()); } Image image = getImage(CElementImageProvider.getIncludeImageDescriptor()); - for (int i = 0; i < potentialIncludes.length; i++) { - String include= potentialIncludes[i]; + for (String include : potentialIncludes) { if (alreadyIncluded.add(include)) { final char openingBracket= angleBrackets ? '<' : '"'; final char closingBracket= angleBrackets ? '>' : '"'; @@ -205,16 +205,16 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { String[] quoteIncludes= extendedInfo.getLocalIncludePath(); if (quoteIncludes != null) { - for (int i = 0; i < quoteIncludes.length; i++) { - IPath includeDir= new Path(quoteIncludes[i]); + for (String quoteInclude : quoteIncludes) { + IPath includeDir= new Path(quoteInclude); collectIncludeFilesFromDirectory(tu, includeDir, prefixPath, includeFiles); } } } String[] allIncludes= info.getIncludePaths(); - for (int i = 0; i < allIncludes.length; i++) { - IPath includeDir= new Path(allIncludes[i]); + for (String allInclude : allIncludes) { + IPath includeDir= new Path(allInclude); collectIncludeFilesFromDirectory(tu, includeDir, prefixPath, includeFiles); } } @@ -228,7 +228,6 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { * @param includeFiles the result list */ private void collectIncludeFilesFromDirectory(ITranslationUnit tu, IPath directory, IPath prefixPath, List includeFiles) { - final boolean isCpp= tu.isCXXLanguage(); final String namePrefix; if (prefixPath.segmentCount() == 0) { namePrefix= ""; //$NON-NLS-1$ @@ -250,8 +249,7 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { final int prefixLength = namePrefix.length(); final IProject project= tu.getCProject().getProject(); File[] files= fileDir.listFiles(); - for (int i = 0; i < files.length; i++) { - File file = files[i]; + for (File file : files) { final String name= file.getName(); if (name.length() >= prefixLength && namePrefix.equalsIgnoreCase(name.substring(0, prefixLength))) { if (file.isFile()) { @@ -275,7 +273,6 @@ public class InclusionProposalComputer implements ICompletionProposalComputer { * @throws CoreException */ private void collectIncludeFilesFromContainer(final ITranslationUnit tu, IContainer parent, IPath prefixPath, final List includeFiles) throws CoreException { - final boolean isCpp= tu.isCXXLanguage(); final String namePrefix; if (prefixPath.segmentCount() == 0) { namePrefix= ""; //$NON-NLS-1$ diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/KeywordCompletionProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/KeywordCompletionProposalComputer.java index 3ca36f22593..b4beb87b895 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/KeywordCompletionProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/KeywordCompletionProposalComputer.java @@ -21,6 +21,7 @@ import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.IDocument; import org.eclipse.jface.text.ITypedRegion; import org.eclipse.jface.text.TextUtilities; +import org.eclipse.jface.text.contentassist.ICompletionProposal; import org.eclipse.swt.graphics.Image; import org.eclipse.cdt.core.dom.ast.IASTCompletionNode; @@ -31,14 +32,13 @@ import org.eclipse.cdt.core.parser.Directives; import org.eclipse.cdt.core.parser.Keywords; import org.eclipse.cdt.ui.CUIPlugin; import org.eclipse.cdt.ui.text.ICPartitions; -import org.eclipse.cdt.ui.text.contentassist.ICompletionProposalComputer; import org.eclipse.cdt.internal.ui.viewsupport.CElementImageProvider; -public class KeywordCompletionProposalComputer extends ParsingBasedProposalComputer implements ICompletionProposalComputer { +public class KeywordCompletionProposalComputer extends ParsingBasedProposalComputer { @Override - protected List computeCompletionProposals( + protected List computeCompletionProposals( CContentAssistInvocationContext context, IASTCompletionNode completionNode, String prefix) throws CoreException { @@ -58,7 +58,7 @@ public class KeywordCompletionProposalComputer extends ParsingBasedProposalCompu // keywords are matched case-sensitive final int relevance = RelevanceConstants.CASE_MATCH_RELEVANCE + RelevanceConstants.KEYWORD_TYPE_RELEVANCE; - List proposals = new ArrayList(); + List proposals = new ArrayList(); if (inPreprocessorDirective(context)) { // TODO split this into a separate proposal computer? diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ParsingBasedProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ParsingBasedProposalComputer.java index f5c0c912e1e..0ecae2ab3db 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ParsingBasedProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/ParsingBasedProposalComputer.java @@ -13,8 +13,8 @@ package org.eclipse.cdt.internal.ui.text.contentassist; import java.util.ArrayList; +import java.util.Collection; import java.util.Collections; -import java.util.Iterator; import java.util.LinkedHashSet; import java.util.List; @@ -38,14 +38,15 @@ public abstract class ParsingBasedProposalComputer implements ICompletionProposa private String fErrorMessage = null; - public List computeCompletionProposals( + public List computeCompletionProposals( ContentAssistInvocationContext context, IProgressMonitor monitor) { try { if (context instanceof CContentAssistInvocationContext) { CContentAssistInvocationContext cContext = (CContentAssistInvocationContext) context; IASTCompletionNode completionNode = cContext.getCompletionNode(); - if (completionNode == null) return Collections.EMPTY_LIST; + if (completionNode == null) + return Collections.emptyList(); String prefix = completionNode.getPrefix(); if (prefix == null) { prefix = cContext.computeIdentifierPrefix().toString(); @@ -58,23 +59,22 @@ public abstract class ParsingBasedProposalComputer implements ICompletionProposa CUIPlugin.log(e); } - return Collections.EMPTY_LIST; + return Collections.emptyList(); } - protected abstract List computeCompletionProposals( + protected abstract List computeCompletionProposals( CContentAssistInvocationContext context, IASTCompletionNode completionNode, String prefix) throws CoreException; - public List computeContextInformation( + public List computeContextInformation( ContentAssistInvocationContext context, IProgressMonitor monitor) { - List proposals= computeCompletionProposals(context, monitor); + Collection proposals= computeCompletionProposals(context, monitor); // remove duplicates - proposals= new ArrayList(new LinkedHashSet(proposals)); - List result= new ArrayList(); - - for (Iterator it= proposals.iterator(); it.hasNext();) { - ICompletionProposal proposal= (ICompletionProposal) it.next(); + + proposals= (new LinkedHashSet(proposals)); + List result= new ArrayList(); + for (ICompletionProposal proposal : proposals) { IContextInformation contextInformation= proposal.getContextInformation(); if (contextInformation != null) { result.add(contextInformation); @@ -106,11 +106,10 @@ public abstract class ParsingBasedProposalComputer implements ICompletionProposa boolean caseMatch= prefix.length() > 0 && match.startsWith(prefix); if (caseMatch) { return RelevanceConstants.CASE_MATCH_RELEVANCE; - } else { - boolean exactNameMatch= match.equalsIgnoreCase(prefix); - if (exactNameMatch) { - return RelevanceConstants.EXACT_NAME_MATCH_RELEVANCE; - } + } + boolean exactNameMatch= match.equalsIgnoreCase(prefix); + if (exactNameMatch) { + return RelevanceConstants.EXACT_NAME_MATCH_RELEVANCE; } return 0; } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/TemplateCompletionProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/TemplateCompletionProposalComputer.java index 37fa52978d0..8ee2f02c4c1 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/TemplateCompletionProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/contentassist/TemplateCompletionProposalComputer.java @@ -18,6 +18,8 @@ import org.eclipse.core.runtime.IProgressMonitor; import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.ITextViewer; import org.eclipse.jface.text.TextUtilities; +import org.eclipse.jface.text.contentassist.ICompletionProposal; +import org.eclipse.jface.text.contentassist.IContextInformation; import org.eclipse.jface.text.templates.TemplateContextType; import org.eclipse.cdt.core.model.ITranslationUnit; @@ -51,25 +53,19 @@ public class TemplateCompletionProposalComputer implements ICompletionProposalCo contextType= new CContextType(); CUIPlugin.getDefault().getTemplateContextRegistry().addContextType(contextType); } - if (contextType != null) - fCTemplateEngine= new TemplateEngine(contextType); - else - fCTemplateEngine= null; + fCTemplateEngine= new TemplateEngine(contextType); contextType= CUIPlugin.getDefault().getTemplateContextRegistry().getContextType(CommentContextType.ID); if (contextType == null) { contextType= new CommentContextType(); CUIPlugin.getDefault().getTemplateContextRegistry().addContextType(contextType); } - if (contextType != null) - fCommentTemplateEngine= new TemplateEngine(contextType); - else - fCommentTemplateEngine= null; + fCommentTemplateEngine= new TemplateEngine(contextType); } /* * @see org.eclipse.cdt.ui.text.contentassist.ICompletionProposalComputer#computeCompletionProposals(org.eclipse.cdt.ui.text.contentassist.ContentAssistInvocationContext, org.eclipse.core.runtime.IProgressMonitor) */ - public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { + public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { ITextViewer viewer= context.getViewer(); int offset= context.getInvocationOffset(); TemplateEngine engine= null; @@ -83,23 +79,23 @@ public class TemplateCompletionProposalComputer implements ICompletionProposalCo } } } catch (BadLocationException x) { - return Collections.EMPTY_LIST; + return Collections.emptyList(); } if (engine != null && context instanceof CContentAssistInvocationContext) { CContentAssistInvocationContext cContext= (CContentAssistInvocationContext)context; ITranslationUnit tUnit = cContext.getTranslationUnit(); if (tUnit == null) { - return Collections.EMPTY_LIST; + return Collections.emptyList(); } engine.reset(); engine.complete(viewer, offset, tUnit); - List result= engine.getResults(); + List result= engine.getResults(); return result; } - return Collections.EMPTY_LIST; + return Collections.emptyList(); } /** @@ -117,8 +113,8 @@ public class TemplateCompletionProposalComputer implements ICompletionProposalCo /* * @see org.eclipse.cdt.ui.text.contentassist.ICompletionProposalComputer#computeContextInformation(org.eclipse.cdt.ui.text.contentassist.ContentAssistInvocationContext, org.eclipse.core.runtime.IProgressMonitor) */ - public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { - return Collections.EMPTY_LIST; + public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) { + return Collections.emptyList(); } /* diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionAssistant.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionAssistant.java index d870f58527d..b5259bd5687 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionAssistant.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionAssistant.java @@ -18,15 +18,8 @@ import java.util.Iterator; import org.eclipse.core.runtime.Assert; import org.eclipse.core.runtime.IStatus; import org.eclipse.core.runtime.Status; - -import org.eclipse.swt.graphics.Color; -import org.eclipse.swt.graphics.Point; -import org.eclipse.swt.graphics.RGB; -import org.eclipse.swt.widgets.Shell; - import org.eclipse.jface.preference.IPreferenceStore; import org.eclipse.jface.preference.PreferenceConverter; - import org.eclipse.jface.text.BadLocationException; import org.eclipse.jface.text.DefaultInformationControl; import org.eclipse.jface.text.IDocument; @@ -40,7 +33,10 @@ import org.eclipse.jface.text.quickassist.QuickAssistAssistant; import org.eclipse.jface.text.source.Annotation; import org.eclipse.jface.text.source.IAnnotationModel; import org.eclipse.jface.text.source.ISourceViewer; - +import org.eclipse.swt.graphics.Color; +import org.eclipse.swt.graphics.Point; +import org.eclipse.swt.graphics.RGB; +import org.eclipse.swt.widgets.Shell; import org.eclipse.ui.IEditorPart; import org.eclipse.ui.texteditor.IDocumentProvider; import org.eclipse.ui.texteditor.ITextEditor; @@ -156,7 +152,7 @@ public class CCorrectionAssistant extends QuickAssistAssistant { return super.showPossibleQuickAssists(); - ArrayList resultingAnnotations= new ArrayList(20); + ArrayList resultingAnnotations= new ArrayList(20); try { Point selectedRange= fViewer.getSelectedRange(); int currOffset= selectedRange.x; @@ -172,7 +168,7 @@ public class CCorrectionAssistant extends QuickAssistAssistant { } catch (BadLocationException e) { CUIPlugin.log(e); } - fCurrentAnnotations= (Annotation[]) resultingAnnotations.toArray(new Annotation[resultingAnnotations.size()]); + fCurrentAnnotations= resultingAnnotations.toArray(new Annotation[resultingAnnotations.size()]); return super.showPossibleQuickAssists(); } @@ -190,7 +186,7 @@ public class CCorrectionAssistant extends QuickAssistAssistant { return document.getLineInformationOfOffset(invocationLocation); } - public static int collectQuickFixableAnnotations(ITextEditor editor, int invocationLocation, boolean goToClosest, ArrayList resultingAnnotations) throws BadLocationException { + public static int collectQuickFixableAnnotations(ITextEditor editor, int invocationLocation, boolean goToClosest, ArrayList resultingAnnotations) throws BadLocationException { IAnnotationModel model= CUIPlugin.getDefault().getDocumentProvider().getAnnotationModel(editor.getEditorInput()); if (model == null) { return invocationLocation; @@ -198,7 +194,7 @@ public class CCorrectionAssistant extends QuickAssistAssistant { ensureUpdatedAnnotations(editor); - Iterator iter= model.getAnnotationIterator(); + Iterator iter= model.getAnnotationIterator(); if (goToClosest) { IRegion lineInfo= getRegionOfInterest(editor, invocationLocation); if (lineInfo == null) { @@ -207,8 +203,8 @@ public class CCorrectionAssistant extends QuickAssistAssistant { int rangeStart= lineInfo.getOffset(); int rangeEnd= rangeStart + lineInfo.getLength(); - ArrayList allAnnotations= new ArrayList(); - ArrayList allPositions= new ArrayList(); + ArrayList allAnnotations= new ArrayList(); + ArrayList allPositions= new ArrayList(); int bestOffset= Integer.MAX_VALUE; while (iter.hasNext()) { Annotation annot= (Annotation) iter.next(); @@ -225,24 +221,23 @@ public class CCorrectionAssistant extends QuickAssistAssistant { return invocationLocation; } for (int i= 0; i < allPositions.size(); i++) { - Position pos= (Position) allPositions.get(i); + Position pos= allPositions.get(i); if (isInside(bestOffset, pos.offset, pos.offset + pos.length)) { resultingAnnotations.add(allAnnotations.get(i)); } } return bestOffset; - } else { - while (iter.hasNext()) { - Annotation annot= (Annotation) iter.next(); - if (CCorrectionProcessor.isQuickFixableType(annot)) { - Position pos= model.getPosition(annot); - if (pos != null && isInside(invocationLocation, pos.offset, pos.offset + pos.length)) { - resultingAnnotations.add(annot); - } + } + while (iter.hasNext()) { + Annotation annot= (Annotation) iter.next(); + if (CCorrectionProcessor.isQuickFixableType(annot)) { + Position pos= model.getPosition(annot); + if (pos != null && isInside(invocationLocation, pos.offset, pos.offset + pos.length)) { + resultingAnnotations.add(annot); } } - return invocationLocation; } + return invocationLocation; } private static void ensureUpdatedAnnotations(ITextEditor editor) { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionProcessor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionProcessor.java index f0860ac63f7..fca6d4cbaeb 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionProcessor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CCorrectionProcessor.java @@ -59,7 +59,7 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { private static ContributedProcessorDescriptor[] getProcessorDescriptors(String contributionId, boolean testMarkerTypes) { IConfigurationElement[] elements= Platform.getExtensionRegistry().getConfigurationElementsFor(CUIPlugin.PLUGIN_ID, contributionId); - ArrayList res= new ArrayList(elements.length); + ArrayList res= new ArrayList(elements.length); for (int i= 0; i < elements.length; i++) { ContributedProcessorDescriptor desc= new ContributedProcessorDescriptor(elements[i], testMarkerTypes); @@ -70,7 +70,7 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { CUIPlugin.log(status); } } - return (ContributedProcessorDescriptor[]) res.toArray(new ContributedProcessorDescriptor[res.size()]); + return res.toArray(new ContributedProcessorDescriptor[res.size()]); } private static ContributedProcessorDescriptor[] getCorrectionProcessors() { @@ -198,9 +198,9 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { ICCompletionProposal[] res= null; if (model != null && annotations != null) { - ArrayList proposals= new ArrayList(10); + ArrayList proposals= new ArrayList(10); IStatus status= collectProposals(context, model, annotations, true, !fAssistant.isUpdatedOffset(), proposals); - res= (ICCompletionProposal[]) proposals.toArray(new ICCompletionProposal[proposals.size()]); + res= proposals.toArray(new ICCompletionProposal[proposals.size()]); if (!status.isOK()) { fErrorMessage= status.getMessage(); CUIPlugin.log(status); @@ -217,8 +217,8 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { return res; } - public static IStatus collectProposals(CorrectionContext context, IAnnotationModel model, Annotation[] annotations, boolean addQuickFixes, boolean addQuickAssists, Collection proposals) { - ArrayList problems= new ArrayList(); + public static IStatus collectProposals(CorrectionContext context, IAnnotationModel model, Annotation[] annotations, boolean addQuickFixes, boolean addQuickAssists, Collection proposals) { + ArrayList problems= new ArrayList(); // collect problem locations and corrections from marker annotations for (int i= 0; i < annotations.length; i++) { @@ -235,7 +235,7 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { } MultiStatus resStatus= null; - IProblemLocation[] problemLocations= (IProblemLocation[]) problems.toArray(new IProblemLocation[problems.size()]); + IProblemLocation[] problemLocations= problems.toArray(new IProblemLocation[problems.size()]); if (addQuickFixes) { IStatus status= collectCorrections(context, problemLocations, proposals); if (!status.isOK()) { @@ -269,7 +269,7 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { return null; } - private static void collectMarkerProposals(SimpleMarkerAnnotation annotation, Collection proposals) { + private static void collectMarkerProposals(SimpleMarkerAnnotation annotation, Collection proposals) { IMarker marker= annotation.getMarker(); IMarkerResolution[] res= IDE.getMarkerHelpRegistry().getResolutions(marker); if (res.length > 0) { @@ -318,10 +318,10 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { private static class SafeCorrectionCollector extends SafeCorrectionProcessorAccess { private final CorrectionContext fContext; - private final Collection fProposals; + private final Collection fProposals; private IProblemLocation[] fLocations; - public SafeCorrectionCollector(CorrectionContext context, Collection proposals) { + public SafeCorrectionCollector(CorrectionContext context, Collection proposals) { fContext= context; fProposals= proposals; } @@ -347,9 +347,9 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { private static class SafeAssistCollector extends SafeCorrectionProcessorAccess { private final CorrectionContext fContext; private final IProblemLocation[] fLocations; - private final Collection fProposals; + private final Collection fProposals; - public SafeAssistCollector(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { + public SafeAssistCollector(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { fContext= context; fLocations= locations; fProposals= proposals; @@ -415,7 +415,7 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { } } - public static IStatus collectCorrections(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { + public static IStatus collectCorrections(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { ContributedProcessorDescriptor[] processors= getCorrectionProcessors(); SafeCorrectionCollector collector= new SafeCorrectionCollector(context, proposals); for (int i= 0; i < processors.length; i++) { @@ -432,19 +432,19 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { private static IProblemLocation[] getHandledProblems(IProblemLocation[] locations, ContributedProcessorDescriptor processor) { // implementation tries to avoid creating a new array boolean allHandled= true; - ArrayList res= null; + ArrayList res= null; for (int i= 0; i < locations.length; i++) { IProblemLocation curr= locations[i]; if (processor.canHandleMarkerType(curr.getMarkerType())) { if (!allHandled) { // first handled problem if (res == null) { - res= new ArrayList(locations.length - i); + res= new ArrayList(locations.length - i); } res.add(curr); } } else if (allHandled) { if (i > 0) { // first non handled problem - res= new ArrayList(locations.length - i); + res= new ArrayList(locations.length - i); for (int k= 0; k < i; k++) { res.add(locations[k]); } @@ -458,10 +458,10 @@ public class CCorrectionProcessor implements IQuickAssistProcessor { if (res == null) { return null; } - return (IProblemLocation[]) res.toArray(new IProblemLocation[res.size()]); + return res.toArray(new IProblemLocation[res.size()]); } - public static IStatus collectAssists(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { + public static IStatus collectAssists(CorrectionContext context, IProblemLocation[] locations, Collection proposals) { ContributedProcessorDescriptor[] processors= getAssistProcessors(); SafeAssistCollector collector= new SafeAssistCollector(context, locations, proposals); collector.process(processors); diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CSelectAnnotationRulerAction.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CSelectAnnotationRulerAction.java index 5065bda4df1..1ed6619eb3b 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CSelectAnnotationRulerAction.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CSelectAnnotationRulerAction.java @@ -111,7 +111,7 @@ public class CSelectAnnotationRulerAction extends SelectMarkerRulerAction { if (model == null) return ; - Iterator iter= model.getAnnotationIterator(); + Iterator iter= model.getAnnotationIterator(); int layer= Integer.MIN_VALUE; while (iter.hasNext()) { @@ -138,21 +138,20 @@ public class CSelectAnnotationRulerAction extends SelectMarkerRulerAction { fHasCorrection= true; layer= annotationLayer; continue; - } else { - AnnotationPreference preference= fAnnotationPreferenceLookup.getAnnotationPreference(annotation); - if (preference == null) - continue; - - String key= preference.getVerticalRulerPreferenceKey(); - if (key == null) - continue; - - if (fStore.getBoolean(key)) { - fPosition= position; - fHasCorrection= false; - layer= annotationLayer; - } } + AnnotationPreference preference= fAnnotationPreferenceLookup.getAnnotationPreference(annotation); + if (preference == null) + continue; + + String key= preference.getVerticalRulerPreferenceKey(); + if (key == null) + continue; + + if (fStore.getBoolean(key)) { + fPosition= position; + fHasCorrection= false; + layer= annotationLayer; + } } } } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/ContributedProcessorDescriptor.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/ContributedProcessorDescriptor.java index 22dce4e7c0d..c21975f4d1e 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/ContributedProcessorDescriptor.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/ContributedProcessorDescriptor.java @@ -15,15 +15,14 @@ import java.util.Arrays; import java.util.HashSet; import java.util.Set; -import org.eclipse.core.runtime.CoreException; -import org.eclipse.core.runtime.IConfigurationElement; -import org.eclipse.core.runtime.IStatus; - import org.eclipse.core.expressions.EvaluationContext; import org.eclipse.core.expressions.EvaluationResult; import org.eclipse.core.expressions.Expression; import org.eclipse.core.expressions.ExpressionConverter; import org.eclipse.core.expressions.ExpressionTagNames; +import org.eclipse.core.runtime.CoreException; +import org.eclipse.core.runtime.IConfigurationElement; +import org.eclipse.core.runtime.IStatus; import org.eclipse.cdt.core.model.ICModelMarker; import org.eclipse.cdt.core.model.ICProject; @@ -37,7 +36,7 @@ public final class ContributedProcessorDescriptor { private Object fProcessorInstance; private Boolean fStatus; private boolean fLastResult; - private final Set fHandledMarkerTypes; + private final Set fHandledMarkerTypes; private static final String ID= "id"; //$NON-NLS-1$ private static final String CLASS= "class"; //$NON-NLS-1$ @@ -55,13 +54,13 @@ public final class ContributedProcessorDescriptor { fHandledMarkerTypes= testMarkerTypes ? getHandledMarkerTypes(element) : null; } - private Set getHandledMarkerTypes(IConfigurationElement element) { - HashSet map= new HashSet(7); + private Set getHandledMarkerTypes(IConfigurationElement element) { + HashSet map= new HashSet(7); IConfigurationElement[] children= element.getChildren(HANDLED_MARKER_TYPES); - for (int i= 0; i < children.length; i++) { - IConfigurationElement[] types= children[i].getChildren(MARKER_TYPE); - for (int k= 0; k < types.length; k++) { - String attribute= types[k].getAttribute(ID); + for (IConfigurationElement element2 : children) { + IConfigurationElement[] types= element2.getChildren(MARKER_TYPE); + for (IConfigurationElement type : types) { + String attribute= type.getAttribute(ID); if (attribute != null) { map.add(attribute); } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandHandler.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandHandler.java index 07972889f3e..d506f7e0b48 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandHandler.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandHandler.java @@ -38,6 +38,7 @@ import org.eclipse.ui.keys.IBindingService; import org.eclipse.cdt.core.model.ITranslationUnit; import org.eclipse.cdt.ui.CUIPlugin; +import org.eclipse.cdt.ui.text.ICCompletionProposal; import org.eclipse.cdt.internal.ui.editor.CEditor; @@ -74,7 +75,7 @@ public class CorrectionCommandHandler extends AbstractHandler { private ICompletionProposal findCorrection(String id, boolean isAssist, ITextSelection selection, ITranslationUnit cu, IAnnotationModel model) { CorrectionContext context= new CorrectionContext(cu, selection.getOffset(), selection.getLength()); - Collection proposals= new ArrayList(10); + Collection proposals= new ArrayList(10); if (isAssist) { CCorrectionProcessor.collectAssists(context, new ProblemLocation[0], proposals); } else { @@ -86,7 +87,7 @@ public class CorrectionCommandHandler extends AbstractHandler { return null; } } - for (Iterator iter= proposals.iterator(); iter.hasNext();) { + for (Iterator iter= proposals.iterator(); iter.hasNext();) { Object curr= iter.next(); if (curr instanceof ICommandAccess) { if (id.equals(((ICommandAccess) curr).getCommandId())) { @@ -98,9 +99,9 @@ public class CorrectionCommandHandler extends AbstractHandler { } private Annotation[] getAnnotations(int offset, boolean goToClosest) throws BadLocationException { - ArrayList resultingAnnotations= new ArrayList(); + ArrayList resultingAnnotations= new ArrayList(); CCorrectionAssistant.collectQuickFixableAnnotations(fEditor, offset, goToClosest, resultingAnnotations); - return (Annotation[]) resultingAnnotations.toArray(new Annotation[resultingAnnotations.size()]); + return resultingAnnotations.toArray(new Annotation[resultingAnnotations.size()]); } private IDocument getDocument() { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandInstaller.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandInstaller.java index 436458bc783..eb3d3d1c10a 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandInstaller.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/CorrectionCommandInstaller.java @@ -14,7 +14,6 @@ package org.eclipse.cdt.internal.ui.text.correction; import java.util.ArrayList; import java.util.Collection; -import java.util.Iterator; import java.util.List; import org.eclipse.ui.IWorkbench; @@ -39,7 +38,7 @@ public class CorrectionCommandInstaller { */ public static final String ASSIST_SUFFIX= ".assist"; //$NON-NLS-1$ - private List fCorrectionHandlerActivations; + private List fCorrectionHandlerActivations; public CorrectionCommandInstaller() { fCorrectionHandlerActivations= null; @@ -56,11 +55,12 @@ public class CorrectionCommandInstaller { if (fCorrectionHandlerActivations != null) { CUIPlugin.getDefault().logErrorMessage("correction handler activations not released"); //$NON-NLS-1$ } - fCorrectionHandlerActivations= new ArrayList(); + fCorrectionHandlerActivations= new ArrayList(); - Collection definedCommandIds= commandService.getDefinedCommandIds(); - for (Iterator iter= definedCommandIds.iterator(); iter.hasNext();) { - String id= (String) iter.next(); + @SuppressWarnings("unchecked") + Collection definedCommandIds= commandService.getDefinedCommandIds(); + for (Object element : definedCommandIds) { + String id= (String) element; if (id.startsWith(COMMAND_PREFIX)) { boolean isAssist= id.endsWith(ASSIST_SUFFIX); CorrectionCommandHandler handler= new CorrectionCommandHandler(editor, id, isAssist); diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/QuickAssistLightBulbUpdater.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/QuickAssistLightBulbUpdater.java index 1fd9ed6d67a..defccd65b7e 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/QuickAssistLightBulbUpdater.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/correction/QuickAssistLightBulbUpdater.java @@ -260,7 +260,7 @@ public class QuickAssistLightBulbUpdater { int currLine= document.getLineOfOffset(offset); // this iterator is not protected, it may throw ConcurrentModificationExceptions - Iterator iter= model.getAnnotationIterator(); + Iterator iter= model.getAnnotationIterator(); while (iter.hasNext()) { Annotation annot= (Annotation) iter.next(); if (CCorrectionProcessor.isQuickFixableType(annot)) { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/AbstractDocCommentProposalComputer.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/AbstractDocCommentProposalComputer.java index 60fe0c0cc90..04af9f98e7a 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/AbstractDocCommentProposalComputer.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/AbstractDocCommentProposalComputer.java @@ -14,6 +14,8 @@ import java.util.List; import org.eclipse.core.resources.IResource; import org.eclipse.core.runtime.IProgressMonitor; +import org.eclipse.jface.text.contentassist.ICompletionProposal; +import org.eclipse.jface.text.contentassist.IContextInformation; import org.eclipse.ui.IEditorPart; import org.eclipse.ui.IWorkbenchPage; import org.eclipse.ui.IWorkbenchWindow; @@ -38,11 +40,11 @@ abstract class AbstractDocCommentProposalComputer implements ICompletionProposal return getConfiguration(owner); } - public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { + public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) { return getConfiguration().createProposalComputer().computeCompletionProposals(context, monitor); } - public List computeContextInformation( + public List computeContextInformation( ContentAssistInvocationContext context, IProgressMonitor monitor) { return getConfiguration().createProposalComputer().computeContextInformation(context, monitor); } diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/DocCommentOwnerManager.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/DocCommentOwnerManager.java index 73d1fc198e1..a7877b5812e 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/DocCommentOwnerManager.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/doctools/DocCommentOwnerManager.java @@ -13,7 +13,6 @@ package org.eclipse.cdt.internal.ui.text.doctools; import java.text.MessageFormat; import java.util.ArrayList; import java.util.HashMap; -import java.util.Iterator; import java.util.List; import java.util.Map; @@ -225,16 +224,15 @@ public class DocCommentOwnerManager { IExtensionRegistry registry = Platform.getExtensionRegistry(); IExtensionPoint indexProviders = registry.getExtensionPoint(CUIPlugin.ID_COMMENT_OWNER); IExtension[] extensions = indexProviders.getExtensions(); - for(int i=0; i i= fMap.values().iterator(); i.hasNext();) { + String cid= i.next(); Element commentNode= data.getOwnerDocument().createElement(ELEMENT_DOC_COMMENT_OWNER); commentNode.setAttribute(ATTRKEY_DCO_ID, cid); - for(Iterator j= fMap.keySet().iterator(); j.hasNext(); ) { - IPath path= (IPath) j.next(); + for(Iterator j= fMap.keySet().iterator(); j.hasNext(); ) { + IPath path= j.next(); String ccid= fMap.get(path); if(cid.equals(ccid)) { Element pathNode= data.getOwnerDocument().createElement(ELEMENT_PATH); diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/CFoldingStructureProviderRegistry.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/CFoldingStructureProviderRegistry.java index 0676dbb77ee..b7b280b6416 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/CFoldingStructureProviderRegistry.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/CFoldingStructureProviderRegistry.java @@ -28,7 +28,7 @@ public class CFoldingStructureProviderRegistry { private static final String EXTENSION_POINT= "foldingStructureProviders"; //$NON-NLS-1$ /** The map of descriptors, indexed by their identifiers. */ - private Map fDescriptors; + private Map fDescriptors; /** * Creates a new instance. @@ -46,7 +46,7 @@ public class CFoldingStructureProviderRegistry { public CFoldingStructureProviderDescriptor[] getFoldingProviderDescriptors() { synchronized (this) { ensureRegistered(); - return (CFoldingStructureProviderDescriptor[]) fDescriptors.values().toArray(new CFoldingStructureProviderDescriptor[fDescriptors.size()]); + return fDescriptors.values().toArray(new CFoldingStructureProviderDescriptor[fDescriptors.size()]); } } @@ -61,7 +61,7 @@ public class CFoldingStructureProviderRegistry { public CFoldingStructureProviderDescriptor getFoldingProviderDescriptor(String id) { synchronized (this) { ensureRegistered(); - return (CFoldingStructureProviderDescriptor) fDescriptors.get(id); + return fDescriptors.get(id); } } @@ -102,7 +102,7 @@ public class CFoldingStructureProviderRegistry { */ public void reloadExtensions() { IExtensionRegistry registry= Platform.getExtensionRegistry(); - Map map= new HashMap(); + Map map= new HashMap(); IConfigurationElement[] elements= registry.getConfigurationElementsFor(CUIPlugin.getPluginId(), EXTENSION_POINT); for (int i= 0; i < elements.length; i++) { diff --git a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/DefaultCFoldingPreferenceBlock.java b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/DefaultCFoldingPreferenceBlock.java index 4e16bd8f5d7..68162b95730 100644 --- a/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/DefaultCFoldingPreferenceBlock.java +++ b/core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/text/folding/DefaultCFoldingPreferenceBlock.java @@ -43,14 +43,14 @@ public class DefaultCFoldingPreferenceBlock implements ICFoldingPreferenceBlock private IPreferenceStore fStore; protected OverlayPreferenceStore fOverlayStore; private OverlayKey[] fKeys; - protected Map fCheckBoxes= new HashMap(); + protected Map fCheckBoxes= new HashMap(); private SelectionListener fCheckBoxListener= new SelectionListener() { public void widgetDefaultSelected(SelectionEvent e) { } public void widgetSelected(SelectionEvent e) { Button button= (Button) e.widget; - String key= (String) fCheckBoxes.get(button); + String key= fCheckBoxes.get(button); fOverlayStore.setValue(key, button.getSelection()); updateEnablement(key); } @@ -65,7 +65,7 @@ public class DefaultCFoldingPreferenceBlock implements ICFoldingPreferenceBlock } private OverlayKey[] createKeys() { - ArrayList overlayKeys= new ArrayList(); + ArrayList overlayKeys= new ArrayList(); overlayKeys.add(new OverlayPreferenceStore.OverlayKey(OverlayPreferenceStore.BOOLEAN, PreferenceConstants.EDITOR_FOLDING_MACROS)); overlayKeys.add(new OverlayPreferenceStore.OverlayKey(OverlayPreferenceStore.BOOLEAN, PreferenceConstants.EDITOR_FOLDING_FUNCTIONS)); @@ -77,7 +77,7 @@ public class DefaultCFoldingPreferenceBlock implements ICFoldingPreferenceBlock overlayKeys.add(new OverlayPreferenceStore.OverlayKey(OverlayPreferenceStore.BOOLEAN, PreferenceConstants.EDITOR_FOLDING_PREPROCESSOR_BRANCHES_ENABLED)); overlayKeys.add(new OverlayPreferenceStore.OverlayKey(OverlayPreferenceStore.BOOLEAN, PreferenceConstants.EDITOR_FOLDING_STATEMENTS)); - return (OverlayKey[]) overlayKeys.toArray(new OverlayKey[overlayKeys.size()]); + return overlayKeys.toArray(new OverlayKey[overlayKeys.size()]); } /* @@ -127,10 +127,10 @@ public class DefaultCFoldingPreferenceBlock implements ICFoldingPreferenceBlock } private void initializeFields() { - Iterator it= fCheckBoxes.keySet().iterator(); + Iterator