Just to point to the code that I am worried about, here are the ccurrences:
./soap/SoapDeSerializer.cpp: ptrval = reinterpret_cast < unsigned long >(Array.m_Array); ./soap/SoapDeSerializer.cpp: reinterpret_cast < void *>(ptrval + nIndex * itemsize); ./soap/SoapDeSerializer.cpp: ptrval = reinterpret_cast < unsigned long >(Array.m_Array); ./soap/SoapDeSerializer.cpp: reinterpret_cast < ./common/ArrayBean.cpp: unsigned long ptrval = reinterpret_cast<unsigned long>(m_value.cta->pObject); ./common/ArrayBean.cpp: pItem = reinterpret_cast<void*>(ptrval+x*itemsize); ./common/ArrayBean.cpp: pItem = reinterpret_cast<void*>(ptrval+x*itemsize); which seems to be casting a pointer to an integer and then recasting it. Can we rewrite that to not do that? What would be the proper code to do that? And are there any other occurrences that I need to worry about? Nadir Amra/Rochester/[EMAIL PROTECTED] wrote on 04/10/2005 01:49:15 AM: > I am trying to run the client tests, and half of the tests are successful. > As I look at some of the failures that are occurring, I see stuff like > > ==> Pointer not set for location referenced. > > I examined one of the failing test cases, the > src\ws-axis\c\tests\auto_build\testcases\tests\ComplexLists.xml testcase. > And I looked at the code that is generated, and am wondering the > following: > > Does the code that is generated convert pointers to integers and > vice-versa? If it does, I think this needs to change. Not sure why it > does so but we cannot assume that on all platforms a pointer fits into an > integer (specially 64-bit machine), and it is bad practice for portability > given that on some platforms pointers are protected, and if you convert it > to some other data type, you can never retrieve the pointer back. > > Does the code perform object copies via memcpy instead of using copy > constructors? Why the heck do we do that? Again, I am not sure if that > is legit in the object-oriented world and I do not think it is a good > practice to do so. > > If either case is true, can someone tell me what I would need to change? > Any help would be greatly appreciated. > > > > >
