[cdt-patch] Re: NullPointer with Indexer Patch

["Alain Magloire" <alain@xxxxxxx>]

> 
> JohnC
> 
> 
> 
> 
> Bogdan Gheorghe/Ottawa/IBM@IBMCA 
> Sent by: cdt-patch-admin@xxxxxxxxxxx
> 09/09/2003 11:41 AM
> Please respond to
> cdt-patch@xxxxxxxxxxx
> 
> 

Yellow

  I beleive this one introduced a NullPointer,

DeltaProcessor.updateDependencies() {
		//Update table
		String fileExtension = element.getResource().getFileExtension();
		if (fileExtension.equals("h") ||
			fileExtension.equals("hh") ||
			fileExtension.equals("hpp")){
	        }	
}

fileExtension can be null, it may not be visible for people running windows.

For exampe and executable ICElement/IBinary will not have any extension.

Need a PR ?

Could have just fix it 8-), but seems to be lots of movement in the area
it helps to limit the patch collision.  Let me know, I'll commit.

StackTrace:
java.lang.NullPointerException
    at org.eclipse.cdt.internal.core.model.DeltaProcessor.updateDependencies(DeltaProcessor.java:524)
    at org.eclipse.cdt.internal.core.model.DeltaProcessor.updateCurrentDeltaAndIndex(DeltaProcessor.java:436)
    at org.eclipse.cdt.internal.core.model.DeltaProcessor.traverseDelta(DeltaProcessor.java:373)
    at org.eclipse.cdt.internal.core.model.DeltaProcessor.traverseDelta(DeltaProcessor.java:383)
    at org.eclipse.cdt.internal.core.model.DeltaProcessor.processResourceDelta(DeltaProcessor.java:356)
    at org.eclipse.cdt.internal.core.model.CModelManager.resourceChanged(CModelManager.java:575)
    at org.eclipse.core.internal.events.NotificationManager$1.run(NotificationManager.java:137)
    at org.eclipse.core.internal.runtime.InternalPlatform.run(InternalPlatform.java:1006)
    at org.eclipse.core.runtime.Platform.run(Platform.java:413)
...