Merge pull request #758 from RayTracing/aarect-override

Aarect override

Author: hollasch

CHANGELOG.md

@@ -12,6 +12,8 @@ Change Log -- Ray Tracing in One Weekend
 ### The Next Week
 
 ### The Rest of Your Life
+  - Fix: Missing `override` keyword for `xz_rect::pdf_value()` and `xz_rect::random()` methods
+    (#748)
   - Fix: Synchronize book and source for `cornell_box()` function.
   - Fix: Introduction of light code was introduced out of sequence (#738, #740)
 

src/TheRestOfYourLife/aarect.h

@@ -55,7 +55,7 @@ class xz_rect : public hittable {
             return true;
         }
 
-        virtual double pdf_value(const point3& origin, const vec3& v) const {
+        virtual double pdf_value(const point3& origin, const vec3& v) const override {
             hit_record rec;
             if (!this->hit(ray(origin, v), 0.001, infinity, rec))
                 return 0;
@@ -67,7 +67,7 @@ class xz_rect : public hittable {
             return distance_squared / (cosine * area);
         }
 
-        virtual vec3 random(const point3& origin) const {
+        virtual vec3 random(const point3& origin) const override {
             auto random_point = point3(random_double(x0,x1), k, random_double(z0,z1));
             return random_point - origin;
         }