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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> +@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




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux