RED

refs 2490

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/storage/eclipse/EclipseResourceStorageService.java

@@ -155,8 +155,8 @@ public class EclipseResourceStorageService implements IEclipseResourceStorageSer
 				}
 			}
 		} catch(CoreException e) {
-			error(ToolingKernelActivator.getDefault(),
-					"Core Exception while scanning Eclipse workspace", e);
+			String msg = "Core Exception while scanning Eclipse workspace";
+			error(ToolingKernelActivator.getDefault(), msg, e);
 		}
 	}
 
@@ -231,15 +231,14 @@ public class EclipseResourceStorageService implements IEclipseResourceStorageSer
 							final ResourceSet resourceSet = topLevelElement.getResourceSet();
 							final List<Resource> toRemove = new ArrayList<Resource>();
 							for(Resource resource : resourceSet.getResources()) {
-								URI fileURI =
-										URI.createPlatformResourceURI(
-												file.getFullPath().toString(), true);
+								String uriString = file.getFullPath().toString();
+								URI fileURI = URI.createPlatformResourceURI(uriString, true);
 								if(resource.getURI().equals(fileURI)) {
 									toRemove.add(resource);
 								}
 							}
 
-							// Remove resource(s) corresponding to removed file
+							// Remove resource(s) corresponding to the removed file
 							topLevelElement.runAsCommand(new Runnable() {
 								@Override
 								public void run() {
@@ -403,6 +402,7 @@ public class EclipseResourceStorageService implements IEclipseResourceStorageSer
 	public String getIntrospectionDescription() {
 		return getIntrospectionLabel() +
 				"\n\nThis service storage implementation registered with the persistency service." +
+				// TODO(VA) Misses a verb in the sentence above.
 				"\nIt searches the Eclipse workspace directory and its sub-directories for model files and" +
 				"\ncreates top-level elements for it." +
 				"\nAll existing files are presented to registered instances of IEclipseResourceStorageLocationProvider," +