Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2016-08-23 Thread Nicolas George
Le septidi 7 fructidor, an CCXXIV, Ronald S. Bultje a écrit : > On Tue, Apr 21, 2015 at 9:43 AM, Nicolas George wrote: > > Le duodi 2 floréal, an CCXXIII, wm4 a écrit : > > > Isn't the problem that realloc() doesn't keep the alignment? > > No. That problem exists too > > > That seems counter-int

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2016-08-23 Thread Ronald S. Bultje
Hi, On Tue, Apr 21, 2015 at 9:43 AM, Nicolas George wrote: > Le duodi 2 floréal, an CCXXIII, wm4 a écrit : > > Isn't the problem that realloc() doesn't keep the alignment? > > No. That problem exists too That seems counter-intuitive. The problem does not exist, but it does exist too? If you'r

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2016-08-23 Thread Michael Niedermayer
On Sun, Jan 10, 2016 at 04:29:47AM +0100, Michael Niedermayer wrote: > On Tue, Apr 21, 2015 at 03:12:36PM +0200, Michael Niedermayer wrote: > > memalign() is not guranteed to be compatible with free() or realloc() > > and for platforms in this category we have --enable-memalign-hack > > (which shou

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2016-01-09 Thread Michael Niedermayer
On Tue, Apr 21, 2015 at 03:12:36PM +0200, Michael Niedermayer wrote: > memalign() is not guranteed to be compatible with free() or realloc() > and for platforms in this category we have --enable-memalign-hack > (which should be enabled automatically if such system is detected) > Trying to somehow h

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2015-04-21 Thread Nicolas George
Le duodi 2 floréal, an CCXXIII, Michael Niedermayer a écrit : > the only case iam aware of was some memory debugger which crashed > when realloc & malloc where mixed (thats clearly a bug in the debugger > as it should never crash, warn / fail maybe but not crash) I was referring to this one: comm

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2015-04-21 Thread Michael Niedermayer
On Tue, Apr 21, 2015 at 03:43:59PM +0200, Nicolas George wrote: > Le duodi 2 floréal, an CCXXIII, wm4 a écrit : > > Isn't the problem that realloc() doesn't keep the alignment? > > No. That problem exists too, but is irrelevant for the current code base: > there has been one bug of that kind about

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2015-04-21 Thread Nicolas George
Le duodi 2 floréal, an CCXXIII, wm4 a écrit : > Isn't the problem that realloc() doesn't keep the alignment? No. That problem exists too, but is irrelevant for the current code base: there has been one bug of that kind about three years ago, AFAIK that is all. The problem that was addressed here

Re: [FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2015-04-21 Thread wm4
On Tue, 21 Apr 2015 15:12:36 +0200 Michael Niedermayer wrote: > memalign() is not guranteed to be compatible with free() or realloc() > and for platforms in this category we have --enable-memalign-hack > (which should be enabled automatically if such system is detected) > Trying to somehow half s

[FFmpeg-devel] [PATCH] avutil/mem: remove av_realloc / av_malloc incompatibility warning

2015-04-21 Thread Michael Niedermayer
memalign() is not guranteed to be compatible with free() or realloc() and for platforms in this category we have --enable-memalign-hack (which should be enabled automatically if such system is detected) Trying to somehow half support systems that can free() memalign memory but not reallocate it see