Perhaps Casey himself or people who are following the project closely can comment. I am at day 067 right now. During the stream Casey embeds the whole update & render part of the GAME_UPDATE_AND_RENDER function inside the Updatable check:

1
2
3

if(Entity->Updatable) {
...
}

This leads to 200 lines of codes moving deeper one level of indentation. Why not instead do the following:

1
2
3
4
5

if(!Entity->Updatable) {
continue;
}
...

This would lead to the code that is much cleaner and understandable, as well as easier to read via diff.

Was this question ever discussed on the stream? Is there some reasoning behind this approach or is it just a habit?

Fair enough, thank you for the reply. I am used to the completely opposite approach: discarding everything that's not fit right in the beginning allows me to follow the rest of the logic easier. But I'll adjust for this project.

And yes, the diff issue could be managed by ignoring whitespaces in the diff tool.