From: axelcl <axelcl>
Date: Mon, 2 Jan 2006 05:12:27 +0000 (+0000)
Subject: Avoid NPE for "mark occurences"
X-Git-Url: http://git.phpeclipse.com

Avoid NPE for "mark occurences"
---

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPEditor.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPEditor.java
index 458a4ac..5dcf343 100644
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPEditor.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPEditor.java
@@ -5584,9 +5584,9 @@ public abstract class PHPEditor extends AbstractDecoratedTextEditor implements I
 	 * @since 3.0
 	 */
 	protected void updateOccurrenceAnnotations(ITextSelection selection) {// ,
-																																				// CompilationUnit
-																																				// astRoot)
-																																				// {
+		// CompilationUnit
+		// astRoot)
+		// {
 
 		if (fOccurrencesFinderJob != null)
 			fOccurrencesFinderJob.cancel();
@@ -5615,7 +5615,7 @@ public abstract class PHPEditor extends AbstractDecoratedTextEditor implements I
 			fMarkOccurrenceModificationStamp = currentModificationStamp;
 		}
 
-		if (fMarkOccurrenceTargetRegion == null || fMarkOccurrenceTargetRegion.getLength()==0) {
+		if (fMarkOccurrenceTargetRegion == null || fMarkOccurrenceTargetRegion.getLength() == 0) {
 			return;
 		}
 
@@ -5628,30 +5628,32 @@ public abstract class PHPEditor extends AbstractDecoratedTextEditor implements I
 				Scanner fScanner = new Scanner();
 				fScanner.setSource(document.get().toCharArray());
 				fScanner.setPHPMode(false);
+				String wordStr;
 				char[] word;
 
-				word = document.get(fMarkOccurrenceTargetRegion.getOffset(), fMarkOccurrenceTargetRegion.getLength()).toCharArray();
-
-				int fToken = ITerminalSymbols.TokenNameEOF;
-				try {
-					fToken = fScanner.getNextToken();
-					while (fToken != ITerminalSymbols.TokenNameEOF) { // && fToken !=
-						// TokenNameERROR) {
-						if (fToken == ITerminalSymbols.TokenNameVariable || fToken == ITerminalSymbols.TokenNameIdentifier) {
-							// global variable
-							if (fScanner.equalsCurrentTokenSource(word)) {
-								matches.add(new Region(fScanner.getCurrentTokenStartPosition(), fScanner.getCurrentTokenEndPosition()
-										- fScanner.getCurrentTokenStartPosition() + 1));
+				wordStr = document.get(fMarkOccurrenceTargetRegion.getOffset(), fMarkOccurrenceTargetRegion.getLength());
+				if (wordStr != null) {
+					word = wordStr.toCharArray();
+					int fToken = ITerminalSymbols.TokenNameEOF;
+					try {
+						fToken = fScanner.getNextToken();
+						while (fToken != ITerminalSymbols.TokenNameEOF) { // && fToken !=
+							// TokenNameERROR) {
+							if (fToken == ITerminalSymbols.TokenNameVariable || fToken == ITerminalSymbols.TokenNameIdentifier) {
+								// global variable
+								if (fScanner.equalsCurrentTokenSource(word)) {
+									matches.add(new Region(fScanner.getCurrentTokenStartPosition(), fScanner.getCurrentTokenEndPosition()
+											- fScanner.getCurrentTokenStartPosition() + 1));
+								}
 							}
+							fToken = fScanner.getNextToken();
 						}
-						fToken = fScanner.getNextToken();
+					} catch (InvalidInputException e) {
+						// ignore errors
+					} catch (SyntaxError e) {
+						// ignore errors
 					}
-				} catch (InvalidInputException e) {
-					// ignore errors
-				} catch (SyntaxError e) {
-					// ignore errors
 				}
-
 			} catch (BadLocationException e1) {
 				// ignore errors
 			} catch (Exception e) {
@@ -5686,9 +5688,9 @@ public abstract class PHPEditor extends AbstractDecoratedTextEditor implements I
 
 		fPostSelectionListenerWithAST = new ISelectionListenerWithAST() {
 			public void selectionChanged(IEditorPart part, ITextSelection selection) { // ,
-																																									// CompilationUnit
-																																									// astRoot)
-																																									// {
+				// CompilationUnit
+				// astRoot)
+				// {
 				updateOccurrenceAnnotations(selection);// , astRoot);
 			}
 		};