Signed-off-by: Felipe Contreras <[email protected]>
---
 revision.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/revision.c b/revision.c
index 0173e01..3ebd558 100644
--- a/revision.c
+++ b/revision.c
@@ -2971,7 +2971,7 @@ static struct commit *get_revision_internal(struct 
rev_info *revs)
        if (revs->max_count) {
                c = get_revision_1(revs);
                if (c) {
-                       while (0 < revs->skip_count) {
+                       while (revs->skip_count > 0) {
                                revs->skip_count--;
                                c = get_revision_1(revs);
                                if (!c)
@@ -2986,9 +2986,8 @@ static struct commit *get_revision_internal(struct 
rev_info *revs)
        if (c)
                c->object.flags |= SHOWN;
 
-       if (!revs->boundary) {
+       if (!revs->boundary)
                return c;
-       }
 
        if (!c) {
                /*
@@ -3034,9 +3033,8 @@ struct commit *get_revision(struct rev_info *revs)
 
        if (revs->reverse) {
                reversed = NULL;
-               while ((c = get_revision_internal(revs))) {
+               while ((c = get_revision_internal(revs)))
                        commit_list_insert(c, &reversed);
-               }
                revs->commits = reversed;
                revs->reverse = 0;
                revs->reverse_output_stage = 1;
-- 
1.8.4-fc

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to