This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: [RFA/JDWP] Transport factory
- From: Tom Tromey <tromey at redhat dot com>
- To: Keith Seitz <keiths at redhat dot com>
- Cc: GCJ Patches <java-patches at gcc dot gnu dot org>, classpath patches <classpath-patches at gnu dot org>
- Date: 03 Jun 2005 14:59:15 -0600
- Subject: Re: [RFA/JDWP] Transport factory
- References: <1117811480.4519.3.camel@lindt.uglyboxes.com>
- Reply-to: tromey at redhat dot com
>>>>> "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