On 27/05/15 02:00, Vittorio Giovara wrote:
> On Tue, May 26, 2015 at 7:50 PM, Andreas Cadhalpun
> <andreas.cadhal...@googlemail.com> 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.

lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to