<div dir="ltr"><div>Phil,<br><br></div>Thanks for your feedback !<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-03-06 20:06 GMT+01:00 Phil Race <span dir="ltr"><<a href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
you placed the test in the java.awt.geom package.<br>
<br>
  25 package java.awt.geom;<br>
<br>
and are accessing internals of that package.<br>
<br>
In jigsaw/modular mode that won't even compile.<br></blockquote><div><br></div><div>Ok it is annoying: <br>as all Path2D fields are package protected, I designed the test using direct access to any fields ...<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
So the test should go in the anonymous package and avoid accessing internals.<br>
It should be possible to use just public API  to verify the arrays  of a shape<br>
being cloned are trimmed .<br></blockquote><div><br></div><div>No, it is not possible to use Shape API to access arrays nor fields (numTypes ...): <br>only getPathIterator() could give me data but it won't tell me if the underlying arrays or fields are correct.<br><br></div><div>Maybe I could use introspection to getDeclaredField(name) and setAccessible(true) to get internal data.<br><br></div><div>Any idea or utility class I could use ? <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Why is it necessary to explicitly add the call to super(); ?<br>
<br>
223             super();<br></blockquote><div><br></div><div>I agree it is not necessary but it explicitely says that I use the empty constructor:<br><span style="font-family:monospace,monospace"><br>    /**<br>     * Constructs a new empty {@code Path2D} object.<br>     * It is assumed that the package sibling subclass that is<br>     * defaulting to this constructor will fill in all values.<br>     *<br>     * @since 1.6<br>     */<br>    /* private protected */<br>    Path2D() {<br>    }</span><br> <br></div><div>Laurent<br></div><br></div></div></div></div></div>