more work on smart pointers

This commit is contained in:
ZappoMan 2015-05-22 21:45:57 -07:00
parent 9988f2d1cd
commit 72ffd01920
7 changed files with 12 additions and 24 deletions

View file

@ -45,6 +45,5 @@ void RenderableBoxEntityItem::render(RenderArgs* args) {
DependencyManager::get<DeferredLightingEffect>()->renderSolidCube(batch, 1.0f, cubeColor);
}
// FIX ME!
//RenderableDebugableEntityItem::render(this, args);
RenderableDebugableEntityItem::render(this, args);
};

View file

@ -22,7 +22,7 @@
#include "RenderableDebugableEntityItem.h"
void RenderableDebugableEntityItem::renderBoundingBox(EntityItemPointer entity, RenderArgs* args,
void RenderableDebugableEntityItem::renderBoundingBox(EntityItem* entity, RenderArgs* args,
float puffedOut, glm::vec4& color) {
Q_ASSERT(args->_batch);
gpu::Batch& batch = *args->_batch;
@ -44,7 +44,7 @@ void RenderableDebugableEntityItem::renderHoverDot(EntityItem* entity, RenderArg
DependencyManager::get<DeferredLightingEffect>()->renderSolidSphere(batch, 0.5f, SLICES, STACKS, blueColor);
}
void RenderableDebugableEntityItem::render(EntityItemPointer entity, RenderArgs* args) {
void RenderableDebugableEntityItem::render(EntityItem* entity, RenderArgs* args) {
bool debugSimulationOwnership = args->_debugFlags & RenderArgs::RENDER_DEBUG_SIMULATION_OWNERSHIP;
if (debugSimulationOwnership) {

View file

@ -16,9 +16,9 @@
class RenderableDebugableEntityItem {
public:
static void renderBoundingBox(EntityItemPointer entity, RenderArgs* args, float puffedOut, glm::vec4& color);
static void renderHoverDot(EntityItemPointer entity, RenderArgs* args);
static void render(EntityItemPointer entity, RenderArgs* args);
static void renderBoundingBox(EntityItem* entity, RenderArgs* args, float puffedOut, glm::vec4& color);
static void renderHoverDot(EntityItem* entity, RenderArgs* args);
static void render(EntityItem* entity, RenderArgs* args);
};
#endif // hifi_RenderableDebugableEntityItem_h

View file

@ -191,12 +191,10 @@ void RenderableModelEntityItem::render(RenderArgs* args) {
if (!didDraw) {
glm::vec4 greenColor(0.0f, 1.0f, 0.0f, 1.0f);
// FIX ME
//RenderableDebugableEntityItem::renderBoundingBox(this, args, 0.0f, greenColor);
RenderableDebugableEntityItem::renderBoundingBox(this, args, 0.0f, greenColor);
}
// FIX ME
//RenderableDebugableEntityItem::render(this, args);
RenderableDebugableEntityItem::render(this, args);
}
Model* RenderableModelEntityItem::getModel(EntityTreeRenderer* renderer) {

View file

@ -103,8 +103,7 @@ EntityItemProperties EntityScriptingInterface::getEntityProperties(QUuid identit
if (_entityTree) {
_entityTree->lockForRead();
// FIX ME!!
EntityItemPointer entity = nullptr; // const_cast<EntityItemPointer>(_entityTree->findEntityByEntityItemID(EntityItemID(identity)));
EntityItemPointer entity = _entityTree->findEntityByEntityItemID(EntityItemID(identity));
if (entity) {
results = entity->getProperties();
@ -163,8 +162,7 @@ void EntityScriptingInterface::deleteEntity(QUuid id) {
if (_entityTree) {
_entityTree->lockForWrite();
// FIX ME!!
EntityItemPointer entity = nullptr; // const_cast<EntityItemPointer>(_entityTree->findEntityByEntityItemID(entityID));
EntityItemPointer entity = _entityTree->findEntityByEntityItemID(entityID);
if (entity) {
if (entity->getLocked()) {
shouldDelete = false;

View file

@ -39,15 +39,10 @@ void EntitySimulation::updateEntities() {
void EntitySimulation::getEntitiesToDelete(VectorOfEntities& entitiesToDelete) {
// FIX ME!!!
/*
for (auto entityItr : _entitiesToDelete) {
EntityItemPointer entity = (*entityItr);
for (auto entity : _entitiesToDelete) {
// this entity is still in its tree, so we insert into the external list
entitiesToDelete.push_back(entity);
++entityItr;
}
*/
_entitiesToDelete.clear();
}

View file

@ -560,8 +560,7 @@ bool EntityTreeElement::findSpherePenetration(const glm::vec3& center, float rad
if (findSphereSpherePenetration(center, radius, entityCenter, entityRadius, penetration)) {
// return true on first valid entity penetration
// FIX ME!!
//*penetratedObject = (void*)(entity);
*penetratedObject = (void*)(entity.get());
return true;
}
@ -641,7 +640,6 @@ EntityItemPointer EntityTreeElement::getEntityWithEntityItemID(const EntityItemI
void EntityTreeElement::cleanupEntities() {
uint16_t numberOfEntities = _entityItems->size();
for (uint16_t i = 0; i < numberOfEntities; i++) {
// FIX ME!!
EntityItemPointer entity = (*_entityItems)[i];
entity->_element = NULL;