Issue 4610: ValueRefCountBase::_add_ref (cxx_revision) Source: Triodia Technologies Pty Ltd (Mr. Michi Henning, michi(at)triodia.com) Nature: Uncategorized Issue Severity: Summary: PortableServer::ValueRefCountBase derives from both CORBA::ValueBase and PortableServer::ServantBase. This create a problem because ValueBase contains virtual ValueBase * _add_ref() = 0; but ServantBase contains virtual void _add_ref(); The easy fix would appear to be to change the ValueBase version to return void. Anyone see a problem with this? (I don't understand why the ValueBase version currently returns a ValueBase * anyway. Any subtle reason for this that I am not aware of?) Resolution: Change _add_ref() on ValueBase to return void. Revised Text: In 1.17.5 (p 1-71) replace virtual ValueBase* _add_ref() = 0; by virtual void _add_ref() = 0; Actions taken: October 9, 2001: received issue July 1, 2003: closed issue Discussion: End of Annotations:===== Date: Tue, 9 Oct 2001 20:04:19 +1000 (EST) From: Michi Henning Reply-To: C++ Revision Task Force To: C++ Revision Task Force cc: issues@omg.org Subject: ValueRefCountBase::_add_ref In-Reply-To: Message-ID: Organization: IONA Technologies MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-UIDL: %I%e9YO#e9@#Ne9l'1e9 Oops, my apologies, I sent the previous e-mail without a subject heading. Here it is once more (with subject heading ;-) Hi, PortableServer::ValueRefCountBase derives from both CORBA::ValueBase and PortableServer::ServantBase. This create a problem because ValueBase contains virtual ValueBase * _add_ref() = 0; but ServantBase contains virtual void _add_ref(); The easy fix would appear to be to change the ValueBase version to return void. Anyone see a problem with this? (I don't understand why the ValueBase version currently returns a ValueBase * anyway. Any subtle reason for this that I am not aware of?) Cheers, Michi. -- Michi Henning +61 7 3324 9633 Chief CORBA Scientist +61 4 1118 2700 (mobile) IONA Technologies +61 7 3324 9799 (fax) Total Business Integration http://www.ooc.com.au/staff/michi