Systemd/coccinelle/no-if-assignments.cocci at 1817be807ff5568ef03d6dd60f315d9c75aad5e6 - picnoir/Systemd - Alternativebit

picnoir/Systemd

Lennart Poettering 3cc2aff1ab tree-wide: don't do assignments within if checks

Turn this:

       if ((r = foo()) < 0) { ...

into this:

       r = foo();
       if (r < 0) { ...

2015-09-09 08:20:20 +02:00

21 lines

200 B

Plaintext

Raw Blame History

 @@
 expression p, q;
 identifier r;
 statement s;
 @@
 - if ((r = q) < p)
 - s
 + r = q;
 + if (r < p)
 + s
 @@
 expression p, q;
 identifier r;
 statement s;
 @@
 - if ((r = q) >= p)
 - s
 + r = q;
 + if (r >= p)
 + s