Fixed compiler warnings.

debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/sourcelookup/CProjectSourceLocation.java

@@ -13,7 +13,6 @@ package org.eclipse.cdt.debug.internal.core.sourcelookup;
 import java.io.File;
 import java.io.IOException;
 import java.io.StringReader;
-import com.ibm.icu.text.MessageFormat;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.LinkedList;
@@ -27,6 +26,7 @@ import org.eclipse.cdt.debug.core.CDebugCorePlugin;
 import org.eclipse.cdt.debug.core.CDebugUtils;
 import org.eclipse.cdt.debug.core.sourcelookup.ICSourceLocation;
 import org.eclipse.cdt.debug.core.sourcelookup.IProjectSourceLocation;
+import org.eclipse.cdt.internal.core.resources.ResourceLookup;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IProject;
 import org.eclipse.core.resources.IResource;
@@ -38,14 +38,14 @@ import org.eclipse.core.runtime.IPath;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Path;
 import org.eclipse.core.runtime.Status;
+import org.eclipse.osgi.util.NLS;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.xml.sax.InputSource;
 import org.xml.sax.SAXException;
 
 /**
- * 
+ * Locates source elements in a C/C++ project. Returns instances of <code>IFile</code>.
- * Locates source elements in a Java project. Returns instances of <code>IFile</code>.
  * 
  * @since Sep 23, 2002
  */
@@ -59,40 +59,39 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	 */
 	private IProject fProject;
 	private IResource[] fFolders;
-	private HashMap fCache = new HashMap( 20 );
+	private HashMap<String, Object> fCache = new HashMap<String, Object>(20);
-	private HashSet fNotFoundCache = new HashSet( 20 );
+	private HashSet<String> fNotFoundCache = new HashSet<String>(20);
 	private boolean fGenerated = true;
 	private boolean fSearchForDuplicateFiles = false;
 
 	public CProjectSourceLocation() {
 	}
 
