Quote:Original post by Grain
BTW: If it helps any, I put a break point in the OctTree constructor and its not being called on this line: if(Child == NULL){Child = new OctTree;} how ever the other one where the allocation works the constructor is called.
Quote:Original post by Basiror
@snk_kid:let me guess the placement new doesn t initialize the VFT pointer right?
the compiler sets up the VFT pointer as i just read on google
void OctTree::add_geometry(vector3d* V, unsigned int num_verts, unsigned int *I, unsigned int num_tris){ OctTree *T1 = new OctTree; verts = V; index = I; NumTris = num_tris; num_tris = 0; unsigned int *new_I = NULL; if(Parent == NULL) { B.add_points(V,num_verts); } if(NumTris > 1) { OctTree *T2 = new OctTree; for(int i = 0; i < 8; i++) { OctTree *T3 = new OctTree; AABB Box = B.eighth(i); for(unsigned int j = 0; j < NumTris; j++) { if( Box.contains(V[I[j*3]]) || Box.contains(V[I[j*3+1]]) || Box.contains(V[I[j*3+2]])) { num_tris++; } } OctTree *T4 = new OctTree; for(unsigned int j = 0, k = 0; j < NumTris; j++) { if( Box.contains(V[I[j*3]]) || Box.contains(V[I[j*3+1]]) || Box.contains(V[I[j*3+2]])) { if(new_I == NULL) new_I = new unsigned int[num_tris]; new_I[k*3] = I[j*3]; new_I[k*3+1] = I[j*3+1]; new_I[k*3+2] = I[j*3+2]; k++; } } OctTree *T5 = new OctTree; if(num_tris > 0) { if(Child == NULL) { Child = new OctTree; } Child->B = Box; Child->Parent = this; Child->add_geometry(V, num_verts, new_I, num_tris); } num_tris = 0; } }} for(unsigned int j = 0, k = 0; j < NumTris; j++) { if( Box.contains(V[I[j*3]]) || Box.contains(V[I[j*3+1]]) || Box.contains(V[I[j*3+2]])) { if(new_I == NULL) new_I = new unsigned int[num_tris]; new_I[k*3] = I[j*3]; new_I[k*3+1] = I[j*3+1]; new_I[k*3+2] = I[j*3+2]; k++; } } OctTree *Child[8]; //in the variable declarations//and then Child = NULL; // in the constructorQuote:Original post by VergOctTree *Child[8]; //in the variable declarations
Child is declared as a pointer to an array; since you haven't defined the array, you're attempting to NULL out memory any old place.
for(unsigned int j = 0, k = 0; j < NumTris; j++) { if( Box.contains(V[I[j*3]]) || Box.contains(V[I[j*3+1]]) || Box.contains(V[I[j*3+2]])) { if(new_I == NULL) new_I = new unsigned int[num_tris*3]; //I forgot that A triangle is 3 unsigned ints. new_I[k*3] = I[j*3]; new_I[k*3+1] = I[j*3+1]; new_I[k*3+2] = I[j*3+2]; k++; } }Quote:Original post by KippesoepQuote:Original post by VergOctTree *Child[8]; //in the variable declarations
Child is declared as a pointer to an array; since you haven't defined the array, you're attempting to NULL out memory any old place.
It's not a pointer to an array, it's an array of 8 pointers.
Quote:Original post by Grain
But why didn’t I get an access violation when I say new_I[j*3+2]?
