Bug 2179 - Virtual methods in ACE_Thread_Manager
Summary: Virtual methods in ACE_Thread_Manager
Status: RESOLVED FIXED
Alias: None
Product: ACE
Classification: Unclassified
Component: ACE Core (show other bugs)
Version: 5.4.6
Hardware: All All
: P3 normal
Assignee: DOC Center Support List (internal)
URL:
Depends on:
Blocks: 2181
  Show dependency tree
 
Reported: 2005-07-03 06:39 CDT by Johnny Willemsen
Modified: 2006-04-19 07:43 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 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