<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/scripts/coccinelle/misc, branch v4.4.136</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.4.136</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.4.136'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2015-10-26T21:44:51Z</updated>
<entry>
<title>coccinelle: Improve checking for missing NULL terminators</title>
<updated>2015-10-26T21:44:51Z</updated>
<author>
<name>Daniel Granat</name>
<email>d.granat@samsung.com</email>
</author>
<published>2015-10-23T15:31:06Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=67afc2110a84872aa6aae44ce6b2ea6534db1438'/>
<id>urn:sha1:67afc2110a84872aa6aae44ce6b2ea6534db1438</id>
<content type='text'>
Extend checking on tables containing structures which are initialized
without specifying member name. Added new tables for checking:
i2c_device_id and platform_device_id.

Signed-off-by: Daniel Granat &lt;d.granat@samsung.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>coccinelle: misc: remove "complex return code" warnings</title>
<updated>2015-10-26T21:10:01Z</updated>
<author>
<name>Johan Hovold</name>
<email>johan@kernel.org</email>
</author>
<published>2015-09-30T22:37:27Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=1a617a8475e8509255cc77f88c148095f4cc05ba'/>
<id>urn:sha1:1a617a8475e8509255cc77f88c148095f4cc05ba</id>
<content type='text'>
This effectively reverts 932058a5d5f9 ("coccinelle: misc: semantic patch
to delete overly complex return code processing").

There can be both symmetry and readability reasons for not wanting to do
the final function call as part of the return statement and to maintain
a clear separation of success and error paths.

Since this is in no way mandated by the coding standard, let's just
remove this semantic patch to avoid having "clean up" patches being
posted over and over in response to these Coccinelle warnings.

Signed-off-by: Johan Hovold &lt;johan@kernel.org&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>coccinelle: misc: move constants to the right</title>
<updated>2015-10-26T19:39:44Z</updated>
<author>
<name>Julia Lawall</name>
<email>Julia.Lawall@lip6.fr</email>
</author>
<published>2015-09-04T19:27:51Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=74a8478f9ea2b3e70640a64db8acd54d4225a2c4'/>
<id>urn:sha1:74a8478f9ea2b3e70640a64db8acd54d4225a2c4</id>
<content type='text'>
Move constants to the right in binary operators.

Signed-off-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.com&gt;
</content>
</entry>
<entry>
<title>coccinelle: simple_return: Add a blank line</title>
<updated>2015-06-03T08:23:33Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-06-02T01:52:20Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=f94c56f4f33dd34551af6bcc1afde5082fdf6e86'/>
<id>urn:sha1:f94c56f4f33dd34551af6bcc1afde5082fdf6e86</id>
<content type='text'>
Insert a blank line in order to improve the readability of the
generated patch and also make it consistent with the other
.cocci files.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>coccinelle: irqf_oneshot.cocci: Improve the generated commit log</title>
<updated>2015-05-25T08:23:07Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-05-24T20:45:54Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=9473a62f779d78bae646e7ef1a792d53ad4ac29e'/>
<id>urn:sha1:9473a62f779d78bae646e7ef1a792d53ad4ac29e</id>
<content type='text'>
Improve the commit log of the generated patch by mentioning the commit
log that makes threaded IRQs without a primary handler to be requested
with the IRQF_ONESHOT flag.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked- by: Valentin Rothberg &lt;valentinrothberg@gmail.com&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>scripts/coccinelle/misc/irqf_oneshot.cocci: Fix grammar</title>
<updated>2015-05-21T06:14:45Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-05-20T11:02:35Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=74de120d8096f72bdf95aba7234428c798d931cd'/>
<id>urn:sha1:74de120d8096f72bdf95aba7234428c798d931cd</id>
<content type='text'>
Correct form is 'always requested'.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>scripts/coccinelle/misc/semicolon.cocci: Use imperative mood</title>
<updated>2015-05-21T06:14:41Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-05-20T11:02:34Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=4341f6e5ce448dd79c3e663513213b936ba34c83'/>
<id>urn:sha1:4341f6e5ce448dd79c3e663513213b936ba34c83</id>
<content type='text'>
According to Documentation/SubmittingPatches:

"Describe your changes in imperative mood, e.g. "make xyzzy do frotz"
instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy
to do frotz", as if you are giving orders to the codebase to change
its behaviour.

So do as recommended.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>coccinelle: ifaddr: Fix the sentence</title>
<updated>2015-05-20T06:01:41Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-05-09T20:09:29Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=dd494ac0de48ded6a7ec0525f253116fde5c7be5'/>
<id>urn:sha1:dd494ac0de48ded6a7ec0525f253116fde5c7be5</id>
<content type='text'>
Make the sentence sensible.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>coccinelle: returnvar: Use imperative mood</title>
<updated>2015-05-20T06:01:41Z</updated>
<author>
<name>Fabio Estevam</name>
<email>fabio.estevam@freescale.com</email>
</author>
<published>2015-05-09T20:09:28Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=fe8c46b632505a880c527bc9ae246e868aa3ece5'/>
<id>urn:sha1:fe8c46b632505a880c527bc9ae246e868aa3ece5</id>
<content type='text'>
According to Documentation/SubmittingPatches:

"Describe your changes in imperative mood, e.g. "make xyzzy do frotz"
instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy
to do frotz", as if you are giving orders to the codebase to change
its behaviour."

So do as recommended.

Signed-off-by: Fabio Estevam &lt;fabio.estevam@freescale.com&gt;
Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>scripts/coccinelle/misc/bugon.cocci: update bug_on conversion warning</title>
<updated>2015-04-15T12:21:31Z</updated>
<author>
<name>Fabian Frederick</name>
<email>fabf@skynet.be</email>
</author>
<published>2015-04-09T20:58:04Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=33236415ece63d5689411f3ddb676eeb93af2698'/>
<id>urn:sha1:33236415ece63d5689411f3ddb676eeb93af2698</id>
<content type='text'>
if()/BUG conversion to BUG_ON must be avoided when there's side effect
in condition. The reason being BUG_ON won't execute the condition when
CONFIG_BUG is not defined.

Inspired-by: J. Bruce Fields &lt;bfields@fieldses.org&gt;
Suggested-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Acked-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Signed-off-by: Fabian Frederick &lt;fabf@skynet.be&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
</feed>
