On Mon 04-04-16 00:35:40, Ted Tso wrote: > If a quota implementation does not have a report function, such as > quotaio_meta, print the same white spaces so that xfstests generic/235 > doesn't fail. The extra white sapce makes it easier to read the > output, and consistency is a good thing in any case. > > Signed-off-by: Theodore Ts'o <tytso@xxxxxxx> Thanks. Makes sense. I have merged the patch. Honza > --- > repquota.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/repquota.c b/repquota.c > index 957dc8d..ea79fc8 100644 > --- a/repquota.c > +++ b/repquota.c > @@ -400,9 +400,10 @@ static void report_it(struct quota_handle *h, int type) > if (h->qh_ops->scan_dquots(h, output) < 0) > return; > dump_cached_dquots(type); > - if (h->qh_ops->report && ofmt == QOF_DEFAULT) { > + if (ofmt == QOF_DEFAULT) { > putchar('\n'); > - h->qh_ops->report(h, flags & FL_VERBOSE); > + if (h->qh_ops->report) > + h->qh_ops->report(h, flags & FL_VERBOSE); > putchar('\n'); > } > if (ofmt == QOF_XML) > -- > 2.5.0 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html