[PATCH] UBI-Utils: Cleanup of libubimirror
Alexander Schmidt
alexs at linux.vnet.ibm.com
Thu Jun 28 12:49:32 EDT 2007
libubimirror has several odd intendations and spacing errors.
Signed-off-by: Alexander Schmidt <alexs at linux.vnet.ibm.com>
---
ubi-utils/src/libubimirror.c | 49 ++++++++++++++++++++-----------------------
1 files changed, 23 insertions(+), 26 deletions(-)
--- mtd-utils.orig/ubi-utils/src/libubimirror.c
+++ mtd-utils/ubi-utils/src/libubimirror.c
@@ -54,10 +54,10 @@ static ssize_t fill_buffer(int fd, unsig
do {
got = read(fd, buf + have, len - have);
- if( got == -1 && errno != EINTR )
+ if (got == -1 && errno != EINTR)
return -1;
have += got;
- } while ( got > 0 && have < len);
+ } while (got > 0 && have < len);
return have;
}
@@ -71,10 +71,10 @@ static ssize_t flush_buffer(int fd, unsi
do {
done = write(fd, buf + have, len - have);
- if( done == -1 && errno != EINTR )
+ if (done == -1 && errno != EINTR)
return -1;
have += done;
- } while ( done > 0 && have < len);
+ } while (done > 0 && have < len);
return have;
}
@@ -88,8 +88,7 @@ static ssize_t flush_buffer(int fd, unsi
* fd_a is source
* fd_b is destination
*/
-static int
-compare_files(int fd_a, int fd_b)
+static int compare_files(int fd_a, int fd_b)
{
unsigned char buf_a[COMPARE_BUF_SIZE], buf_b[COMPARE_BUF_SIZE];
ssize_t len_a, len_b;
@@ -97,61 +96,59 @@ compare_files(int fd_a, int fd_b)
for (;;) {
len_a = fill_buffer(fd_a, buf_a, sizeof(buf_a));
- if (len_a == -1){
+ if (len_a == -1) {
rc = compare_error;
break;
}
len_b = fill_buffer(fd_b, buf_b, sizeof(buf_b));
- if (len_b == -1){
+ if (len_b == -1) {
rc = compare_different;
break;
}
- if( len_a != len_b ){
+ if (len_a != len_b) {
rc = compare_different;
break;
}
- if( len_a == 0 ){ /* Size on both filies equal and EOF */
+ if (len_a == 0) { /* Size on both files equal and EOF */
rc = compare_equal;
break;
}
- if( memcmp(buf_a, buf_b, len_a) != 0 ){
+ if (memcmp(buf_a, buf_b, len_a) != 0 ) {
rc = compare_different;
break;
}
}
/* Position both files at the beginning */
- if( lseek(fd_a, 0, SEEK_SET) == -1 ||
- lseek(fd_b, 0, SEEK_SET) == -1 )
+ if (lseek(fd_a, 0, SEEK_SET) == -1 ||
+ lseek(fd_b, 0, SEEK_SET) == -1)
rc = seek_error;
return rc;
}
-static int
-copy_files(int fd_in, int fd_out)
+static int copy_files(int fd_in, int fd_out)
{
unsigned char buf_a[COMPARE_BUF_SIZE];
ssize_t len_a, len_b;
unsigned long long update_size, copied;
- if( ubi_vol_get_used_bytes(fd_in, &update_size) == -1 ||
- ubi_vol_update(fd_out, update_size) == -1 )
+ if (ubi_vol_get_used_bytes(fd_in, &update_size) == -1 ||
+ ubi_vol_update(fd_out, update_size) == -1)
return update_error;
- for( copied = 0; copied < update_size; copied += len_b ){
+ for (copied = 0; copied < update_size; copied += len_b ) {
len_a = fill_buffer(fd_in, buf_a, sizeof(buf_a));
if (len_a == -1)
return read_error;
if (len_a == 0) /* Reach EOF */
return 0;
len_b = flush_buffer(fd_out, buf_a, len_a);
- if( len_b != len_a )
+ if (len_b != len_a)
return write_error;
}
return 0;
}
-int
-ubimirror(uint32_t devno, int seqnum, uint32_t *ids, ssize_t ids_size,
- char *err_buf, size_t err_buf_size)
+int ubimirror(uint32_t devno, int seqnum, uint32_t *ids, ssize_t ids_size,
+ char *err_buf, size_t err_buf_size)
{
int rc = 0;
uint32_t src_id;
@@ -173,7 +170,7 @@ ubimirror(uint32_t devno, int seqnum, ui
return ubi_error;
fd_in = ubi_vol_open(ulib, devno, src_id, O_RDONLY);
- if (fd_in == -1){
+ if (fd_in == -1) {
EBUF("open error source volume %d", ids[i]);
rc = open_error;
goto err;
@@ -190,7 +187,7 @@ ubimirror(uint32_t devno, int seqnum, ui
goto err;
}
rc = compare_files(fd_in, fd_out);
- if (rc < 0 ){
+ if (rc < 0) {
EBUF("compare error volume %d and %d", src_id, ids[i]);
goto err;
}
@@ -199,11 +196,11 @@ ubimirror(uint32_t devno, int seqnum, ui
EBUF("mirror error volume %d to %d", src_id, ids[i]);
goto err;
}
- if( (rc = ubi_vol_close(fd_out)) == -1 ){
+ if ((rc = ubi_vol_close(fd_out)) == -1) {
EBUF("close error volume %d", ids[i]);
rc = close_error;
goto err;
- }else
+ } else
fd_out = -1;
}
err:
More information about the linux-mtd
mailing list