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

[Cyril Hrubis]

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