ccanlint: mark unused parameters. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
tools: use tal/grab_file Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
tools: use tal instead of talloc. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
tools: use rbuf instead of grab_file. Slowly removing the talloc dependency. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
tools: don't assume modules are immediately below ccan/ dir. Removing this assumption should allow nested modules. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
ccanlint: fix _info documentation addition. Firstly, we should always ask before altering files. Secondly, we are not in the module directory: we need to use m->info_file->fullname. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
ccanlint: use ccan/autodata Gets rid of generated file.
ccanlint: remove argument to -k/--keep It's much easier to simply say "keep all", and it simplifies the code quite a bit.
ccanlint: exit with non-zero exit status if a test fails. This means we change some minor tests to "never fail", eg. whitespace or documentation tests. Note that pass/fail is independent of the score for a test.
ccanlint: fix _info option handling Two places were using a bogus ccanlint struct: the side-effect was that valgrind options didn't work, nor did license fixups. Make REGISTER_TEST do the extern declaration, and remove the bogus ones.
ccanlint: score_file_error() takes printf-format We simply build up the error string in score_file_error; a bit different but simpler than current behaviour. We keep around struct file_error because some tests need it.
ccanlint: rename files to match keys