On 3/13/18 5:16 PM, David Lloyd wrote:
OK, done.  It's a little bigger now so I'm attaching it.

Thanks for adding the new test.   Looks okay and some minor comment.

+        try {
   :
+        } catch (Throwable e) {
+            System.err.println("\nTEST FAILED:");
+            e.printStackTrace();
+            throw new RuntimeException("TEST FAILED: " + e.toString());
+        }


You can take out this try-catch (Basic1.java isn't be the best example to 
reference that is old and needs update).

I assume your colleague at Red Hat will sponsor it for you?

Mandy

Reply via email to