[PATCH 5/8] Replace all the occurrences of AVERROR_EIO with AVERROR(EIO), and mark for deletion AVERROR_EIO at the next major bump.

Stefano Sabatini stefano.sabatini-lala
Thu Mar 11 00:37:39 CET 2010


---
 libavdevice/alsa-audio-common.c |    6 +++---
 libavdevice/vfwcap.c            |    6 +++---
 libavformat/ape.c               |    4 ++--
 libavformat/cafdec.c            |    8 ++++----
 libavformat/flacdec.c           |    2 +-
 libavformat/soxdec.c            |    2 +-
 libavutil/error.h               |    2 +-
 7 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/libavdevice/alsa-audio-common.c b/libavdevice/alsa-audio-common.c
index e125b2d..5bd0b49 100644
--- a/libavdevice/alsa-audio-common.c
+++ b/libavdevice/alsa-audio-common.c
@@ -74,7 +74,7 @@ av_cold int ff_alsa_open(AVFormatContext *ctx, snd_pcm_stream_t mode,
     if (res < 0) {
         av_log(ctx, AV_LOG_ERROR, "cannot open audio device %s (%s)\n",
                audio_device, snd_strerror(res));
-        return AVERROR_IO;
+        return AVERROR(EIO);
     }
 
     res = snd_pcm_hw_params_malloc(&hw_params);
@@ -153,7 +153,7 @@ fail:
     snd_pcm_hw_params_free(hw_params);
 fail1:
     snd_pcm_close(h);
-    return AVERROR_IO;
+    return AVERROR(EIO);
 }
 
 av_cold int ff_alsa_close(AVFormatContext *s1)
@@ -175,7 +175,7 @@ int ff_alsa_xrun_recover(AVFormatContext *s1, int err)
         if (err < 0) {
             av_log(s1, AV_LOG_ERROR, "cannot recover from underrun (snd_pcm_prepare failed: %s)\n", snd_strerror(err));
 
-            return AVERROR_IO;
+            return AVERROR(EIO);
         }
     } else if (err == -ESTRPIPE) {
         av_log(s1, AV_LOG_ERROR, "-ESTRPIPE... Unsupported!\n");
diff --git a/libavdevice/vfwcap.c b/libavdevice/vfwcap.c
index 9171457..3aaa5db 100644
--- a/libavdevice/vfwcap.c
+++ b/libavdevice/vfwcap.c
@@ -240,7 +240,7 @@ static int vfw_read_header(AVFormatContext *s, AVFormatParameters *ap)
 
     if(!ap->time_base.den) {
         av_log(s, AV_LOG_ERROR, "A time base must be specified.\n");
-        return AVERROR_IO;
+        return AVERROR(EIO);
     }
 
     ctx->s = s;
@@ -248,7 +248,7 @@ static int vfw_read_header(AVFormatContext *s, AVFormatParameters *ap)
     ctx->hwnd = capCreateCaptureWindow(NULL, 0, 0, 0, 0, 0, HWND_MESSAGE, 0);
     if(!ctx->hwnd) {
         av_log(s, AV_LOG_ERROR, "Could not create capture window.\n");
-        return AVERROR_IO;
+        return AVERROR(EIO);
     }
 
     /* If atoi fails, devnum==0 and the default device is used */
@@ -390,7 +390,7 @@ fail_bi:
 
 fail_io:
     vfw_read_close(s);
-    return AVERROR_IO;
+    return AVERROR(EIO);
 }
 
 static int vfw_read_packet(AVFormatContext *s, AVPacket *pkt)
diff --git a/libavformat/ape.c b/libavformat/ape.c
index a881138..b5f93cf 100644
--- a/libavformat/ape.c
+++ b/libavformat/ape.c
@@ -338,9 +338,9 @@ static int ape_read_packet(AVFormatContext * s, AVPacket * pkt)
     uint32_t extra_size = 8;
 
     if (url_feof(s->pb))
