Bug 1852

Summary: Exception specification should be removed from operations and attributes.
Product: TAO Reporter: Ossama Othman <ossama.othman>
Component: IDL CompilerAssignee: Johnny Willemsen <jwillemsen>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P3    
Version: 1.4.1   
Hardware: All   
OS: All   
URL: http://www.omg.org/issues/issue4265.txt
Bug Depends on:    
Bug Blocks: 2181, 1278, 2064    

Description Ossama Othman 2004-06-16 01:32:03 CDT 
According to Section 1.20 "Mapping for Operations and Attributes" in the latest
C++ mapping, operations and attributes should not have exception specifications.
 TAO_IDL still generates exception specifications for operations and attributes.

We'll also have to remove the code in TAO that overrides the run-time's
unexpected exception handler.
Comment 1 Jeff Parsons 2004-06-28 10:01:50 CDT 
accepted.
Comment 2 Jeff Parsons 2004-08-25 09:14:18 CDT 
accepted again.
Comment 3 Ossama Othman 2004-09-05 02:08:25 CDT 
To be clear, the same goes for the skeleton operations, as described in Section
1.38 "Implementing Operations" of the C++ mapping.
Comment 4 Johnny Willemsen 2005-03-07 15:16:38 CST 
This will impact footprint also:

I just thought about the fact that the footprint build uses emulated C++ 
exceptions so I can't see the impact of the ACE_THROW_SPEC change I did 
earlier. For the collocated case I had to remove some ACE_THROW_SPEC from some 
method definitions I tested this on a Linux system quickly with GCC 3.3.4. I 
tested things on tao/Adapter_Registry.{h,cpp}. What I did was to remove 
ACE_THROW_SPEC(CORBA::SystemException) from the dispatch method.

With ACE_THROW_SPEC the size is: 2927 bytes
Without: 2823. 

So, about 100 bytes reduction, which is about 3% just by not using 
ACE_THROW_SPEC. So the bugzilla bug 1852 to not generate ACE_THROW_SPEC does 
have also benefit our footprint when using native C++ exceptions.
Comment 5 Johnny Willemsen 2005-08-22 03:39:38 CDT 
This will also give some behaviour change. At the moment an exception is thrown
from the deep that is not in the exception specification list we now convert
this for most runtimes to an unknown exception using the exception handler. We
will loose this functionality when dropping exception specifications.
Comment 6 Johnny Willemsen 2005-10-20 03:36:07 CDT 
should we make this a tao_idl compiler option which is enabled by default?
Comment 7 Jeff Parsons 2005-10-20 06:45:04 CDT 
Yes, once we complete the corresponding changes to the rest of TAO+CIAO.
Comment 8 Johnny Willemsen 2007-02-05 14:22:20 CST 
taking this over, merging Ossama changes to the core to handle this, if this
works I will zap ACE_THROW_SPEC from all code
Comment 9 Johnny Willemsen 2007-02-05 15:17:57 CST 
changed ready in my workspace
Comment 10 Johnny Willemsen 2007-04-11 06:00:51 CDT 
fixed