I ended up doing this to make dealing with #2197 saner. The multi-arch stuff itself is not ready for merging but moving to STL stuff is something we'll want to do anyway going forward, regardless of what/when happens with the multi-arch stuff, and this'll be easier to review separately anyhow.
Not adding any new tests here because elfdeps is reasonably covered already (as in, made itself known while working on this :sweat_smile: ) You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/3581 -- Commit Summary -- * Natively allocate elfdeps main structure * Isolate dependency additions elfdeps behind addDep() * Convert elfdeps dependency storage to STL containers * Handle dependency tokens as STL strings inside add*Dep() * Use a C++ string for skipSoname() convenience * Use an STL string for temporary soname storages * Convert other elfdeps data structure strings to STL strings -- File Changes -- M tools/elfdeps.cc (141) -- Patch Links -- https://github.com/rpm-software-management/rpm/pull/3581.patch https://github.com/rpm-software-management/rpm/pull/3581.diff -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/3581 You are receiving this because you are subscribed to this thread. Message ID: <rpm-software-management/rpm/pull/3...@github.com>
_______________________________________________ Rpm-maint mailing list Rpm-maint@lists.rpm.org https://lists.rpm.org/mailman/listinfo/rpm-maint