-        return AVERROR_IO;
+        return AVERROR(EIO);
     if (ape->currentframe > ape->totalframes)
-        return AVERROR_IO;
+        return AVERROR(EIO);
 
     url_fseek (s->pb, ape->frames[ape->currentframe].pos, SEEK_SET);
 
diff --git a/libavformat/cafdec.c b/libavformat/cafdec.c
index c4aad1b..df29716 100644
--- a/libavformat/cafdec.c
+++ b/libavformat/cafdec.c
@@ -293,13 +293,13 @@ static int read_packet(AVFormatContext *s, AVPacket *pkt)
     int64_t left      = CAF_MAX_PKT_SIZE;
 
     if (url_feof(pb))
-        return AVERROR_IO;
+        return AVERROR(EIO);
 
     /* don't read past end of data chunk */
     if (caf->data_size > 0) {
         left = (caf->data_start + caf->data_size) - url_ftell(pb);
         if (left <= 0)
-            return AVERROR_IO;
+            return AVERROR(EIO);
     }
 
     pkt_frames = caf->frames_per_packet;
@@ -317,12 +317,12 @@ static int read_packet(AVFormatContext *s, AVPacket *pkt)
             pkt_size   = caf->num_bytes - st->index_entries[caf->packet_cnt].pos;
             pkt_frames = st->duration   - st->index_entries[caf->packet_cnt].timestamp;
         } else {
-            return AVERROR_IO;
+            return AVERROR(EIO);
         }
     }
 
     if (pkt_size == 0 || pkt_frames == 0 || pkt_size > left)
-        return AVERROR_IO;
+        return AVERROR(EIO);
 
     res = av_get_packet(pb, pkt, pkt_size);
     if (res < 0)
diff --git a/libavformat/flacdec.c b/libavformat/flacdec.c
index 067d4fa..c9ad191 100644
--- a/libavformat/flacdec.c
+++ b/libavformat/flacdec.c
@@ -70,7 +70,7 @@ static int flac_read_header(AVFormatContext *s,
             }
             if (get_buffer(s->pb, buffer, metadata_size) != metadata_size) {
                 av_freep(&buffer);
-                return AVERROR_IO;
+                return AVERROR(EIO);
             }
             break;
         /* skip metadata block for unsupported types */
diff --git a/libavformat/soxdec.c b/libavformat/soxdec.c
index ceaedec..0151efe 100644
--- a/libavformat/soxdec.c
+++ b/libavformat/soxdec.c
@@ -97,7 +97,7 @@ static int sox_read_header(AVFormatContext *s,
         char *comment = av_malloc(comment_size+1);
         if (get_buffer(pb, comment, comment_size) != comment_size) {
             av_freep(&comment);
-            return AVERROR_IO;
+            return AVERROR(EIO);
         }
         comment[comment_size] = 0;
 
diff --git a/libavutil/error.h b/libavutil/error.h
index 7536fcf..ee4ba8b 100644
--- a/libavutil/error.h
+++ b/libavutil/error.h
@@ -44,8 +44,8 @@
 
 #if LIBAVUTIL_VERSION_MAJOR < 51
 #define AVERROR_UNKNOWN     AVERROR(EINVAL)  /**< unknown error */
-#endif
 #define AVERROR_IO          AVERROR(EIO)     /**< I/O error */
+#endif
 #define AVERROR_NUMEXPECTED AVERROR(EDOM)    /**< Number syntax expected in filename. */
 #define AVERROR_INVALIDDATA AVERROR(EINVAL)  /**< invalid data found */
 #define AVERROR_NOMEM       AVERROR(ENOMEM)  /**< not enough memory */
-- 
1.6.6.1


--YiEDa0DAkWCtVeE4
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="0006-Mark-AVERROR_ENOMEM-to-be-deleted-at-the-next-major-.patch"




More information about the ffmpeg-devel mailing list