Bug 259

Summary:	ORB_Core: "naked" new() operator is used instead of ACE_NEW* macros
Product:	TAO	Reporter:	Ossama Othman <ossama.othman>
Component:	ORB	Assignee:	DOC Center Support List (internal) <tao-support>
Status:	NEW ---
Severity:	minor
Priority:	P4
Version:	1.0.2
Hardware:	All
OS:	All

Description Ossama Othman 1999-08-25 13:48:11 CDT

There are many instances in TAO that use the raw C++ new() operator instead of
the appropriate ACE_NEW macro (e.g., ACE_NEW(), ACE_NEW_RETURN(),
ACE_NEW_THROW_EX(), etc.).  TAO should be updated to use ACE_NEW* macros.

Use commands such as the following to find most of the instances:

   find . -type f -name '*.cpp' | xargs egrep -n 'return new.?|=.?new.?'

and

   find . -type f -name '*.i' | xargs egrep -n 'return new.?|=.?new.?'

Comment 1 Carlos O'Ryan 1999-08-27 10:44:59 CDT

Ossama: since you reported the bug could you please keep track of it?

Comment 2 Ossama Othman 1999-08-27 12:12:59 CDT

Okay, I'll handle it as soon as I'm done with the first release of the TP4
support and the scalability benchmarks.

Comment 3 Ossama Othman 1999-09-20 19:41:59 CDT

I nailed a few but there are still *alot* left.  This is going to be ugly. :(

Comment 4 Ossama Othman 1999-11-09 11:34:59 CST

I'm not going to have time to work on this.  Can someone else please take it?

Comment 5 Ossama Othman 1999-11-16 15:50:59 CST

I've been asked to make tao-support accept this bug.

Comment 6 Ossama Othman 1999-11-16 16:15:59 CST

Assigning it to Bala.

Comment 7 Nanbor Wang 1999-11-24 12:58:59 CST

I can do it along the way

Comment 8 Ossama Othman 2001-11-14 23:08:58 CST

Didn't Michael Kircher address this bug?

Comment 9 Nanbor Wang 2001-11-15 06:47:32 CST

I guess he did  all of ACE. The availabilty of the following 
___________________________________________________________
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_except,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_except,
Typecode_Constants.cpp:    new CORBA::TypeCode(CORBA::tk_except,
Typecode_Constants.cpp:  CORBA::_tc_ORBid = new CORBA::TypeCode (CORBA::tk_alias
,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_objref,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_objref,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_except,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:   new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:   new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_struct,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_struct,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:    new CORBA::TypeCode(CORBA::tk_except,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_except,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_alias,
Typecode_Constants.cpp:    new CORBA::TypeCode (CORBA::tk_objref,
__________________________________________________________________

is an example that it is not fixed :-)

Comment 10 Johnny Willemsen 2007-09-20 03:08:58 CDT

to pool