From 9965fc25fcc92dc76d1cd4cf9595dc3dc9ebc586 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Mon, 10 Nov 2008 16:50:42 +1100 Subject: [PATCH] Rename string to str, and split into three modules. --- ccan/grab_file/_info.c | 51 ++++++ ccan/grab_file/grab_file.c | 50 ++++++ ccan/grab_file/grab_file.h | 58 +++++++ ccan/{string => grab_file}/test/run-grab.c | 0 ccan/{string => str}/_info.c | 8 +- ccan/str/str.h | 46 +++++ ccan/{string => str}/test/run.c | 38 +---- ccan/str_talloc/_info.c | 52 ++++++ .../string.c => str_talloc/str_talloc.c} | 51 +----- ccan/str_talloc/str_talloc.h | 64 +++++++ ccan/str_talloc/test/run.c | 52 ++++++ ccan/string/string.h | 159 ------------------ ccan/talloc/talloc.h | 2 +- tools/ccan_depends.c | 2 +- tools/ccanlint/Makefile | 3 +- tools/ccanlint/file_analysis.c | 4 +- tools/ccanlint/has_main_header.c | 2 +- tools/ccanlint/has_tests.c | 2 - tools/ccanlint/idempotent.c | 2 +- tools/ccanlint/trailing_whitespace.c | 2 +- tools/create_dep_tar.c | 3 +- tools/depends.c | 4 +- tools/doc_extract.c | 4 +- tools/namespacize.c | 4 +- tools/run_tests.c | 2 +- 25 files changed, 400 insertions(+), 265 deletions(-) create mode 100644 ccan/grab_file/_info.c create mode 100644 ccan/grab_file/grab_file.c create mode 100644 ccan/grab_file/grab_file.h rename ccan/{string => grab_file}/test/run-grab.c (100%) rename ccan/{string => str}/_info.c (75%) create mode 100644 ccan/str/str.h rename ccan/{string => str}/test/run.c (65%) create mode 100644 ccan/str_talloc/_info.c rename ccan/{string/string.c => str_talloc/str_talloc.c} (52%) create mode 100644 ccan/str_talloc/str_talloc.h create mode 100644 ccan/str_talloc/test/run.c delete mode 100644 ccan/string/string.h diff --git a/ccan/grab_file/_info.c b/ccan/grab_file/_info.c new file mode 100644 index 00000000..4aac31ae --- /dev/null +++ b/ccan/grab_file/_info.c @@ -0,0 +1,51 @@ +#include +#include +#include "config.h" + +/** + * grab_file - file helper routines + * + * This contains simple functions for getting the contents of a file. + * + * Example: + * #include "grab_file/grab_file.h" + * #include + * #include + * #include + * + * int main(int argc, char *argv[]) + * { + * size_t len; + * char *file; + * + * file = grab_file(NULL, argv[1], &len); + * if (!file) + * err(1, "Could not read file %s", argv[1]); + * if (strlen(file) != len) + * printf("File contains NUL characters\n"); + * else if (len == 0) + * printf("File contains nothing\n"); + * else if (strchr(file, '\n')) + * printf("File contains multiple lines\n"); + * else + * printf("File contains one line\n"); + * talloc_free(file); + * + * return 0; + * } + * + * Licence: LGPL (2 or any later version) + */ +int main(int argc, char *argv[]) +{ + if (argc != 2) + return 1; + + if (strcmp(argv[1], "depends") == 0) { + printf("ccan/talloc\n"); + printf("ccan/noerr\n"); + return 0; + } + + return 1; +} diff --git a/ccan/grab_file/grab_file.c b/ccan/grab_file/grab_file.c new file mode 100644 index 00000000..6f2c9fe1 --- /dev/null +++ b/ccan/grab_file/grab_file.c @@ -0,0 +1,50 @@ +#include "grab_file.h" +#include "talloc/talloc.h" +#include "noerr/noerr.h" +#include +#include +#include +#include + +void *grab_fd(const void *ctx, int fd, size_t *size) +{ + int ret; + size_t max = 16384, s; + char *buffer; + + if (!size) + size = &s; + *size = 0; + + buffer = talloc_array(ctx, char, max+1); + while ((ret = read(fd, buffer + *size, max - *size)) > 0) { + *size += ret; + if (*size == max) + buffer = talloc_realloc(ctx, buffer, char, max*=2 + 1); + } + if (ret < 0) { + talloc_free(buffer); + buffer = NULL; + } else + buffer[*size] = '\0'; + + return buffer; +} + +void *grab_file(const void *ctx, const char *filename, size_t *size) +{ + int fd; + char *buffer; + + if (!filename) + fd = dup(STDIN_FILENO); + else + fd = open(filename, O_RDONLY, 0); + + if (fd < 0) + return NULL; + + buffer = grab_fd(ctx, fd, size); + close_noerr(fd); + return buffer; +} diff --git a/ccan/grab_file/grab_file.h b/ccan/grab_file/grab_file.h new file mode 100644 index 00000000..5f7e37c6 --- /dev/null +++ b/ccan/grab_file/grab_file.h @@ -0,0 +1,58 @@ +#ifndef CCAN_GRAB_FILE_H +#define CCAN_GRAB_FILE_H +#include // For size_t + +/** + * grab_fd - read all of a file descriptor into memory + * @ctx: the context to tallocate from (often NULL) + * @fd: the file descriptor to read from + * @size: the (optional) size of the file + * + * This function reads from the given file descriptor until no more + * input is available. The content is talloced off @ctx, and the size + * of the file places in @size if it's non-NULL. For convenience, the + * byte after the end of the content will always be NUL. + * + * Example: + * // Return all of standard input, as lines. + * char **read_as_lines(void) + * { + * char **lines, *all; + * + * all = grab_fd(NULL, 0, NULL); + * if (!all) + * return NULL; + * lines = strsplit(NULL, all, "\n", NULL); + * talloc_free(all); + * return lines; + * } + */ +void *grab_fd(const void *ctx, int fd, size_t *size); + +/** + * grab_file - read all of a file (or stdin) into memory + * @ctx: the context to tallocate from (often NULL) + * @filename: the file to read (NULL for stdin) + * @size: the (optional) size of the file + * + * This function reads from the given file until no more input is + * available. The content is talloced off @ctx, and the size of the + * file places in @size if it's non-NULL. For convenience, the byte + * after the end of the content will always be NUL. + * + * Example: + * // Return all of a given file, as lines. + * char **read_as_lines(const char *filename) + * { + * char **lines, *all; + * + * all = grab_file(NULL, filename, NULL); + * if (!all) + * return NULL; + * lines = strsplit(NULL, all, "\n", NULL); + * talloc_free(all); + * return lines; + * } + */ +void *grab_file(const void *ctx, const char *filename, size_t *size); +#endif /* CCAN_GRAB_FILE_H */ diff --git a/ccan/string/test/run-grab.c b/ccan/grab_file/test/run-grab.c similarity index 100% rename from ccan/string/test/run-grab.c rename to ccan/grab_file/test/run-grab.c diff --git a/ccan/string/_info.c b/ccan/str/_info.c similarity index 75% rename from ccan/string/_info.c rename to ccan/str/_info.c index 873d3607..4baea773 100644 --- a/ccan/string/_info.c +++ b/ccan/str/_info.c @@ -3,13 +3,13 @@ #include "config.h" /** - * string - string helper routines + * str - string helper routines * - * This is a grab bag of modules for string operations, designed to enhance + * This is a grab bag of functions for string operations, designed to enhance * the standard string.h. * * Example: - * #include "string/string.h" + * #include "str/str.h" * * int main(int argc, char *argv[]) * { @@ -30,8 +30,6 @@ int main(int argc, char *argv[]) return 1; if (strcmp(argv[1], "depends") == 0) { - printf("ccan/talloc\n"); - printf("ccan/noerr\n"); return 0; } diff --git a/ccan/str/str.h b/ccan/str/str.h new file mode 100644 index 00000000..f633bc75 --- /dev/null +++ b/ccan/str/str.h @@ -0,0 +1,46 @@ +#ifndef CCAN_STR_H +#define CCAN_STR_H +#include +#include + +/** + * streq - Are two strings equal? + * @a: first string + * @b: first string + * + * This macro is arguably more readable than "!strcmp(a, b)". + * + * Example: + * if (streq(str, "")) + * printf("String is empty!\n"); + */ +#define streq(a,b) (strcmp((a),(b)) == 0) + +/** + * strstarts - Does this string start with this prefix? + * @str: string to test + * @prefix: prefix to look for at start of str + * + * Example: + * if (strstarts(str, "foo")) + * printf("String %s begins with 'foo'!\n", str); + */ +#define strstarts(str,prefix) (strncmp((str),(prefix),strlen(prefix)) == 0) + +/** + * strends - Does this string end with this postfix? + * @str: string to test + * @postfix: postfix to look for at end of str + * + * Example: + * if (strends(str, "foo")) + * printf("String %s end with 'foo'!\n", str); + */ +static inline bool strends(const char *str, const char *postfix) +{ + if (strlen(str) < strlen(postfix)) + return false; + + return streq(str + strlen(str) - strlen(postfix), postfix); +} +#endif /* CCAN_STR_H */ diff --git a/ccan/string/test/run.c b/ccan/str/test/run.c similarity index 65% rename from ccan/string/test/run.c rename to ccan/str/test/run.c index c50854ab..20711ad0 100644 --- a/ccan/string/test/run.c +++ b/ccan/str/test/run.c @@ -38,7 +38,7 @@ int main(int argc, char *argv[]) } } - plan_tests(n * n * 5 + 19); + plan_tests(n * n * 5); for (i = 0; i < n; i++) { for (j = 0; j < n; j++) { unsigned int k, identical = 0; @@ -79,41 +79,5 @@ int main(int argc, char *argv[]) } } - split = strsplit(NULL, "hello world", " ", &n); - ok1(n == 3); - ok1(streq(split[0], "hello")); - ok1(streq(split[1], "")); - ok1(streq(split[2], "world")); - ok1(split[3] == NULL); - talloc_free(split); - - split = strsplit(NULL, "hello world", " ", NULL); - ok1(streq(split[0], "hello")); - ok1(streq(split[1], "")); - ok1(streq(split[2], "world")); - ok1(split[3] == NULL); - talloc_free(split); - - split = strsplit(NULL, "hello world", "o ", NULL); - ok1(streq(split[0], "hell")); - ok1(streq(split[1], "")); - ok1(streq(split[2], "")); - ok1(streq(split[3], "w")); - ok1(streq(split[4], "rld")); - ok1(split[5] == NULL); - - ctx = split; - split = strsplit(ctx, "hello world", "o ", NULL); - ok1(talloc_parent(split) == ctx); - talloc_free(ctx); - - str = strjoin(NULL, substrings, ", "); - ok1(streq(str, "far, bar, baz, b, ba, z, ar, ")); - ctx = str; - str = strjoin(ctx, substrings, ""); - ok1(streq(str, "farbarbazbbazar")); - ok1(talloc_parent(str) == ctx); - talloc_free(ctx); - return exit_status(); } diff --git a/ccan/str_talloc/_info.c b/ccan/str_talloc/_info.c new file mode 100644 index 00000000..3186b2ba --- /dev/null +++ b/ccan/str_talloc/_info.c @@ -0,0 +1,52 @@ +#include +#include +#include "config.h" + +/** + * str_talloc - string helper routines which use talloc + * + * This is a grab bag of fnctions for string operations, designed to enhance + * the standard string.h; these are separated from the non-talloc-needing + * string utilities in "str.h". + * + * Example: + * #include "str_talloc/str_talloc.h" + * #include "talloc/talloc.h" + * #include "grab_file/grab_file.h" + * #include + * + * // Dumb demo program to double-linespace a file. + * int main(int argc, char *argv[]) + * { + * char *textfile; + * char **lines; + * + * // Grab lines in file. + * textfile = grab_file(NULL, argv[1], NULL); + * if (!textfile) + * err(1, "Failed reading %s", argv[1]); + * lines = strsplit(textfile, textfile, "\n", NULL); + * + * // Join them back together with two linefeeds. + * printf("%s", strjoin(textfile, lines, "\n\n")); + * + * // Free everything, just because we can. + * talloc_free(textfile); + * return 0; + * } + * + * Licence: LGPL (2 or any later version) + */ +int main(int argc, char *argv[]) +{ + if (argc != 2) + return 1; + + if (strcmp(argv[1], "depends") == 0) { + printf("ccan/talloc\n"); + printf("ccan/noerr\n"); + return 0; + } + + return 1; +} diff --git a/ccan/string/string.c b/ccan/str_talloc/str_talloc.c similarity index 52% rename from ccan/string/string.c rename to ccan/str_talloc/str_talloc.c index f0d0fd4f..7438f480 100644 --- a/ccan/string/string.c +++ b/ccan/str_talloc/str_talloc.c @@ -2,15 +2,9 @@ #include #include #include -#include #include -#include "string.h" +#include "str_talloc.h" #include "talloc/talloc.h" -#include -#include -#include -#include -#include "noerr/noerr.h" char **strsplit(const void *ctx, const char *string, const char *delims, unsigned int *nump) @@ -47,46 +41,3 @@ char *strjoin(const void *ctx, char *strings[], const char *delim) } return ret; } - -void *grab_fd(const void *ctx, int fd, size_t *size) -{ - int ret; - size_t max = 16384, s; - char *buffer; - - if (!size) - size = &s; - *size = 0; - - buffer = talloc_array(ctx, char, max+1); - while ((ret = read(fd, buffer + *size, max - *size)) > 0) { - *size += ret; - if (*size == max) - buffer = talloc_realloc(ctx, buffer, char, max*=2 + 1); - } - if (ret < 0) { - talloc_free(buffer); - buffer = NULL; - } else - buffer[*size] = '\0'; - - return buffer; -} - -void *grab_file(const void *ctx, const char *filename, size_t *size) -{ - int fd; - char *buffer; - - if (!filename) - fd = dup(STDIN_FILENO); - else - fd = open(filename, O_RDONLY, 0); - - if (fd < 0) - return NULL; - - buffer = grab_fd(ctx, fd, size); - close_noerr(fd); - return buffer; -} diff --git a/ccan/str_talloc/str_talloc.h b/ccan/str_talloc/str_talloc.h new file mode 100644 index 00000000..9828cd85 --- /dev/null +++ b/ccan/str_talloc/str_talloc.h @@ -0,0 +1,64 @@ +#ifndef CCAN_STR_TALLOC_H +#define CCAN_STR_TALLOC_H +#include +#include + +/** + * strsplit - Split string into an array of substrings + * @ctx: the context to tallocate from (often NULL) + * @string: the string to split + * @delims: delimiters where lines should be split. + * @nump: optional pointer to place resulting number of lines + * + * This function splits a single string into multiple strings. The + * original string is untouched: an array is allocated (using talloc) + * pointing to copies of each substring. Multiple delimiters result + * in empty substrings. By definition, no delimiters will appear in + * the substrings. + * + * The final char * in the array will be NULL, so you can use this or + * @nump to find the array length. + * + * Example: + * unsigned int count_long_lines(const char *text) + * { + * char **lines; + * unsigned int i, long_lines = 0; + * + * // Can only fail on out-of-memory. + * lines = strsplit(NULL, string, "\n", NULL); + * for (i = 0; lines[i] != NULL; i++) + * if (strlen(lines[i]) > 80) + * long_lines++; + * talloc_free(lines); + * return long_lines; + * } + */ +char **strsplit(const void *ctx, const char *string, const char *delims, + unsigned int *nump); + +/** + * strjoin - Join an array of substrings into one long string + * @ctx: the context to tallocate from (often NULL) + * @strings: the NULL-terminated array of strings to join + * @delim: the delimiter to insert between the strings + * + * This function joins an array of strings into a single string. The + * return value is allocated using talloc. Each string in @strings is + * followed by a copy of @delim. + * + * Example: + * // Append the string "--EOL" to each line. + * char *append_to_all_lines(const char *string) + * { + * char **lines, *ret; + * unsigned int i, num, newnum; + * + * lines = strsplit(NULL, string, "\n", NULL); + * ret = strjoin(NULL, lines, "-- EOL\n"); + * talloc_free(lines); + * return ret; + * } + */ +char *strjoin(const void *ctx, char *strings[], const char *delim); +#endif /* CCAN_STR_TALLOC_H */ diff --git a/ccan/str_talloc/test/run.c b/ccan/str_talloc/test/run.c new file mode 100644 index 00000000..6da48bd0 --- /dev/null +++ b/ccan/str_talloc/test/run.c @@ -0,0 +1,52 @@ +#include "str_talloc/str_talloc.h" +#include +#include +#include "str_talloc/str_talloc.c" +#include "tap/tap.h" + +/* FIXME: ccanize */ +#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof(arr[0])) + +int main(int argc, char *argv[]) +{ + unsigned int i, j, n; + + plan_tests(19); + split = strsplit(NULL, "hello world", " ", &n); + ok1(n == 3); + ok1(streq(split[0], "hello")); + ok1(streq(split[1], "")); + ok1(streq(split[2], "world")); + ok1(split[3] == NULL); + talloc_free(split); + + split = strsplit(NULL, "hello world", " ", NULL); + ok1(streq(split[0], "hello")); + ok1(streq(split[1], "")); + ok1(streq(split[2], "world")); + ok1(split[3] == NULL); + talloc_free(split); + + split = strsplit(NULL, "hello world", "o ", NULL); + ok1(streq(split[0], "hell")); + ok1(streq(split[1], "")); + ok1(streq(split[2], "")); + ok1(streq(split[3], "w")); + ok1(streq(split[4], "rld")); + ok1(split[5] == NULL); + + ctx = split; + split = strsplit(ctx, "hello world", "o ", NULL); + ok1(talloc_parent(split) == ctx); + talloc_free(ctx); + + str = strjoin(NULL, substrings, ", "); + ok1(streq(str, "far, bar, baz, b, ba, z, ar, ")); + ctx = str; + str = strjoin(ctx, substrings, ""); + ok1(streq(str, "farbarbazbbazar")); + ok1(talloc_parent(str) == ctx); + talloc_free(ctx); + + return exit_status(); +} diff --git a/ccan/string/string.h b/ccan/string/string.h deleted file mode 100644 index 3bc7adfd..00000000 --- a/ccan/string/string.h +++ /dev/null @@ -1,159 +0,0 @@ -#ifndef CCAN_STRING_H -#define CCAN_STRING_H -#include -#include - -/** - * streq - Are two strings equal? - * @a: first string - * @b: first string - * - * This macro is arguably more readable than "!strcmp(a, b)". - * - * Example: - * if (streq(str, "")) - * printf("String is empty!\n"); - */ -#define streq(a,b) (strcmp((a),(b)) == 0) - -/** - * strstarts - Does this string start with this prefix? - * @str: string to test - * @prefix: prefix to look for at start of str - * - * Example: - * if (strstarts(str, "foo")) - * printf("String %s begins with 'foo'!\n", str); - */ -#define strstarts(str,prefix) (strncmp((str),(prefix),strlen(prefix)) == 0) - -/** - * strends - Does this string end with this postfix? - * @str: string to test - * @postfix: postfix to look for at end of str - * - * Example: - * if (strends(str, "foo")) - * printf("String %s end with 'foo'!\n", str); - */ -static inline bool strends(const char *str, const char *postfix) -{ - if (strlen(str) < strlen(postfix)) - return false; - - return streq(str + strlen(str) - strlen(postfix), postfix); -} - -/** - * strsplit - Split string into an array of substrings - * @ctx: the context to tallocate from (often NULL) - * @string: the string to split - * @delims: delimiters where lines should be split. - * @nump: optional pointer to place resulting number of lines - * - * This function splits a single string into multiple strings. The - * original string is untouched: an array is allocated (using talloc) - * pointing to copies of each substring. Multiple delimiters result - * in empty substrings. By definition, no delimiters will appear in - * the substrings. - * - * The final char * in the array will be NULL, so you can use this or - * @nump to find the array length. - * - * Example: - * unsigned int count_long_lines(const char *text) - * { - * char **lines; - * unsigned int i, long_lines = 0; - * - * // Can only fail on out-of-memory. - * lines = strsplit(NULL, string, "\n", NULL); - * for (i = 0; lines[i] != NULL; i++) - * if (strlen(lines[i]) > 80) - * long_lines++; - * talloc_free(lines); - * return long_lines; - * } - */ -char **strsplit(const void *ctx, const char *string, const char *delims, - unsigned int *nump); - -/** - * strjoin - Join an array of substrings into one long string - * @ctx: the context to tallocate from (often NULL) - * @strings: the NULL-terminated array of strings to join - * @delim: the delimiter to insert between the strings - * - * This function joins an array of strings into a single string. The - * return value is allocated using talloc. Each string in @strings is - * followed by a copy of @delim. - * - * Example: - * // Append the string "--EOL" to each line. - * char *append_to_all_lines(const char *string) - * { - * char **lines, *ret; - * unsigned int i, num, newnum; - * - * lines = strsplit(NULL, string, "\n", NULL); - * ret = strjoin(NULL, lines, "-- EOL\n"); - * talloc_free(lines); - * return ret; - * } - */ -char *strjoin(const void *ctx, char *strings[], const char *delim); - -/** - * grab_fd - read all of a file descriptor into memory - * @ctx: the context to tallocate from (often NULL) - * @fd: the file descriptor to read from - * @size: the (optional) size of the file - * - * This function reads from the given file descriptor until no more - * input is available. The content is talloced off @ctx, and the size - * of the file places in @size if it's non-NULL. For convenience, the - * byte after the end of the content will always be NUL. - * - * Example: - * // Return all of standard input, as lines. - * char **read_as_lines(void) - * { - * char **lines, *all; - * - * all = grab_fd(NULL, 0, NULL); - * if (!all) - * return NULL; - * lines = strsplit(NULL, all, "\n", NULL); - * talloc_free(all); - * return lines; - * } - */ -void *grab_fd(const void *ctx, int fd, size_t *size); - -/** - * grab_file - read all of a file (or stdin) into memory - * @ctx: the context to tallocate from (often NULL) - * @filename: the file to read (NULL for stdin) - * @size: the (optional) size of the file - * - * This function reads from the given file until no more input is - * available. The content is talloced off @ctx, and the size of the - * file places in @size if it's non-NULL. For convenience, the byte - * after the end of the content will always be NUL. - * - * Example: - * // Return all of a given file, as lines. - * char **read_as_lines(const char *filename) - * { - * char **lines, *all; - * - * all = grab_file(NULL, filename, NULL); - * if (!all) - * return NULL; - * lines = strsplit(NULL, all, "\n", NULL); - * talloc_free(all); - * return lines; - * } - */ -void *grab_file(const void *ctx, const char *filename, size_t *size); -#endif /* CCAN_STRING_H */ diff --git a/ccan/talloc/talloc.h b/ccan/talloc/talloc.h index 2c55a12a..380009f3 100644 --- a/ccan/talloc/talloc.h +++ b/ccan/talloc/talloc.h @@ -27,7 +27,7 @@ #include #include #include "config.h" -#include "ccan/typesafe_cb/typesafe_cb.h" +#include "typesafe_cb/typesafe_cb.h" /* this uses a little trick to allow __LINE__ to be stringified diff --git a/tools/ccan_depends.c b/tools/ccan_depends.c index ca8c4a61..0511648e 100644 --- a/tools/ccan_depends.c +++ b/tools/ccan_depends.c @@ -2,7 +2,7 @@ #include #include #include -#include "string/string.h" +#include "str/str.h" #include "talloc/talloc.h" int main(int argc, char *argv[]) diff --git a/tools/ccanlint/Makefile b/tools/ccanlint/Makefile index 4cc50440..a084a224 100644 --- a/tools/ccanlint/Makefile +++ b/tools/ccanlint/Makefile @@ -23,7 +23,8 @@ tools/ccanlint/ccanlint: \ $(OBJS) \ tools/ccanlint/ccanlint.o \ tools/ccanlint/file_analysis.o \ - ccan/string/string.o ccan/talloc/talloc.o ccan/noerr/noerr.o + ccan/str_talloc/str_talloc.o ccan/grab_file/grab_file.o \ + ccan/talloc/talloc.o ccan/noerr/noerr.o ccanlint-clean: $(RM) tools/ccanlint/generated-init-tests diff --git a/tools/ccanlint/file_analysis.c b/tools/ccanlint/file_analysis.c index bee99597..d9f03dac 100644 --- a/tools/ccanlint/file_analysis.c +++ b/tools/ccanlint/file_analysis.c @@ -1,7 +1,9 @@ #include "ccanlint.h" #include "get_file_lines.h" #include -#include +#include +#include +#include #include #include #include diff --git a/tools/ccanlint/has_main_header.c b/tools/ccanlint/has_main_header.c index 75e3f91d..0647f434 100644 --- a/tools/ccanlint/has_main_header.c +++ b/tools/ccanlint/has_main_header.c @@ -8,7 +8,7 @@ #include #include #include -#include +#include #include #include diff --git a/tools/ccanlint/has_tests.c b/tools/ccanlint/has_tests.c index 7a187ab2..4a8d14b3 100644 --- a/tools/ccanlint/has_tests.c +++ b/tools/ccanlint/has_tests.c @@ -8,9 +8,7 @@ #include #include #include -#include #include -#include static char test_is_not_dir[] = "test is not a directory"; diff --git a/tools/ccanlint/idempotent.c b/tools/ccanlint/idempotent.c index 44565575..78ebd2ce 100644 --- a/tools/ccanlint/idempotent.c +++ b/tools/ccanlint/idempotent.c @@ -1,6 +1,6 @@ #include "ccanlint.h" #include -#include +#include #include #include #include diff --git a/tools/ccanlint/trailing_whitespace.c b/tools/ccanlint/trailing_whitespace.c index 3aeec6e5..35f37380 100644 --- a/tools/ccanlint/trailing_whitespace.c +++ b/tools/ccanlint/trailing_whitespace.c @@ -1,7 +1,7 @@ /* Trailing whitespace test. Almost embarrassing, but trivial. */ #include "ccanlint.h" #include -#include +#include static char *report_on_trailing_whitespace(const char *line) { diff --git a/tools/create_dep_tar.c b/tools/create_dep_tar.c index 78bdea6c..ecf043c5 100644 --- a/tools/create_dep_tar.c +++ b/tools/create_dep_tar.c @@ -4,7 +4,8 @@ #include #include #include -#include "ccan/string/string.h" +#include "ccan/grab_file/grab_file.h" +#include "ccan/str_talloc/str_talloc.h" #include "ccan/talloc/talloc.h" #include "tools/_infotojson/database.h" diff --git a/tools/depends.c b/tools/depends.c index d215441e..c631b9bc 100644 --- a/tools/depends.c +++ b/tools/depends.c @@ -1,5 +1,7 @@ #include "talloc/talloc.h" -#include "string/string.h" +#include "str/str.h" +#include "grab_file/grab_file.h" +#include "str_talloc/str_talloc.h" #include "tools.h" #include #include diff --git a/tools/doc_extract.c b/tools/doc_extract.c index 0a5752c1..fb7024a7 100644 --- a/tools/doc_extract.c +++ b/tools/doc_extract.c @@ -9,7 +9,9 @@ #include #include #include "talloc/talloc.h" -#include "string/string.h" +#include "str/str.h" +#include "str_talloc/str_talloc.h" +#include "grab_file/grab_file.h" static char **grab_doc(const char *fname) { diff --git a/tools/namespacize.c b/tools/namespacize.c index 6393d72e..3fa04028 100644 --- a/tools/namespacize.c +++ b/tools/namespacize.c @@ -10,7 +10,9 @@ #include #include #include -#include "ccan/string/string.h" +#include "ccan/str/str.h" +#include "ccan/str_talloc/str_talloc.h" +#include "ccan/grab_file/grab_file.h" #include "ccan/talloc/talloc.h" #include "tools.h" diff --git a/tools/run_tests.c b/tools/run_tests.c index b9a6f66c..3ae75237 100644 --- a/tools/run_tests.c +++ b/tools/run_tests.c @@ -6,7 +6,7 @@ #include #include "ccan/tap/tap.h" #include "ccan/talloc/talloc.h" -#include "ccan/string/string.h" +#include "ccan/str/str.h" #include "tools.h" /* FIXME: Use build bug later. */ -- 2.39.2