Quoting Luca Barbato (2014-11-06 13:20:22) > On 06/11/14 11:02, Anton Khirnov wrote: > > Quoting Luca Barbato (2014-11-06 10:54:48) > >> On 06/11/14 06:56, Anton Khirnov wrote: > >>> Quoting Luca Barbato (2014-11-05 18:42:58) > >>>> On 05/11/14 18:41, Anton Khirnov wrote: > >>>>> It's not symmetric though. You still need to call write_header for > >>>> > >>>> What you'd like better? drop the **oc or add the write header part ? > >>>> > >>> > >>> Just forbid preallocated contexts. > >>> > >> > >> So who wants a custom avio has to use the normal boilerplate though. > >> > >> (noticed when updating it) > >> > >> Either ways work for me though. > >> > > > > If you want custom avio, then your current patch will explode horribly > > on failure. So yes, those people should write the extra ten lines. > > I think I have a patch to fix that somewhere (by adding a close callback > to the whole deal) >
The question then becomes, why would you want to call avformat_open_output() on a preallocated context with a preallocated IO context? It's pretty much a no-op then. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
