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> --- 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 -- 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