[FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

[Michael Niedermayer]

On Tue, Aug 05, 2014 at 04:23:14PM +0200, Clément Bœsch wrote:
> On Tue, Jul 22, 2014 at 01:51:46AM +0200, Michael Niedermayer wrote:
> [...]
> > > +typedef struct AVMBInfo_MB {
> > 
> >     > +    int8_t source; /* -1/+1 XXX: set exact relative ref frame instead of "direction" */
> > 
> > i suspect this is too small for long term references
> 
> Would signed 16-bit enough?

practically, probably
theoretically, i dont know


> 
> > 
> > 
> > > +    uint32_t type; /* how much codec specific can this be? */
> > > +    uint8_t w, h;
> > > +    uint16_t src_x, src_y;
> > > +    uint16_t dst_x, dst_y;
> > > +} AVMBInfo_MB;
> > 
> > as side data this is not optimal as it depends on native endianness
> > making exchange of side data over the network or serialization
> > into file storage more difficult
> 
> Any better suggestion that side-data?

probably not


> 
> > 
> > also QP, intra prediction mode, ... is missing 
> > 
> 
> Shouldn't we have QP in a separate side-data type?

thats possible too


[..]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140805/9727aca3/attachment.asc>