Message ID | 20180302184704.22399-3-jeffm@suse.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 2.03.2018 20:46, jeffm@suse.com wrote: > From: Jeff Mahoney <jeffm@suse.com> > > In print_single_qgroup_table we check the loop index against > BTRFS_QGROUP_CHILD, but what we really mean is "last column." Since > we have an enum value to indicate the last value, use that instead > of assuming that BTRFS_QGROUP_CHILD is always last. > > Signed-off-by: Jeff Mahoney <jeffm@suse.com> Reviewed-by: Nikolay Borisov <nborisov@suse.com> > --- > qgroup.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qgroup.c b/qgroup.c > index 11659e83..67bc0738 100644 > --- a/qgroup.c > +++ b/qgroup.c > @@ -267,7 +267,7 @@ static void print_single_qgroup_table(struct btrfs_qgroup *qgroup) > continue; > print_qgroup_column(qgroup, i); > > - if (i != BTRFS_QGROUP_CHILD) > + if (i != BTRFS_QGROUP_ALL - 1) > printf(" "); > } > printf("\n"); > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/qgroup.c b/qgroup.c index 11659e83..67bc0738 100644 --- a/qgroup.c +++ b/qgroup.c @@ -267,7 +267,7 @@ static void print_single_qgroup_table(struct btrfs_qgroup *qgroup) continue; print_qgroup_column(qgroup, i); - if (i != BTRFS_QGROUP_CHILD) + if (i != BTRFS_QGROUP_ALL - 1) printf(" "); } printf("\n");