Skip to content

Commit df5807a

Browse files
hollaschhollasch
authored
Merge pull request #906 from RayTracing/fix-scene-order
Fix book 2 scene ordering in main.cc
2 parents d09b3b7 + abea2a3 commit df5807a

File tree

2 files changed

+31
-23
lines changed

2 files changed

+31
-23
lines changed

‎books/RayTracingTheNextWeek.html

Lines changed: 27 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1270,7 +1270,8 @@
12701270

12711271
switch (0) {
12721272
case 1: random_spheres(scene_desc); break;
1273-
default: two_spheres(scene_desc); break;
1273+
default:
1274+
case 2: two_spheres(scene_desc); break;
12741275
}
12751276

12761277
scene_desc.render();
@@ -1651,8 +1652,8 @@
16511652
case 1: random_spheres(scene_desc); break;
16521653
case 2: two_spheres(scene_desc); break;
16531654
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
1654-
case 3: two_perlin_spheres(scene_desc); break;
1655-
default: earth(scene_desc); break;
1655+
default:
1656+
case 3: earth(scene_desc); break;
16561657
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
16571658
}
16581659

@@ -1820,9 +1821,11 @@
18201821
...
18211822
switch (0) {
18221823
case 1: random_spheres(scene_desc); break;
1823-
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
18241824
case 2: two_spheres(scene_desc); break;
1825-
default: two_perlin_spheres(scene_desc); break;
1825+
case 3: earth(scene_desc); break;
1826+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
1827+
default:
1828+
case 4: two_perlin_spheres(scene_desc); break;
18261829
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
18271830
}
18281831

@@ -2516,10 +2519,11 @@
25162519
switch (0) {
25172520
case 1: random_spheres(scene_desc); break;
25182521
case 2: two_spheres(scene_desc); break;
2519-
case 3: two_perlin_spheres(scene_desc); break;
2522+
case 3: earth(scene_desc); break;
2523+
case 4: two_perlin_spheres(scene_desc); break;
25202524
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
2521-
case 4: earth(scene_desc); break;
2522-
default: simple_light(scene_desc); break;
2525+
default:
2526+
case 5: simple_light(scene_desc); break;
25232527
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
25242528
}
25252529

@@ -2679,11 +2683,12 @@
26792683
switch (0) {
26802684
case 1: random_spheres(scene_desc); break;
26812685
case 2: two_spheres(scene_desc); break;
2682-
case 3: two_perlin_spheres(scene_desc); break;
2683-
case 4: earth(scene_desc); break;
2684-
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
2686+
case 3: earth(scene_desc); break;
2687+
case 4: two_perlin_spheres(scene_desc); break;
26852688
case 5: simple_light(scene_desc); break;
2686-
default: cornell_box(scene_desc); break;
2689+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
2690+
default:
2691+
case 6: cornell_box(scene_desc); break;
26872692
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
26882693
}
26892694

@@ -3224,12 +3229,13 @@
32243229
switch (0) {
32253230
case 1: random_spheres(scene_desc); break;
32263231
case 2: two_spheres(scene_desc); break;
3227-
case 3: two_perlin_spheres(scene_desc); break;
3228-
case 4: earth(scene_desc); break;
3232+
case 3: earth(scene_desc); break;
3233+
case 4: two_perlin_spheres(scene_desc); break;
32293234
case 5: simple_light(scene_desc); break;
3230-
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
32313235
case 6: cornell_box(scene_desc); break;
3232-
default: cornell_smoke(scene_desc); break;
3236+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
3237+
default:
3238+
case 7: cornell_smoke(scene_desc); break;
32333239
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
32343240
}
32353241

@@ -3346,14 +3352,15 @@
33463352
switch (0) {
33473353
case 1: random_spheres(scene_desc); break;
33483354
case 2: two_spheres(scene_desc); break;
3349-
case 3: two_perlin_spheres(scene_desc); break;
3350-
case 4: earth(scene_desc); break;
3355+
case 3: earth(scene_desc); break;
3356+
case 4: two_perlin_spheres(scene_desc); break;
33513357
case 5: simple_light(scene_desc); break;
33523358
case 6: cornell_box(scene_desc); break;
3353-
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
33543359
case 7: cornell_smoke(scene_desc); break;
3360+
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++ highlight
33553361
case 8: final_scene(scene_desc); break;
3356-
default: default_scene(scene_desc); break;
3362+
default:
3363+
case 9: default_scene(scene_desc); break;
33573364
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
33583365
}
33593366

‎src/TheNextWeek/main.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -328,13 +328,14 @@ int main() {
328328
switch (0) {
329329
case 1: random_spheres(scene_desc); break;
330330
case 2: two_spheres(scene_desc); break;
331-
case 3: two_perlin_spheres(scene_desc); break;
332-
case 4: earth(scene_desc); break;
331+
case 3: earth(scene_desc); break;
332+
case 4: two_perlin_spheres(scene_desc); break;
333333
case 5: simple_light(scene_desc); break;
334334
case 6: cornell_box(scene_desc); break;
335335
case 7: cornell_smoke(scene_desc); break;
336336
case 8: final_scene(scene_desc); break;
337-
default: default_scene(scene_desc); break;
337+
default:
338+
case 9: default_scene(scene_desc); break;
338339
}
339340

340341
scene_desc.render();

0 commit comments

Comments
 (0)