Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-28 Thread Tomas Härdin
ons 2022-09-28 klockan 13:06 +0200 skrev Andreas Rheinhardt: > Tomas Härdin: > > tis 2022-09-27 klockan 17:23 +0200 skrev Tomas Härdin: > > > mån 2022-09-26 klockan 16:24 +0200 skrev Tomas Härdin: > > > > mån 2022-09-26 klockan 14:25 +0200 skrev Andreas Rheinhardt: > > > > > Anton Khirnov: > > > >

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-28 Thread Andreas Rheinhardt
Tomas Härdin: > tis 2022-09-27 klockan 17:23 +0200 skrev Tomas Härdin: >> mån 2022-09-26 klockan 16:24 +0200 skrev Tomas Härdin: >>> mån 2022-09-26 klockan 14:25 +0200 skrev Andreas Rheinhardt: Anton Khirnov: > Quoting Andreas Rheinhardt (2022-07-14 14:51:07) >> Anton Khirnov: >>>

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-28 Thread Tomas Härdin
tis 2022-09-27 klockan 17:23 +0200 skrev Tomas Härdin: > mån 2022-09-26 klockan 16:24 +0200 skrev Tomas Härdin: > > mån 2022-09-26 klockan 14:25 +0200 skrev Andreas Rheinhardt: > > > Anton Khirnov: > > > > Quoting Andreas Rheinhardt (2022-07-14 14:51:07) > > > > > Anton Khirnov: > > > > > > Quoting

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-27 Thread Tomas Härdin
mån 2022-09-26 klockan 16:24 +0200 skrev Tomas Härdin: > mån 2022-09-26 klockan 14:25 +0200 skrev Andreas Rheinhardt: > > Anton Khirnov: > > > Quoting Andreas Rheinhardt (2022-07-14 14:51:07) > > > > Anton Khirnov: > > > > > Quoting Andreas Rheinhardt (2022-07-12 16:12:16) > > > > > > Anton really

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-26 Thread Tomas Härdin
mån 2022-09-26 klockan 14:25 +0200 skrev Andreas Rheinhardt: > Anton Khirnov: > > Quoting Andreas Rheinhardt (2022-07-14 14:51:07) > > > Anton Khirnov: > > > > Quoting Andreas Rheinhardt (2022-07-12 16:12:16) > > > > > Anton really dislikes the av_fast_* naming and instead wants > > > > > this to b

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-26 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Anton Khirnov: >> Quoting Andreas Rheinhardt (2022-07-14 14:51:07) >>> Anton Khirnov: Quoting Andreas Rheinhardt (2022-07-12 16:12:16) > Anton really dislikes the av_fast_* naming and instead wants this to be > called av_realloc_array_reuse(). I don't care either

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-09-26 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2022-07-14 14:51:07) >> Anton Khirnov: >>> Quoting Andreas Rheinhardt (2022-07-12 16:12:16) Anton really dislikes the av_fast_* naming and instead wants this to be called av_realloc_array_reuse(). I don't care either way. Any more opinions

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-08-17 Thread Anton Khirnov
Quoting Tomas Härdin (2022-07-21 23:23:25) > tis 2022-07-12 klockan 16:12 +0200 skrev Andreas Rheinhardt: > > > > Anton really dislikes the av_fast_* naming and instead wants this to > > be > > called av_realloc_array_reuse(). I don't care either way. Any more > > opinions on this (or on the patch

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-21 Thread Tomas Härdin
tis 2022-07-12 klockan 16:12 +0200 skrev Andreas Rheinhardt: > > Anton really dislikes the av_fast_* naming and instead wants this to > be > called av_realloc_array_reuse(). I don't care either way. Any more > opinions on this (or on the patch itself)? That's going to cause some "impedance mismat

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2022-07-14 14:51:07) > Anton Khirnov: > > Quoting Andreas Rheinhardt (2022-07-12 16:12:16) > >> Anton really dislikes the av_fast_* naming and instead wants this to be > >> called av_realloc_array_reuse(). I don't care either way. Any more > >> opinions on this (or on th

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-14 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2022-07-12 16:12:16) >> Anton really dislikes the av_fast_* naming and instead wants this to be >> called av_realloc_array_reuse(). I don't care either way. Any more >> opinions on this (or on the patch itself)? > > If people dislike _reuse(), I am open

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-14 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2022-07-12 16:12:16) > Anton really dislikes the av_fast_* naming and instead wants this to be > called av_realloc_array_reuse(). I don't care either way. Any more > opinions on this (or on the patch itself)? If people dislike _reuse(), I am open to other reasonable sug

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-12 Thread Andreas Rheinhardt
Andreas Rheinhardt: > From: Andreas Rheinhardt > > This is an array-equivalent of av_fast_realloc(). Its advantages > compared to using av_fast_realloc() for allocating arrays are as > follows: > > a) It performs its own overflow checks for the multiplication that is > implicit in array allocati

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-06 Thread Tomas Härdin
ons 2022-07-06 klockan 16:46 +0200 skrev Andreas Rheinhardt: > Tomas Härdin: > > tis 2022-07-05 klockan 22:26 +0200 skrev Andreas Rheinhardt: > > > From: Andreas Rheinhardt > > > > > > This is an array-equivalent of av_fast_realloc(). Its advantages > > > compared to using av_fast_realloc() for a

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-06 Thread Andreas Rheinhardt
Tomas Härdin: > tis 2022-07-05 klockan 22:26 +0200 skrev Andreas Rheinhardt: >> From: Andreas Rheinhardt >> >> This is an array-equivalent of av_fast_realloc(). Its advantages >> compared to using av_fast_realloc() for allocating arrays are as >> follows: >> >> a) It performs its own overflow chec

Re: [FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-06 Thread Tomas Härdin
tis 2022-07-05 klockan 22:26 +0200 skrev Andreas Rheinhardt: > From: Andreas Rheinhardt > > This is an array-equivalent of av_fast_realloc(). Its advantages > compared to using av_fast_realloc() for allocating arrays are as > follows: > > a) It performs its own overflow checks for the multiplica

[FFmpeg-devel] [PATCH 3/8] avutil/mem: Add av_fast_realloc_array()

2022-07-05 Thread Andreas Rheinhardt
From: Andreas Rheinhardt This is an array-equivalent of av_fast_realloc(). Its advantages compared to using av_fast_realloc() for allocating arrays are as follows: a) It performs its own overflow checks for the multiplication that is implicit in array allocations. (And it only needs to perform t