Revision 553febf4d8d473b7849eda2735ea77bbcb4f7c80 authored by Filip Roséen on 20 February 2016, 21:41:07 UTC, committed by Jean-Baptiste Kempf on 05 March 2016, 21:15:12 UTC
Manually managing memory comes at a cost of both maintainability (in
terms of safety) and performance, as such I have replaced `p_indexes`
with a `std::vector` with equivalent functionality.

Three helper member-functions have been introduced in order to clean up
the usage of the functionality, as well as removal of two now obsolete
member-variables. A `typedef` has also been introduced to aid future
development.

The changes in `mkv.cpp` are due to the fact that it needs access to the
indexes present in `matroska_segment_c`; this should be refactored away
in the future.

Also fixed a bug where you would access index out of bounds if there
are no known indexes.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
1 parent 881299b
History
File Mode Size
.gitignore -rw-r--r-- 63 bytes
Makefile.am -rw-r--r-- 2.6 KB
cachegen.c -rw-r--r-- 2.9 KB
darwinvlc.m -rw-r--r-- 9.9 KB
override.c -rw-r--r-- 7.9 KB
rootwrap.c -rw-r--r-- 7.0 KB
vlc.c -rw-r--r-- 8.9 KB
vlc_win32_rc.rc.in -rw-r--r-- 995 bytes
winvlc.c -rw-r--r-- 13.6 KB

back to top