Re: [PATCH v8 1/5] dir_iterator: add tests for dir_iterator API

2017-04-12 Thread Stefan Beller
On Wed, Apr 5, 2017 at 6:39 PM, Daniel Ferreira wrote: > +int cmd_main(int argc, const char **argv) { Style (minor nit, in case resend is needed): We only put the { brace on the same line for statements (if, for, while, switch), not for function declarations. I tried to find a good rationale f

[PATCH v8 1/5] dir_iterator: add tests for dir_iterator API

2017-04-05 Thread Daniel Ferreira
Create t/helper/test-dir-iterator.c, which prints relevant information about a directory tree iterated over with dir_iterator. Create t/t0065-dir-iterator.sh, which tests that dir_iterator does iterate through a whole directory tree. Signed-off-by: Daniel Ferreira --- Makefile