Re: [FFmpeg-devel] [PATCH 2/2] avformat/dvenc: Don't zero unnecessarily

2020-01-26 Thread Michael Niedermayer
On Sun, Jan 26, 2020 at 10:11:03AM +0100, Paul B Mahol wrote: > lgtm will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Its not that you shouldnt use gotos but rather that you should write readable code and code with gotos often but not always is le

Re: [FFmpeg-devel] [PATCH 2/2] avformat/dvenc: Don't zero unnecessarily

2020-01-26 Thread Paul B Mahol
lgtm On 1/26/20, Andreas Rheinhardt wrote: > The muxing context has already been zeroed when it was allocated, hence > it is unnecessary to do it again. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/dvenc.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/libavformat/dvenc

[FFmpeg-devel] [PATCH 2/2] avformat/dvenc: Don't zero unnecessarily

2020-01-25 Thread Andreas Rheinhardt
The muxing context has already been zeroed when it was allocated, hence it is unnecessary to do it again. Signed-off-by: Andreas Rheinhardt --- libavformat/dvenc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c index b89ad4d1c8..c71e532771 100644