>> Its implementation of the check “ALLOC_WITH_MULTIPLY” considers only an other >> search pattern so far. >> >> * Do you find it worthwhile to add a prefix like “devm_” to the used >> regular expression? >> >> * Would like to improve any related scripts for the semantic patch language >> (Coccinelle software) a bit more? > I don't understand why you're asking this.
Software developers and code reviewers have got different opinions about such checks and their relevance. > I'm talking about the _output_ of checkpatch, This information is clear at first glance. > not about the script itself. But it will not provide the warning you might be looking for while you seem to find my source code analysis approach and notifications improvable. I assume that you might be interested in corresponding extensions for the involved search patterns. > But undoubtedly your patch is motivated by the output of said tool. This tool implemented some checks. > Hence you should mention that. Additional tools take also care for similar software development concerns, don't they? Can it be appropriate to omit the reference to only one Perl script for related use cases? Regards, Markus
