29 May 15, 09:51AM
While I actually appreciate this attempt, I wonder why you don't try to put down the the overall pipeline first and then drill down into each aspect from there.
On your 'irritations' about the state of the code, I'd like to point out the following in order to prevent people from getting the wrong picture:
Wouter as the initiator of the cube engine has stated himself that this code was never meant to be a real open-source project and he has willingly programmed as it comforted him best, regardless the readability to other people. Unfortunately he's one of those programmers that don't comment much. Yeah, that is considered bad style throughout literature, but hey it obviously did work for him and so did it later for eihrul and others. There should be a readme somewhere next to the source files, where this is written down.
Not that I'm defending anyone here, I just don't like how you make grown programmers look like they didn't know what they were doing and totally disregard from what spirit this code was born and how it has been 'patched up' with new functionality which wasn't even planned on the initial design. That's where things always get messy. :P
One particular remark about what you criticise, though.
I'm not big into C/C++, but I don't see what's wrong with #ifdefine-ing included files. I wish I could do this on another coding language that I'm working with. And what's wrong with keeping major parameters and the main loop in one file, namely cube.h?
On your 'irritations' about the state of the code, I'd like to point out the following in order to prevent people from getting the wrong picture:
Wouter as the initiator of the cube engine has stated himself that this code was never meant to be a real open-source project and he has willingly programmed as it comforted him best, regardless the readability to other people. Unfortunately he's one of those programmers that don't comment much. Yeah, that is considered bad style throughout literature, but hey it obviously did work for him and so did it later for eihrul and others. There should be a readme somewhere next to the source files, where this is written down.
Not that I'm defending anyone here, I just don't like how you make grown programmers look like they didn't know what they were doing and totally disregard from what spirit this code was born and how it has been 'patched up' with new functionality which wasn't even planned on the initial design. That's where things always get messy. :P
One particular remark about what you criticise, though.
I'm not big into C/C++, but I don't see what's wrong with #ifdefine-ing included files. I wish I could do this on another coding language that I'm working with. And what's wrong with keeping major parameters and the main loop in one file, namely cube.h?