Bug 2179

Summary: Virtual methods in ACE_Thread_Manager
Product: ACE Reporter: Johnny Willemsen <jwillemsen>
Component: ACE CoreAssignee: DOC Center Support List (internal) <tao-support>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P3    
Version: 5.4.6   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 2181    

Description Johnny Willemsen 2005-07-03 06:39:52 CDT
ACE_Thread_Manager has a virtual spawn_i and destructor. looks not useful. Needs
to be looked at.
Comment 1 Steve Huston 2005-07-25 14:35:09 CDT
I faintly recalled something inheriting from ACE_Thread_Manager, making the
virtual stuff useful. However, I can't find it, and the Doxygen docs don't list one.
I think you're right - this may be able to be changed. However, I think it'd be
best to wait til after 5.5 is out, and then search the ChangeLogs for why these
were made virtual.
Comment 2 Johnny Willemsen 2006-04-11 06:57:18 CDT
change seems to be done as part of the entry below. Maybe this was used in the
past but it seems we don't have derived classes anymore so we could zap this to
my idea

Wed Jan 15 02:06:21 1997  Douglas C. Schmidt  <schmidt@flamenco.cs.wustl.edu>

        * ace/Thread_Manager.h: Made the spawn_i() method virtual and
          moved it into the protected section of the class so that it can
          be over-ridden by subclasses.  Thanks to Tim Harrison for
          suggesting this.
Comment 3 Johnny Willemsen 2006-04-19 07:33:53 CDT
Wed Apr 19 12:11:12 UTC 2006  Johnny Willemsen  <jwillemsen@remedy.nl>
Comment 4 Johnny Willemsen 2006-04-19 07:43:20 CDT
fixed