tools: more intelligent caching for compile _info.
authorRusty Russell <rusty@rustcorp.com.au>
Mon, 10 Sep 2012 05:03:01 +0000 (14:33 +0930)
committerRusty Russell <rusty@rustcorp.com.au>
Mon, 10 Sep 2012 05:03:01 +0000 (14:33 +0930)
We let the get_deps() caller hand us the compiled _info filename, but what
about recursive dependencies?  These we re-generate every time.

So fix this: hand a generator callback to get_deps(), expose the one
which simply compiles it, and add a ccanlint one which looks up the
manifest to see if we have one already.

Before:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
31

After:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
17

tools/ccan_depends.c
tools/ccanlint/ccanlint.h
tools/ccanlint/file_analysis.c
tools/ccanlint/tests/depends_exist.c
tools/ccanlint/tests/examples_compile.c
tools/depends.c
tools/namespacize.c
tools/tools.h

index 4dda6fa274d65965daaceb4707b377b8bc1989ff..c0b0f94b1ce12fcaa7e6581c2a5a7105de3e8b6a 100644 (file)
@@ -38,7 +38,7 @@ int main(int argc, char *argv[])
 
        if (compile)
                deps = get_deps(talloc_autofree_context(), argv[1],
-                               recurse, NULL);
+                               recurse, compile_info);
        else
                deps = get_safe_ccan_deps(talloc_autofree_context(),
                                          argv[1], recurse);
