small army needed - test major patch

[SUSaiyan]

I tried testing the rockbox.ipod v19 for the nano, but when rockbox tries to boot, I get a blank screen with the message:

Data abort
00014310

and it freezes.  I have to reboot, and manually go into diskmode and put the CVS rockbox.ipod back into the root to boot completely.
I hope that all made sense!  

i have that exact same problem, though the last line actually says "at 00014310" (woo, nitpicking). if its any help, ive got a 4Gb nano, and its black (not that that matters  )

[RaeNye]

Testing on X5, here're some glitches:

General Settings->Display->Browse .(r)wps files does nothing (browse fonts/plugins/themes do work)

General Settings->Display->Remote-LCD Settings->Not Done yet!
(instead of Caption Backlight)

General Settings->Display->Remote-LCD Settings->LCD Mode now displays on/off instead of normal/inverse (or whatever it was before...)

Recording Settings has both the submenu File Split Options and its four children (Measure, What to do, Time, Filesize).

[RaeNye]

And the most important thing -
I really like the check mark besides the active setting, but you can't cancel a setting anymore... settings are saved when pressing back even if you didn't click to set the check mark on the current value.

Another thing - some settings happen immediately (e.g., volume) even without applying the check mark, but others (e.g. LCD contrast) aren't applied until you exit the menu item.

[Llorean]

I think he's going to change it so moving the cursor doesn't select a setting, backing out doesn't do anything but back out, and pressing select moves the checkmark (after which if you back out, it's saved). I *think*.

[soap]

Oh, and on the wiki page the attached rockbox.iriver says it is for H100, but it's actually for H120.  I tried to edit it, but couldn't see how to change the comment without uploading a new file.

I changed the wiki entry to reflect H120.

[Rincewind]

I think he's going to change it so moving the cursor doesn't select a setting, backing out doesn't do anything but back out, and pressing select moves the checkmark (after which if you back out, it's saved). I *think*.

I hope not. Because I really like it that it is possible to change a setting without pressing select. My H120 has a quite loud click when I press select, so I try not to push it if I don't need to.
Also, If I interpret your post correctly, this would mean that it takes one more button press to change a setting. I don't think that's a good idea. Why should the current behavior be changed at all?

[Llorean]

It could be possible that a right-press moves the checkmark instead of "Select" I imagine. I think the problem was that with the screen like it is now it acts in "unexpected" ways when you try to cancel out of the screen and the selection changes, or if it doesn't show some users the moved checkmark then they don't have visual confirmation of a settings change.

I dunno. I think "selection" should be a separate function from "navigation" but that's just me. Like the common concept of setting a radio button from a series of options and then clicking "Okay" after. I'm certainly not oppossed to a one-button solution, but the current highlighted entry shouldn't be considered the selection in my opinion.

[Rincewind]

the current highlighted setting is only the selection for settings that change instantly and that's good, because otherwise you have click navi every time you want to see how something like contrast changes.

for every other setting, the current highlighted only applies when leave the screen (not canceling).

Maybe a nice feature would be an indicator what the previous setting was. This indicator changes with the cursor on dynamic settings and stays at the value that you would get by canceling with the other settings.

(sorry for the grammer, I'm in a hurry)

[JdGordon]

thanks all for testing.
RE: the bugs on the first page, I'll hopefully fix them tonight ro tomorow.
RE: the setting seelction screen. I'm quite open to ideas for the setting selection screen. however, I want to try and please the majority (actually, I lie, I want to please the core dev guys...). iirc "they" don't like the current way where left backs out and cancels, So i figured this would be a fairly reasonable compromise.
now, I have no problems adding settings to deal with the settings screen, e.g I would gladly add an option for 1 button selection/cancelation in the screen, but the core devs are fairly against that (so i might just sneak ti in and not tell )

OK, now the important bits...
Each setting can have a callback function associated which "applies" whatever value is curently selected, most dont need this (and some it is plain wrong to do this), so aside from the contrtast and "sound items" (e.g volume) which others need this?

[AlexP]

Good work!

RE: which items need immediate update, I can't think of anything other than contrast and sound, but unfortunately I managed to drop my H140 yesterday when accessing the disk, and now am having disk problems!    More unfortunately still, Toshiba seem to have stopped selling the 1.8" ATA disks (everywhere I found in the UK says discontinued) - I guess in preparation for release of the perpendicular disks.

So no more testing for me for a while!

[JdGordon]

thats terrible... ive dropped mine a few times, but fortunatly it still works.... ebay has heaps of 1.8" discs...

[JdGordon]

Replying to BibBambi, message # 13...

contrast: odd... the default must be set to high elsewhere in the code, because i use DEFAULT_CONTRAST_SETTING which is set in the target config... will investigate further later.

menu not redrwaing status bar: yes, ive known about this for a while, but keep forgetting to fix, done now.

recording setting qaulities missing: fixed.
backlight fade in: fixed
invert display strings: fixed
In remote LCD settings, the forth item is Not Done Yet!: fixed
remote invert: fixed
scrolling settings: hmm... ill look into this later, remind me if i forget please :p
dircache reminder... yes, ill put that back soon... its being stupid and not going in easily!
current playlist: fix later

[JdGordon]

I tried testing the rockbox.ipod v19 for the nano, but when rockbox tries to boot, I get a blank screen with the message:

Data abort
00014310

and it freezes.  I have to reboot, and manually go into diskmode and put the CVS rockbox.ipod back into the root to boot completely.
I hope that all made sense!  

righto, damn this is going to be a pain to debug.. ill grab my sisters nano and figure it out... that magic number does help tho, it narrows it down to about 100 lines of code :p

edit: damn... whatever it is isnt crashing the sim.. and the function is a loop, so this is gonna be a huge PITA to figure out

[JdGordon]

Testing on X5, here're some glitches:

General Settings->Display->Browse .(r)wps files does nothing (browse fonts/plugins/themes do work)

are u sure you have rwps's ?
 the rest is fixed..

ill post a patch will all the fixes later tonight

[Rincewind]

the scrolling options could need a callback, so that you can see directly what changes when you change delays and speed. This requires a sample scrolling line in the selection screen, of course.

On the selection thing:
If it gets decided to make settings selection a two button press thing (with SELCECT to select an action but don't applying it, and LEFT or RIGHT to leave the screen afterwards), then I hope you make it easy to change this back (maybe an #ifdef or a special action that you can define in your keymap instead of the default one?) because I think that this behaviour is one thing that rockbox does different from other firmwares and that makes navigation very fast.

I liked the original button handling - before the new settings display - better (with LEFT to apply a setting and go back; and STOP to cancel). But the current one is ok, too (with RIGHT to apply setting; and LEFT and STOP to cancel). Actually, I never use the cancel button, whereever it is, because it gives a 1 sec delay showing the "cancelled" splash. I just apply the setting where the cursor was when I entered the screen (Yes, I can remember if it was "on" or "off" )

It would be a sad thing if Rockbox gets dumbed down to make it "usable" for people that expect everything to work just in the same way than their crappy retail firmware.

PS: If change something, don't forget the iriver remotes, They have a different button layout without a 4-way selection pad.