From: Paul Brossier <piem@piem.org>
Date: Wed, 12 Dec 2018 14:27:37 +0000 (+0100)
Subject: [io] abort sink_vorbis_do on error, remove unused variable
X-Git-Url: https://git.aubio.org/?a=commitdiff_plain;h=4b9481f974ca00a27aff49e15cbcdca02de676c6;p=aubio.git

[io] abort sink_vorbis_do on error, remove unused variable
---

diff --git a/src/io/sink_vorbis.c b/src/io/sink_vorbis.c
index da0584a1..57df1da7 100644
--- a/src/io/sink_vorbis.c
+++ b/src/io/sink_vorbis.c
@@ -30,8 +30,6 @@
 #include <errno.h> // errno
 #include <time.h> // time
 
-#define VORBIS_READSIZE 1024
-
 struct _aubio_sink_vorbis_t {
   FILE *fid;            // file id
   ogg_stream_state os;  // stream
@@ -224,6 +222,7 @@ void aubio_sink_vorbis_do(aubio_sink_vorbis_t *s, fvec_t *write_data,
     return;
   } else if (!buffer) {
     AUBIO_WRN("sink_vorbis: failed fetching buffer of size %d\n", write);
+    return;
   } else {
     for (c = 0; c < s->channels; c++) {
       for (v = 0; v < write; v++) {
@@ -247,6 +246,7 @@ void aubio_sink_vorbis_do_multi(aubio_sink_vorbis_t *s, fmat_t *write_data,
     return;
   } else if (!buffer) {
     AUBIO_WRN("sink_vorbis: failed fetching buffer of size %d\n", write);
+    return;
   } else {
     for (c = 0; c < s->channels; c++) {
       for (v = 0; v < write; v++) {