eu07/maszyna
16
0
Fork 0
mirror of https://github.com/MaSzyna-EU07/maszyna.git synced 2026-07-22 13:59:19 +02:00
Code Issues 123 Wiki Activity

change iFlags to unsigned (shifting signed values is UB)

Browse Source
This commit is contained in:
milek7
2018-02-24 16:27:07 +01:00
parent a2881dfa9c
commit 2bd18519ed
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
Model3d.cpp
View File

@@ -748,12 +748,12 @@ int TSubModel::count_children() {
1 + Child->count_siblings() ); 1 + Child->count_siblings() );
} }
int TSubModel::FlagsCheck() uint32_t TSubModel::FlagsCheck()
{ // analiza koniecznych zmian pomiędzy submodelami { // analiza koniecznych zmian pomiędzy submodelami
// samo pomijanie glBindTexture() nie poprawi wydajności // samo pomijanie glBindTexture() nie poprawi wydajności
// ale można sprawdzić, czy można w ogóle pominąć kod do tekstur (sprawdzanie // ale można sprawdzić, czy można w ogóle pominąć kod do tekstur (sprawdzanie
// replaceskin) // replaceskin)
int i = 0; uint32_t i = 0;
if (Child) if (Child)
{ // Child jest renderowany po danym submodelu { // Child jest renderowany po danym submodelu
if (Child->m_material) // o ile ma teksturę if (Child->m_material) // o ile ma teksturę
@@ -1318,7 +1318,7 @@ void TSubModel::serialize(std::ostream &s,
sn_utils::ls_int32(s, (int32_t)get_container_pos(names, pName)); sn_utils::ls_int32(s, (int32_t)get_container_pos(names, pName));
sn_utils::ls_int32(s, (int)b_Anim); sn_utils::ls_int32(s, (int)b_Anim);
sn_utils::ls_int32(s, iFlags); sn_utils::ls_uint32(s, iFlags);
sn_utils::ls_int32(s, (int32_t)get_container_pos(transforms, *fMatrix)); sn_utils::ls_int32(s, (int32_t)get_container_pos(transforms, *fMatrix));
sn_utils::ls_int32(s, iNumVerts); sn_utils::ls_int32(s, iNumVerts);
@@ -1446,7 +1446,7 @@ void TSubModel::deserialize(std::istream &s)
b_Anim = (TAnimType)sn_utils::ld_int32(s); b_Anim = (TAnimType)sn_utils::ld_int32(s);
iFlags = sn_utils::ld_int32(s); iFlags = sn_utils::ld_uint32(s);
iMatrix = sn_utils::ld_int32(s); iMatrix = sn_utils::ld_int32(s);
iNumVerts = sn_utils::ld_int32(s); iNumVerts = sn_utils::ld_int32(s);

10
Model3d.h
View File

@@ -76,7 +76,7 @@ public: // chwilowo
TAnimType b_Anim{ at_None }; TAnimType b_Anim{ at_None };
private: private:
int iFlags{ 0x0200 }; // bit 9=1: submodel został utworzony a nie ustawiony na wczytany plik uint32_t iFlags{ 0x0200 }; // bit 9=1: submodel został utworzony a nie ustawiony na wczytany plik
// flagi informacyjne: // flagi informacyjne:
// bit 0: =1 faza rysowania zależy od wymiennej tekstury 0 // bit 0: =1 faza rysowania zależy od wymiennej tekstury 0
// bit 1: =1 faza rysowania zależy od wymiennej tekstury 1 // bit 1: =1 faza rysowania zależy od wymiennej tekstury 1
@@ -178,7 +178,7 @@ public:
inline void Hide() { iVisible = 0; }; inline void Hide() { iVisible = 0; };
void create_geometry( std::size_t &Dataoffset, gfx::geometrybank_handle const &Bank ); void create_geometry( std::size_t &Dataoffset, gfx::geometrybank_handle const &Bank );
int FlagsCheck(); uint32_t FlagsCheck();
void WillBeAnimated() void WillBeAnimated()
{ {
if (this) if (this)
@@ -192,7 +192,7 @@ public:
void Name_Material( std::string const &Name ); void Name_Material( std::string const &Name );
void Name( std::string const &Name ); void Name( std::string const &Name );
// Ra: funkcje do budowania terenu z E3D // Ra: funkcje do budowania terenu z E3D
int Flags() const { return iFlags; }; uint32_t Flags() const { return iFlags; };
void UnFlagNext() { iFlags &= 0x00FFFFFF; }; void UnFlagNext() { iFlags &= 0x00FFFFFF; };
void ColorsSet( glm::vec3 const &Ambient, glm::vec3 const &Diffuse, glm::vec3 const &Specular ); void ColorsSet( glm::vec3 const &Ambient, glm::vec3 const &Diffuse, glm::vec3 const &Specular );
// sets light level (alpha component of illumination color) to specified value // sets light level (alpha component of illumination color) to specified value
@@ -222,7 +222,7 @@ class TModel3d
private: private:
TSubModel *Root; // drzewo submodeli TSubModel *Root; // drzewo submodeli
int iFlags; // Ra: czy submodele mają przezroczyste tekstury uint32_t iFlags; // Ra: czy submodele mają przezroczyste tekstury
public: // Ra: tymczasowo public: // Ra: tymczasowo
int iNumVerts; // ilość wierzchołków (gdy nie ma VBO, to m_nVertexCount=0) int iNumVerts; // ilość wierzchołków (gdy nie ma VBO, to m_nVertexCount=0)
gfx::geometrybank_handle m_geometrybank; gfx::geometrybank_handle m_geometrybank;
@@ -251,7 +251,7 @@ public:
void LoadFromBinFile(std::string const &FileName, bool dynamic); void LoadFromBinFile(std::string const &FileName, bool dynamic);
bool LoadFromFile(std::string const &FileName, bool dynamic); bool LoadFromFile(std::string const &FileName, bool dynamic);
void SaveToBinFile(std::string const &FileName); void SaveToBinFile(std::string const &FileName);
int Flags() const { return iFlags; }; uint32_t Flags() const { return iFlags; };
void Init(); void Init();
std::string NameGet() const { return m_filename; }; std::string NameGet() const { return m_filename; };
int TerrainCount() const; int TerrainCount() const;