You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by de...@apache.org on 2010/03/05 03:57:25 UTC

svn commit: r919274 - /geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java

Author: delos
Date: Fri Mar  5 02:57:25 2010
New Revision: 919274

URL: http://svn.apache.org/viewvc?rev=919274&view=rev
Log:
GERONIMODEVTOOLS-283 improve implementation to avoid possible failure with JDK6

Modified:
    geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java

Modified: geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java
URL: http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java?rev=919274&r1=919273&r2=919274&view=diff
==============================================================================
--- geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java (original)
+++ geronimo/devtools/eclipse-plugin/trunk/plugins/org.apache.geronimo.st.ui/src/main/java/org/apache/geronimo/st/ui/refactoring/DeploymentPlanHandler.java Fri Mar  5 02:57:25 2010
@@ -72,7 +72,7 @@
 			//ch doesn't contains XML declare statement at the beginning of deployment plan
 			//get the character number of first line
 			int xmlDeclareLength = getXMLDeclareLength();
-			wtn.setOffset(xmlDeclareLength + start - 1);
+			wtn.setOffset(xmlDeclareLength + start);
 		} catch (IOException e) {
 			e.printStackTrace();
 		}
@@ -149,7 +149,7 @@
 		do {
 			current = br.read();
 			offset++;
-		} while (current != '\n');
+		} while (current != '>');
 
 		br.close();