videoaggregator: Document why buffer_vinfo is needed

This commit is contained in:
Nirbheek Chauhan 2015-03-24 19:05:00 +05:30 committed by Sebastian Dröge
parent 802a29eb48
commit dceb8aefd1

View file

@ -61,6 +61,12 @@ struct _GstVideoAggregatorPad
GstVideoInfo info; GstVideoInfo info;
GstBuffer *buffer; GstBuffer *buffer;
/* The caps on the pad may not match the buffer above because of two reasons:
* 1) When caps change, the info above will get updated, but the buffer might
* not since it might be pending on the GstAggregatorPad
* 2) We might reject the new buffer in fill_queues() and reuse a previous
* buffer which has older GstVideoInfo
* Hence, we need to maintain a GstVideoInfo for mapping buffers separately */
GstVideoInfo buffer_vinfo; GstVideoInfo buffer_vinfo;
GstVideoFrame *aggregated_frame; GstVideoFrame *aggregated_frame;