Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Ricardo Constantino
Ok, nevermind. I see the srt and webvttenc samples use \h already, so that's what you'll want. On 11 October 2015 at 16:53, Ricardo Constantino wrote: > > On 11 October 2015 at 15:37, Clément Bœsch wrote: > >> nbsp → \h in ASS > > > nevcairiel suggested not relying on \h being parsed as nbsp in

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Ricardo Constantino
On 11 October 2015 at 15:37, Clément Bœsch wrote: > nbsp → \h in ASS nevcairiel suggested not relying on \h being parsed as nbsp in .srt with VSFilter and Libass, but maybe using the unicode char itself. Unless FFmpeg converts \h to the char code in srtenc or something? Aegisub converts \h to r

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 04:29:03PM +0100, Ricardo Constantino wrote: > Bare ampersand characters are still accepted, even though out-of-spec. > Also fixes adjacent tags not being parsed. > > Fixes trac #4915 > > Signed-off-by: Ricardo Constantino > --- > libavcodec/webvttdec.c | 13 +++-

[FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-09 Thread Ricardo Constantino
Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/libavcodec/webvttde