]> git.ozlabs.org Git - ccan/blobdiff - tools/ccanlint/ccanlint.h
tools: split out manifest from ccanlint
[ccan] / tools / ccanlint / ccanlint.h
index ee12ce04fb81083c228e2ac47604c27ed39e25b8..6fcca5fa8eb26fbf40153e244a27c4d9e66e0922 100644 (file)
@@ -1,61 +1,83 @@
 #ifndef CCAN_LINT_H
 #define CCAN_LINT_H
+#include "config.h"
 #include <ccan/list/list.h>
+#include <ccan/dgraph/dgraph.h>
+#include <ccan/autodata/autodata.h>
 #include <stdbool.h>
 #include "../doc_extract.h"
+#include "../manifest.h"
+#include "licenses.h"
 
-#define REGISTER_TEST(name, ...) extern struct ccanlint name
-#include "generated-init-tests"
-#undef REGISTER_TEST
+AUTODATA_TYPE(ccanlint_tests, struct ccanlint);
+#define REGISTER_TEST(test) AUTODATA(ccanlint_tests, &test)
 
-#define REGISTER_TEST(name, ...) 
+/* 0 == Describe failed tests.
+   1 == Describe results for partial failures.
+   2 == One line per test, plus details of failures.
 
-struct manifest {
-       char *basename;
-       struct ccan_file *info_file;
+   Mainly for debugging ccanlint:
+   3 == Describe every object built.
+   4 == Describe every action. */
+extern int verbose;
 
-       struct list_head c_files;
-       struct list_head h_files;
-
-       struct list_head run_tests;
-       struct list_head api_tests;
-       struct list_head compile_ok_tests;
-       struct list_head compile_fail_tests;
-       struct list_head other_test_files;
-
-       struct list_head other_files;
+/* Error in a particular file: stored off score->per_file_errors. */
+struct file_error {
+       struct list_node list;
+       struct ccan_file *file;
+       unsigned int line;
 };
 
-struct manifest *get_manifest(void);
+/* The score for an individual test. */
+struct score {
+       /* Starts as false: if not set to true, ccanlint exits non-zero.
+        * Thus it is usually set for compilation or other serious failures. */
+       bool pass;
+       /* Starts at 0 and 1 respectively. */
+       unsigned int score, total;
+       /* The error message to print. */
+       char *error;
+       /* Per file errors, set by score_file_error() */
+       struct list_head per_file_errors;
+};
 
 struct ccanlint {
-       struct list_node list;
+       /* More concise unique name of test. */
+       const char *key;
 
        /* Unique name of test */
        const char *name;
 
-       /* Total score that this test is worth.  0 means compulsory tests. */
-       unsigned int total_score;
+       /* Can we run this test?  Return string explaining why, if not. */
+       const char *(*can_run)(struct manifest *m);
 
-       /* If this returns non-NULL, it means the check failed. */
-       void *(*check)(struct manifest *m);
+       /* Should we stop immediately if test fails? */
+       bool compulsory;
 
-       /* The non-NULL return from check is passed to one of these: */
+       /* If timeleft is set to 0, means it timed out.
+        * score is the result, and a talloc context freed after all our
+        * depends are done. */
+       void (*check)(struct manifest *m,
+                     unsigned int *timeleft, struct score *score);
 
-       /* So, what did this get out of the total_score?  (NULL means 0). */
-       unsigned int (*score)(struct manifest *m, void *check_result);
+       /* Can we do something about it? (NULL if not) */
+       void (*handle)(struct manifest *m, struct score *score);
 
-       /* Verbose description of what was wrong. */
-       const char *(*describe)(struct manifest *m, void *check_result);
+       /* Options from _info. */
+       char **options;
+       /* If not set, we'll give an error if they try to set options. */
+       bool takes_options;
 
-       /* Can we do something about it? (NULL if not) */
-       void (*handle)(struct manifest *m, void *check_result);
+       /* Space-separated list of dependency keys. */
+       const char *needs;
 
        /* Internal use fields: */
-       /* Who depends on us? */
-       struct list_head dependencies;
-       /* How many things do we (still) depend on? */
-       unsigned int num_depends;
+       /* We are a node in a dependency graph. */
+       struct dgraph_node node;
+       /* Did we skip a dependency?  If so, must skip this, too. */
+       const char *skip;
+       /* Have we already run this? */
+       bool done;
 };
 
 /* Ask the user a yes/no question: the answer is NO if there's an error. */
@@ -95,30 +117,12 @@ struct line_info {
        struct pp_conditions *cond;
 };
 
-struct ccan_file {
-       struct list_node list;
-
-       char *name;
-
-       /* Pristine version of the original file.
-        * Use get_ccan_file_lines to fill this. */
-       const char *contents;
-       size_t contents_size;
-
-       /* Use get_ccan_file_lines / get_ccan_line_info to fill these. */
-       unsigned int num_lines;
-       char **lines;
-       struct line_info *line_info;
-
-       struct list_head *doc_sections;
-};
-
-/* Use this rather than accessing f->lines directly: loads on demand. */
-char **get_ccan_file_lines(struct ccan_file *f);
-
 /* Use this rather than accessing f->lines directly: loads on demand. */
 struct line_info *get_ccan_line_info(struct ccan_file *f);
 
+/* Use this rather than accessing f->simplified directly: loads on demand. */
+const char *get_ccan_simplified(struct ccan_file *f);
+
 enum line_compiled {
        NOT_COMPILED,
        COMPILED,
@@ -139,12 +143,26 @@ char *get_symbol_token(void *ctx, const char **line);
 /* Similarly for ->doc_sections */
 struct list_head *get_ccan_file_docs(struct ccan_file *f);
 
+/* Get NULL-terminated array options for this file for this test */
+char **per_file_options(const struct ccanlint *test, struct ccan_file *f);
+
+/* Append message about this file (and line, if non-zero) to the score->error */
+void score_file_error(struct score *, struct ccan_file *f, unsigned line,
+                     const char *errorfmt, ...);
+
+/* Start a command in the background. */
+void run_command_async(const void *ctx, unsigned int time_ms,
+                      const char *fmt, ...);
+
+/* Async version of compile_and_link. */
+void compile_and_link_async(const void *ctx, unsigned int time_ms,
+                           const char *cfile, const char *ccandir,
+                           const char *objs, const char *compiler,
+                           const char *cflags,
+                           const char *libs, const char *outfile);
 
-/* Call the reporting on every line in the file.  sofar contains
- * previous results. */
-char *report_on_lines(struct list_head *files,
-                     char *(*report)(const char *),
-                     char *sofar);
+/* Get results of a command, returning ctx (and free it). */
+void *collect_command(bool *ok, char **output);
 
 /* Normal tests. */
 extern struct ccanlint trailing_whitespace;
@@ -155,4 +173,22 @@ struct dependent {
        struct ccanlint *dependent;
 };
 
+/* Is this test excluded (cmdline or _info). */
+bool is_excluded(const char *name);
+
+/* Called to add options from _info, once it's located. */
+void add_info_options(struct ccan_file *info);
+
+/* Are we happy to compile stuff, or just non-intrusive tests? */
+extern bool safe_mode;
+
+/* Did the user want to keep all the results? */
+extern bool keep_results;
+
+/* Compiler and CFLAGS, from config.h if available. */
+extern const char *compiler, *cflags;
+
+/* Contents of config.h (or NULL if not found) */
+extern const char *config_header;
+
 #endif /* CCAN_LINT_H */