• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Rbutil
  • Assigned To
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by tomers - 2009-04-11
Last edited by tomers - 2009-10-01

FS#10124 - Fix some rbutil compilation warnings

This patch eliminates some of the warnings that are shown during compilation of the Rockbox applications.

The patch includes:
- Wrapping ‘if’ blocks with curly brackets.
- Remove ‘typedef’ from struct declaration when nothing is typedef’ed to the struct.

Closed by  tomers
2009-10-01 05:27
Reason for closing:  Wont Fix

can you please explain this patch? I don’t get any warnings for the changed code at all, adding braces for one line statements isn’t required at all and the typedef thing is in imported code which shouldn’t be changed unless necessary IMO. Besides, we have one-line-statements without braces in quite a lot of places. If you get warnings here you should get a ton of other warnings as well.

Please read attached console log file to look at the warnings I got.
My system details (compilers versions, Linux kernel versions) are here:


Available keyboard shortcuts


Task Details

Task Editing