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

wm4 nfxjfg at googlemail.com
Wed Oct 28 09:55:56 CET 2015


On Tue, 27 Oct 2015 20:21:11 -0400
Ganesh Ajjanagadde <gajjanagadde at gmail.com> 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: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  ffprobe.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ffprobe.c b/ffprobe.c
> index 02e2c48..0374cd9 100644
> --- a/ffprobe.c
> +++ b/ffprobe.c
> @@ -1890,9 +1890,9 @@ static void show_frame(WriterContext *w, AVFrame *frame, AVStream *stream,
>              print_str("side_data_type", name ? name : "unknown");
>              print_int("side_data_size", sd->size);
>              if (sd->type == AV_FRAME_DATA_DISPLAYMATRIX && sd->size >= 9*4) {
> -                abort();
>                  writer_print_integers(w, "displaymatrix", sd->data, 9, " %11d", 3, 4, 1);
>                  print_int("rotation", av_display_rotation_get((int32_t *)sd->data));
> +                abort();
>              }
>              writer_print_section_footer(w);
>          }

Why the heck is there an abort()?


More information about the ffmpeg-devel mailing list