- Status Closed
- Percent Complete
- Task Type Patches
- Category Bootloader
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#8827 - ipodpatcher errorlevels fix
At the moment, ipodpatcher doesn’t seem to have any standard errorlevels/exit values. In some cases, both success and failure return the exit value of ‘0’. Some situations even return arbitrary values of ‘4’ or ‘5’. I redid main.c for ipodpatcher such that it returned -1 if there were iPod detection/access issues, 0 if the chosen action was successful, and 1 if unsuccessful. A “switch/case” was used instead of the plethora of “if/else” statements such that the code is easier to follow and redundant code was reordered. Strings containing “ipod”, etc. were also corrected (e.g. to “iPod” and “iTunes”) so I’ve also included a small patch for ipodpatcher.c. The flow of the program and all the actions are still the same (so it will still work exactly the same as before); things have just been re-ordered a bit and cleaned up with proper errorlevels/exit values.
2011-06-05 11:03
Reason for closing: Rejected
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
no real name
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
For more colourful versions:
main.c: http://pastebin.ca/962459 ipodpatcher.c.patch: http://pastebin.ca/962513
Your main.c contains tabs and it would’ve been nicer to have it as diff for easier tracking of the relevant changes.
About your renaming of ipod → iPod etc: there is currently some mixture about the writing of Ipod around. The manual uses Ipod, the website iPod, ipodpatcher uses ipod. IMO we should be consistent with the naming, but until we have agreed on a specific way of writing renaming isn’t a good idea. Feel free to join the discussion.
There’s the diff with spaces instead of tabs.
As for the renaming of ipod→iPod, etc., if the manual uses “Ipod”, etc., I suppose it’d be better to keep it how it is right now (i.e. ignore ipodpatcher.c.patch). My opinion is that the canonic name (”iPod” and “iTunes”) should be used to avoid confusion for new users, but if it has already been set in stone like such, not worth the effort of changing everything.
Er, fixed syntax error.
The naming hasn’t been “set in stone” – it was just agreed to use Ipod etc. for the manual some while back. Recently the discussion came up if that shouldn’t be changed back to iPod. While I got used to Ipod I have no strong opinion for (or against) that capitalization, I rather have a strong concern that we should be consistent.
Feel free to join on IRC and discuss the issue.