I love that they modeled the walls of the room with giant spheres... a wonderful hack to avoid having to support plane primitives while still producing something that looks like a Cornell box.
This is a sight more readable than some much more sanely formatted C++ programs I've read. What I've learned from this that it's often the simplicity of the program that's more important than the formatting. (Although formatting is of course still important and anyone who inflicts code like this on their coworkers doesn't reach my standard of "decent human being.")
Actually, I think that this style is readable. The problem, though, it that it is hard to modify. Which would be a real trouble for code that needs to be maintained.
Yes, this would be tons more philanthropic and if it were just readable. Just some simple line breaks and variable names that meant something. It's not impressive that you can just cram stuff into 99 lines.
Actually, the code seemed to be a mess but was quite readable. I read it through. You could almost see all the code in once glance which is when you pretty much don't need formatting anymore, just some time to initially absorb the code.
erand48 returns random doubles from [0.0, 1.0). The argument to erand48() is just a state vector (instead of the usual unsafe convention of having the RNG store its state internally). You can replace it with a call to any other equivalent random number generator.
