[FFmpeg-devel] [PATCH 19/23] dnn/dnn_backend_native_layer_conv2d: Check allocation

[Guo, Yejun]

> -----Original Message-----
> From: Pedro Arthur <bygrandao at gmail.com>
> Sent: 2021年3月11日 20:43
> To: Guo, Yejun <yejun.guo at intel.com>
> Cc: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 19/23]
> dnn/dnn_backend_native_layer_conv2d: Check allocation
> 
> Em qui., 11 de mar. de 2021 às 04:29, Guo, Yejun <yejun.guo at intel.com>
> escreveu:
> >
> >
> >
> > > -----Original Message-----
> > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> > > Andreas Rheinhardt
> > > Sent: 2021年3月11日 5:55
> > > To: ffmpeg-devel at ffmpeg.org
> > > Cc: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> > > Subject: [FFmpeg-devel] [PATCH 19/23]
> > > dnn/dnn_backend_native_layer_conv2d: Check allocation
> > >
> > > Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> > > ---
> > > Why does DNN actually not use the ordinary error codes?
> >
> > DNN_ERROR/DNN_SUCCESS is introduced at the very beginning, @Pedro any
> > comment if we need to revisit the error code? thanks.
> 
> I believe it was used for dnn specific errors and at some point were mixed with
> ordinary errors.
> I agree we should use ordinary error codes for ordinary errors.

I see, thanks.