On Sat, Aug 16, 2014 at 01:48:27AM +0200, wm4 wrote: > On Sat, 16 Aug 2014 01:44:59 +0200 > Michael Niedermayer <michae...@gmx.at> wrote: > > > On Sat, Aug 16, 2014 at 01:30:44AM +0200, wm4 wrote: > > > On Sat, 16 Aug 2014 01:15:42 +0200 > > > Michael Niedermayer <michae...@gmx.at> wrote: > > > > > > > Signed-off-by: Michael Niedermayer <michae...@gmx.at> > > > > --- > > > > doc/examples/resampling_audio.c | 3 +-- > > > > doc/examples/scaling_video.c | 3 +-- > > > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > > > > > diff --git a/doc/examples/resampling_audio.c > > > > b/doc/examples/resampling_audio.c > > > > index 8a43b09..f35e7e1 100644 > > > > --- a/doc/examples/resampling_audio.c > > > > +++ b/doc/examples/resampling_audio.c > > > > @@ -199,8 +199,7 @@ int main(int argc, char **argv) > > > > fmt, dst_ch_layout, dst_nb_channels, dst_rate, > > > > dst_filename); > > > > > > > > end: > > > > - if (dst_file) > > > > - fclose(dst_file); > > > > + fclose(dst_file); > > > > > > If dst_file can be NULL, then I don't think this is safe. > > > > how can dst_file be NULL ? > > but if preferred, iam happy to leave it as it is, one could argue > > it to be more rubust against future changes > > Oh, sorry, it wasn't clear from the context. Nevermind.
patch applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship naturally arises out of democracy, and the most aggravated form of tyranny and slavery out of the most extreme liberty. -- Plato
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
