On Mon, Aug 03, 2015 at 12:35:59PM -0400, Tony Camuso wrote:
> Disable all error reporting. Useful when semantic parsing checks are
> being done elsewhere or all you need is a tokenizer.
>
> Signed-off-by: Tony Camuso <tcamuso@xxxxxxxxxx>
> ---
> lib.c | 27 +++++++++++++++++++++++----
> lib.h | 1 +
> 2 files changed, 24 insertions(+), 4 deletions(-)
>
> diff --git a/lib.c b/lib.c
> index 8dc5bcf..08f6a6b 100644
> --- a/lib.c
> +++ b/lib.c
...
> @@ -130,6 +135,12 @@ void info(struct position pos, const char * fmt, ...)
> static void do_error(struct position pos, const char * fmt, va_list args)
> {
> static int errors = 0;
> +
> + if (Wall_off) {
> + max_warnings = 0;
> + return;
> + }
> +
Is this really needed?
The same check in do_warn should be enough to quiet all messages, isn't it?
> @@ -479,6 +491,13 @@ static char **handle_onoff_switch(char *arg, char **next, const struct warning w
> char *p = arg + 1;
> unsigned i;
>
> + if (!strcmp(p, "all_off")) {
> + for (i = 0; i < n; i++)
> + *warnings[i].flag = WARNING_FORCE_OFF;
> + Wall_off = 1;
> + return NULL;
> + }
> +
Can't you simply set max_warnings to 0 already here
instead of doing it in do_warn()?
> @@ -127,6 +127,7 @@ extern int Wtypesign;
> extern int Wundef;
> extern int Wuninitialized;
> extern int Wvla;
> +extern int Wall_off;
I think it should be better to use a much more explicit name,
something like "ignore_all_warnings" for example.
Also it would be nice to document this new option in the man page.
Regards,
Luc
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
