Bug 1952 - Reference Counting Is Mandatory
Summary: Reference Counting Is Mandatory
Status: RESOLVED FIXED
Alias: None
Product: TAO
Classification: Unclassified
Component: POA (show other bugs)
Version: 1.4.2
Hardware: All All
: P3 normal
Assignee: Johnny Willemsen
URL:
Depends on:
Blocks: 1278
  Show dependency tree
 
Reported: 2004-09-29 12:23 CDT by fp
Modified: 2005-05-26 05:18 CDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description fp 2004-09-29 12:23:05 CDT 
According to the latest C++ mapping, reference counting is mandatory; the
RefCountServantBase class is now a legacy artifact.

PortableServer::ServantBase, by way of TAO_Abstract_ServantBase, currently
implements the _add_ref and _remove_ref operations as no-ops.
Comment 1 Irfan Pyarali 2004-09-30 12:22:31 CDT 
accepted
Comment 2 Johnny Willemsen 2005-05-25 07:25:12 CDT 
Reassigned, I will handle this.
Comment 3 Johnny Willemsen 2005-05-25 07:50:37 CDT 
accept, have this ready on my laptop.
Comment 4 Johnny Willemsen 2005-05-26 05:18:24 CDT 
Fixed, thanks for reporting this.

Thu May 26 10:17:12 UTC 2005  Johnny Willemsen  <jwillemsen@remedy.nl>

        According to the latest C++ mapping, reference counting is mandatory;
        the RefCountServantBase class is now a legacy artifact. Thanks to
        Frank Pilhofer <fpilhofe at mc dot com> for reporting this.
        This fixes bugzilla [1952].

        * tao/Abstract_Servant_Base.{h,cpp}:
          Made the add_ref/remove_ref methods pure virtual and added
          _refcount_value

        * tao/PortableServer/Servant_Base.{h,cpp}:
          Added reference counting to TAO_ServantBase and removed
          TAO_RefCountServantBase

        * tao/PortableServer/PS_ForwardC.h:
          Make RefCountServantBase a noop struct

        * tao/PortableServer/diffs/PS_Forward.diff:
          Updated