Attachment 112804 Details for Bug 247701 – Check to make sure the initializer is for this variable

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Check to make sure the initializer is for this variable

bug247701patch (text/plain), 1013 bytes, created by Carl Anderson

on 2008-09-17 14:24:09 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Carl Anderson

Created: 2008-09-17 14:24:09 EDT

Size: 1013 bytes

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jem.workbench
>Index: workbench/org/eclipse/jem/internal/adapters/jdom/JavaFieldJDOMAdaptor.java
>===================================================================
>RCS file: /cvsroot/webtools/jeetools/plugins/org.eclipse.jem.workbench/workbench/org/eclipse/jem/internal/adapters/jdom/JavaFieldJDOMAdaptor.java,v
>retrieving revision 1.14
>diff -u -r1.14 JavaFieldJDOMAdaptor.java
>--- workbench/org/eclipse/jem/internal/adapters/jdom/JavaFieldJDOMAdaptor.java	10 Apr 2007 01:22:55 -0000	1.14
>+++ workbench/org/eclipse/jem/internal/adapters/jdom/JavaFieldJDOMAdaptor.java	17 Sep 2008 18:05:31 -0000
>@@ -126,7 +126,7 @@
> 					int endPos = source.indexOf(',');//$NON-NLS-1$
> 					if (endPos == -1)
> 						endPos = source.length() - 1;
>-					if (equalsPos != -1) {
>+					if (equalsPos != -1 && equalsPos < endPos) {
> 						// Copy from after "=" to before ";" or ","
> 						result = source.substring(equalsPos + 1, endPos);
> 						result = result.trim();

Actions: View | Diff

Attachments on bug 247701: 112804