On Mon, May 02, 2016 at 10:09:48AM -0400, Noah Massey wrote:
> > + if (!(flags & BALANCE_START_FILTERS) && !(flags & BALANCE_START_NOWARN)) {
> > + int delay = 10;
> > +
> > + printf("WARNING:\n\n");
> > + printf("\tFull balance without filters requested. This operation is very\n");
> > + printf("\tintense and takes potentially very long. It is recommended to\n");
> > + printf("\tuse the balance filters to narrow down the balanced data.\n");
> > + printf("\tUse 'btrfs balance start --full-balance' option to skip this\n");
> > + printf("\twarning. The operation will start in %d seconds.\n", delay);
> > + printf("\tUse Ctrl-C to stop it.\n");
> > + while (delay) {
> > + sleep(1);
> > + printf("%2d", delay--);
> > + fflush(stdout);
> > + }
>
> Shouldn't the sleep be after the fflush?
Yes it looks better when there's a delay after '1' appears. Care to send
a patch?
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
