From: Antonio Ospite <ao2@ao2.it>
Date: Tue, 27 Feb 2018 22:42:32 +0000 (+0100)
Subject: am7xxx-play: remove any usage of ENOTSUP
X-Git-Tag: v0.1.7~15
X-Git-Url: https://git.ao2.it/libam7xxx.git/commitdiff_plain/18978589ed6ae1ddef52ae892b2c224b545ec29a?ds=inline;hp=-c

am7xxx-play: remove any usage of ENOTSUP

There's really no need to use ENOTSUP here, the return value is not
checked against the specific value, but just for the sign.

Replacing it with EINVAL makes the code more aligned with the other
error paths and also allows to remove a conditional define.
---

18978589ed6ae1ddef52ae892b2c224b545ec29a
diff --git a/examples/am7xxx-play.c b/examples/am7xxx-play.c
index 7bad4f7..82954fc 100644
--- a/examples/am7xxx-play.c
+++ b/examples/am7xxx-play.c
@@ -36,13 +36,6 @@
 
 #include <am7xxx.h>
 
-/* On some systems ENOTSUP is not defined, fallback to its value on
- * linux which is equal to EOPNOTSUPP which is 95
- */
-#ifndef ENOTSUP
-#define ENOTSUP 95
-#endif
-
 static unsigned int run = 1;
 
 struct video_input_ctx {
@@ -114,7 +107,7 @@ static int video_input_init(struct video_input_ctx *input_ctx,
 		}
 	if (video_index == -1) {
 		fprintf(stderr, "cannot find any video streams\n");
-		ret = -ENOTSUP;
+		ret = -EINVAL;
 		goto cleanup;
 	}
 
@@ -122,7 +115,7 @@ static int video_input_init(struct video_input_ctx *input_ctx,
 	input_codec_ctx = input_format_ctx->streams[video_index]->codec;
 	if (input_codec_ctx == NULL) {
 		fprintf(stderr, "input codec context is not valid\n");
-		ret = -ENOTSUP;
+		ret = -EINVAL;
 		goto cleanup;
 	}
 
@@ -130,7 +123,7 @@ static int video_input_init(struct video_input_ctx *input_ctx,
 	input_codec = avcodec_find_decoder(input_codec_ctx->codec_id);
 	if (input_codec == NULL) {
 		fprintf(stderr, "input_codec is NULL!\n");
-		ret = -ENOTSUP;
+		ret = -EINVAL;
 		goto cleanup;
 	}
 
@@ -138,7 +131,7 @@ static int video_input_init(struct video_input_ctx *input_ctx,
 	ret = avcodec_open2(input_codec_ctx, input_codec, NULL);
 	if (ret < 0) {
 		fprintf(stderr, "cannot open input codec\n");
-		ret = -ENOTSUP;
+		ret = -EINVAL;
 		goto cleanup;
 	}
 
@@ -245,7 +238,7 @@ static int video_output_init(struct video_output_ctx *output_ctx,
 	output_codec = avcodec_find_encoder(output_codec_ctx->codec_id);
 	if (output_codec == NULL) {
 		fprintf(stderr, "cannot find output codec!\n");
-		ret = -ENOTSUP;
+		ret = -EINVAL;
 		goto cleanup;
 	}