I updated the parsing code to also include the new check for future Blender versions.
@blender-bot build
Using bool was indeed a bit weird, looks like that was a leftover from an earlier version of the patch. I changed it to an int that still contains L_ENDIAN or B_ENDIAN now. Changing it to…
Some things that don't seem to be tested by your regression tests yet. Would be good to test at least some of these, because they don't look all trivial.
A comment would be useful here, not sure what calc_faces is supposed to do.
Can you move this function as a separate commit now already?
Use version_node_add_empty and related functions in new versioning code, to avoid depending on the current node declaration.
I think I'd put CD_PROP_INT16_2D right before CD_PROP_INT32_2D instead.
I wonder if we need to be careful in the case when CD_TEXTURE_MLOOPCOL is stored in a .blend file, not sure what that was or when it was removed.
