<div dir="ltr">Hi<div><br></div><div style="">GCC 6.0 previews give the indentation warning below</div><div style=""><br></div><div style=""><div>../../../../../rtems/c/src/libchip/network/sonic.c:837:11: warning: statement is indented as if it were guarded by... [-Wmisleading-indentation]</div><div><br></div><div><div>on this code from sonic.c. </div></div><div><br></div><div><div>      for (i = 0 ; i < 2 ; i++) {</div><div>        if ((*sc->read_register)( rp, SONIC_REG_RRP ) ==</div><div>            (*sc->read_register)( rp, SONIC_REG_RSA ))</div><div>          (*sc->write_register)(</div><div>            rp,</div><div>            SONIC_REG_RRP,</div><div>            (*sc->read_register)( rp, SONIC_REG_REA )</div><div>          );</div><div>          (*sc->write_register)(</div><div>             rp,</div><div>             SONIC_REG_RRP,</div><div>             (*sc->read_register)(rp, SONIC_REG_RRP) - sizeof(ReceiveResource_t)</div><div>          );</div><div>      }</div><div><br></div></div><div style="">From the comments, this code was written for RTEMS. Assuming it</div><div style="">works (and it did on both a DMV177 and erc32 Tharsys board), then</div><div style="">I am prone to add braces to the if and move the second write register</div><div style="">to the left. </div><div style=""><br></div><div style="">That at least makes the code match what the compiler thinks was written.</div><div style=""><br></div><div style="">Any thoughts?</div><div style=""><br></div><div style="">--joel</div></div></div>