Paul B Mahol <[email protected]> writes: > On 3/20/12, Maans Rullgaard <[email protected]> wrote: >> Paul B Mahol <[email protected]> writes: >> >>> static av_always_inline void bytestream2_skip(GetByteContext *g, >>> - unsigned int size) >>> + int size) >>> { >>> g->buffer += FFMIN(g->buffer_end - g->buffer, size); >>> } >> >> Think about what happens with a large negative offset. > > Nothing better than with very big positive offset.
But a lot worse. With the signed argument type, it allows setting the position before the start of the buffer. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
