On Wed, 31 Jul 2013 12:09:20 +0200, Luca Barbato <[email protected]> wrote: > On 31/07/13 08:59, Anton Khirnov wrote: > > > > On Wed, 31 Jul 2013 01:51:28 +0200, Luca Barbato <[email protected]> wrote: > >> Codecs with internal parallelism need to set the pkt_pts/pkt_dts > >> by themselves. > >> --- > >> libavcodec/avcodec.h | 5 +++++ > >> libavcodec/utils.c | 6 ++++-- > >> 2 files changed, 9 insertions(+), 2 deletions(-) > >> > >> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > >> index 1c37e95..3e62d6e 100644 > >> --- a/libavcodec/avcodec.h > >> +++ b/libavcodec/avcodec.h > >> @@ -750,6 +750,11 @@ typedef struct RcOverride{ > >> */ > >> #define CODEC_CAP_VARIABLE_FRAME_SIZE 0x10000 > >> > >> +/** > >> + * Codec sets by its own the frame pkt_pts and pkt_dts. > >> + */ > >> +#define CODEC_CAP_PKT_TS 0x20000 > >> + > > > > I wonder if we should split caps into those that are useful to the caller > > and > > those like this one, that are for internal use only. > > It could be a good idea shall we offset the internal caps (e.g. start > from the bottom) ? >
I'd rather have a completely separate field, so the defines themselves could be hidden in a private header. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
