A simple code mistake in format method

[GoogleCodeExporter]

In have found a mistake in the code of JNLua 1.0.3, thanks to FindBug.

Class: com.naef.jnlua.DefaultJavaReflector.java line: 441.

if (!luaState.isJavaObject(3, componentType)) {
    throw new LuaRuntimeException(
        String.format(
            "attempt to write array of %s at index %d with %s value",
            componentType.getCanonicalName(),
            luaState.typeName(3)));
}

The format string have two %s and one %d, but only two string are passed in 
parametter.
The missing parameter is the value of the index '3' as follow:

if (!luaState.isJavaObject(3, componentType)) {
    throw new LuaRuntimeException(
        String.format(
            "attempt to write array of %s at index %d with %s value",
            componentType.getCanonicalName(), 3,
            luaState.typeName(3)));
}

Marc

Original issue reported on code.google.com by marc.aub...@gmail.com on 11 Feb 2013 at 3:44

[GoogleCodeExporter]

Thanks! Fixed on the trunk and the 0.9 branch.

Original comment by an...@naef.com on 28 Jul 2013 at 10:26

• Changed state: Fixed