These are the locations where for_write seems to be called for (as far as I understand the code):
source/blender/blenkernel/intern/DerivedMesh.cc:406 source/blender/blenkernel/intern/data_trans…
Without the templates, it will not be all that shorter, because, in situations where for_write_only may be set, it is often as part of a pattern
It's not that it is a lot of work, it's just I don't want to do all the requested changes and then watch the PR sit in limbo for another 3 months.
I am going to break it out further if and when…
Thanks for the updated comparison. It's nice that we can still get a good speedup. Unfortunately, this patch is somewhat hard to act on because it contains so many different changes (at least it…
Comparing against 2024/03/04 trunk (which includes the improvements from blender/blender#118772):
Playback:
--
It's not like it''s going to be accepted in either case (I have 8 open PRs counting this one, and no activity on any of them.)