This is a personal experiment of mine, it's not really something that can be tested well by others unfortunately, because I hard-coded all kinds of assumptions in the code.
Will close this to…
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.