>>>>> "Mario" == Mario Torre <[EMAIL PROTECTED]> writes:

Mario> Before committing, I wait for approval,

Looks pretty good... I just have a couple nits to point out.

Mario> +    try {
Mario> +      segment = (CubicSegment) super.clone();

In our coding style the "{" goes on its own line.
There are a few cases of this.

Mario> +    } catch (CloneNotSupportedException cnse) {
Mario> +      throw new InternalError(cnse.toString());
Mario> +    }

In this case it doesn't matter a whole lot, but in general it is
better to chain exceptions.

Tom

Reply via email to