[FFmpeg-devel] [PATCH 2/2] Update fate ogg_vp3 expected result

2016-02-02 Thread Pierre Choffet
Since the Matroska tag format has been modified, the target expected file checksum after conversion has been modified too. --- tests/ref/lavf-fate/ogg_vp3 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/ref/lavf-fate/ogg_vp3 b/tests/ref/lavf-fate/ogg_vp3 index 9e9cc7

[FFmpeg-devel] [PATCH 1/2] Add TargetTypeValue in Matroska tag prefix

2016-02-02 Thread Pierre Choffet
Previously, the Matroska tag names were structured like this: [/] This lead to an issue when is not available: the different levels tags overwrite each other when they have the same name. This patch transforms the name prefix into: [-]/ As the TargetTypeValue has default value in case it ha

[FFmpeg-devel] Reworked patch

2016-02-02 Thread Pierre Choffet
Here is my second attempt to push this patch. The changes since last time are: - The global code structure follows Nicolas George suggestions, with an if-else statement. - The silent truncate has been fixed: tag name length is not arbitrarily limited anymore. - The fate broken test has been upda

[FFmpeg-devel] Improve Matroska tag name prefix

2016-01-27 Thread Pierre Choffet
The current implementation of the Matroska tags reader prefixes tag names with TargetType[1] only. In files with Tag having no TargetType element, all tags which share the same name but apply at different levels overwrite each other. This patch adds the TargetTypeValue[1] in the prefix. This e

[FFmpeg-devel] [PATCH] Add TargetTypeValue in Matroska tag prefix

2016-01-27 Thread Pierre Choffet
Previously, the Matroska tag names were structured like this: [/] This lead to an issue when is not available: the different levels tags overwrite each other when they have the same name. This patch transforms the name prefix into: [-]/ As the TargetTypeValue has default value in case it ha