On 2009-07-27 15:37, ogg.k.ogg.k at googlemail.com wrote: > With latest feedback applied. > Thanks > > 2 remarks I forgot last time: > + sub->rects = av_mallocz(sizeof(*sub->rects)); IMHO, av_malloc is enough > + for (d = 0; d < sizeof(pic->data) / sizeof(pic->data[0]); > d++) { see FF_ARRAY_ELEMS macro. Ben