[FFmpeg-devel] [PATCH V1 3/5] lavfi/dnn/dnn_backend_native: Refine the coding style

mypopy at gmail.com mypopy at gmail.com
Tue Aug 20 05:01:08 EEST 2019


On Tue, Aug 20, 2019 at 9:31 AM Guo, Yejun <yejun.guo at intel.com> wrote:
>
>
>
> > -----Original Message-----
> > From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf Of
> > Jun Zhao
> > Sent: Monday, August 19, 2019 9:01 PM
> > To: ffmpeg-devel at ffmpeg.org
> > Cc: Jun Zhao <barryjzhao at tencent.com>
> > Subject: [FFmpeg-devel] [PATCH V1 3/5] lavfi/dnn/dnn_backend_native: Refine
> > the coding style
> >
> > From: Jun Zhao <barryjzhao at tencent.com>
> >
> >     We perfer the coding style like:
> >
> >     /* some stuff */
> >     if (error) {
> >         /* error handling */
> >         return -(errorcode);
> >     }
> >     /* normal actions */
> >     do_something()
>
> as mentioned in http://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/247887.html, I've finished
> some more patches at https://github.com/guoyejun/ffmpeg/commits/dnn0809 waiting for send out set by set.
>
> The next patch I plan to send is https://github.com/guoyejun/ffmpeg/commit/02567319a640bb3d150917dfc6cf5e047a4bb776,
> which includes the changes in this patch, and it includes more changes to support operand concept in native mode.
>
> It would be convenient for me to not split that patch, could this patch be ignored? thanks.
>
Please continue this patch set, will hold on this patch. Thanks


More information about the ffmpeg-devel mailing list