]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
common/safe_io.c: reduce scope of some ssize_t variables
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 7 May 2013 15:25:26 +0000 (17:25 +0200)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Fri, 10 May 2013 17:19:58 +0000 (19:19 +0200)
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/common/safe_io.c

index a7be648ebf41e5cf9a69907c53a068e2615bc2dc..ac99db04ad3eeca0ef4a9fa4bf69c49df9fb0995 100644 (file)
 
 ssize_t safe_read(int fd, void *buf, size_t count)
 {
-       ssize_t r;
        size_t cnt = 0;
 
        while (cnt < count) {
-               r = read(fd, buf, count - cnt);
+               ssize_t r = read(fd, buf, count - cnt);
                if (r <= 0) {
                        if (r == 0) {
                                // EOF
@@ -53,10 +52,8 @@ ssize_t safe_read_exact(int fd, void *buf, size_t count)
  
 ssize_t safe_write(int fd, const void *buf, size_t count)
 {
-       ssize_t r;
-
        while (count > 0) {
-               r = write(fd, buf, count);
+               ssize_t r = write(fd, buf, count);
                if (r < 0) {
                        if (errno == EINTR)
                                continue;
@@ -70,12 +67,11 @@ ssize_t safe_write(int fd, const void *buf, size_t count)
 
 ssize_t safe_pread(int fd, void *buf, size_t count, off_t offset)
 {
-       ssize_t r;
        size_t cnt = 0;
        char *b = (char*)buf;
 
        while (cnt < count) {
-               r = pread(fd, b + cnt, count - cnt, offset + cnt);
+               ssize_t r = pread(fd, b + cnt, count - cnt, offset + cnt);
                if (r <= 0) {
                        if (r == 0) {
                                // EOF
@@ -103,10 +99,8 @@ ssize_t safe_pread_exact(int fd, void *buf, size_t count, off_t offset)
 
 ssize_t safe_pwrite(int fd, const void *buf, size_t count, off_t offset)
 {
-       ssize_t r;
-
        while (count > 0) {
-               r = pwrite(fd, buf, count, offset);
+               ssize_t r = pwrite(fd, buf, count, offset);
                if (r < 0) {
                        if (errno == EINTR)
                                continue;