I copied the init code from existing node_socket.cc, which also [does nothing here](e1b1b2a8b4/source/blender/n…
The null in the name indicates the end-of-line terminal. It's still a wrapper for real strings only.
@mod_moder I switched to StringRefNull because that guarantees a null-terminated string…
Wrt StringRefNull (can't find the comment any more):
Problem is that this thing does not accept nullptr (or rather: accepts it and asserts internally). That just puts the burden of null checks…
I'd like to stick to the BKE naming scheme for freeing/copying ID datablocks here
As concluded in #109288 the draw_color callback should now support nullptr/None for both the context and the bNode pointer. This is technically a breaking change (note added to #110272) but very unlikely to cause trouble since very few addons implement interface classes and even fewer would have context-based colors.
But "self" would be the interface, not a panel
I'm guessing bNodeTreeInterfacePanel::name
Yeah i'm just using a string identifier now, for the sake of simplicity and compatibility.
StringRef does not have c_str. Or was this referring to something else? Gitea seems to have lost track of what this comment refers to ...
Should i be using StringRefNull for anything that…