#10 crypto/kernel/alloc.c:102:5: warning: suggest braces around

open
nobody
None
5
2012-11-30
2012-11-30
Daniel Holbert
No

If ENABLE_DEBUGGING isn't turned on, then this chunk of code ends up with an empty un-braced "else" statement:
http://srtp.cvs.sourceforge.net/viewvc/srtp/srtp/crypto/kernel/alloc.c?view=annotate#l101

and it triggers this build warning:
crypto/kernel/alloc.c:102:5: warning: suggest braces around empty body in an ‘else’ statement [-Wempty-body]

Filing this bug on adding braces around the "else" condition, to fix that build warning.

Discussion

  • Daniel Holbert
    Daniel Holbert
    2012-11-30

    Ah, I just misunderstood the UI. "Add a file" didn't look like a button, so I didn't click on it until now -- but it turns out it is a button. :)

    So -- I'm attaching the patch on this tracker-page here, as well, since that seems like what I should've originally done. Sorry for being a sourceforge-issue-tracker n00b.

     
  • Daniel Holbert
    Daniel Holbert
    2012-12-01

    fix: add braces around "else" condition