Bug 1301 - Missing emulated exception support in CosTime::UTO impl.
Summary: Missing emulated exception support in CosTime::UTO impl.
Status: RESOLVED WONTFIX
Alias: None
Product: TAO
Classification: Unclassified
Component: Time Service (show other bugs)
Version: 1.2.4
Hardware: All All
: P3 normal
Assignee: DOC Center Support List (internal)
URL:
Depends on: 2256
Blocks:
  Show dependency tree
 
Reported: 2002-09-02 20:59 CDT by Ossama Othman
Modified: 2006-08-16 03:51 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 Ossama Othman 2002-09-02 20:59:37 CDT 
TAO's CosTime::UTO implementation is missing emulated exception support in 
some places in the implementation.  Specifically, there a number of calls 
to "this->time()" in the implementation.  The time() method is actually 
defined in IDL meaning that the emulated exception enabled implementation will 
have a default argument that accesses the CORBA::Environment in TSS.  This can 
greatly affect the performance of these methods due to *multiple* TSS accesses.

The appropriate emulated exception macro (e.g. ACE_ENG_SINGLE_ARG_PARAMETER) 
should be passed to such calls to avoid incurring unnecessary TSS accesses.

See the file:
  orbsvcs/orbsvcs/Time/TAO_UTO.cpp
Comment 1 Ossama Othman 2002-09-02 21:04:31 CDT 
There is an absurd amount of debugging code in 
orbsvcs/orbsvcs/Time/Timer_Helper.cpp that incurs a large number of TSS 
accessed in the emulated exception case.  Those should be fixed, too.
Comment 2 Ossama Othman 2002-09-15 15:41:27 CDT 
Accepted.
Comment 3 Johnny Willemsen 2006-08-16 03:51:46 CDT 
we don't support emulated exceptions anymore, so closing this one as won't fix