Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-28 Thread Yayoi Ukai
>> >> rstrip_spaces_buf(dst); >> av_bprintf(dst, "\\N"); >> line_start = 1; >> @@ -90,6 +105,15 @@ void ff_htmlmarkup_to_ass(AVCodecContext *avctx, >> AVBPrint *dst, const char *in) >> av_bprint_chars(dst, *in, 1); >> break; >>

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-27 Thread Yayoi Ukai
On Sat, Aug 15, 2015 at 11:08 AM, Clément Bœsch wrote: > On Sat, Aug 08, 2015 at 12:52:04PM -0700, Yayoi Ukai wrote: > [...] >> >> -while (dst->len >= 2 && !strncmp(&dst->str[dst->len - 2], "\\N", 2)) >> >> -dst->len -= 2; >> >> -dst->str[dst->len] = 0; >> >> -rstrip_spaces_buf

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-15 Thread Clément Bœsch
On Sat, Aug 08, 2015 at 12:52:04PM -0700, Yayoi Ukai wrote: [...] > >> -while (dst->len >= 2 && !strncmp(&dst->str[dst->len - 2], "\\N", 2)) > >> -dst->len -= 2; > >> -dst->str[dst->len] = 0; > >> -rstrip_spaces_buf(dst); > > > > why did you completely remove this chunk? > > It

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-08 Thread Yayoi Ukai
On Sat, Aug 8, 2015 at 8:17 AM, Clément Bœsch wrote: >> Subject: Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML >> parsing code > > typo in the commit message: srtdec > > On Fri, Aug 07, 2015 at 11:03:28PM -0700, Yayoi wrote: >> This code will be u

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-08 Thread Clément Bœsch
> Subject: Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML > parsing code typo in the commit message: srtdec On Fri, Aug 07, 2015 at 11:03:28PM -0700, Yayoi wrote: > This code will be used in SAMI decoder in a later commit. > --- > libavcodec/Makefi