index 8ebf68d276e8f2bc4042bf5694e1a1b534cbd1e1..954bb30a0b9d06782bdd1c3ecc817666c21b1673 100644 (file)
@@ -164,6 +164,9 @@ void compile_and_link_async(const void *ctx, unsigned int time_ms,
 /* Get results of a command, returning ctx (and free it). */
 void *collect_command(bool *ok, char **output);
 
+/* Find manifest for this dir and return compiled _info filename. */
+char *get_or_compile_info(const void *ctx, const char *dir);
+
 /* Normal tests. */
 extern struct ccanlint trailing_whitespace;
 
index 8760630a7549e22a4d981c24fe69fab407d37043..575425df79e0c10ef993ae7f92747e70f3d8bdaf 100644 (file)
@@ -394,3 +394,13 @@ void score_file_error(struct score *score, struct ccan_file *f, unsigned line,
                score->error = talloc_append_string(score->error,
                                    "... more (use -vv to see them all)\n");
 }
+
+char *get_or_compile_info(const void *ctx, const char *dir)
+{
+       struct manifest *m = get_manifest(NULL, dir);
+
+       if (!m->info_file->compiled[COMPILE_NORMAL])
+               m->info_file->compiled[COMPILE_NORMAL] = compile_info(m, dir);
+
+       return m->info_file->compiled[COMPILE_NORMAL];
+}
index 27bbfc4ed0f2c28a13868fba99987c2c4b9463b7..7712c48d1847738e903200bbd2dc9c106e09750d 100644 (file)
@@ -57,8 +57,7 @@ static void check_depends_exist(struct manifest *m,
        if (safe_mode)
                deps = get_safe_ccan_deps(m, m->dir, true);
        else
-               deps = get_deps(m, m->dir, true,
-                               &m->info_file->compiled[COMPILE_NORMAL]);
+               deps = get_deps(m, m->dir, true, get_or_compile_info);
 
        for (i = 0; deps[i]; i++) {
                if (!strstarts(deps[i], "ccan/"))
index 6602c54683ed3b215dfa1dd9a5780267b11ba5cb..7fb60845c6604af2e1b5ecbe7cb90b1bd2c1a286 100644 (file)
@@ -62,8 +62,7 @@ static void add_dep(struct manifest ***deps, const char *basename)
        if (m->info_file) {
                char **infodeps;
 
-               infodeps = get_deps(m, m->dir, false,
-                                   &m->info_file->compiled[COMPILE_NORMAL]);
+               infodeps = get_deps(m, m->dir, false, get_or_compile_info);
 
                for (i = 0; infodeps[i]; i++) {
                        if (strstarts(infodeps[i], "ccan/"))
index 7f232233b32b90bda37a9b5f35422e3c399ce6b0..6958c37bd2608c2125b1987aeb17b5d50d193a77 100644 (file)
@@ -38,7 +38,7 @@ lines_from_cmd(const void *ctx, const char *format, ...)
 
 /* Be careful about trying to compile over running programs (parallel make).
  * temp_file helps here. */
-static char *compile_info(const void *ctx, const char *dir)
+char *compile_info(const void *ctx, const char *dir)
 {
        char *info_c_file, *info, *ccandir, *compiled, *output;
        size_t len;
@@ -71,17 +71,16 @@ static char *compile_info(const void *ctx, const char *dir)
        return NULL;
 }
 
-static char **get_one_deps(const void *ctx, const char *dir, char **infofile)
+static char **get_one_deps(const void *ctx, const char *dir,
+                          char *(*get_info)(const void *ctx, const char *dir))
 {
        char **deps, *cmd;
+       char *infofile = get_info(ctx, dir);
 
-       if (!*infofile) {
-               *infofile = compile_info(ctx, dir);
-               if (!*infofile)
-                       errx(1, "Could not compile _info for '%s'", dir);
-       }
+       if (!infofile)
+               return NULL;
 
-       cmd = talloc_asprintf(ctx, "%s depends", *infofile);
+       cmd = talloc_asprintf(ctx, "%s depends", infofile);
        deps = lines_from_cmd(cmd, "%s", cmd);
        if (!deps)
                err(1, "Could not run '%s'", cmd);
@@ -118,7 +117,7 @@ static char *replace(const void *ctx, const char *src,
 /* This is a terrible hack.  We scan for ccan/ strings. */
 static char **get_one_safe_deps(const void *ctx,
                                const char *dir,
-                               char **infofile)
+                               char *(*unused)(const void *, const char *))
 {
        char **deps, **lines, *raw, *fname;
        unsigned int i, n;
@@ -178,17 +177,17 @@ static bool have_dep(char **deps, const char *dep)
 /* Gets all the dependencies, recursively. */
 static char **
 get_all_deps(const void *ctx, const char *dir,
-            char **infofile,
-            char **(*get_one)(const void *, const char *, char **))
+            char *(*get_info)(const void *ctx, const char *dir),
+            char **(*get_one)(const void *, const char *,
+                              char *(*get_info)(const void *, const char *)))
 {
        char **deps;
        unsigned int i;
 
-       deps = get_one(ctx, dir, infofile);
+       deps = get_one(ctx, dir, get_info);
        for (i = 0; i < talloc_array_length(deps)-1; i++) {
                char **newdeps;
                unsigned int j;
-               char *subinfo = NULL;
                char *subdir;
 
                if (!strstarts(deps[i], "ccan/"))
@@ -197,7 +196,7 @@ get_all_deps(const void *ctx, const char *dir,
                subdir = talloc_asprintf(ctx, "%s/%s",
                                         talloc_dirname(ctx, dir),
                                         deps[i] + strlen("ccan/"));
-               newdeps = get_one(ctx, subdir, &subinfo);
+               newdeps = get_one(ctx, subdir, get_info);
 
                /* Should be short, so brute-force out dups. */
                for (j = 0; j < talloc_array_length(newdeps)-1; j++) {
@@ -240,6 +239,9 @@ static char **uniquify_deps(char **deps)
 {
        unsigned int i, j, num;
 
+       if (!deps)
+               return NULL;
+
        num = talloc_array_length(deps) - 1;
        for (i = 0; i < num; i++) {
                for (j = i + 1; j < num; j++) {
@@ -255,22 +257,16 @@ static char **uniquify_deps(char **deps)
        return talloc_realloc(NULL, deps, char *, num + 1);
 }
 
-char **get_deps(const void *ctx, const char *dir,
-               bool recurse, char **infofile)
+char **get_deps(const void *ctx, const char *dir, bool recurse,
+               char *(*get_info)(const void *ctx, const char *dir))
 {
-       char *temp = NULL, **ret;
-       if (!infofile)
-               infofile = &temp;
+       char **ret;
 
        if (!recurse) {
-               ret = get_one_deps(ctx, dir, infofile);
+               ret = get_one_deps(ctx, dir, get_info);
        } else
-               ret = get_all_deps(ctx, dir, infofile, get_one_deps);
+               ret = get_all_deps(ctx, dir, get_info, get_one_deps);
 
-       if (infofile == &temp && temp) {
-               unlink(temp);
-               talloc_free(temp);
-       }
        return uniquify_deps(ret);
 }
 
index 100d79428a1a1de6877c298d48c11e3bab5a0245..931d05f321da8eff3316b57a3abebfd739377637 100644 (file)
@@ -458,7 +458,7 @@ static void adjust_dir(const char *dir)
 
        verbose("Adjusting %s\n", dir);
        verbose_indent();
-       for (deps = get_deps(parent, dir, false, NULL); *deps; deps++) {
+       for (deps = get_deps(parent, dir, false, compile_info); *deps; deps++) {
                char *depdir;
                struct adjusted *adj = NULL;
                struct replace *repl;
@@ -496,7 +496,7 @@ static void adjust_dependents(const char *dir)
                if (access(info, R_OK) != 0)
                        continue;
 
-               for (deps = get_deps(*file, *file, false, NULL);
+               for (deps = get_deps(*file, *file, false, compile_info);
                     *deps; deps++) {
                        if (!strstarts(*deps, "ccan/"))
                                continue;
index 8d1ae04887493206107990ec810c209e475f1cb0..e78d13904c82b1c36c263ddb57a5fdb267779139 100644 (file)
 
 #define COVERAGE_CFLAGS "-fprofile-arcs -ftest-coverage"
 
+/* This compiles up the _info file into a temporary. */
+char *compile_info(const void *ctx, const char *dir);
+
 /* This actually compiles and runs the info file to get dependencies. */
 char **get_deps(const void *ctx, const char *dir, bool recurse,
-               char **infofile);
+               char *(*get_info)(const void *ctx, const char *dir));
 
 /* This is safer: just looks for ccan/ strings in info */
 char **get_safe_ccan_deps(const void *ctx, const char *dir, bool recurse);