Mark Harris 4ccafaca1c avformat/id3v2enc: use UTF-16 in id3v2.3 APIC frame only if non-ASCII
This makes the encoding of picture descriptions consistent with the
encoding of other text id3 tags and works better with iTunes, which
does not display pictures with some UTF-16 picture descriptions
(including a UTF-16 empty string, i.e. BOM + terminator).  It also
saves a few bytes.

Example:
 ffmpeg -f lavfi -i sine=b=4 -f lavfi -i smptebars -map 0:a -map 1:v \
   -codec:a libmp3lame -codec:v mjpeg -id3v2_version 3 \
   -metadata:s:v comment="Cover (front)" -t 3 -y out.mp3

This example does not set a picture description (-metadata:s:v title=)
so an empty string is written in the id3v2.3 APIC frame.  Without this
patch, UTF-16 is used and the cover art does not display in iTunes.
With the patch the cover art is displayed.  (Note that iTunes does not
display or have a way to set picture descriptions, only the picture
itself, but nevertheless has trouble skipping some UTF-16 descriptions.)

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
2013-08-11 13:06:40 +02:00
..
2013-06-09 01:10:36 +02:00
2013-06-09 01:10:36 +02:00
2013-07-30 15:29:12 +02:00
2013-07-01 21:37:16 +00:00
2013-06-28 09:04:16 +02:00
2013-08-02 13:05:16 +02:00
2013-06-29 18:02:37 +02:00
2013-06-20 21:49:02 +02:00
2013-07-02 18:52:45 +02:00
2013-06-15 18:50:11 -03:00
2013-07-07 18:50:03 +02:00
2013-08-02 21:14:59 +02:00
2013-08-09 23:02:48 +02:00
2013-07-06 22:27:35 +02:00
2013-08-05 02:53:16 +02:00
2013-08-05 10:14:42 +02:00
2013-06-06 20:29:38 +02:00
2013-06-16 18:56:56 +00:00
2013-06-02 11:25:56 +02:00