
25 Mar
2008
25 Mar
'08
3:29 p.m.
Markus Klotzbücher wrote:
Jean-Christophe PLAGNIOL-VILLARD plagnioj@jcrosoft.com writes:
- reg16 = in_le16(hose_cfg_base + PEX_LTSSM_STAT);
- if (reg16 < 0x16) {
printf("No link\n");
hose->indirect_type = INDIRECT_TYPE_NO_PCIE_LINK;
- } else {
printf("link\n");
- }
please remove the bracket on the else
Come on, this is being ridiculously pedantic.
Absolutely agree.
In my opinion, extra braces should just acceptable and even encouraged.
jdl