[PATCH] Tidy while loop in end_compressed_writeback

[Sahil Kang <sahil.kang@asilaycomputing.com>]

Hi,

This is my first patch so it's a minor code change.
I think removing the early continue from the loop makes the function a 
little easier to follow.
Please have a look and I'd appreciate any feedback.

Thanks,
Sahil


 From 98afe83a570180e841fefe3fd48d450accc42ea3 Mon Sep 17 00:00:00 2001
From: Sahil Kang <sahil.kang@asilaycomputing.com>
Date: Wed, 19 Apr 2017 04:47:00 -0700
Subject: [PATCH] Tidy while loop in end_compressed_writeback

Instead of continuing early in the loop when ret is 0,
we can decrement/increment nr_pages/index by 1 at the ending.
The for loop will not execute when ret is 0 anyway.

Signed-off-by: Sahil Kang <sahil.kang@asilaycomputing.com>
---
  fs/btrfs/compression.c | 7 ++-----
  1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index c473c42..c653297 100644
--- a/fs/btrfs/compression.c
+++ b/fs/btrfs/compression.c
@@ -238,17 +238,14 @@ static noinline void 
end_compressed_writeback(struct inode *inode,
          ret = find_get_pages_contig(inode->i_mapping, index,
                       min_t(unsigned long,
                       nr_pages, ARRAY_SIZE(pages)), pages);
-        if (ret == 0) {
-            nr_pages -= 1;
-            index += 1;
-            continue;
-        }
          for (i = 0; i < ret; i++) {
              if (cb->errors)
                  SetPageError(pages[i]);
              end_page_writeback(pages[i]);
              page_cache_release(pages[i]);
          }
+
+        ret = ret ? ret : 1; /* set ret to 1 if it's 0 */
          nr_pages -= ret;
          index += ret;
      }
-- 
2.1.4