Re: [FFmpeg-devel] [PATCH] avutil/mem: Fix invalid use of av_alloc_size

2018-11-26 Thread Michael Niedermayer
On Sun, Nov 25, 2018 at 05:55:37PM -0800, Mark Harris wrote: > On 2018-11-25 17:29, James Almer wrote: > > On 11/25/2018 10:01 PM, Michael Niedermayer wrote: > >> On Sat, Nov 24, 2018 at 01:02:02PM -0800, Mark Harris wrote: > >>> The alloc_size attribute is valid only on functions that return a >

Re: [FFmpeg-devel] [PATCH] avutil/mem: Fix invalid use of av_alloc_size

2018-11-25 Thread Mark Harris
On 2018-11-25 17:29, James Almer wrote: > On 11/25/2018 10:01 PM, Michael Niedermayer wrote: >> On Sat, Nov 24, 2018 at 01:02:02PM -0800, Mark Harris wrote: >>> The alloc_size attribute is valid only on functions that return a >>> pointer. GCC 9 (not yet released) warns about invalid usage: >>>

Re: [FFmpeg-devel] [PATCH] avutil/mem: Fix invalid use of av_alloc_size

2018-11-25 Thread James Almer
On 11/25/2018 10:01 PM, Michael Niedermayer wrote: > On Sat, Nov 24, 2018 at 01:02:02PM -0800, Mark Harris wrote: >> The alloc_size attribute is valid only on functions that return a >> pointer. GCC 9 (not yet released) warns about invalid usage: >> >> ./libavutil/mem.h:342:1: warning:

Re: [FFmpeg-devel] [PATCH] avutil/mem: Fix invalid use of av_alloc_size

2018-11-25 Thread Michael Niedermayer
On Sat, Nov 24, 2018 at 01:02:02PM -0800, Mark Harris wrote: > The alloc_size attribute is valid only on functions that return a > pointer. GCC 9 (not yet released) warns about invalid usage: > > ./libavutil/mem.h:342:1: warning: 'alloc_size' attribute ignored on a > function returning int'

[FFmpeg-devel] [PATCH] avutil/mem: Fix invalid use of av_alloc_size

2018-11-24 Thread Mark Harris
The alloc_size attribute is valid only on functions that return a pointer. GCC 9 (not yet released) warns about invalid usage: ./libavutil/mem.h:342:1: warning: 'alloc_size' attribute ignored on a function returning int' [-Wattributes] 342 | av_alloc_size(2, 3) int av_reallocp_array(void