In the CORBA code I see this:

 [...]

    catch (Exception ex)
      {
        throw new MARSHAL(ex + ":" + ex.getMessage());
      }

This is weird since Throwable.toString() (i.e. the default) includes
getMessage.  I think it would be a bit better to just use getMessage
here and add a package-private constructor to allow exception
chaining:

   throw new MARSHAL(ex.getMessage(), ex);

Comments?

Tom


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

Reply via email to