Skip to content

Commit cb31968

Browse files
committed
Fix error in WbView3d::getModelNameAndScale
1 parent c9311da commit cb31968

File tree

2 files changed

+20
-10
lines changed

2 files changed

+20
-10
lines changed

Generals/Code/Tools/WorldBuilder/src/wbview3d.cpp

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1250,15 +1250,18 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
12501250
break;
12511251
}
12521252

1253-
AsciiString modelName("No Model Name");
1253+
AsciiString modelName;
12541254
*scale = 1.0f;
1255-
Int i;
1255+
1256+
#ifdef LOAD_TEST_ASSETS
12561257
char buffer[ _MAX_PATH ];
1258+
12571259
if (strncmp(TEST_STRING, pMapObj->getName().str(), strlen(TEST_STRING)) == 0)
12581260
{
12591261
/* Handle test art models here */
12601262
strcpy(buffer, pMapObj->getName().str());
12611263

1264+
Int i;
12621265
for (i=0; buffer[i]; i++) {
12631266
if (buffer[i] == '/') {
12641267
i++;
@@ -1267,7 +1270,9 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
12671270
}
12681271
modelName = buffer+i;
12691272
}
1270-
else
1273+
#endif
1274+
1275+
if (modelName.isEmpty())
12711276
{
12721277
modelName = "No Model Name"; // must be this while GDF exists (it's the default)
12731278
const ThingTemplate *tTemplate;
@@ -1280,8 +1285,8 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
12801285
modelName = getBestModelName(tTemplate, state);
12811286
*scale = tTemplate->getAssetScale();
12821287

1283-
} // end if
1284-
} // end else
1288+
}
1289+
}
12851290
return modelName;
12861291
}
12871292

GeneralsMD/Code/Tools/WorldBuilder/src/wbview3d.cpp

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1307,15 +1307,18 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
13071307
break;
13081308
}
13091309

1310-
AsciiString modelName("No Model Name");
1310+
AsciiString modelName;
13111311
*scale = 1.0f;
1312-
Int i;
1312+
1313+
#ifdef LOAD_TEST_ASSETS
13131314
char buffer[ _MAX_PATH ];
1315+
13141316
if (strncmp(TEST_STRING, pMapObj->getName().str(), strlen(TEST_STRING)) == 0)
13151317
{
13161318
/* Handle test art models here */
13171319
strcpy(buffer, pMapObj->getName().str());
13181320

1321+
Int i;
13191322
for (i=0; buffer[i]; i++) {
13201323
if (buffer[i] == '/') {
13211324
i++;
@@ -1324,7 +1327,9 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
13241327
}
13251328
modelName = buffer+i;
13261329
}
1327-
else
1330+
#endif
1331+
1332+
if (modelName.isEmpty())
13281333
{
13291334
modelName = "No Model Name"; // must be this while GDF exists (it's the default)
13301335
const ThingTemplate *tTemplate;
@@ -1337,8 +1342,8 @@ AsciiString WbView3d::getModelNameAndScale(MapObject *pMapObj, Real *scale, Body
13371342
modelName = getBestModelName(tTemplate, state);
13381343
*scale = tTemplate->getAssetScale();
13391344

1340-
} // end if
1341-
} // end else
1345+
}
1346+
}
13421347
return modelName;
13431348
}
13441349

0 commit comments

Comments
 (0)