This is a debugging feature, yes, it's not meant for the final PR. Since tt has served its purpose so I'll go ahead and remove it. I'll go through and mark other similar things with comments.
You're right, yeesh, can't believe I did that. I think I'll hide the whole struct behind __cplusplus, as far as I know there isn't any C code using it anyway.
That's just a debugging thing to make hidden attributes visible in the UI. I'll get rid of it.
It's for debugging, yes; sometimes I accidentally leave some in place. This is another place that's supposed to be reverted prior to merging. I actually did submit a separate PR, it was…
It's a hash set with a flat array to speed up iteration. I'm going to rewrite it in C++. I've already been forced to use blender::Map internally as part of my effort to remove all usages of…
I'm not getting errors in atomics on master, currently trying to track down what exactly is causing it.
I should have explained this more clearly in a comment, sorry; the autosave stuff is actually not part of the PR. The idea was to make testing less painful to users but remove the code prior to…
It solves an error actually. I'm getting a sign conversion error with MSVC's bundled clang.