On Fri, Aug 28, 2015 at 08:33:07PM -0700, Yayoi wrote:
[...]
> @@ -82,18 +89,18 @@ static int sami_paragraph_to_ass(AVCodecContext *avctx,
> const char *src)
> if (*p == '<') {
> if (!av_strncasecmp(p, "<P", 2) && (p[2] == '>' ||
> av_isspace(p[2])))
> break;
> - if (!av_strncasecmp(p, "<BR", 3))
> + }
> + if (!av_strncasecmp(p, "<BR", 3)) {
> av_bprintf(dst, "\\N");
> - p++;
> - while (*p && *p != '>')
> - p++;
> - if (!*p)
> - break;
> - if (*p == '>')
> + p += 3;
> + while (*p && *p != '>')
> + p++;
*p can be 0 after this
> p++;
so when you execute this, shit happens. That's what I was trying to
explain in the previous patch.
> - continue;
> }
> - if (!av_isspace(*p))
> + if (*p == '\n') {
> + av_bprintf(dst, "\\N");
> + }
Is that really on purpose? <BR> is used to force a line break, but is a
real line break supposed to have the same purpose?
[...]
--
Clément B.
pgpAd51xOCU29.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
