Cleanup: use some better variable names.
authorCarl Hetherington <cth@carlh.net>
Thu, 9 Feb 2023 21:42:11 +0000 (22:42 +0100)
committerCarl Hetherington <cth@carlh.net>
Fri, 10 Feb 2023 09:31:15 +0000 (10:31 +0100)
src/lib/player.cc

index b1dcfaec5f9fbb3d1736719742c9bf3098b30773..76e7239aa83f02c7dc7699023a1fd958ff8fc478 100644 (file)
@@ -378,13 +378,13 @@ Player::setup_pieces ()
                return v && v->use() && v->frame_type() != VideoFrameType::THREE_D_LEFT && v->frame_type() != VideoFrameType::THREE_D_RIGHT;
        };
 
-       for (auto i = _pieces.begin(); i != _pieces.end(); ++i) {
-               if (ignore_overlap((*i)->content->video)) {
+       for (auto piece = _pieces.begin(); piece != _pieces.end(); ++piece) {
+               if (ignore_overlap((*piece)->content->video)) {
                        /* Look for content later in the content list with in-use video that overlaps this */
-                       auto const period = DCPTimePeriod((*i)->content->position(), (*i)->content->end(film));
-                       for (auto j = std::next(i); j != _pieces.end(); ++j) {
-                               if (ignore_overlap((*j)->content->video)) {
-                                       (*i)->ignore_video = DCPTimePeriod((*j)->content->position(), (*j)->content->end(film)).overlap(period);
+                       auto const period = DCPTimePeriod((*piece)->content->position(), (*piece)->content->end(film));
+                       for (auto later_piece = std::next(piece); later_piece != _pieces.end(); ++later_piece) {
+                               if (ignore_overlap((*later_piece)->content->video)) {
+                                       (*piece)->ignore_video = DCPTimePeriod((*later_piece)->content->position(), (*later_piece)->content->end(film)).overlap(period);
                                }
                        }
                }