- Status Closed
- Percent Complete
- Task Type Patches
- Category Operating System/Drivers
- Assigned To No-one
- Operating System Another
- Severity Low
- Priority Very Low
- Reported Version Version 3.2
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10128 - Sansa AMS debug cosmetic fix
This is a cosmetic fix for the Sansa AMS ports debug screen. DBOP_DIN displays 8 hex characters until the record button is pressed, then it displays only 4 hex characters (first 4 charactersare "0"). Changed the sprintf parameter from %4x to %8x so that the leading zeros are displayed.
Closed by kugel.
2009-04-20 20:11
Reason for closing: Fixed
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
2009-04-20 20:11
Reason for closing: Fixed
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
Fixed in r20757.
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
It should only display 4 character, since dbop is a short :/
Since dbop is a short, then why would it displaying 8 characters? Could there be someplace it is expanded?
According to amiconn it's expanded due to the sign bit. Currently dbop_din is a signed short. Since the sign bit changes due to button presses, the sign bit gets added again before the actual short data. The attached patch just changes it to unsigned. Please test and report back.
The patch works on the e200v2, although I had to change button-target.h to get it to build. Attached is the updated patch.