[RFA/JDWP] Transport factory

Tom Tromey tromey@redhat.com
Fri Jun 3 21:17:00 GMT 2005


>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:

Keith> +    IllegalArgumentException e;
Keith> +    e = new IllegalArgumentException ("transport \"" + name + "\" not found");
Keith> +    throw new TransportException (e);

It is more ordinary to just make an IllegalArgumentException and throw
it.  Does this need to be caught somewhere else in the JDWP code
somehow?  If so, then wrapping an IllegalArgumentException seems a bit
weird...

Otherwise this looks fine.

Tom



More information about the Java-patches mailing list