-	public CProjectSourceLocation( IProject project ) {
+	public CProjectSourceLocation(IProject project) {
-		setProject( project );
+		setProject(project);
 		fGenerated = true;
 	}
 
-	public CProjectSourceLocation( IProject project, boolean generated ) {
+	public CProjectSourceLocation(IProject project, boolean generated) {
-		setProject( project );
+		setProject(project);
 		fGenerated = generated;
 	}
 
 	/* (non-Javadoc)
 	 * @see org.eclipse.cdt.debug.core.sourcelookup.ICSourceLocation#findSourceElement(String)
 	 */
-	public Object findSourceElement( String name ) throws CoreException
+	public Object findSourceElement(String name) throws CoreException {
-	{
 		Object result = null;
-		if ( !isEmpty( name ) && getProject() != null && !notFoundCacheLookup( name ) ) {
+		if (!isEmpty(name) && getProject() != null && !notFoundCacheLookup(name)) {
-			result = cacheLookup( name );
+			result = cacheLookup(name);
-			if ( result == null ) { 
+			if (result == null) { 
-				result = doFindSourceElement( name );
+				result = doFindSourceElement(name);
-				if ( result != null ) {
+				if (result != null) {
-					cacheSourceElement( name, result );
+					cacheSourceElement(name, result);
 				}
 			}
-			if ( result == null ) {
+			if (result == null) {
-				cacheNotFound( name );
+				cacheNotFound(name);
 			}
 		}
 		return result;
@@ -101,12 +100,12 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	/* (non-Javadoc)
 	 * @see org.eclipse.core.runtime.IAdaptable#getAdapter(Class)
 	 */
-	public Object getAdapter( Class adapter ) {
+	public Object getAdapter(@SuppressWarnings("rawtypes") Class adapter) {
-		if ( adapter.equals( ICSourceLocation.class ) )
+		if (adapter.equals(ICSourceLocation.class))
 			return this;
-		if ( adapter.equals( CProjectSourceLocation.class ) )
+		if (adapter.equals(CProjectSourceLocation.class))
 			return this;
-		if ( adapter.equals( IProject.class ) )
+		if (adapter.equals(IProject.class))
 			return getProject();
 		return null;
 	}
@@ -116,7 +115,7 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	 * 
 	 * @param project the project
 	 */
-	private void setProject( IProject project ) {
+	private void setProject(IProject project) {
 		fProject = project;
 	}
 
@@ -129,64 +128,66 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 		return fProject;
 	}
 
-	private Object doFindSourceElement( String name ) {
+	private Object doFindSourceElement(String name) {
-		File file = new File( name );
+		File file = new File(name);
-		return ( file.isAbsolute() ) ? findFileByAbsolutePath( file ) : findFileByRelativePath( name );
+		return file.isAbsolute() ? findFileByAbsolutePath(file) : findFileByRelativePath(name);
 	}
 
-	private Object findFileByAbsolutePath( File file ) {
+	private Object findFileByAbsolutePath(File file) {
-		LinkedList list = new LinkedList();
+		LinkedList<IFile> list = new LinkedList<IFile>();
-		if ( file.exists() ) {
+		if (file.exists()) {
-			IPath path = new Path( file.getAbsolutePath() );
+			IPath path = new Path(file.getAbsolutePath());
-			IFile[] wsFiles = CDebugCorePlugin.getWorkspace().getRoot().findFilesForLocation( path );
+			IFile[] wsFiles = ResourceLookup.findFilesForLocation(path);
-			for ( int i = 0; i < wsFiles.length; ++i )
+			for (int i = 0; i < wsFiles.length; ++i) {
-				if ( wsFiles[i].getProject().equals( getProject() ) && wsFiles[i].exists() ) {
+				if (wsFiles[i].getProject().equals(getProject()) && wsFiles[i].exists()) {
-					if ( !searchForDuplicateFiles() )
+					if (!searchForDuplicateFiles())
 						return wsFiles[i];
-					list.add( wsFiles[i] );
+					list.add(wsFiles[i]);
-				}
-		}
-		return ( list.size() > 0 ) ? ( ( list.size() == 1 ) ? list.getFirst() : list ) : null;
-	}
-
-	private Object findFileByRelativePath( String fileName ) {
-		IResource[] folders = getFolders();
-		LinkedList list = new LinkedList();
-		for ( int i = 0; i < folders.length; ++i ) {
-			if ( list.size() > 0 && !searchForDuplicateFiles() )
-				break;
-			IPath path = folders[i].getLocation();
-			if ( path != null ) {
-				path = path.append( fileName );
-				File file = new File( path.toOSString() );
-				if ( file.exists() ) {
-					IFile[] wsFiles = CDebugCorePlugin.getWorkspace().getRoot().findFilesForLocation( path );
-					for ( int j = 0; j < wsFiles.length; ++j )
-						if ( wsFiles[j].exists() ) {
-							if ( !searchForDuplicateFiles() )
-								return wsFiles[j];
-							list.add( wsFiles[j] );
-						}
 				}
 			}
 		}
-		return ( list.size() > 0 ) ? ( ( list.size() == 1 ) ? list.getFirst() : list ) : null;
+		return (list.size() > 0) ? ((list.size() == 1) ? list.getFirst() : list) : null;
 	}
 
-	private Object cacheLookup( String name ) {
+	private Object findFileByRelativePath(String fileName) {
-		return fCache.get( name );
+		IResource[] folders = getFolders();
-	}
+		LinkedList<IFile> list = new LinkedList<IFile>();
-	
+		for (int i = 0; i < folders.length; ++i) {
-	private boolean notFoundCacheLookup( String name ) {
+			if (list.size() > 0 && !searchForDuplicateFiles())
-		return fNotFoundCache.contains( name );
+				break;
-	}
+			IPath path = folders[i].getLocation();
-	
+			if (path != null) {
-	private void cacheSourceElement( String name, Object element ) {
+				path = path.append(fileName);
-		fCache.put( name, element );
+				File file = new File(path.toOSString());
+				if (file.exists()) {
+					IFile[] wsFiles = ResourceLookup.findFilesForLocation(path);
+					for (int j = 0; j < wsFiles.length; ++j) {
+						if (wsFiles[j].exists()) {
+							if (!searchForDuplicateFiles())
+								return wsFiles[j];
+							list.add(wsFiles[j]);
+						}
+					}
+				}
+			}
+		}
+		return list.size() > 0 ? (list.size() == 1 ? list.getFirst() : list) : null;
 	}
 
-	private void cacheNotFound( String name ) {
+	private Object cacheLookup(String name) {
-		fNotFoundCache.add( name );
+		return fCache.get(name);
+	}
+	
+	private boolean notFoundCacheLookup(String name) {
+		return fNotFoundCache.contains(name);
+	}
+	
+	private void cacheSourceElement(String name, Object element) {
+		fCache.put(name, element);
+	}
+
+	private void cacheNotFound(String name) {
+		fNotFoundCache.add(name);
 	}
 
 	public void dispose() {
@@ -197,27 +198,25 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	/* (non-Javadoc)
 	 * @see org.eclipse.cdt.debug.core.sourcelookup.ICSourceLocation#getMemento()
 	 */
-	public String getMemento() throws CoreException
+	public String getMemento() throws CoreException {
-	{
         Document document = null;
         Throwable ex = null;
-        try 
+        try  {
-		{
             document = DocumentBuilderFactory.newInstance().newDocumentBuilder().newDocument();
-            Element node = document.createElement( ELEMENT_NAME );
+            Element node = document.createElement(ELEMENT_NAME);
-            document.appendChild( node );
+            document.appendChild(node);
-    		node.setAttribute( ATTR_PROJECT, getProject().getName() );
+    		node.setAttribute(ATTR_PROJECT, getProject().getName());
-    		node.setAttribute( ATTR_GENERIC, Boolean.valueOf( isGeneric() ).toString() );
+    		node.setAttribute(ATTR_GENERIC, Boolean.valueOf(isGeneric()).toString());
-			return CDebugUtils.serializeDocument( document );
+			return CDebugUtils.serializeDocument(document);
-        } catch( ParserConfigurationException e ) 
+        } catch (ParserConfigurationException e) {
-		{
         	ex = e;
-        } catch( IOException e ) {
+        } catch (IOException e) {
 			ex = e;
-		} catch( TransformerException e ) {
+		} catch (TransformerException e) {
 			ex = e;
 		}
-		abort( MessageFormat.format( InternalSourceLookupMessages.CProjectSourceLocation_0, new String[] { getProject().getName() } ), ex ); //$NON-NLS-1$
+		abort(NLS.bind(InternalSourceLookupMessages.CProjectSourceLocation_0,
+				getProject().getName()), ex);
 		// execution will not reach here
 		return null;
 	}
@@ -225,55 +224,47 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	/* (non-Javadoc)
 	 * @see org.eclipse.cdt.debug.core.sourcelookup.ICSourceLocation#initializeFrom(java.lang.String)
 	 */
-	public void initializeFrom( String memento ) throws CoreException
+	public void initializeFrom(String memento) throws CoreException {
-	{
 		Exception ex = null;
-		try
+		try {
-		{
 			Element root = null;
 			DocumentBuilder parser = DocumentBuilderFactory.newInstance().newDocumentBuilder();
-			StringReader reader = new StringReader( memento );
+			StringReader reader = new StringReader(memento);
-			InputSource source = new InputSource( reader );
+			InputSource source = new InputSource(reader);
-			root = parser.parse( source ).getDocumentElement();
+			root = parser.parse(source).getDocumentElement();
 
-			String name = root.getAttribute( ATTR_PROJECT );
+			String name = root.getAttribute(ATTR_PROJECT);
-			if ( isEmpty( name ) ) {
+			if (isEmpty(name)) {
-				abort( InternalSourceLookupMessages.CProjectSourceLocation_1, null ); //$NON-NLS-1$
+				abort(InternalSourceLookupMessages.CProjectSourceLocation_1, null);
+			} else {
+				IProject project = ResourcesPlugin.getWorkspace().getRoot().getProject(name);
+				setProject(project);
 			}
-			else
+			String isGeneric = root.getAttribute(ATTR_GENERIC);
-			{
+			if (isGeneric == null || isGeneric.trim().length() == 0)
-				IProject project = ResourcesPlugin.getWorkspace().getRoot().getProject( name );
-				setProject( project );
-			}
-			String isGeneric = root.getAttribute( ATTR_GENERIC );
-			if ( isGeneric == null || isGeneric.trim().length() == 0 )
 				isGeneric = Boolean.FALSE.toString();
-			setGenerated( isGeneric.equals( Boolean.TRUE.toString() ) );
+			setGenerated(isGeneric.equals(Boolean.TRUE.toString()));
 			return;
-		} catch( ParserConfigurationException e ) {
+		} catch (ParserConfigurationException e) {
 			ex = e;
-		} catch( SAXException e ) {
+		} catch (SAXException e) {
 			ex = e;
-		} catch( IOException e ) {
+		} catch (IOException e) {
 			ex = e;
 		}
-		abort( InternalSourceLookupMessages.CProjectSourceLocation_2, ex ); //$NON-NLS-1$
+		abort(InternalSourceLookupMessages.CProjectSourceLocation_2, ex);
 	}
 
 	/**
 	 * Throws an internal error exception
 	 */
-	private void abort( String message, Throwable e ) throws CoreException
+	private void abort(String message, Throwable e) throws CoreException {
-	{
+		IStatus s = new Status(IStatus.ERROR, CDebugCorePlugin.getUniqueIdentifier(),
-		IStatus s = new Status( IStatus.ERROR,
+				CDebugCorePlugin.INTERNAL_ERROR, message, e);
-								CDebugCorePlugin.getUniqueIdentifier(),
+		throw new CoreException(s);
-								CDebugCorePlugin.INTERNAL_ERROR,
-								message,
-								e );
-		throw new CoreException( s );
 	}
 
-	private boolean isEmpty( String string ) {
+	private boolean isEmpty(String string) {
 		return string == null || string.length() == 0;
 	}
 
@@ -284,53 +275,50 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 		return fGenerated;
 	}
 
-	public void setGenerated( boolean b ) {
+	public void setGenerated(boolean b) {
 		fGenerated = b;
 	}
 
 	/* (non-Javadoc)
 	 * @see java.lang.Object#equals(java.lang.Object)
 	 */
-	public boolean equals( Object obj ) {
+	public boolean equals(Object obj) {
-		if ( obj instanceof IProjectSourceLocation && getProject() != null )
+		if (obj instanceof IProjectSourceLocation && getProject() != null)
-			return getProject().equals( ((IProjectSourceLocation)obj).getProject() );
+			return getProject().equals(((IProjectSourceLocation) obj).getProject());
 		return false;
 	}
 
 	private void initializeFolders() {
-		final LinkedList list = new LinkedList();
+		final LinkedList<IResource> list = new LinkedList<IResource>();
-		if ( getProject() != null && getProject().exists() ) {
+		if (getProject() != null && getProject().exists()) {
-			list.add( getProject() );
+			list.add(getProject());
 			try {
-				getProject().accept( 
+				getProject().accept(
 						new IResourceProxyVisitor() {
-								public boolean visit( IResourceProxy proxy ) throws CoreException
+							public boolean visit(IResourceProxy proxy) throws CoreException {
-								{
+								switch (proxy.getType()) {
-									switch( proxy.getType() ) {
+									case IResource.FILE:
-										case IResource.FILE:
+										return false;
-											return false;
+									case IResource.FOLDER:
-										case IResource.FOLDER:
+										list.addLast(proxy.requestResource());
-											list.addLast( proxy.requestResource() );
+										return true;
-											return true;
-									}
-									return true;
 								}
-	
+								return true;
-							}, 
+							}
-						IResource.NONE );
+						}, 
-			} catch( CoreException e ) {
+						IResource.NONE);
+			} catch (CoreException e) {
 			}
 		}
-		synchronized( this ) {
+		synchronized (this) {
-			if ( fFolders == null ) 
+			if (fFolders == null) {
-			{
+				fFolders = list.toArray(new IResource[list.size()]);
-				fFolders = (IResource[])list.toArray( new IResource[list.size()] );
 			}
 		}
 	}
 
 	protected IResource[] getFolders() {
-		if ( fFolders == null )
+		if (fFolders == null)
 			initializeFolders();
 		return fFolders;
 	}
@@ -345,13 +333,13 @@ public class CProjectSourceLocation implements IProjectSourceLocation {
 	/* (non-Javadoc)
 	 * @see org.eclipse.cdt.debug.core.sourcelookup.ICSourceLocation#setSearchForDuplicateFiles(boolean)
 	 */
-	public void setSearchForDuplicateFiles( boolean search ) {
+	public void setSearchForDuplicateFiles(boolean search) {
 		fCache.clear();
 		fNotFoundCache.clear();
 		fSearchForDuplicateFiles = search;
 	}
 
 	public String toString() {
-		return ( getProject() != null ) ? fProject.toString() : ""; //$NON-NLS-1$
+		return getProject() != null ? fProject.toString() : ""; //$NON-NLS-1$
 	}
 }