On Wed, Jan 2, 2019 at 6:45 PM James Almer <[email protected]> wrote:
> On 1/2/2019 2:18 PM, Vittorio Giovara wrote: > > On Wed, Jan 2, 2019 at 4:13 PM Vittorio Giovara < > [email protected]> > > wrote: > > > >> > >> > >> On Fri, Dec 28, 2018 at 3:17 AM Guo, Yejun <[email protected]> wrote: > >> > > > > AVRegionOfInterest { > > size_t top/left/right/bottom > > } > > > > AVRegionOfInterestSet { > > int rois_nb; > > AVRegionOfInterest *rois; > > This will go south as soon as you start copying, referencing and freeing > frames and/or frame side data. > > All side data types need to be a contiguous array of bytes in memory > with no pointers. > Hmm you're right, but embedding an entire array is pretty bad too, especially because it locks the struct size... Do you have any alternative ideas for this? -- Vittorio _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
