Code cleanup.

diff --git a/ext/vcedit.c b/ext/vcedit.c
index b6c7dfd8d0d0db18e361b0fd79b37fa74a1664cb..24622e855974eafc7a6c7765c7edfc21a017d302 100644 (file)
--- a/ext/vcedit.c
+++ b/ext/vcedit.c
@@ -133,7 +133,6 @@ vcedit_clear_internals (vcedit_state *state)
        }
 
        state->serial = 0;
        }
 
        state->serial = 0;

-       state->prevW = state->extrapage = state->eosin = 0;

 }
 
 void
 }
 
 void


@@ -168,6 +167,8 @@ _v_writestring (oggpack_buffer *o, char *s, int len)
 static int
 _commentheader_out (vorbis_comment *vc, char *vendor, ogg_packet *op)
 {
 static int
 _commentheader_out (vorbis_comment *vc, char *vendor, ogg_packet *op)
 {

+       int i;
+

        oggpack_buffer opb;
 
        oggpack_writeinit (&opb);
        oggpack_buffer opb;
 
        oggpack_writeinit (&opb);


@@ -183,16 +184,13 @@ _commentheader_out (vorbis_comment *vc, char *vendor, ogg_packet *op)
        /* comments */
        oggpack_write (&opb, vc->comments, 32);
 
        /* comments */
        oggpack_write (&opb, vc->comments, 32);
 

-       if (vc->comments) {
-               int i;
-
-               for (i = 0; i < vc->comments; i++) {
-                       if (vc->user_comments[i]) {
-                               oggpack_write (&opb, vc->comment_lengths[i], 32);
-                               _v_writestring (&opb, vc->user_comments[i],
-                                               vc->comment_lengths[i]);
-                       } else
-                               oggpack_write (&opb, 0, 32);
+       for (i = 0; i < vc->comments; i++) {
+               if (!vc->user_comments[i])
+                       oggpack_write (&opb, 0, 32);
+               else {
+                       oggpack_write (&opb, vc->comment_lengths[i], 32);
+                       _v_writestring (&opb, vc->user_comments[i],
+                                       vc->comment_lengths[i]);

                }
        }
 
                }
        }
 


@@ -214,13 +212,12 @@ _commentheader_out (vorbis_comment *vc, char *vendor, ogg_packet *op)
 static int
 _blocksize (vcedit_state *s, ogg_packet *p)
 {
 static int
 _blocksize (vcedit_state *s, ogg_packet *p)
 {

-       int this = vorbis_packet_blocksize (s->vi, p);
-       int ret = (this + s->prevW) / 4;
+       int this, ret = 0;

 
 

-       if (!s->prevW) {
-               s->prevW = this;
-               return 0;
-       }
+       this = vorbis_packet_blocksize (s->vi, p);
+
+       if (s->prevW)
+               ret = (this + s->prevW) / 4;

 
        s->prevW = this;
 
 
        s->prevW = this;
 


@@ -418,8 +415,7 @@ vcedit_write (vcedit_state *state)
                return -1;
        }
 
                return -1;
        }
 

-       state->eosin = 0;
-       state->extrapage = 0;
+       state->prevW = state->extrapage = state->eosin = 0;

 
        header_main.bytes = state->mainlen;
        header_main.packet = state->mainbuf;
 
        header_main.bytes = state->mainlen;
        header_main.packet = state->mainbuf;