Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-10-10 Thread James Darnley
On 2015-10-09 14:46, Nicolas George wrote: > Le quartidi 4 vendémiaire, an CCXXIV, James Darnley a écrit : >> I can. You should find it attached to this email. I cleaned it up and >> put two test cases of data into the file. You will need Lua and the >> Lua-iconv module. If your package

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-10-09 Thread Nicolas George
Le quartidi 4 vendémiaire, an CCXXIV, James Darnley a écrit : > I can. You should find it attached to this email. I cleaned it up and > put two test cases of data into the file. You will need Lua and the > Lua-iconv module. If your package manager doesn't have that see here: >

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-25 Thread James Darnley
On 2015-09-24 20:37, Nicolas George wrote: > Le tridi 3 vendémiaire, an CCXXIV, James Darnley a écrit : >> I don't know what to say here. I know the encodings needed for iconv >> because I arrived at them by brute force. I wrote a short Lua script to >> iterate over a list of encodings supported

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-24 Thread Nicolas George
Le duodi 2 vendémiaire, an CCXXIV, James Darnley a écrit : > It is not supposed to replace any invalid bytes with a "random" > character. That sounds like it will only make the problem worse with > that lossy removal of data. This is trying to fix incorrect > interpretation of bytes. > > This

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-24 Thread James Darnley
On 2015-09-24 12:36, Nicolas George wrote: > Le duodi 2 vendémiaire, an CCXXIV, James Darnley a écrit : >> It is not supposed to replace any invalid bytes with a "random" >> character. That sounds like it will only make the problem worse with >> that lossy removal of data. This is trying to fix

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-24 Thread Nicolas George
Le tridi 3 vendémiaire, an CCXXIV, James Darnley a écrit : > As far as I understand the iconv API, it doesn't appear to do this for > you. So adding this feature would require writing code to handle more > errors returned from the iconv() function. That means a more > complicated argument

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-23 Thread James Darnley
On 2015-09-22 20:22, Nicolas George wrote: > Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : >> +@item -metadata_iconv_code_page_list @var{code_page_list} >> +Force the metadata from input files to be converted through the given >> codepages >> +using iconv. This allows the user

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-23 Thread Nicolas George
Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : > At present it only converts global metadata as that is what I wanted to do. > It > should be possible to extend it so that the conversion can be different for > different files or streams. > --- > doc/ffmpeg.texi | 6 +++ >

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-23 Thread James Darnley
On 2015-09-23 16:56, Nicolas George wrote: > Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : >> At present it only converts global metadata as that is what I wanted to do. >> It >> should be possible to extend it so that the conversion can be different for >> different files or

[FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-22 Thread James Darnley
At present it only converts global metadata as that is what I wanted to do. It should be possible to extend it so that the conversion can be different for different files or streams. --- doc/ffmpeg.texi | 6 +++ ffmpeg.c| 15 ++ ffmpeg.h| 1 + ffmpeg_opt.c| 149

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-22 Thread Nicolas George
Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : > At present it only converts global metadata as that is what I wanted to do. > It > should be possible to extend it so that the conversion can be different for > different files or streams. > --- > doc/ffmpeg.texi | 6 +++ >