bartmassey / ancient-raytracer Goto Github PK

Right now there's a useless texture::value() which can never be called. This is a symptom of deeper issues with the construction and optimized rendering in the raytracer: the code wants to first build a model, then pick lighting and camera, then use optimized tracing that has all that information baked together. The class hierarchy for this is not right, and maybe it can't be — may be a situation where template types (not available when the code was first written) are necessary.

After multiple tries at untangling it all I've given up. Someone smarter than me should fix it.

Rendering a non-square frame squashes rather than clipping appropriately.

The code has some capability of being compiled with INLINE for inlining or NOOUTLINE for not. Unfortunately, information about what this is and how to use it is lost in the sands of time. Needs to be either documented or removed.

In modern C++ the standard library contains a nice vector implementation. Perhaps should replace our point class with that.