Uttered Andrew Morton <[email protected]>, spake thus: > > if (uskb) { > > netlink_unicast(dev->nls, uskb, 0, 0); > > } > > Unneeded {} Speaking strictly as a language lawyer, they are not needed. However, for maintainability (and best practices) they are essential. They make the scope of the test explicit and ensure that a one-line- change really doesn't affect more lines that needed. Think context diffs here. Cheers!
Attachment:
pgpIr7et7NGvL.pgp
Description: PGP signature
- Follow-Ups:
- Re: connector.c
- From: Matthias Urlichs <[email protected]>
- Re: connector.c
- Prev by Date: Re: NFS client latencies
- Next by Date: sysfs for IPMI, for new mm kernels
- Previous by thread: cn_queue.c
- Next by thread: Re: connector.c
- Index(es):
 |