Messages in this thread Patch in this message | | | From | Sudip Mukherjee <> | Subject | [PATCH 11/11] parport: avoid assignment in if | Date | Tue, 27 Oct 2015 20:12:37 +0530 |
| |
From: Sudip Mukherjee <sudip@vectorindia.org>
It is not an usual practise to assign some value to a variable in the if test condition.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> --- drivers/parport/share.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c index b68f194..3308427 100644 --- a/drivers/parport/share.c +++ b/drivers/parport/share.c @@ -1120,7 +1120,8 @@ int parport_claim(struct pardevice *dev) /* Preempt any current device */ write_lock_irqsave(&port->cad_lock, flags); - if ((oldcad = port->cad) != NULL) { + oldcad = port->cad; + if (oldcad) { if (oldcad->preempt) { if (oldcad->preempt(oldcad->private)) goto blocked; -- 1.9.1
| |