[FFmpeg-devel] [PATCH 2/2] dnn_backend_native: check operand index

Guo, Yejun yejun.guo at intel.com
Wed Jun 17 06:23:34 EEST 2020



> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Carl
> Eugen Hoyos
> Sent: 2020年6月16日 18:57
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 2/2] dnn_backend_native: check operand
> index
> 
> 
> 
> > Am 16.06.2020 um 10:18 schrieb Guo, Yejun <yejun.guo at intel.com>:
> >
> > I personally like to always add '{}' to avoid possible mistakes when we change
> code in the future. Some coding styles also recommend this method.
> >
> > I checked at http://ffmpeg.org/developer.html#Coding-Rules and do not find
> an explicit description. But I just remembered that someone also mentioned this
> some time ago. I'll change to this style in v2 if no support from others for my
> style.
> 
> I support your style but it is only the file maintainer‘s decision.

thanks, I'll keep the V1 patch and push soon.

> 
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email ffmpeg-devel-request at ffmpeg.org with
> subject "unsubscribe".


More information about the ffmpeg-devel mailing list