Merge pull request #2976 from hbina/fix_clang_warnings
Fixed a bunch of clang warningspull/2984/head^2
commit
c9244a4ae3
|
@ -250,7 +250,7 @@ aiNode* COBImporter::BuildNodes(const Node& root,const Scene& scin,aiScene* fill
|
||||||
const Mesh& ndmesh = (const Mesh&)(root);
|
const Mesh& ndmesh = (const Mesh&)(root);
|
||||||
if (ndmesh.vertex_positions.size() && ndmesh.texture_coords.size()) {
|
if (ndmesh.vertex_positions.size() && ndmesh.texture_coords.size()) {
|
||||||
|
|
||||||
typedef std::pair<unsigned int,Mesh::FaceRefList> Entry;
|
typedef std::pair<const unsigned int,Mesh::FaceRefList> Entry;
|
||||||
for(const Entry& reflist : ndmesh.temp_map) {
|
for(const Entry& reflist : ndmesh.temp_map) {
|
||||||
{ // create mesh
|
{ // create mesh
|
||||||
size_t n = 0;
|
size_t n = 0;
|
||||||
|
|
|
@ -323,7 +323,7 @@ public:
|
||||||
// oh well.
|
// oh well.
|
||||||
bool have_param = false, have_point = false;
|
bool have_param = false, have_point = false;
|
||||||
IfcVector3 point;
|
IfcVector3 point;
|
||||||
for(const Entry sel :entity.Trim1) {
|
for(const Entry& sel :entity.Trim1) {
|
||||||
if (const ::Assimp::STEP::EXPRESS::REAL* const r = sel->ToPtr<::Assimp::STEP::EXPRESS::REAL>()) {
|
if (const ::Assimp::STEP::EXPRESS::REAL* const r = sel->ToPtr<::Assimp::STEP::EXPRESS::REAL>()) {
|
||||||
range.first = *r;
|
range.first = *r;
|
||||||
have_param = true;
|
have_param = true;
|
||||||
|
@ -340,7 +340,7 @@ public:
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
have_param = false, have_point = false;
|
have_param = false, have_point = false;
|
||||||
for(const Entry sel :entity.Trim2) {
|
for(const Entry& sel :entity.Trim2) {
|
||||||
if (const ::Assimp::STEP::EXPRESS::REAL* const r = sel->ToPtr<::Assimp::STEP::EXPRESS::REAL>()) {
|
if (const ::Assimp::STEP::EXPRESS::REAL* const r = sel->ToPtr<::Assimp::STEP::EXPRESS::REAL>()) {
|
||||||
range.second = *r;
|
range.second = *r;
|
||||||
have_param = true;
|
have_param = true;
|
||||||
|
|
|
@ -1421,11 +1421,11 @@ void MDLImporter::InternReadFile_3DGS_MDL7( )
|
||||||
avOutList[i].reserve(3);
|
avOutList[i].reserve(3);
|
||||||
|
|
||||||
// buffer to held the names of all groups in the file
|
// buffer to held the names of all groups in the file
|
||||||
const size_t buffersize( AI_MDL7_MAX_GROUPNAMESIZE*pcHeader->groups_num );
|
const size_t buffersize(AI_MDL7_MAX_GROUPNAMESIZE*pcHeader->groups_num);
|
||||||
char* aszGroupNameBuffer = new char[ buffersize ];
|
char* aszGroupNameBuffer = new char[ buffersize ];
|
||||||
|
|
||||||
// read all groups
|
// read all groups
|
||||||
for (unsigned int iGroup = 0; iGroup < (unsigned int)pcHeader->groups_num;++iGroup) {
|
for (unsigned int iGroup = 0; iGroup < (unsigned int)pcHeader->groups_num; ++iGroup) {
|
||||||
MDL::IntGroupInfo_MDL7 groupInfo((BE_NCONST MDL::Group_MDL7*)szCurrent,iGroup);
|
MDL::IntGroupInfo_MDL7 groupInfo((BE_NCONST MDL::Group_MDL7*)szCurrent,iGroup);
|
||||||
szCurrent = (const unsigned char*)(groupInfo.pcGroup+1);
|
szCurrent = (const unsigned char*)(groupInfo.pcGroup+1);
|
||||||
|
|
||||||
|
|
|
@ -685,7 +685,7 @@ bool XGLImporter::ReadMesh(TempScope& scope)
|
||||||
}
|
}
|
||||||
|
|
||||||
// finally extract output meshes and add them to the scope
|
// finally extract output meshes and add them to the scope
|
||||||
typedef std::pair<unsigned int, TempMaterialMesh> pairt;
|
typedef std::pair<const unsigned int, TempMaterialMesh> pairt;
|
||||||
for(const pairt& p : bymat) {
|
for(const pairt& p : bymat) {
|
||||||
aiMesh* const m = ToOutputMesh(p.second);
|
aiMesh* const m = ToOutputMesh(p.second);
|
||||||
scope.meshes_linear.push_back(m);
|
scope.meshes_linear.push_back(m);
|
||||||
|
|
Loading…
Reference in New Issue