On Mon, Dec 12, 2011 at 01:02:58AM +0530, Shitiz Dragooon Garg wrote: > On Mon, Dec 12, 2011 at 12:59 AM, Kostya Shishkov > <[email protected]> wrote: > > On Mon, Dec 12, 2011 at 12:49:40AM +0530, Shitiz Garg wrote: > >> --- > >> libavcodec/qtrle.c | 62 > >> +++++++++++++++++++++++++++++++++++++++++++++++++++- > >> 1 files changed, 61 insertions(+), 1 deletions(-) > > > > Maybe Diego has something to say about the message and I wonder how it > > behaves > > on all samples from our collection but LGTM. > > I ran it on every file from fate/qtrle folder and it seems to work. So > I think it should be fine
I meant every sample in http://samples.libav.org/V-codecs/QTRLE/ And maybe you should mention in commit message why zero values are not allowed (because it leads to negative skips). _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
