Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-4362556

Modifications to org.omg.CORBA APIs for standard exceptions

    XMLWordPrintable

    Details

    • Type: Enhancement
    • Status: Closed
    • Priority: P3
    • Resolution: Fixed
    • Affects Version/s: 1.4.0
    • Fix Version/s: 1.4.0
    • Component/s: other-libs
    • Labels:
    • Subcomponent:
    • Resolved In Build:
      beta
    • CPU:
      generic
    • OS:
      generic
    • Verification:
      Not verified

      Backports

        Description

        The following changes must be made to various org.omg.CORBA API for exceptions
        to achieve spec compliance:

        Files affected:
        UnknownUserException.java
        WrongTransaction.java
        CompletionStatus.java
        INV_POLICY.java

        All files are in the org.omg.CORBA package.

        Specifications used:

        IDL to Java ptc-00-01-08
        CORBA 99-10-07

        Detail on proposed changes:

        -------

        File: UnknownUserException.java

        OLD:
        No UnknownUserExceptionHelper or UnknownUserExceptionHolder classes

        Proposed:
        Add them as required by the IDL to Java spec section 1.15.1

        User exceptions should have generated Helper and Holder classes

        -------

        File: WrongTransaction.java

        OLD:
        public class WrongTransaction extends UserException {

        Proposed:
        public final class WrongTransaction extends UserException {

        IDL to Java section 1.15.1 requires user exceptions to be final.

        OLD:
        No WrongTransactionHelper or WrongTransactionHolder classes

        Proposed:
        Add them as required by the IDL to Java spec section 1.15.1

        User exceptions should have generated Helper and Holder classes

        -------

        File: CompletionStatus.java

        OLD:
        public class CompletionStatus implements org.omg.CORBA.portable.IDLEntity

        Proposed:
        public final class CompletionStatus

        IDL to Java section 1.15.2 shows the code for the proposed change.
        CompletitionStatus isn't an IDLEntity since it isn't defined from IDL.

        OLD:
        public static CompletionStatus from_int(int i) throws org.omg.CORBA.BAD_PARAM {

        Proposed:
        public static CompletionStatus from_int(int i) {

        IDL to Java section 1.15.2 shows the code for the proposed change.
        A BAD_PARAM exception is a RuntimeException, and shouldn't be in the throws list.
        This must also change in the javadoc comment.

        OLD:
        protected CompletionStatus(int _value) {

        Proposed:
        private CompletionStatus(int _value) {

        IDL to Java section 1.15.2 shows the code for the proposed change.
        The constructor should be private.

        -------

        File: INV_POLICY.java

        OLD:
        public class INV_POLICY extends SystemException {

        Proposed:
        public final class INV_POLICY extends SystemException {

        All system exceptions should be final as described in IDL to Java section 1.15.2.



        *******************************************************************************
        muthu.anbumani@eng 2001-08-06
        - Code inspection ok for org/omg/CORBA/UnknownUserException.java
        - Code inspection ok for org/omg/CORBA/WrongTransaction.java
        - Code inspection ok for org/omg/CORBA/CompletionStatus.java
        - Code inspection ok for org/omg/CORBA/INV_POLICY.java

        - Build: j2sdk-1_4_0-beta_refresh-bin-b74-solsparc-01_aug_2001
        - Spec Referred
          http://javaweb.eng/~kenc/toHome/papers/objects/OMG/ptc/00-01-08.pdf
        ********************************************************************************

          Attachments

            Issue Links

              Activity

                People

                Assignee:
                eandersosunw Everett Anderson (Inactive)
                Reporter:
                eandersosunw Everett Anderson (Inactive)
                Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:
                  Imported:
                  Indexed: