Re: [FFmpeg-devel] [PATCH] ffprobe: move abort() call to after the print statements

2015-10-28 Thread Ganesh Ajjanagadde
On Wed, Oct 28, 2015 at 5:14 AM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 09:55:56AM +0100, wm4 wrote: >> On Tue, 27 Oct 2015 20:21:11 -0400 >> Ganesh Ajjanagadde wrote: >> >> > This fixes warning C4702 "unreachable code" reported in MSVC: >> > http://fate.ffmpeg.org/log.cgi?time=2015

Re: [FFmpeg-devel] [PATCH] ffprobe: move abort() call to after the print statements

2015-10-28 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 09:55:56AM +0100, wm4 wrote: > On Tue, 27 Oct 2015 20:21:11 -0400 > Ganesh Ajjanagadde wrote: > > > This fixes warning C4702 "unreachable code" reported in MSVC: > > http://fate.ffmpeg.org/log.cgi?time=20151026165907&log=compile&slot=x86_64-msvc12-windows-native, > > and a

Re: [FFmpeg-devel] [PATCH] ffprobe: move abort() call to after the print statements

2015-10-28 Thread wm4
On Tue, 27 Oct 2015 20:21:11 -0400 Ganesh Ajjanagadde wrote: > This fixes warning C4702 "unreachable code" reported in MSVC: > http://fate.ffmpeg.org/log.cgi?time=20151026165907&log=compile&slot=x86_64-msvc12-windows-native, > and also likely fixes the intended behavior. > > Signed-off-by: Ganes

[FFmpeg-devel] [PATCH] ffprobe: move abort() call to after the print statements

2015-10-27 Thread Ganesh Ajjanagadde
This fixes warning C4702 "unreachable code" reported in MSVC: http://fate.ffmpeg.org/log.cgi?time=20151026165907&log=compile&slot=x86_64-msvc12-windows-native, and also likely fixes the intended behavior. Signed-off-by: Ganesh Ajjanagadde --- ffprobe.c | 2 +- 1 file changed, 1 insertion(+), 1 d