Now that node_intern.hh is a C++ header, we can use C++ types
there. This patch replaces the linked list of dragged links with a
vector. Also, the list of drag operator custom data, nldrag, doesn't
seem to need to be a list at all, so I just made it a unique pointer.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- cleanup-nldrag-types (branched from master)
- Build Status
Buildable 18718 Build 18718: arc lint + arc unit
Event Timeline
Comment Actions
Actually, I honestly don't know why bNodeLinkDrag::links has to be a vector rather than just a single item either. Can you even drag multiple links at the same time? Or maybe I'm missing something.
| source/blender/editors/space_node/space_node.cc | ||
|---|---|---|
| 537 | That change looks wrong. Now both spaces may point to the same runtime data I think. I think it's good to copy the runtime data, and ignore some elements explicitly. Long-term it's unlikely that we never want to copy any runtime data over. | |
Comment Actions
- null new region's runtime struct when copying
Unless there is a good reason to copy anything now, I don't think it's necessary
actually.