Messages in this thread | | | Subject | Re: iio: Rename a jump label in iio_buffer_store_watermark() | From | SF Markus Elfring <> | Date | Sun, 25 Sep 2016 17:17:25 +0200 |
| |
> It's not an inappropriate identifier as it stands. The point is > that it could be better.
Thanks for your interest in clarifying further improvement possibilities.
>>> Which tool is spitting it out? >> >> Are you looking for any special tool? > I was wondering how you identified these particular > issues as I wanted to know the logic behind the test.
Do you get any related ideas from information in a message like "Source code review around jump label usage" (from 2015-12-11)?
https://lkml.org/lkml/2015/12/11/378 https://lkml.kernel.org/r/<566ABCD9.1060404@users.sourceforge.net>
>> Which test do you mean? > Whatever you used to find these jump labels
There is a general possibility that dedicated scripts for the semantic patch language can also adjust jump labels as I suggested it for five functions in this software.
> (the patch series description suggested it was a static checker).
The corresponding five patches are just a result of a source code review by the means of a current text editor wit extra support for programming.
How do you think about to improve the capabilities of tools for advanced static source code analysis any further?
Regards, Markus
| |