29c5c69 | Hugo Beauzée-Luyssen | 10 March 2016, 14:36:54 UTC | Revert "lib: Add libvlc_dialog.h to vlc.h" This reverts commit 1e0f167c78121a4c790bd92e762e8e306cf5075a. | 10 March 2016, 14:37:07 UTC |
b42a03f | Francois Cartegnie | 10 March 2016, 10:52:02 UTC | demux: ts: set filter only after pcr repicking | 10 March 2016, 10:52:23 UTC |
994b93c | Francois Cartegnie | 10 March 2016, 10:49:50 UTC | demux: ts: simplify CAM setting And comment | 10 March 2016, 10:52:23 UTC |
c021c52 | Thomas Guillem | 10 March 2016, 10:36:51 UTC | libvlc: re-order dialog callback arguments The opaque p_data is now the first argument for the sake of consistency. | 10 March 2016, 10:45:27 UTC |
b37866a | Thomas Guillem | 10 March 2016, 10:36:24 UTC | dialog: re-order callback arguments The opaque p_data is now the first argument for the sake of consistency. | 10 March 2016, 10:45:27 UTC |
86ca39d | Francois Cartegnie | 10 March 2016, 09:50:15 UTC | demux: ts: create missing dvb subs (refs #5267) | 10 March 2016, 10:07:27 UTC |
1e0f167 | Hugo Beauzée-Luyssen | 10 March 2016, 09:36:06 UTC | lib: Add libvlc_dialog.h to vlc.h | 10 March 2016, 09:36:26 UTC |
42d180b | Steve Lhomme | 10 March 2016, 08:34:37 UTC | directdraw: more debug info for overlay issues Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 10 March 2016, 09:23:43 UTC |
6e44b5d | Sean McGovern | 10 March 2016, 00:02:43 UTC | contrib/fontconfig: require at least 2.10.92 Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 10 March 2016, 09:23:43 UTC |
569f928 | Francois Cartegnie | 10 March 2016, 09:21:57 UTC | demux: dash: add missing config.h | 10 March 2016, 09:21:57 UTC |
3925299 | Francois Cartegnie | 10 March 2016, 09:17:03 UTC | demux: dash: add missing config.h | 10 March 2016, 09:17:22 UTC |
518c88e | Petri Hintukainen | 10 March 2016, 08:49:35 UTC | avahi: remove domain from rtsp service type | 10 March 2016, 08:49:35 UTC |
42568e3 | Petri Hintukainen | 01 March 2016, 08:36:09 UTC | avahi: remove useless avahi_threaded_poll_stop() from error path Error path is never triggered after avahi_threaded_poll_start(). Even if it was, thread should probably be stopped before freeing avahi client ... | 10 March 2016, 08:26:20 UTC |
3cbc233 | Petri Hintukainen | 01 March 2016, 08:34:22 UTC | avahi: remove AvahiServiceBrowser from sys Only last created service browser was stored. avahi_client_free() should free also all service browsers. | 10 March 2016, 08:16:27 UTC |
d6b6f0f | Francois Cartegnie | 09 March 2016, 16:57:26 UTC | demux: smooth: fix include order | 09 March 2016, 20:50:21 UTC |
e87ac8f | Francois Cartegnie | 09 March 2016, 16:55:18 UTC | demux: hls: fix include order | 09 March 2016, 20:50:21 UTC |
e9c488a | Francois Cartegnie | 09 March 2016, 16:53:17 UTC | demux: dash: fix include order | 09 March 2016, 20:50:21 UTC |
f1a57d0 | Francois Cartegnie | 09 March 2016, 16:52:31 UTC | demux: adaptive: fix include order | 09 March 2016, 20:50:21 UTC |
f47aace | Felix Paul Kühne | 09 March 2016, 18:02:50 UTC | libvlc: add missing include in umbrella header | 09 March 2016, 18:03:16 UTC |
8cf54bf | Felix Paul Kühne | 09 March 2016, 16:25:56 UTC | contrib/live555: update to current version | 09 March 2016, 17:56:51 UTC |
8d119e0 | Filip Roséen | 09 March 2016, 17:19:15 UTC | mkv: fix build failure when compiled with llvm LLVM treats a typedef with forward-declaration inside a local struct as a forward-declaration for a nested type of that struct, effectively erroring on the implementation of the dispatchers. This patch fixes that issue by forcing the name to referr to a type in the sourrounding scope (by introducing a proper forward-declaration). Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 17:26:40 UTC |
c7ccd77 | Jean-Baptiste Kempf | 09 March 2016, 17:14:01 UTC | Contribs: update libass to 0.13.2 | 09 March 2016, 17:14:01 UTC |
02d5157 | Filip Roséen | 09 March 2016, 12:23:38 UTC | mkv: fixed C++03 restriction regarding IndexFinder It somehow slipped my mind that C++03 disallows local types as template-arguments, which makes one of the earlier commits invalid when compiled as C++03. This patch correctly moves the type in question to the global namespace (inside an anonymous namespace to not pollute the global linkage scope) so that everything is 100% legal C++03. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 12:32:04 UTC |
b9341d8 | Patrick Ho | 09 March 2016, 03:55:26 UTC | Fixed vlm show to display next launch for schedules with endless repeats Currently vlm show does not show the next launch of endlessly repeated schedules after the first launch Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 12:00:22 UTC |
e41757a | Filip Roséen | 09 March 2016, 11:50:09 UTC | mkv: minor clean-up + c-style casts => c++ casts Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
86e13b3 | Filip Roséen | 09 March 2016, 11:50:08 UTC | mkv: made MkvTree match legacy code in terms of indentation Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
166bbc1 | Filip Roséen | 09 March 2016, 11:50:07 UTC | mkv: use EbmlTypeDispatcher in matroska_segment_c::ParseChapterAtom Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
dba3e1d | Filip Roséen | 09 March 2016, 11:50:06 UTC | mkv: removed unnecessary variable and cleaned up expression Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
2e185a2 | Filip Roséen | 09 March 2016, 11:50:05 UTC | mkv: added debug-helper to matroska_segment_c::ParseInfo Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
f85f390 | Filip Roséen | 09 March 2016, 11:50:04 UTC | mkv: removed unnecessary dynamic allocation Replaced the dynamic allocation with an equivalent construction on the stack (we do not need dynamic life time since it is unconditionally destroyed at the end of the block). Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
fc393f4 | Filip Roséen | 09 March 2016, 11:50:03 UTC | mkv: replaced reallocation of EbmlParser with EbmlParser::reconstruct Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:58:11 UTC |
6487b49 | Filip Roséen | 09 March 2016, 11:50:02 UTC | mkv: Added reconstruct function to EbmlParser The two overloads of EbmlParser::reconstruct has been added due to the fact that there are _a lot_ of places in the code following the below: delete ep; ep = new EbmlParser (a, b, c, b); The above will, unless the compiler feels cocky and optimizes it to what is included in this patch. First free the memory used, and then allocate new memory for a new EbmlParser. Instead of doing what is effectively a reallocation, this patch introduces EbmlParser::reconstruct which will reconstruct the object in-place (without actually having to reallocate the underlying storage). Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:59 UTC |
1b41c8e | Filip Roséen | 09 March 2016, 11:50:01 UTC | mkv: tweaked diagnostic when seeking Outputting the percentage when we do not have any (-1.f) is of no use, this patch properly makes use diagnose what we actually have (either a fixed location, or the percentage). Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:29 UTC |
4f65458 | Filip Roséen | 09 March 2016, 11:50:00 UTC | mkv: removed legacy code from matroska_segment.cpp The macro and the disabled code blocks are not required, as such they have been removed. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:29 UTC |
d6f4169 | Filip Roséen | 09 March 2016, 11:49:59 UTC | mkv: replaced loop with (already present) helper BlockFindTrackIndex does what the explicit loop is doing; we should use it. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:29 UTC |
cc84e98 | Filip Roséen | 09 March 2016, 11:49:58 UTC | mkv: use binary- instead of linear-search in Seek Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:29 UTC |
0a7d6eb | Filip Roséen | 09 March 2016, 11:49:56 UTC | mkv: introduced helper functions related to indexes Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:57:29 UTC |
6a735f6 | Filip Roséen | 09 March 2016, 11:33:21 UTC | mkv: Use StringDispatcher in matroska_segment_c::TrackInit Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:52:40 UTC |
4a80d93 | Filip Roséen | 09 March 2016, 11:33:20 UTC | mkv: Added StringDispatcher Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:52:26 UTC |
597b289 | Steve Lhomme | 08 March 2016, 15:45:21 UTC | i420_yuy2: fix the line remaining offset for other YUY2 variants Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 11:45:04 UTC |
befae51 | Steve Lhomme | 09 March 2016, 09:20:12 UTC | yuy2_i420: only handle visible pixels Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 10:59:58 UTC |
bcf4484 | Steve Lhomme | 09 March 2016, 09:20:11 UTC | yuy2_i420: only adjust the line ending when the line was actually used -- must have been tested with samples where the shift was 0 this fixes http://streams.videolan.org/samples/V-codecs/V422.AVI playing on a I420 surface Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 10:59:58 UTC |
351cd5a | Steve Lhomme | 09 March 2016, 10:16:52 UTC | directdraw: in overlay mode we need to use DDSCAPS_FLIP and DDSCAPS_FRONTBUFFER We use DDSCAPS_BACKBUFFER to do overlay. According to the documentation we need these 2 flags on the front surface (the only one we have). https://msdn.microsoft.com/en-us/library/windows/desktop/gg426110%28v=vs.85%29.aspx#DDSCAPS_BACKBUFFER https://msdn.microsoft.com/en-us/library/windows/desktop/gg426110%28v=vs.85%29.aspx#DDSCAPS_FRONTBUFFER Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 09 March 2016, 10:55:53 UTC |
88521ef | Rémi Denis-Courmont | 09 March 2016, 05:04:22 UTC | url: test previous commit | 09 March 2016, 05:04:22 UTC |
99d97ef | Rémi Denis-Courmont | 09 March 2016, 05:03:08 UTC | url: validate host name and path This avoids passing embedding unexpected garbage into network requests, leading to weird errors, or in the worst cases, injection (e.g. white spaces in HTTP/1.x requests). | 09 March 2016, 05:03:08 UTC |
736e7e7 | Steve Lhomme | 08 March 2016, 15:45:22 UTC | i420_yuy2: only handle visible pixels Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 16:51:49 UTC |
e29200c | Steve Lhomme | 08 March 2016, 15:45:20 UTC | i420_rgb: only handle visible pixels fixes a crash similar to #16686 when an RGB output is used instead of YUYV Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 16:51:49 UTC |
ffcc062 | Filip Roséen | 08 March 2016, 14:11:57 UTC | mkv: use EbmlTypeDispatcher in matroska_segment_c::BlockGet | 08 March 2016, 16:51:49 UTC |
f5cdfd6 | Filip Roséen | 08 March 2016, 14:11:56 UTC | mkv: Use EbmlTypeDispatcher in `matroska_segment_c::ParseInfo` | 08 March 2016, 16:51:49 UTC |
2d8949d | Filip Roséen | 08 March 2016, 14:11:55 UTC | mkv: Cleaned up code to issue debug diagnostics - helper function to print diagnostics within MetaDataHandlers introduced - fixed so that `matroska_segment_c::ParseTracks` is using `MkvTree` for diagnostics | 08 March 2016, 16:51:49 UTC |
0503738 | Filip Roséen | 08 March 2016, 14:11:54 UTC | mkv: Use EbmTypeDispatcher in `matroska_segment_c::{ParseTrackEntry, ParseTracks}` Introduced `EbmlTypeDispatcher` to the functions in question, making them more readable (as well as faster). This commit is a very good example of how the new header `Ebml_dispatcher.hpp` allows for better and faster code. Also removed some legacy code that has been disabled due to, by matroska, deprecated types. | 08 March 2016, 16:51:49 UTC |
421416d | Filip Roséen | 08 March 2016, 14:11:53 UTC | mkv: Added EbmlTypeDispatcher The EbmlTypeDispatcher is a type used to "dispatch" an object of type EbmlElement* to an appropriate callback depending on the dynamic type of the referred to object. It can effectivelly replace the massive if-else branches found throughout the module by not only making the code easier to understand and maintain, but also by making it _a lot_ faster (benchmarks show a speed increase between 450 and 700% in terms of lookup). | 08 March 2016, 16:51:49 UTC |
155fbc6 | Thomas Guillem | 08 March 2016, 14:06:18 UTC | mediacodec: fix crash when reporting a hw error | 08 March 2016, 14:07:43 UTC |
bfb5a9c | Steve Lhomme | 08 March 2016, 13:52:09 UTC | i420_yuy2: fix using buffer areas not visible fixes #16686 Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 13:58:46 UTC |
ca3e55c | Steve Lhomme | 08 March 2016, 13:52:08 UTC | i420_yuy2: remove useless lines Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 13:58:46 UTC |
b26c633 | Steve Lhomme | 08 March 2016, 10:25:04 UTC | directdraw: list the preferred formats not supported in debug builds Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 10:35:11 UTC |
c51d1a8 | Steve Lhomme | 08 March 2016, 10:25:03 UTC | directdraw: better debugging of the pixel format used for surfaces Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 10:35:11 UTC |
21d8776 | Petri Hintukainen | 08 March 2016, 07:56:50 UTC | bluray: fix copy-paste error Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 08 March 2016, 08:25:24 UTC |
e3d09f9 | Hugo Beauzée-Luyssen | 07 March 2016, 16:15:15 UTC | Only add items in the root playlist to the recent elements | 07 March 2016, 16:15:15 UTC |
6abe049 | Steve Lhomme | 07 March 2016, 14:15:04 UTC | video_output: better logging of filters needed Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 14:15:32 UTC |
7cdabac | Jean-Baptiste Kempf | 07 March 2016, 13:36:36 UTC | Qt: fix potential crash in UpdateVout() | 07 March 2016, 13:36:36 UTC |
f4055f8 | Thomas Guillem | 07 March 2016, 13:31:25 UTC | contrib: update libdsm | 07 March 2016, 13:34:39 UTC |
ca78be7 | Thomas Guillem | 07 March 2016, 13:30:48 UTC | dsm: update with last API changes libdsm functions return now a negative number in case of error. Add NT_STATUS error check to detect permission errors in order to ask for credentials only in that case. | 07 March 2016, 13:34:39 UTC |
106a325 | Guillaume Gomez | 04 March 2016, 01:36:13 UTC | Add shortcut to set focus on playlist search field Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com> Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:12:58 UTC |
fcedf52 | Michael Tänzer | 04 March 2016, 20:45:44 UTC | ogg codecs: use checked realloc() instead of xrealloc() Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:07:47 UTC |
37a9c77 | Michael Tänzer | 03 March 2016, 19:02:49 UTC | oggspots: Add OggSpots codec module Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:05:37 UTC |
98448d3 | Steve Lhomme | 07 March 2016, 12:53:27 UTC | mkv: revert 8317c24270d45b4938acaf58104e18f5039eade0 This reverts commit 8317c24270d45b4938acaf58104e18f5039eade0. it introduced issues with CPU intensive files dropping a lot of frames Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:04:00 UTC |
ce5d53a | Petri Hintukainen | 07 March 2016, 10:50:10 UTC | sftp: Factorize common cleanup code in Open() and Close() Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:02:52 UTC |
2a7f738 | Petri Hintukainen | 07 March 2016, 10:42:59 UTC | sftp: fix browsing home directory ( ex. vlc sftp://127.0.0.1 ) Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:02:14 UTC |
61ffa3f | Petri Hintukainen | 07 March 2016, 10:37:50 UTC | avahi: remove unused default port Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:02:14 UTC |
32e98b0 | Petri Hintukainen | 07 March 2016, 10:37:34 UTC | avahi: add RTSP Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 07 March 2016, 13:02:14 UTC |
b6a3dd5 | Francois Cartegnie | 06 March 2016, 22:21:04 UTC | demux: ts: add debuging for arib descriptors From untranslated TR-B14 | 06 March 2016, 22:36:59 UTC |
24f3724 | Francois Cartegnie | 06 March 2016, 21:33:37 UTC | demux: ts: rename transport type field and add ignored one, fully blacklist dsm cc | 06 March 2016, 21:33:37 UTC |
a9aea03 | Francois Cartegnie | 06 March 2016, 21:04:18 UTC | demux: ts: show psip current program progress | 06 March 2016, 21:04:18 UTC |
7283809 | Francois Cartegnie | 06 March 2016, 21:03:11 UTC | demux: ts: remove pmt access control test Should already have been evaluated on pat | 06 March 2016, 21:03:11 UTC |
c7497d0 | Francois Cartegnie | 06 March 2016, 20:22:27 UTC | demux: ts: remove indirect eit callbacks | 06 March 2016, 20:22:27 UTC |
c91def4 | Francois Cartegnie | 06 March 2016, 20:02:55 UTC | epg: fix current replacement and add tests | 06 March 2016, 20:13:43 UTC |
f28e899 | Francois Cartegnie | 06 March 2016, 18:29:31 UTC | demux: ts: use fail proof realloc | 06 March 2016, 20:13:43 UTC |
7ce220a | Francois Cartegnie | 06 March 2016, 16:54:17 UTC | demux: ts: fix current event/running event TS 101 211, undefined status event should be handled as running. Should fix channels without real event status like France 5 | 06 March 2016, 20:13:43 UTC |
8521672 | Francois Cartegnie | 29 February 2016, 13:43:17 UTC | demux: ts: simplify eit offset (arib) and add network time | 06 March 2016, 20:13:43 UTC |
b051a4b | Jean-Baptiste Kempf | 03 March 2016, 12:21:30 UTC | posix fs: define vlc_cloexec when mkostemp does not exist On Android, for 64 bits, O_CLOEXEC is defined, accept4 exists, but not mkostemp. Therefore, in vlc_mkstemp we go to the fallback that calls vlc_cloexec, but we don't define it, because we have accept4 Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:40:57 UTC |
2e5cbf3 | Michael Tänzer | 03 March 2016, 19:02:48 UTC | ogg: Add OggSpots video codec support Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:39:08 UTC |
c7ff5cc | Filip Roséen | 04 March 2016, 16:04:19 UTC | mkv: fixed seek request diagnostic The diagnostic's intention was to print where we are going to seek in procent, though the actual output was previously in the range [0.0, 1.0]. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:21:32 UTC |
d3f83dc | Filip Roséen | 04 March 2016, 16:04:18 UTC | mkv: Reimplemented MkvTree and moved it to `Util.hpp` `MkvTree` can be used all across the module, meaning that it better belongs in `Util.hpp` than with static linkage in `matroska_segment_parse.cpp`. - A helper function `MkvTree_va` has been introduced so that one can easily call the function from functions that accepts a variadic number of arguments (using va_list). - The implementation has been optimized so that we do not need to allocate storage for the format string on every invocation if it would fit inside a fixed length buffer. - An error check has been added to signal if we run out of memory. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:21:10 UTC |
5fc6cd6 | Filip Roséen | 04 March 2016, 16:04:17 UTC | mkv: removed unnecessary memory-allocation in `matroska_segment_parse.cpp` Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:21:07 UTC |
7d32c88 | Filip Roséen | 04 March 2016, 16:04:16 UTC | mkv: Added error diagnostic if i_track does not match seekpoint Added an if-check to make sure that we do not crash on the lines that follow if we, for some reason, end up in a state where there is no matching seekpoint for the specified `i_track`. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:21:04 UTC |
fce2945 | Filip Roséen | 04 March 2016, 16:04:15 UTC | mkv: replaced (manual) linked-list with std::vector in `matroska_segment_c::Seek` Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:20:57 UTC |
2eca52d | Filip Roséen | 04 March 2016, 16:04:14 UTC | mkv: removed manual memory management from Tags + SimpleTags In order to provide better runtime safety manual management of dynamic memory has been removed from `class Tags` and `class SimpleTags`. This includes introducing usage of `std::vector` as well as `std::string`, while also removing data-members that are no longer required. - SimpleTag::b_default has been removed since it was written to (at once place), but never read; the write has been replaced by a comment. - The ParseSimpleTag function has been changed to return `bool` instead of a pointer to signal whether parsing was successful. The result will be written to `pout_simple` (instead of being indirectly returned through a pointer). Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:20:49 UTC |
e527210 | Filip Roséen | 04 March 2016, 16:04:13 UTC | mkv: removed unnecessary dynamic memory allocations As written in the commit title, a few uses of dynamic memory management has been removed, effectivelly fixing a memory leak as well as making the code cleaner/faster. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:20:45 UTC |
19c66b8 | Filip Roséen | 04 March 2016, 16:04:12 UTC | mkv: `MKV_CHECKED_PTR_DECL` introduced to `matroska_segment.cpp` See previous commit for information regarding the macro. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:20:41 UTC |
d8b179e | Filip Roséen | 04 March 2016, 16:04:11 UTC | mkv: Introduced MKV_CHECKED_PTR_DECL `MKV_IS_ID` is a macro to check whether the object hiding behind a pointer has a certain dynamic type and it is being used all over the module. There are however a lot of cases that is followed by a static_cast to the appropriate type; `MKV_CHECKED_PTR_DECL` has been introduced to shorten code that has such usage. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:20:28 UTC |
553febf | Filip Roséen | 20 February 2016, 21:41:07 UTC | mkv: replaced manual memory-management with std::vector in matroska_segment_c 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> | 05 March 2016, 21:15:12 UTC |
881299b | Filip Roséen | 04 March 2016, 16:04:09 UTC | mkv: removed indirection when initializing std::string There really is no need to allocate memory for a temporary buffer, copy the data to this buffer, and then use the buffer to initialize a std::string, before freeing the buffer. One of the constructors of std::string can handle the above for us. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:03 UTC |
d55cd99 | Filip Roséen | 04 March 2016, 16:04:08 UTC | mkv: stream_io_callback.cpp c-style casts => c++ casts Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:03 UTC |
18021cd | Filip Roséen | 04 March 2016, 16:04:07 UTC | mkv: virtual_segment.cpp c-style casts => c++ casts Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |
a567b9e | Filip Roséen | 04 March 2016, 16:04:06 UTC | mkv: util.cpp c-style casts => c++ casts Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |
c3b8f19 | Filip Roséen | 04 March 2016, 16:04:05 UTC | mkv: mkv.cpp c-style casts => c++ casts (+fix) - c-style casts have been replaced with safer c++ casts - removed pointless cast after usage of `va_arg` since the type of the expression `va_arg` expands to is documented to be what has been passed as parameter. Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |
a0d9858 | Filip Roséen | 04 March 2016, 16:04:04 UTC | mkv: chapter_command.cpp c-style casts => c++ casts Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |
8fc136d | Filip Roséen | 04 March 2016, 16:04:03 UTC | mkv: matroska_segment_parse.cpp c-style cast => c++ cast Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |
692c8e9 | Filip Roséen | 04 March 2016, 16:04:02 UTC | mkv: matroska_segment.cpp c-style cast => c++ cast Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org> | 05 March 2016, 21:14:02 UTC |