Skip to content

Commit

Permalink
Fix an unused variable warning in SnapshotTest.setTexturesAfterRestore (
Browse files Browse the repository at this point in the history
#1559)

The CI scripts do RelWithDebInfo builds now, so the assert() will get eliminated since it's a release build
  • Loading branch information
ericwa authored and kduske committed Dec 21, 2016
1 parent f7fe54e commit b2d1aa2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/src/View/SnapshotTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,13 @@ namespace TrenchBroom {
document->addNode(brush, document->currentParent());

const Assets::Texture* texture = document->textureManager().texture("coffin1");
assert(texture->usageCount() == 6u);
ASSERT_EQ(6u, texture->usageCount());

for (Model::BrushFace* face : brush->faces())
assert(face->texture() == texture);
ASSERT_EQ(texture, face->texture());

document->translateObjects(Vec3(1, 1, 1));
assert(texture->usageCount() == 6u);
ASSERT_EQ(6u, texture->usageCount());

document->undoLastCommand();
ASSERT_EQ(6u, texture->usageCount());
Expand Down

0 comments on commit b2d1aa2

Please sign in to comment.