Re: [PATCH v3] coccinelle: semantic patch to check for inappropriate do_div() calls

[Markus Elfring]

> +@initialize:python@
â
> +def filter_out_safe_constants(str):
â
> +def construct_warnings(str, suggested_fun):

* I suggest once more to adjust the dependency specifications for the usage
  of these functions by SmPL rules.

* Can the local variable âmsgâ be omitted?


> +coccilib.org.print_todo(p[0], construct_warnings("div64_ul"))

I suggest again to move the prefix âdiv64_â into the string literal
of the function implementation.


The SmPL code for two disjunctions could become shorter.

Regards,
Markus