Hi,

On Sat, Apr 9, 2011 at 8:57 AM, Anton Khirnov <an...@khirnov.net> wrote:
> ---
>  libavformat/avio.h |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)

This mostly looks OK.

If you (optionally) want to document more:

>     unsigned long checksum;
>     unsigned char *checksum_ptr;
>     unsigned long (*update_checksum)(unsigned long checksum, const uint8_t 
> *buf, unsigned int size);

This allows generating or validating a CRC-or-similar-style checksum
over input / output data. Callers should normally not need to touch
the function pointer directly, but would rather just use the utility
functions provided.

>     int (*read_pause)(void *opaque, int pause);
>     int64_t (*read_seek)(void *opaque, int stream_index,
>                          int64_t timestamp, int flags);

And for these, normally applications would have no business touching
them directly. If they do, it's likely a bug.

Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to