[[ Bug ]] Ensure java param string is parsed correctly

livecodeali · livecodeali · commit 2927baeb25d5 · 2017-03-08T13:44:04.000Z
diff --git a/libfoundation/src/foundation-java-private.cpp b/libfoundation/src/foundation-java-private.cpp
@@ -112,6 +112,9 @@ static bool __NextArgument(MCStringRef p_arguments, MCRange& x_range)
     {
         if (!MCStringFirstIndexOfChar(p_arguments, ';', x_range . offset, kMCStringOptionCompareExact, t_length))
             return false;
+        
+        // Consume the ;
+        t_length++;
     }
     
     x_range . length = t_length;
diff --git a/tests/lcb/stdlib/java.lcb b/tests/lcb/stdlib/java.lcb
@@ -233,4 +233,15 @@ public handler TestNullJObjectParam()
 	MCUnitTestHandlerThrows(NullParamJObject, "type mismatch when passing NULL as non-optional JObject")
 end handler
 
-end module
+foreign handler StringCodepointAt(in pSequence as JObject, in pIndex as CInt) returns CInt binds to "java:java.lang.Character>codePointAt(Ljava/lang/CharSequence;I)I!static"
+
+public handler TestNativeTypeAfterObjectType()
+	unsafe
+		variable tCodepoint as Integer
+		put StringCodepointAt(StringToJString("a"), 0) into tCodepoint
+	
+		test "called foreign handler with signature Ljava/lang/CharSequence;I" when tCodepoint is 97
+	end unsafe
+end handler
+
+end module

Original file line number	Diff line number	Diff line change
`@@ -112,6 +112,9 @@ static bool __NextArgument(MCStringRef p_arguments, MCRange& x_range)`
`112`	`112`	`{`
`113`	`113`	`if (!MCStringFirstIndexOfChar(p_arguments, ';', x_range . offset, kMCStringOptionCompareExact, t_length))`
`114`	`114`	`return false;`
	`115`	`+`
	`116`	`+ // Consume the ;`
	`117`	`+ t_length++;`
`115`	`118`	`}`
`116`	`119`
`117`	`120`	`x_range . length = t_length;`