Re: [FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-23 Thread Michael Niedermayer
On Thu, Feb 23, 2017 at 07:57:41AM -0500, Ronald S. Bultje wrote: > Hi, > > On Thu, Feb 23, 2017 at 2:19 AM, wm4 wrote: > > > On Wed, 22 Feb 2017 19:16:46 +0100 > > Michael Niedermayer wrote: > > > > > This enables the extra error messages in case of DEBUG or high assrtion > > levels. > > > Hig

Re: [FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-23 Thread wm4
On Thu, 23 Feb 2017 07:57:41 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Thu, Feb 23, 2017 at 2:19 AM, wm4 wrote: > > > On Wed, 22 Feb 2017 19:16:46 +0100 > > Michael Niedermayer wrote: > > > > > This enables the extra error messages in case of DEBUG or high assrtion > > levels. > > >

Re: [FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-23 Thread Ronald S. Bultje
Hi, On Thu, Feb 23, 2017 at 2:19 AM, wm4 wrote: > On Wed, 22 Feb 2017 19:16:46 +0100 > Michael Niedermayer wrote: > > > This enables the extra error messages in case of DEBUG or high assrtion > levels. > > High assertion levels imply slow checks in inner loops so any extra > error should > > be

Re: [FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-22 Thread wm4
On Wed, 22 Feb 2017 19:16:46 +0100 Michael Niedermayer wrote: > This enables the extra error messages in case of DEBUG or high assrtion > levels. > High assertion levels imply slow checks in inner loops so any extra error > should > be insignificant. > Is it preferred to have a separate switch

Re: [FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-22 Thread Paul B Mahol
On 2/22/17, Michael Niedermayer wrote: > This enables the extra error messages in case of DEBUG or high assrtion > levels. > High assertion levels imply slow checks in inner loops so any extra error > should > be insignificant. > Is it preferred to have a separate switch for ff_elog() so it doesnt

[FFmpeg-devel] [PATCH 1/2] avutil/internal: Add ff_elog()

2017-02-22 Thread Michael Niedermayer
This enables the extra error messages in case of DEBUG or high assrtion levels. High assertion levels imply slow checks in inner loops so any extra error should be insignificant. Is it preferred to have a separate switch for ff_elog() so it doesnt depend on DEBUG/assertion level ? Signed-off-by: M