> +/*
> + * LRC lyrics file format decoder
> + * Copyright (c) 2014 StarBrilliant <m13...@hotmail.com>

For copyright purposes we generally use real names.

> +#include "libavcodec/internal.h"

Ew. What are you using from this?

> +#include "libavutil/bprint.h"
> +#include "libavutil/dict.h"
> +
> +typedef struct LRCContext {
> +    FFDemuxSubtitlesQueue q;
> +    int ts_offset;

Is int enough for a timestamp difference?

> +    if(sscanf(p, "[-%d:%d.%d]", &mm, &ss, &cs) == 3) {\

'-' is a printf/scanf modifier, I'm not sure this will work.

> +        *start = -(mm*60000LL + ss*1000LL + cs*10LL); // just in case 
> negative pts

Why are you mangling the timestamps? Negative timestamps are technically valid.

- Derek

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to