[vlc-commits] demux:mkv: avoid writing class members directly

Steve Lhomme git at videolan.org
Mon Jan 22 11:09:55 CET 2018


vlc | branch: master | Steve Lhomme <robux4 at ycbcr.xyz> | Mon Jan 22 10:52:13 2018 +0100| [75a0e9ef44c8d47e47ed3c7755cfce846c3b868f] | committer: Steve Lhomme

demux:mkv: avoid writing class members directly

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=75a0e9ef44c8d47e47ed3c7755cfce846c3b868f
---

 modules/demux/mkv/matroska_segment_seeker.cpp | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/modules/demux/mkv/matroska_segment_seeker.cpp b/modules/demux/mkv/matroska_segment_seeker.cpp
index c8f62215f4..78873a186f 100644
--- a/modules/demux/mkv/matroska_segment_seeker.cpp
+++ b/modules/demux/mkv/matroska_segment_seeker.cpp
@@ -271,16 +271,12 @@ SegmentSeeker::get_seekpoints_around( mtime_t target_pts, track_ids_t const& pri
 
             if( cluster.fpos > points.first.fpos )
             {
-                points.first.fpos = cluster.fpos;
-                points.first.pts  = cluster.pts;
+                points.first = Seekpoint( cluster.fpos, cluster.pts );
 
                 // do we need to update the max point? //
 
                 if( points.second.fpos < points.first.fpos )
-                {
-                    points.second.fpos = cluster.fpos + cluster.size;
-                    points.second.pts  = cluster.pts  + cluster.duration;
-                }
+                    points.second = Seekpoint( cluster.fpos + cluster.size, cluster.pts + cluster.duration );
             }
         }
     }



More information about the vlc-commits mailing list