diff options
author | parsons <parsons@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2003-02-05 19:41:35 +0000 |
---|---|---|
committer | parsons <parsons@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2003-02-05 19:41:35 +0000 |
commit | 34f0b59322570d3c67a10abf73d314ad50ae9a1c (patch) | |
tree | d39f47f0116f62789cab7a60069748006b7f3cd3 | |
parent | f2384a7c0d6e92a2e6d33f0b80eef93a30b0b106 (diff) | |
download | ATCD-34f0b59322570d3c67a10abf73d314ad50ae9a1c.tar.gz |
ChangeLogTag: Wed Feb 5 13:34:44 2003 Jeff Parsons <j.parsons@vanderbilt.edu>
-rw-r--r-- | TAO/ChangeLog | 15 | ||||
-rw-r--r-- | TAO/orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp | 11 |
2 files changed, 24 insertions, 2 deletions
diff --git a/TAO/ChangeLog b/TAO/ChangeLog index db7e4eb0f91..b8087f48bb5 100644 --- a/TAO/ChangeLog +++ b/TAO/ChangeLog @@ -1,3 +1,18 @@ +Wed Feb 5 13:34:44 2003 Jeff Parsons <j.parsons@vanderbilt.edu> + + * orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp (create_attribute): + + Changed the behavior to eliminate the creation of a corresponding + repository entry for the operations asssociated with an + attribute in the C++ mapping. On the backend, this mainly affects + describe_interface(). Since the latter operation's return type, + CORBA::FullInterfaceDescription, contains fields for both + attributes and operations, it doesn't seem necessary to include + the _get_* and _set_* operations in the operation list. It also + seems that this behavior will make the TAO IFR more + iteroperable with IFRs from other ORB vendors. Thanks to + Matej Sekoranja <matej.sekoranja@ijs.si> for pointing this out. + Wed Feb 5 10:02:10 2003 Ossama Othman <ossama@uci.edu> * orbsvcs/orbsvcs/LoadBalancing/LB_CPU_Monitor.cpp (loads): diff --git a/TAO/orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp b/TAO/orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp index 38f67afc524..84548eaa77a 100644 --- a/TAO/orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp +++ b/TAO/orbsvcs/orbsvcs/IFRService/InterfaceDef_i.cpp @@ -457,7 +457,9 @@ TAO_InterfaceDef_i::describe_interface_i (ACE_ENV_SINGLE_ARG_DECL) op_key); if (status == 0) - key_queue.enqueue_tail (op_key); + { + key_queue.enqueue_tail (op_key); + } } } @@ -652,6 +654,11 @@ TAO_InterfaceDef_i::create_attribute_i ( mode); // Create the set and/or get operations for this attribute. + // This may not be necessary, and it seems that it may be + // better if the FullInterfaceDescription returned from + // describe_interface does not contain these operations - + // other IFRs from other vendors don't behave that way. +/* this->create_attr_ops (id, name, version, @@ -659,7 +666,7 @@ TAO_InterfaceDef_i::create_attribute_i ( mode ACE_ENV_ARG_PARAMETER); ACE_CHECK_RETURN (CORBA::AttributeDef::_nil ()); - +*/ #if 0 // CCM specific. CORBA::ULong i = 0; |