Version 0.8.10.8

dart/editor/tools/plugins/com.google.dart.engine/src/com/google/dart/engine/internal/index/MemoryIndexStoreImpl.java

 /*
  * Copyright (c) 2013, the Dart project authors.
  * 
  * Licensed under the Eclipse Public License v1.0 (the "License"); you may not use this file except
  * in compliance with the License. You may obtain a copy of the License at
  * 
  * http://www.eclipse.org/legal/epl-v10.html
  * 
  * Unless required by applicable law or agreed to in writing, software distributed under the License
  * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
@@ skipping 195 matching lines @@
     context = unwrapContext(context);
     new MemoryIndexReader(this, context, input).read();
   }
 
   @Override
   public void recordRelationship(Element element, Relationship relationship, Location location) {
     if (element == null || location == null) {
       return;
     }
     location = location.clone();
+    // at the index level we don't care about Member(s)
+    if (element instanceof Member) {
+      element = ((Member) element).getBaseElement();
+    }
     // prepare information
     AnalysisContext elementContext = element.getContext();
     AnalysisContext locationContext = location.getElement().getContext();
     Source elementSource = element.getSource();
     Source locationSource = location.getElement().getSource();
     // sanity check
     if (locationContext == null) {
       return;
     }
+    if (locationSource == null) {
+      return;
+    }
     if (elementContext == null && !(element instanceof NameElementImpl)
         && !(element instanceof UniverseElementImpl)) {
       return;
     }
+    if (elementSource == null && !(element instanceof NameElementImpl)
+        && !(element instanceof UniverseElementImpl)) {
+      return;
+    }
     // may be already removed in other thread
     if (removedContexts.containsKey(elementContext)) {
       return;
     }
     if (removedContexts.containsKey(locationContext)) {
       return;
     }
-    // at the index level we don't care about Member(s)
-    if (element instanceof Member) {
-      element = ((Member) element).getBaseElement();
-    }
     // record: key -> location(s)
     ElementRelationKey key = getCanonicalKey(element, relationship);
     {
       Set<Location> locations = keyToLocations.remove(key);
       if (locations == null) {
         locations = Sets.newSetFromMap(new IdentityHashMap<Location, Boolean>(4));
       } else {
         keyCount--;
       }
       keyToLocations.put(key, locations);
@@ skipping 31 matching lines @@
         locations = Lists.newArrayList();
         sourceToLocations.put(locationSource, locations);
       }
       locations.add(location);
     }
   }
 
   @Override
   public void removeContext(AnalysisContext context) {
     context = unwrapContext(context);
+    if (context == null) {
+      return;
+    }
+    // mark as removed
     removedContexts.put(context, WEAK_SET_VALUE);
     removeSources(context, null);
     // remove context
     contextToSourceToKeys.remove(context);
     contextToSourceToLocations.remove(context);
   }
 
   @Override
   public void removeSource(AnalysisContext context, Source source) {
     context = unwrapContext(context);
+    if (context == null) {
+      return;
+    }
     // remove locations defined in source
     clearSource(context, source);
     // remove keys for elements defined in source
     Map<Source, Set<ElementRelationKey>> sourceToKeys = contextToSourceToKeys.get(context);
     if (sourceToKeys != null) {
       Set<ElementRelationKey> keys = sourceToKeys.remove(source);
       if (keys != null) {
         for (ElementRelationKey key : keys) {
           canonicalKeys.remove(key);
           Set<Location> locations = keyToLocations.remove(key);
           if (locations != null) {
             keyCount--;
             locationCount -= locations.size();
           }
         }
         sourceCount--;
       }
     }
   }
 
   @Override
   public void removeSources(AnalysisContext context, SourceContainer container) {
     context = unwrapContext(context);
+    if (context == null) {
+      return;
+    }
     // remove sources #1
     Map<Source, Set<ElementRelationKey>> sourceToKeys = contextToSourceToKeys.get(context);
     if (sourceToKeys != null) {
       List<Source> sources = Lists.newArrayList(sourceToKeys.keySet());
       for (Source source : sources) {
         if (container == null || container.contains(source)) {
           removeSource(context, source);
         }
       }
     }
@@ skipping 22 matching lines @@
   private ElementRelationKey getCanonicalKey(Element element, Relationship relationship) {
     ElementRelationKey key = new ElementRelationKey(element, relationship);
     ElementRelationKey canonicalKey = canonicalKeys.get(key);
     if (canonicalKey == null) {
       canonicalKey = key;
       canonicalKeys.put(key, canonicalKey);
     }
     return canonicalKey;
   }
 }