All public and specific tests are ok, the functionality is not disturbed.

Audrius.

Wolfgang Baer wrote:

Hi,

I check this in. Audrius approved it off list.

Wolfgang

Wolfgang Baer wrote:
Hi,

this patch fixes most of the reverse japi issues. Things that
remain (will be seen if the patch is applied after a japi run)
will require deep knowledge of the Corba stuff. So I will leave
that to Audrius :-)

Most stuff in the patch is that some classes does not extend
from _DynAnyStub and therefore the methods are copied into this
classes from there. There are also some methods which must be
abstract and therefore the default implementation to be removed or
moved into the implementation classes.

Posted as RFC to give Audrius a change to look at it. Everything
compiles and the Corba example still works.

2006-04-08  Wolfgang Baer  <[EMAIL PROTECTED]>

        * org/omg/PortableServer/ServantLocatorPOA.java:
        (preinvoke, postinvoke): Remove default implementation.
        * org/omg/PortableServer/ServantActivatorPOA.java:
        (incarnate, etherealize): Remove default implementation.        
        * org/omg/PortableInterceptor/ObjectReferenceFactory.java:
        Extends from ValueBase and not from ObjectReferenceFactoryOperations.
        (make_object): Moved method from ObjectReferenceFactoryOperations.
        * org/omg/PortableInterceptor/ObjectReferenceFactoryOperations.java:
        Removed unspecified interface.  
        * org/omg/DynamicAny/_DynAnyStub.java:
        (_DynAnyStub(Delegate)): Removed constructor.
        * org/omg/DynamicAny/_DynArrayStub.java,
        * org/omg/DynamicAny/_DynAnyFactoryStub.java,
        * org/omg/DynamicAny/_DynEnumStub.java,
        * org/omg/DynamicAny/_DynFixedStub.java,
        * org/omg/DynamicAny/_DynSequenceStub.java,
        * org/omg/DynamicAny/_DynStructStub.java,
        * org/omg/DynamicAny/_DynUnionStub.java,
        * org/omg/DynamicAny/_DynValueStub.java:
        Extend from ObjectImpl and not from _DynAnyStub.
        (type, next, destroy, copy, rewind, assign, component_count,
        current_component, equal, from_any, get_any, get_boolean, get_char,
        get_double, get_dyn_any, get_float, get_long, get_longlong, get_octet,
        get_reference, get_short, get_string, get_typecode, get_ulong,
        get_ulonglong, get_ushort, get_val, get_wchar, get_wstring, insert_any,
        insert_boolean, insert_char, insert_double, insert_dyn_any, 
insert_float,
        insert_long, insert_longlong, insert_octet, insert_reference,
        insert_short, insert_string, insert_typecode, insert_ulong,
        insert_ulonglong, insert_ushort, insert_val, insert_wchar,
        insert_wstring, seek, to_any): New methods copied from _DynAnyStub.     
        
        * org/omg/CosNaming/_BindingIteratorStub.java:
        (_BindingIteratorStub(Delegate)): Made package private.
        * org/omg/CosNaming/_NamingContextExtStub.java:
        (_NamingContextExtStub(Delegate)): Made package private.
        * org/omg/CosNaming/_NamingContextStub.java:
        (_NamingContextStub(Delegate)): Made package private.
        (throw4, throw5): Likewise.
        * gnu/CORBA/NamingService/NameParser.java (resolve):
        Adapt to package private constructor. Use _set_delegate instead.        
        * org/omg/CosNaming/NamingContextOperations.java: Do not extend 
IDLEntity.
        * org/omg/CORBA/ORB.java:
        (create_recursive_sequence_tc): Made abstract.
        (get_default_context): Likewise.
        * gnu/CORBA/OrbRestricted.java:
        (create_recursive_sequence_tc): New moved method.
        (get_default_context): Likewise.        
        * org/omg/CORBA/ParameterMode.java:
        (PARAM_IN, PARAM_OUT, PARAM_INOUT): Made final. 

The patch does not include the removal of
org/omg/PortableInterceptor/ObjectReferenceFactoryOperations.java

Wolfgang






Reply via email to