On 3/20/12, Maans Rullgaard <[email protected]> wrote: > 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.
Exactly, take look at bytestream2_init*. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
