Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-22 16:41:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Purge by_ref_cached_

Browse Source
This commit is contained in:
Dion Moult
2021-12-01 15:18:04 +11:00
committed by Thomas Krijnen
parent 5ad6d455f9
commit 01bc29c5b2
2 changed files with 0 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcparse/IfcFile.h
-5
View File
@@ -135,7 +135,6 @@ private:
entities_by_type_t bytype_excl; entities_by_type_t bytype_excl;
entities_by_ref_t byref; entities_by_ref_t byref;
entities_by_ref_excl_t byref_excl; entities_by_ref_excl_t byref_excl;
ref_map_t by_ref_cached_;
entity_by_guid_t byguid; entity_by_guid_t byguid;
entity_entity_map_t entity_file_map; entity_entity_map_t entity_file_map;
@@ -265,10 +264,6 @@ public:
return return_value; return return_value;
} }
/// Marks entity as modified so that potential cache for it is invalidated.
/// @todo Currently the whole cache is invalidated. Implement more fine-grained invalidation.
void mark_entity_as_modified(int id);
unsigned int FreshId() { return ++MaxId; } unsigned int FreshId() { return ++MaxId; }
unsigned int getMaxId() { return MaxId; } unsigned int getMaxId() { return MaxId; }
src/ifcparse/IfcParse.cpp
-14
View File
@@ -1400,8 +1400,6 @@ void IfcEntityInstanceData::setArgument(size_t i, Argument* a, IfcUtil::Argument
if (this->file) { if (this->file) {
register_inverse_visitor visitor(*this->file, *this); register_inverse_visitor visitor(*this->file, *this);
apply_individual_instance_visitor(copy, i).apply(visitor); apply_individual_instance_visitor(copy, i).apply(visitor);
this->file->mark_entity_as_modified(id_);
} }
attributes_[i] = copy; attributes_[i] = copy;
@@ -1732,11 +1730,6 @@ aggregate_of_instance::ptr IfcFile::traverse_breadth_first(IfcUtil::IfcBaseClass
return IfcParse::traverse_breadth_first(instance, max_level); return IfcParse::traverse_breadth_first(instance, max_level);
} }
void IfcFile::mark_entity_as_modified(int /*id*/)
{
by_ref_cached_.clear();
}
void IfcFile::addEntities(aggregate_of_instance::ptr es) { void IfcFile::addEntities(aggregate_of_instance::ptr es) {
for( aggregate_of_instance::it i = es->begin(); i != es->end(); ++ i ) { for( aggregate_of_instance::it i = es->begin(); i != es->end(); ++ i ) {
addEntity(*i); addEntity(*i);
@@ -1984,10 +1977,6 @@ IfcUtil::IfcBaseClass* IfcFile::addEntity(IfcUtil::IfcBaseClass* entity, int id)
build_inverses_(new_entity); build_inverses_(new_entity);
} }
// @todo the id isn't actually used here, but instead
// clears the entire inverse cache map.
mark_entity_as_modified(0);
return new_entity; return new_entity;
} }
@@ -2123,7 +2112,6 @@ void IfcFile::process_deletion_() {
{ {
byref_excl.erase(name); byref_excl.erase(name);
} }
by_ref_cached_.erase(name);
} }
} }
} }
@@ -2216,8 +2204,6 @@ void IfcFile::process_deletion_() {
} }
} }
by_ref_cached_.clear();
batch_deletion_ids_.clear(); batch_deletion_ids_.clear();
} }