platypusguy / jacobin

A more than minimal JVM written in Go and capable of running Java 21 classes.
http://www.jacobin.org
Mozilla Public License 2.0
683 stars 29 forks source link

JACOBIN-547 Incremental fixes to support stringer-1 #208

Closed texadactyl closed 2 months ago

texadactyl commented 2 months ago
texadactyl commented 2 months ago

Created a G function for String.replace(oldchar, newchar) in spite of what the commit text said! (:

texadactyl commented 2 months ago

stringer-1 status at this point

Failing sometime after calling the OpenJDK JBM lib fn Strings.split(). class=java/lang/String, meth=split(Ljava/lang/String;)[Ljava/lang/String;

This one is a bit complicated. Golang has a strings package simple-split function which does not entertain regular expressions. Unfortunately, for stringer-1, the Java String.split function defines one parameter as a regular expression.

@platypusguy

Over to you, if you like. (:

texadactyl commented 2 months ago

jacotest just now:

17:15:34 Success in 91 test cases
17:15:34 No Execution timeout errors
17:15:34 Execution failure errors in 50 test cases
17:15:34 Wrote test case summary to: /home/elkins/BASIS/jacotest.go/reports/Summary_2024-07-10_17.13.06_jacobin.txt
17:15:34 Elapsed time = 2m28s
17:15:34 Looking for test cases with changed results .....
    cur  >>  stringer-1  jacobin   2024-07-10  22:15:27.297  passed  n/a
    prv  >>  stringer-1  jacobin   2024-07-10  21:08:23.776  failed  java.lang.NullPointerException: ARRAYLENGTH: Invalid (null) value for *object.Object
17:15:34 Number of test cases with changed results = 1
texadactyl commented 2 months ago

@platypusguy

So, IMO: