On 27.05.2015 14:37, Luca Barbato wrote: > On 27/05/15 02:00, Vittorio Giovara wrote: >> On Tue, May 26, 2015 at 7:50 PM, Andreas Cadhalpun >> <[email protected]> wrote: >>>>>> +int ffio_read_size(AVIOContext *s, unsigned char *buf, int size) >>>>>> +{ >>>>>> + int ret = avio_read(s, buf, size); >>>>>> + if (ret != size) >>>>>> + return AVERROR_INVALIDDATA; >>> >>> I like the idea of having such a function and I think most uses of avio_read >>> should be replaced with it. >>> What do you think about making it public and recommending it over using >>> avio_read directly? >> >> I'd be ok with it, let's hear if there is any objection from the crowd. >> > > API sugar is nice as long you do not have people lagging in updating the > library. > > I guess it could be exposed as we release but not before.
Why delaying it? New API like av_vda_default_init2 or av_small_strptime get's added all the time. Best regards, Andreas _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
