On Tue, Oct 22, 2019 at 1:46 AM Andreas Rheinhardt <
[email protected]> wrote:

> James Almer:
> > On 10/15/2019 8:17 AM, Andreas Rheinhardt wrote:
> >> Several members of structures denote offsets inside dynamic buffers and
> >> therefore always fit into an int. So it is unnecessary to use an int64_t
> >> for them.
> >
> > True, but they are set using avio_tell() and then passed to avio_seek(),
> > both of which expect int64_t. This change may result in some pedantic
> > compilers warning about that conversion.
> >
> > I don't really see the benefit in doing his. The struct size gains are
> > minimal.
> >
> Ok, I'll remove it.
>
> - Andreas
>
> None of the remaining patches need any modifications. So this patch can
just be skipped.

- Andreas
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to