Skip to content

Commit 08933c3

Browse files
chore(dx8fvf): Remove unused vertex_size parameter from FVFInfoClass (TheSuperHackers#2615)
1 parent aea77d0 commit 08933c3

5 files changed

Lines changed: 11 additions & 10 deletions

File tree

Generals/Code/Libraries/Source/WWVegas/WW3D2/dx8vertexbuffer.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ VertexBufferClass::VertexBufferClass(unsigned type_, unsigned FVF, unsigned shor
8080
{
8181
WWASSERT(VertexCount);
8282
WWASSERT(type==BUFFER_TYPE_DX8 || type==BUFFER_TYPE_SORTING);
83+
WWASSERT(FVF != 0);
8384
fvf_info=W3DNEW FVFInfoClass(FVF);
8485

8586
_VertexBufferCount++;

GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dx8fvf.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ static unsigned Get_FVF_Vertex_Size(unsigned FVF)
4848
return D3DXGetFVFVertexSize(FVF);
4949
}
5050

51-
FVFInfoClass::FVFInfoClass(unsigned FVF_, unsigned vertex_size)
51+
FVFInfoClass::FVFInfoClass(unsigned FVF_)
5252
:
5353
FVF(FVF_),
54-
fvf_size(FVF!=0 ? Get_FVF_Vertex_Size(FVF) : vertex_size)
54+
fvf_size(Get_FVF_Vertex_Size(FVF))
5555
{
5656
location_offset=0;
5757
blend_offset=location_offset;

GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dx8fvf.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ class FVFInfoClass : public W3DMPO
258258
unsigned diffuse_offset;
259259
unsigned specular_offset;
260260
public:
261-
FVFInfoClass(unsigned FVF, unsigned vertex_size=0);
261+
FVFInfoClass(unsigned FVF);
262262

263263
unsigned Get_Location_Offset() const { return location_offset; }
264264
unsigned Get_Normal_Offset() const { return normal_offset; }

GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dx8vertexbuffer.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ static int _VertexBufferTotalSize;
7474
//
7575
// ----------------------------------------------------------------------------
7676

77-
VertexBufferClass::VertexBufferClass(unsigned type_, unsigned FVF, unsigned short vertex_count_, unsigned vertex_size)
77+
VertexBufferClass::VertexBufferClass(unsigned type_, unsigned FVF, unsigned short vertex_count_)
7878
:
7979
VertexCount(vertex_count_),
8080
type(type_),
@@ -83,8 +83,8 @@ VertexBufferClass::VertexBufferClass(unsigned type_, unsigned FVF, unsigned shor
8383
WWMEMLOG(MEM_RENDERER);
8484
WWASSERT(VertexCount);
8585
WWASSERT(type==BUFFER_TYPE_DX8 || type==BUFFER_TYPE_SORTING);
86-
WWASSERT((FVF!=0 && vertex_size==0) || (FVF==0 && vertex_size!=0));
87-
fvf_info=W3DNEW FVFInfoClass(FVF,vertex_size);
86+
WWASSERT(FVF != 0);
87+
fvf_info=W3DNEW FVFInfoClass(FVF);
8888

8989
_VertexBufferCount++;
9090
_VertexBufferTotalVertices+=VertexCount;
@@ -307,9 +307,9 @@ SortingVertexBufferClass::~SortingVertexBufferClass()
307307

308308
// bool dynamic=false,bool softwarevp=false);
309309

310-
DX8VertexBufferClass::DX8VertexBufferClass(unsigned FVF, unsigned short vertex_count_, UsageType usage, unsigned vertex_size)
310+
DX8VertexBufferClass::DX8VertexBufferClass(unsigned FVF, unsigned short vertex_count_, UsageType usage)
311311
:
312-
VertexBufferClass(BUFFER_TYPE_DX8, FVF, vertex_count_, vertex_size),
312+
VertexBufferClass(BUFFER_TYPE_DX8, FVF, vertex_count_),
313313
VertexBuffer(nullptr)
314314
{
315315
Create_Vertex_Buffer(usage);

GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dx8vertexbuffer.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ class VertexBufferClass : public W3DMPO, public RefCountClass
7979
//W3DMPO_GLUE(VertexBufferClass)
8080

8181
protected:
82-
VertexBufferClass(unsigned type, unsigned FVF, unsigned short VertexCount, unsigned vertex_size=0);
82+
VertexBufferClass(unsigned type, unsigned FVF, unsigned short VertexCount);
8383
virtual ~VertexBufferClass() override;
8484
public:
8585

@@ -211,7 +211,7 @@ class DX8VertexBufferClass : public VertexBufferClass
211211
USAGE_NPATCHES=4
212212
};
213213

214-
DX8VertexBufferClass(unsigned FVF, unsigned short VertexCount, UsageType usage=USAGE_DEFAULT, unsigned vertex_size=0); // Vertex size not used with FVF formats
214+
DX8VertexBufferClass(unsigned FVF, unsigned short VertexCount, UsageType usage=USAGE_DEFAULT);
215215
DX8VertexBufferClass(const Vector3* vertices, const Vector3* normals, const Vector2* tex_coords, unsigned short VertexCount,UsageType usage=USAGE_DEFAULT);
216216
DX8VertexBufferClass(const Vector3* vertices, const Vector3* normals, const Vector4* diffuse, const Vector2* tex_coords, unsigned short VertexCount,UsageType usage=USAGE_DEFAULT);
217217
DX8VertexBufferClass(const Vector3* vertices, const Vector4* diffuse, const Vector2* tex_coords, unsigned short VertexCount,UsageType usage=USAGE_DEFAULT);

0 commit comments

Comments
 (0)