[vlc-devel] [PATCH 0/4] demux/mp4: add support for internatilized metadata

[Filip Roséen]

These patches are primarily aimed at fixing the below linked ticket, while also
making sure that the implementation of metadata-handling is somewhat cleaner,
and that the string-box can properly be handled in the future.

 - https://trac.videolan.org/vlc/ticket/17608

Here's a sample session using `-Irc` with the sample attached to the ticket in
question:

    % vlc-devel -Irc 'https://trac.videolan.org/vlc/raw-attachment/ticket/17608/sample.mov' -q
    VLC media player 3.0.0-git Vetinari (revision 2.2.0-git-9771-g81d5d9b)
    VLC media player 3.0.0-git Vetinari
    Command Line Interface initialized. Type `help' for help.
    > playlist
    +----[ Playlist - Undefined ]
    | 2 - Playlist
    |   4 - SÚck THÌS (00:00:00) [played 1 time]
    | 3 - Media Library
    +----[ End of playlist ]
    > info 4
    +----[ Meta data ]
    |
    | filename: sample.mov
    | Title (fra): hello world
    | copyright: BjörŃ i SKõGẽÑ
    | Artist (fra): räfp
    | artist: RèFp
    | title: SÚck THÌS
    |
    +----[ end of stream info ]
    >


Given that mp4 has a lot of boxes, here's another one:

                      ____.----.
            ____.----'          \
            \                    \
             \                    \
              \                    \
               \          ____.----'`--.__
                \___.----'          |     `--.____
               /`-._                |       __.-' \
              /     `-._            ___.---'       \
             /          `-.____.---'                \
            /            / | \                       \
           /            /  |  \                   _.--'
           `-.         /   |   \            __.--'
              `-._    /    |    \     __.--'     |
                | `-./     |     \_.-'           |
                |          |                     |
                |          |                     |
                |          |                     |
                |          |                     |
                |          |                     |   VK
                |          |                     |
         _______|          |                     |_______________
                `-.        |                  _.-'
                   `-.     |           __..--'
                      `-.  |      __.-'
                         `-|__.--'

    src: http://www.chris.com/ascii/index.php?art=objects/boxes


Filip Roséen (4):
  demux/mp4: store length of payload in MP4_Box_data_string_t
  demux/mp4: meta: add more versatile helpers
  demux/mp4: meta: replace usage of legacy helpers
  demux/mp4: fix 17608: add support for parsing international strings

 modules/demux/mp4/libmp4.c |   1 +
 modules/demux/mp4/libmp4.h |   1 +
 modules/demux/mp4/meta.c   | 176 ++++++++++++++++++++++++++++++---------------
 3 files changed, 119 insertions(+), 59 deletions(-)

-- 
2.10.2