[LTP] [PATCH v4 2/9] lib: split tst_fill_file() to create new tst_fill_fd()

Cyril Hrubis
2019-02-21T12:04:42Z

Hi!
> > -int tst_fill_file(const char *path, char pattern, size_t bs, size_t bcount)
> > +int tst_fill_fd(int fd, char pattern, size_t bs, size_t bcount)
> > {
> > - int fd;
> > - size_t counter;
> > + size_t i;
>
> Nitpicking perhaps but gratuitous variable renames don't make patches
> containing other changes easier to read. Nor, to be honest, does
> "counter" seem any more descriptive then i (since i is more idiomatic
> and therefore quicker to read).

It seems you misread the patch as we are replacing the counter with i
here and I asked for that since i is the idiomatic way of naming loop
variables, so all the blame for this change goes to me :-).

--
Cyril Hrubis
chrubis@suse.cz