In old ANSI-C you can't have expressions before variable declarations

(C++ allows this)

1 files changed, 4 insertions, 2 deletions

diff --git a/rsync/buf.c b/rsync/buf.c
index c978fff..fdd67ee 100644
--- a/rsync/buf.c
+++ b/rsync/buf.c
@@ -67,121 +67,123 @@ struct rs_filebuf {
 
 
 
 rs_filebuf_t *rs_filebuf_new(FILE *f, size_t buf_len) 
 {
     rs_filebuf_t *pf = rs_alloc_struct(rs_filebuf_t);
 
     pf->buf = rs_alloc(buf_len, "file buffer");
     pf->buf_len = buf_len;
     pf->f = f;
 
     return pf;
 }
 
 
 void rs_filebuf_free(rs_filebuf_t *fb) 
 {
     if ( fb->buf )
         free ( fb->buf );
         rs_bzero(fb, sizeof *fb);
         free(fb);
 }
 
 
 /*
  * If the stream has no more data available, read some from F into
  * BUF, and let the stream use that.  On return, SEEN_EOF is true if
  * the end of file has passed into the stream.
  */
 rs_result rs_infilebuf_fill(rs_job_t *job, rs_buffers_t *buf,
                             void *opaque)
 {
-    job=job;
     int                     len;
     rs_filebuf_t            *fb = (rs_filebuf_t *) opaque;
     FILE                    *f = fb->f;
+
+    job=job; // fix unused warning
         
     /* This is only allowed if either the buf has no input buffer
      * yet, or that buffer could possibly be BUF. */
     if (buf->next_in != NULL) {
         assert(buf->avail_in <= fb->buf_len);
         assert(buf->next_in >= fb->buf);
         assert(buf->next_in <= fb->buf + fb->buf_len);
     } else {
         assert(buf->avail_in == 0);
     }
 
     if (buf->eof_in || (buf->eof_in = feof(f))) {
         rs_trace("seen end of file on input");
         buf->eof_in = 1;
         return RS_DONE;
     }
 
     if (buf->avail_in)
         /* Still some data remaining.  Perhaps we should read
            anyhow? */
         return RS_DONE;
         
     len = fread(fb->buf, 1, fb->buf_len, f);
     if (len < 0) {
         if (ferror(f)) {
             rs_error("error filling buf from file: %s",
                      strerror(errno));
             return RS_IO_ERROR;
         } else {
             rs_error("no error bit, but got %d return when trying to read",
                      len);
             return RS_IO_ERROR;
         }
     }
     buf->avail_in = len;
     buf->next_in = fb->buf;
 
     return RS_DONE;
 }
 
 
 /*
  * The buf is already using BUF for an output buffer, and probably
  * contains some buffered output now.  Write this out to F, and reset
  * the buffer cursor.
  */
 rs_result rs_outfilebuf_drain(rs_job_t *job, rs_buffers_t *buf, void *opaque)
 {
-    job=job;
     int present;
     rs_filebuf_t *fb = (rs_filebuf_t *) opaque;
     FILE *f = fb->f;
 
+    job=job; // fix unused warning
+
     /* This is only allowed if either the buf has no output buffer
      * yet, or that buffer could possibly be BUF. */
     if (buf->next_out == NULL) {
         assert(buf->avail_out == 0);
                 
         buf->next_out = fb->buf;
         buf->avail_out = fb->buf_len;
                 
         return RS_DONE;
     }
         
     assert(buf->avail_out <= fb->buf_len);
     assert(buf->next_out >= fb->buf);
     assert(buf->next_out <= fb->buf + fb->buf_len);
 
     present = buf->next_out - fb->buf;
     if (present > 0) {
         int result;
                 
         assert(present > 0);
 
         result = fwrite(fb->buf, 1, present, f);
         if (present != result) {
             rs_error("error draining buf to file: %s",
                      strerror(errno));
             return RS_IO_ERROR;
         }
 
         buf->next_out = fb->buf;
         buf->avail_out = fb->buf_len;
     }