Bug 1951 - Renaming PortableServer::RefCountServantBase::_ref_count to _refcount_value
Summary: Renaming PortableServer::RefCountServantBase::_ref_count to _refcount_value
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:20 CDT by fp
Modified: 2005-01-09 05:40 CST (History)
1 user (show)

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:20:31 CDT 
According to the latest C++ mapping document, PortableServer::Servant ought
to have an operation "CORBA::ULong _refcount_value ()", returning the current
reference count. RefCountServantBase has the _ref_count operation, but with
the wrong name and signature.
Comment 1 Irfan Pyarali 2004-09-30 12:21:58 CDT 
accepted
Comment 2 Johnny Willemsen 2004-11-30 06:59:41 CST 
I will take care of this.
Comment 3 Johnny Willemsen 2004-11-30 06:59:50 CST 
accept
Comment 4 Johnny Willemsen 2004-12-01 06:34:33 CST 
Is fixed in the POArefactor2 branch and will get probably in the x.4.4 release.
Comment 5 Johnny Willemsen 2005-01-09 05:40:49 CST 
This has been fixed and committed to cvs head. It will appear in the upcoming 
x.4.4 release. Thanks for reporting this.

Sun Jan  9 10:30:12 UTC 2005  Johnny Willemsen  <jwillemsen@remedy.nl>

        * tao/PortableServer/Servant_Base.{h,cpp,i}:
        * tao/PortableServer/Local_Servant_Base.{h,cpp,inl}:
          Moved TAO_Local_ServantBase to its own file. Also changed
          TAO_RefCountServantBase::_ref_count to
          TAO_RefCountServantBase::_refcount_value, the first was TAO
          specific, the second is defined by the corba spec. This fixes
          bugzilla [1951]. Thanks to Frank Pilhofer <fp at mc dot com>
          for reporting this

        * tao/Makefile.am:
          Updated for change above