>>>>> "Keith" == Keith Seitz <[EMAIL PROTECTED]> 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


_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to