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

Ganesh Ajjanagadde gajjanagadde at gmail.com
Wed Oct 28 01:21:11 CET 2015


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);
         }
-- 
2.6.2



More information about the ffmpeg-devel mailing list