Tom Trommey> 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); ---------------- We cannot use the package private constructor because the exceptions are thrown from gnu.CORBA and other package where all our functionality is concentrated. But, indeed, it is a good suggestion to initialise the cause of the thrown CORBA exception. I have found another solution: use the public initCause methodbefore throwing an exception.
Audrius. * gnu/CORBA/Functional_ORB.java, gnu/CORBA/IOR_Delegate.java, gnu/CORBA/Simple_delegate.java, gnu/CORBA/universalHolder.java, gnu/CORBA/CDR/aligningInputStream.java, gnu/CORBA/CDR/aligningOutputStream.java, gnu/CORBA/CDR/cdrInput.java, gnu/CORBA/CDR/cdrOutput.java, gnu/CORBA/CDR/encapsulatedOutput.java, gnu/CORBA/GIOP/ErrorMessage.java, gnu/CORBA/GIOP/MessageHeader.java, gnu/CORBA/GIOP/cxCodeSet.java, gnu/CORBA/GIOP/v1_2/RequestHeader.java, org/omg/CosNaming/BindingIteratorHelper.java, org/omg/CosNaming/BindingListHelper.java, org/omg/CosNaming/BindingTypeHelper.java, org/omg/CosNaming/NameComponentHelper.java, org/omg/CosNaming/NameHelper.java, org/omg/CosNaming/NamingContextExtHelper.java, org/omg/CosNaming/NamingContextHelper.java: Initialise an instances of org.omg.SystemException, thrown due other exception, with InitCause.
Tom.patch
Description: Binary data
_______________________________________________ Classpath-patches mailing list [email protected] http://lists.gnu.org/mailman/listinfo/classpath-patches
