Differential D16546
BLO: Use blender::Map in OldNewMap. Authored by Jacques Lucke (JacquesLucke) on Nov 18 2022, 6:24 PM. Tags None Subscribers None
Details
OldNewMap used to have its own map implementation. Given that the file uses C++ now, it is easy to use a C++ map implementation instead. This simplifies the code a lot. Going forward it might make sense to remove the OldNewMap abstraction or to split it up in two (currently, NewAddress.nr has two different meanings in different contexts which is confusing). No functional changes are expected.
Diff Detail
Event TimelineComment Actions Big +1 Just out of curiosity. did you check if there was any measurable performances difference? Comment Actions
I tried to measure it, but in any reasonable large file there was so much noise that I couldn't measure a performance difference with this patch. |