Attachment 52216 Details for Bug 161400 – Fix for the reported use case

[patch] Fix for the reported use case

patch.txt (text/plain), 1.29 KB, created by Dani Megert

on 2006-10-18 08:58:09 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Dani Megert

Created: 2006-10-18 08:58:09 EDT

Size: 1.29 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/parser/Scanner.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/parser/Scanner.java,v
>retrieving revision 1.179
>diff -u -r1.179 Scanner.java
>--- compiler/org/eclipse/jdt/internal/compiler/parser/Scanner.java	29 Mar 2006 02:47:34 -0000	1.179
>+++ compiler/org/eclipse/jdt/internal/compiler/parser/Scanner.java	18 Oct 2006 12:59:27 -0000
>@@ -853,10 +853,10 @@
> 				unicodePtr = this.withoutUnicodePtr;
> 				offset = this.currentPosition;
> 				this.startPosition = this.currentPosition;
>-				try {
>-					checkIfUnicode = ((this.currentCharacter = this.source[this.currentPosition++]) == '\\')
>-						&& (this.source[this.currentPosition] == 'u');
>-				} catch(IndexOutOfBoundsException e) {
>+				if (currentPosition++ < source.length) {
>+					this.currentCharacter = this.source[this.currentPosition];
>+					checkIfUnicode = currentCharacter == '\\' && (currentCharacter == 'u');
>+				} else{
> 					if (this.tokenizeWhiteSpace && (whiteStart != this.currentPosition - 1)) {
> 						// reposition scanner in case we are interested by spaces as tokens
> 						this.currentPosition--;

Actions: View | Diff

Attachments on bug 161400: 52214 | 52216 | 52217