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