[?] Writing Vertex and Index data to mesh.

[sozoku]

Mesh ends up with incorrect/null values. What is wrong/what isnt there?

Really would appreciate some help, getting headaches from filling a ID3DXMesh.


Sozo/Moot.

[Tyrar]

invalid vertex format?

[sozoku]

Originally Posted by HeavyHacker

invalid vertex format?

Im pretty sure the vertex format isn't the problem as I created the mesh with the same FVF as the vertices being copied.

Edit:
It came up like this, so it maybe a pointer thing when using memcpy.

Mesh {
32;
12.619567;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,
0.000000;0.000000;0.000000;,

[tnd0]

looks like you are trying to copy data from a dynamic array. dynamic arrays are stored as pointers and their data do not necessarily have to be appended to eachother, meaning each element is stored 'somewhere randomly' in memory and your copy function expects all elements to be at "element1 + sizeof(element) * elementindex". use constant arrays instead.

[sozoku]

Originally Posted by tnd0

looks like you are trying to copy data from a dynamic array. dynamic arrays are stored as pointers and their data do not necessarily have to be appended to eachother, meaning each element is stored 'somewhere randomly' in memory and your copy function expects all elements to be at "element1 + sizeof(element) * elementindex". use constant arrays instead.

Epvp was down a bit last night so I didn't get to update.

Well I have half solved the problem, quite similar to what you said but then again not much, it was something to do with

This works on an off. So what your saying about it needing it to be constant makes most sense. Thanks for that going to try that now.

Edit: Seems like that I'm short of knowledge to pursue what you advise. :U
Edit2: I'm also seeing that I'm taking account the offset to the start of the stream, which I've fixed. And also relating to what you have said, when locking data, do I need to take into account flags to achieve constant void?
Edit3: So far:


Now for some reason I'm crashing.
Edit4:
Got it to stop crashing by just adding a & before vb. But as I get the mesh, I can help but feel that the vertices are inverted in some way. Maybe relating to the indices.
<-- What is supposed to be a ring.

[tnd0]

show how you fill the Vertex and Index arrays. and stop using screenshots, there's a [ CODE] tag for that.

[sozoku]

pDevice->GetFVF(&ppFVF);
				D3DXCreateMeshFVF(primCount,NumVertices,0,ppFVF,pDevice,&mesh); //D3DXMESH_SYSTEMMEM
				pDevice->GetStreamSource(0,&ppStreamData,&poffsetinbytes,&pstride);
				pDevice->GetIndices(&ppIndexData);
				
				ppStreamData->GetDesc(&pDescv);
				mesh->LockVertexBuffer( 0, &VertexPtr );
				ppStreamData->Lock(poffsetinbytes,pDescv.Size, (void**)&Vb,0); //ppStreamData->Lock(poffsetinbytes,pDescv.Size, (void**)Vb,0);
				memcpy( VertexPtr, Vb, (size_t)NumVertices * pstride );
				mesh->UnlockVertexBuffer();
				ppStreamData->Unlock();

				ppIndexData->GetDesc(&pDesc);
				UINT size = (pDesc.Format == D3DFMT_INDEX32) ? sizeof( UINT ) : sizeof( USHORT );
				mesh->LockIndexBuffer( 0, &IndexPtr );
				ppIndexData->Lock(0,pDesc.Size, &Ib,0); //ppIndexData->Lock(0,pDesc.Size,(void**)Ib,0);
				memcpy( IndexPtr, Ib, (size_t)NumVertices * size );
				mesh->UnlockIndexBuffer();
				ppIndexData->Unlock();

Technically, I'm getting vertex and index data from the current stream via a DIP hook and filter... :|

[tnd0]

I was asking how you fill your 'Vb' and 'Ib' var. post more source, i cant help you with that information.

[sozoku]

Originally Posted by tnd0

I was asking how you fill your 'Vb' and 'Ib' var. post more source, i cant help you with that information.

I dont think you understand. Its getting vb and ib from a drawindexedprimitive call.

[MrSm!th]

Originally Posted by tnd0

looks like you are trying to copy data from a dynamic array. dynamic arrays are stored as pointers and their data do not necessarily have to be appended to eachother, meaning each element is stored 'somewhere randomly' in memory and your copy function expects all elements to be at "element1 + sizeof(element) * elementindex". use constant arrays instead.

Wat?

[tnd0]

Originally Posted by MrSm!th

Wat?

Imagine you have 2 arrays, a dynamic and a static one, both arrays of integer.



first line: integer(&SomeStaticArray) = 0x00423EDC
second line: integer(&SomeDynamicArray) = 0x00423EEC
third line: integer(&SomeDynamicArray[0]) = 0x0063C2A8

now imagine you want to copy those data to your GPU memory via memcpy. what does memcopy? you pass 2 pointers, your source and your destination and the length. copying the static array with a pointer to the array is fine because the data are all located there. copying the dynamic array by passing a pointer to it, you will only copy the pointer to the first element and then some zero's if you're lucky, or get an AV else.

thats the difference.

I dont think you understand. Its getting vb and ib from a drawindexedprimitive call.

you dont want my help, do ya? the code you posted looks fine as far as one can tell, unless your variables are set up wrong, which I cant tell because I dont see how you fill your buffers. drawindexedprimitve does not fill a buffer, it takes a pointer to a dxstruct which gets passed to the gpu. if you hooked it and grabbed that buffer, it looks like you only grabbed the pointer so you gotta read what that pointer points to.

[sozoku]

Originally Posted by tnd0

Imagine you have 2 arrays, a dynamic and a static one, both arrays of integer.



first line: integer(&SomeStaticArray) = 0x00423EDC
second line: integer(&SomeDynamicArray) = 0x00423EEC
third line: integer(&SomeDynamicArray[0]) = 0x0063C2A8

now imagine you want to copy those data to your GPU memory via memcpy. what does memcopy? you pass 2 pointers, your source and your destination and the length. copying the static array with a pointer to the array is fine because the data are all located there. copying the dynamic array by passing a pointer to it, you will only copy the pointer to the first element and then some zero's if you're lucky, or get an AV else.

thats the difference.



you dont want my help, do ya? the code you posted looks fine as far as one can tell, unless your variables are set up wrong, which I cant tell because I dont see how you fill your buffers. drawindexedprimitve does not fill a buffer, it takes a pointer to a dxstruct which gets passed to the gpu. if you hooked it and grabbed that buffer, it looks like you only grabbed the pointer so you gotta read what that pointer points to.

" it looks like you only grabbed the pointer so you gotta read what that pointer points to." Seems so.

"you dont want my help, do ya?" Who said that? I just said that I haven't got anything relating to it in my code beyond that point.

In terms of variables :

D3DINDEXBUFFER_DESC *pIDesc;
D3DVERTEXBUFFER_DESC *pVDesc;
D3DMATERIAL9* pMaterials=NULL; 
LPD3DXBUFFER materialBuffer;
DWORD numMaterials;
ID3DXMesh* mesh;
IDirect3DIndexBuffer9* ppIndexData;
IDirect3DVertexBuffer9* ppStreamData;
LPDIRECT3DVERTEXBUFFER9 m_vb;
LPDIRECT3DINDEXBUFFER9 m_ib;
DWORD meshstride;
DWORD numvert;
DWORD numfaces;
LPVOID *ppbData=NULL;
BYTE* pVertices=NULL;
DWORD ppFVF;
UINT poffsetinbytes; 
UINT pstride;
LPVOID VertexPtr;
LPVOID IndexPtr;
const void *Ib;
const void *Vb;
D3DINDEXBUFFER_DESC pDesc;
D3DVERTEXBUFFER_DESC pDescv;
BOOLEAN record = FALSE;
D3DXMATERIAL mat;
IDirect3DTexture9 *g_texture=NULL;

^^ *some of this is irrelevant as it may be copied from other code lazily or not applicable to the problem.

[MrSm!th]

What do you mean by "dynamic array"? A linked list?
Because arrays allocated with new or std::vectors are located in a linear order.

edit:

Oh wait, did you mean a double pointer? I mean, a pointer to a pointer or a dynamically allocated two-dimensional array?
Like

int **ptr = new int*[10];
ptr[0] = new int[10];

Well that would indeed cause problems (even though i wouldn't call that just a "dynamic array", sincen it is actually a two-dimensional array), but I can't imagine how you figured that out of his first code snippet oO

[sozoku]

Seems like I found a somewhat similar problem on a Russian article comment. This would explain why very simple meshes work fine but others come out patchy as the proper transformation matrices, vertex shaders and so on haven't been applied. If I'm not mistaken.

Now to see if I can get Processvertices() going on in my code...