Bug 1621 - Un-deprecate CORBA::Request::env()
Summary: Un-deprecate CORBA::Request::env()
Status: NEW
Alias: None
Product: TAO
Classification: Unclassified
Component: ORB (show other bugs)
Version: 1.3.4
Hardware: All All
: P3 normal
Assignee: DOC Center Support List (internal)
URL:
Depends on:
Blocks:
 
Reported: 2003-10-27 05:41 CST by Oliver M. Kellogg
Modified: 2006-12-06 06:10 CST (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 Oliver M. Kellogg 2003-10-27 05:41:25 CST 
In the course of the corba_env_clean overhaul, we
deprecated the method env() in the CORBA::Request
interface (see $TAO_ROOT/tao/DynamicInterface/Request.h)

However, when using the DII, the env() method may still
be required.

Thus it would be helpful to either bring the env() method
back to life for DII users, or to give a recipe on how
DII users can circumvent its lack.
Comment 1 Nanbor Wang 2003-11-05 19:55:44 CST 
Assigning  the bug to me. 

Oliver, I am not yet clear on the motivation why this is needed by the application. If you can 
provide me a use case scenraio on why this is needed, it would be awesome. 

Thanks
Comment 2 Oliver M. Kellogg 2003-11-06 00:04:39 CST 
Bala,

The question arose when I attempted to port CorbaScript
(http://corbaweb.lifl.fr/CorbaScript/index.html) to newer
versions of TAO.
For more detail, see
http://groups.google.com/groups?q=env+%2B+corbascript+group:comp.soft-
sys.ace&hl=en&lr=&ie=UTF-8&group=comp.soft-
sys.ace&safe=off&selm=3F18033C.8855C9E%40sysde.eads.net&rnum=2

It seems that all other CorbaScript supported ORBs support the
CORBA::Request::env() operation.

Thanks for looking into this.
Comment 3 Nanbor Wang 2003-11-06 12:04:37 CST 
Oliver-

I took a look at the spec and looks like the env () is required. Time to fix this. !
Comment 4 Oliver M. Kellogg 2003-11-06 20:07:41 CST 
Bala,

> I took a look at the spec and looks like the env () is required.

Yes. One interesting thing is, CORBA 3.0 actually _does_ deprecate
the operation. (At least I found no mention of env() in the CORBA
3.0.1 PIDL file for CORBA::Request.)
Now I'm not sure which CORBA version TAO is targeting, but I think
it would be friendly to CORBA 2.x users to support it.
Comment 5 Nanbor Wang 2003-11-07 07:21:17 CST 
Oliver-

The latest C++ mapping spec requires it anyway. We have to go by that. Please 
see

http://www.omg.org/docs/formal/03-06-03.pdf

sections 1.30, specifically 1.30.2 on pages 1.119 and 1.120

HTH
Comment 6 Nanbor Wang 2004-01-03 09:31:02 CST 
Accepting this
Comment 7 Nanbor Wang 2004-08-25 08:15:17 CDT 
This should be less than a day's worth of work. We need to pass the environment pointer and use that 
while making requests. Should be easy to fix it as soon as we can get to it.
Comment 8 Oliver M. Kellogg 2004-10-12 02:06:51 CDT 
Hi Bala,

You wrote:

> [...] We need to pass the environment pointer and use that 
> while making requests.

I gave this a shot but I was not sure where the environment
pointer comes from in native exception builds
(the ACE_ENV_ macros resolve to thin air in this case?)

Thanks.
Comment 9 Johnny Willemsen 2006-12-06 06:10:49 CST 
to pool