Skip to content

Commit

Permalink
gobj: use range for where possible in PreparedGraphicsObjects
Browse files Browse the repository at this point in the history
This makes the code easier to read and maintain.
  • Loading branch information
rdb committed Nov 9, 2019
1 parent 9201d75 commit f1b235e
Showing 1 changed file with 13 additions and 38 deletions.
51 changes: 13 additions & 38 deletions panda/src/gobj/preparedGraphicsObjects.cxx
Expand Up @@ -274,11 +274,7 @@ release_all_textures() {


int num_textures = (int)_prepared_textures.size() + (int)_enqueued_textures.size(); int num_textures = (int)_prepared_textures.size() + (int)_enqueued_textures.size();


Textures::iterator tci; for (TextureContext *tc : _prepared_textures) {
for (tci = _prepared_textures.begin();
tci != _prepared_textures.end();
++tci) {
TextureContext *tc = (*tci);
tc->get_texture()->clear_prepared(tc->get_view(), this); tc->get_texture()->clear_prepared(tc->get_view(), this);
tc->_object = nullptr; tc->_object = nullptr;


Expand Down Expand Up @@ -599,11 +595,7 @@ release_all_geoms() {


int num_geoms = (int)_prepared_geoms.size() + (int)_enqueued_geoms.size(); int num_geoms = (int)_prepared_geoms.size() + (int)_enqueued_geoms.size();


Geoms::iterator gci; for (GeomContext *gc : _prepared_geoms) {
for (gci = _prepared_geoms.begin();
gci != _prepared_geoms.end();
++gci) {
GeomContext *gc = (*gci);
gc->_geom->clear_prepared(this); gc->_geom->clear_prepared(this);
gc->_geom = nullptr; gc->_geom = nullptr;


Expand Down Expand Up @@ -774,11 +766,7 @@ release_all_shaders() {


int num_shaders = (int)_prepared_shaders.size() + (int)_enqueued_shaders.size(); int num_shaders = (int)_prepared_shaders.size() + (int)_enqueued_shaders.size();


Shaders::iterator sci; for (ShaderContext *sc : _prepared_shaders) {
for (sci = _prepared_shaders.begin();
sci != _prepared_shaders.end();
++sci) {
ShaderContext *sc = (*sci);
sc->_shader->clear_prepared(this); sc->_shader->clear_prepared(this);
sc->_shader = nullptr; sc->_shader = nullptr;


Expand Down Expand Up @@ -952,11 +940,8 @@ release_all_vertex_buffers() {


int num_vertex_buffers = (int)_prepared_vertex_buffers.size() + (int)_enqueued_vertex_buffers.size(); int num_vertex_buffers = (int)_prepared_vertex_buffers.size() + (int)_enqueued_vertex_buffers.size();


Buffers::iterator vbci; for (BufferContext *bc : _prepared_vertex_buffers) {
for (vbci = _prepared_vertex_buffers.begin(); VertexBufferContext *vbc = (VertexBufferContext *)bc;
vbci != _prepared_vertex_buffers.end();
++vbci) {
VertexBufferContext *vbc = (VertexBufferContext *)(*vbci);
vbc->get_data()->clear_prepared(this); vbc->get_data()->clear_prepared(this);
vbc->_object = nullptr; vbc->_object = nullptr;


Expand All @@ -973,9 +958,8 @@ release_all_vertex_buffers() {
++bci) { ++bci) {
BufferList &buffer_list = (*bci).second; BufferList &buffer_list = (*bci).second;
nassertr(!buffer_list.empty(), num_vertex_buffers); nassertr(!buffer_list.empty(), num_vertex_buffers);
BufferList::iterator li; for (BufferContext *bc : buffer_list) {
for (li = buffer_list.begin(); li != buffer_list.end(); ++li) { VertexBufferContext *vbc = (VertexBufferContext *)bc;
VertexBufferContext *vbc = (VertexBufferContext *)(*li);
_released_vertex_buffers.push_back(vbc); _released_vertex_buffers.push_back(vbc);
} }
} }
Expand Down Expand Up @@ -1150,11 +1134,8 @@ release_all_index_buffers() {


int num_index_buffers = (int)_prepared_index_buffers.size() + (int)_enqueued_index_buffers.size(); int num_index_buffers = (int)_prepared_index_buffers.size() + (int)_enqueued_index_buffers.size();


Buffers::iterator ibci; for (BufferContext *bc : _prepared_index_buffers) {
for (ibci = _prepared_index_buffers.begin(); IndexBufferContext *ibc = (IndexBufferContext *)bc;
ibci != _prepared_index_buffers.end();
++ibci) {
IndexBufferContext *ibc = (IndexBufferContext *)(*ibci);
ibc->get_data()->clear_prepared(this); ibc->get_data()->clear_prepared(this);
ibc->_object = nullptr; ibc->_object = nullptr;


Expand All @@ -1171,10 +1152,9 @@ release_all_index_buffers() {
++bci) { ++bci) {
BufferList &buffer_list = (*bci).second; BufferList &buffer_list = (*bci).second;
nassertr(!buffer_list.empty(), num_index_buffers); nassertr(!buffer_list.empty(), num_index_buffers);
BufferList::iterator li; for (BufferContext *bc : buffer_list) {
for (li = buffer_list.begin(); li != buffer_list.end(); ++li) { IndexBufferContext *ibc = (IndexBufferContext *)bc;
IndexBufferContext *vbc = (IndexBufferContext *)(*li); _released_index_buffers.push_back(ibc);
_released_index_buffers.push_back(vbc);
} }
} }
_index_buffer_cache.clear(); _index_buffer_cache.clear();
Expand Down Expand Up @@ -1337,12 +1317,7 @@ release_all_shader_buffers() {


int num_shader_buffers = (int)_prepared_shader_buffers.size() + (int)_enqueued_shader_buffers.size(); int num_shader_buffers = (int)_prepared_shader_buffers.size() + (int)_enqueued_shader_buffers.size();


Buffers::iterator bci; for (BufferContext *bc : _prepared_shader_buffers) {
for (bci = _prepared_shader_buffers.begin();
bci != _prepared_shader_buffers.end();
++bci) {

BufferContext *bc = (BufferContext *)(*bci);
((ShaderBuffer *)bc->_object)->clear_prepared(this); ((ShaderBuffer *)bc->_object)->clear_prepared(this);
bc->_object = nullptr; bc->_object = nullptr;
_released_shader_buffers.push_back(bc); _released_shader_buffers.push_back(bc);
Expand Down

0 comments on commit f1b235e

Please sign in to comment.