When you're setting the mesh based on the shape it will check all of the if statements because you didn't use elif (negligible performance-wise but yk). I think a match statement would make the most sense if you don't find something else. Also for some things, instead of checking when they have changed in _process to update them, you could use setter functions and signals (where applicable)
Yes, about the match thing: I tried that but didn't get it to work. I prolly could have, but the Colored Collision Shapes were a thing I needed for a project, so I didn't want to spend a bunch of time on making it as clean and as performant as possible. Same thing with checking for changes in _process: I never really used set/get and didn't want to look into it.
When you're setting the mesh based on the shape it will check all of the if statements because you didn't use elif (negligible performance-wise but yk). I think a match statement would make the most sense if you don't find something else.
Also for some things, instead of checking when they have changed in _process to update them, you could use setter functions and signals (where applicable)
Yes, about the match thing: I tried that but didn't get it to work. I prolly could have, but the Colored Collision Shapes were a thing I needed for a project, so I didn't want to spend a bunch of time on making it as clean and as performant as possible. Same thing with checking for changes in _process: I never really used set/get and didn't want to look into it.
@@S4AAS fair, just making suggestions
nice.