# Build fix: remove `boost_system` from CMake components `Boost.System` became header-only in Boost 1.69. Boost 1.90.0 no longer ships a compiled library or CMake config for it, so `find_package(Boost REQUIRED COMPONENTS system ...)` fails. ## Fix `cmake/CMakeLists.txt`: ```diff - set(BOOST_COMPONENTS system program_options regex thread date_time iostreams) + set(BOOST_COMPONENTS program_options regex thread date_time iostreams) ``` The headers are still available; no linking is needed. # Build fix: add `template` keyword for dependent template member calls Calling a template member function through a dependent expression (e.g. `storage->has_attribute_value(...)` where `storage`'s type depends on a template parameter) requires the `template` keyword to disambiguate from a less-than comparison. ## Error ``` src/ifcparse/IfcParse.cpp:1856:67: error: expected primary-expression before '>' token 1856 | if (storage->has_attribute_value(attr_index)) { | ^ ``` Six identical errors at lines 1856, 1865, 1896, 1905, 1934, 1943. ## Fix `src/ifcparse/IfcParse.cpp`: ```diff -storage->has_attribute_value(attr_index) +storage->template has_attribute_value(attr_index) -storage->has_attribute_value(attr_index) +storage->template has_attribute_value(attr_index) ``` Applied at all six call sites in `in_memory_file_storage::read_from_stream`. # Linker fix: missing explicit template instantiations for `InstanceStreamer` `InstanceStreamer` is a class template with methods defined in `IfcParse.cpp`, not the header. Without explicit instantiations, the linker can't find the symbols when the SWIG wrapper loads. ## Error ``` ImportError: undefined symbol: _ZN8IfcParse16InstanceStreamerINS_10FileReaderINS_14FullBufferImplEEEEC1EPS3_PNS_7IfcFileE (IfcParse::InstanceStreamer>::InstanceStreamer(FileReader*, IfcFile*)) ``` ## Fix Cannot use `template class InstanceStreamer<...>` because some constructors have `static_assert` guards that reject certain reader types. Instead, instantiate each member function individually per reader type, only including the constructors valid for that type. `src/ifcparse/IfcParse.cpp` (after the last `InstanceStreamer` method definition): ```cpp // FullBufferImpl template IfcParse::InstanceStreamer>::InstanceStreamer(IfcParse::IfcFile*); template IfcParse::InstanceStreamer>::InstanceStreamer(const std::string&, bool, IfcParse::IfcFile*); template IfcParse::InstanceStreamer>::InstanceStreamer(void*, int, IfcParse::IfcFile*); template IfcParse::InstanceStreamer>::InstanceStreamer(FileReader*, IfcParse::IfcFile*); // ... plus ensure_header, initialize_header, hasSemicolon, semicolonCount, // pushPage, bypassTypes, readInstance // PushedSequentialImpl — same pattern, different valid constructors // MMapFileReader (ifdef USE_MMAP) — same pattern ``` # Linker fix: `FullBufferImpl` missing buffer constructor SWIG's `stream_from_string` calls `InstanceStreamer>(void*, int, IfcFile*)`, but the `(void*, int)` constructor previously hit a `static_assert` for `FullBufferImpl` — it only allowed `PushedSequentialImpl`. ## Error ``` ImportError: undefined symbol: _ZN8IfcParse16InstanceStreamerINS_10FileReaderINS_14FullBufferImplEEEEC1EPviPNS_7IfcFileE (InstanceStreamer>::InstanceStreamer(void*, int, IfcFile*)) ``` ## Fix Three changes to make `FullBufferImpl` support buffer-based and default construction: `src/ifcparse/FileReader.h` — add buffer constructor to `FullBufferImpl`: ```diff class IFC_PARSE_API FullBufferImpl { public: explicit FullBufferImpl(const std::string& fn); + FullBufferImpl(void* data, size_t length); ``` `src/ifcparse/FileReader.h` — add `FileReader(void*, size_t)` forwarding constructor: ```diff + FileReader(void* data, size_t length) + : cursor_(0) { + if constexpr (std::is_same_v) { + impl_ = std::make_shared(data, length); + } else { + static_assert(...); + } + } ``` `src/ifcparse/FileReader.cpp` — implement the constructor: ```cpp FullBufferImpl::FullBufferImpl(void* data, size_t length) : buf_(static_cast(data), static_cast(data) + length) , size_(length) { } ``` `src/ifcparse/IfcParse.cpp` — extend the two `InstanceStreamer` constructors to accept `FullBufferImpl`: ```diff // InstanceStreamer(IfcFile*): + } else if constexpr (std::is_same_v>) { + owned_stream_ = std::make_unique(nullptr, (size_t)0); // InstanceStreamer(void*, int, IfcFile*): + } else if constexpr (std::is_same_v>) { + owned_stream_ = std::make_unique(data, (size_t)length); ``` # Runtime fix: segfault in `parse_context::push()` due to vector reallocation `parse_context_pool` stores nodes in a `std::vector`. During parsing, `load()` takes a `parse_context&` parameter and calls `context.push()`, which calls `pool_->make()`. If the pool's vector reallocates (via `emplace_back`), all existing references into the vector — including the `context` reference held by the caller — become dangling. Subsequent access through the dangling reference causes a segfault. Triggered by larger IFC files (e.g. `ISSUE_159_kleine_Wohnung_R22.ifc`, 9.5 MB) that cause enough pool growth to trigger reallocation. ## Error ``` Thread 1 received signal SIGSEGV, Segmentation fault. 0x... in IfcParse::parse_context::push() #1 in_memory_file_storage::load(...) // context& is dangling after reallocation #2 in_memory_file_storage::load(...) // parent call #3 InstanceStreamer::readInstance() ``` ## Fix `src/ifcparse/storage.h` — change the pool container from `std::vector` to `std::deque`, which does not invalidate references on `push_back`/`emplace_back`: ```diff +#include struct parse_context_pool { - std::vector nodes_; + std::deque nodes_; ```