Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Welcome to the Rockbox Technical Forums!

+  Rockbox Technical Forums
|-+  Rockbox General
| |-+  Rockbox General Discussion
| | |-+  small army needed - test major patch
« previous next »
  • Print
Pages: 1 [2] 3 4

Author Topic: small army needed - test major patch  (Read 21577 times)

Offline SUSaiyan

  • Member
  • *
  • Posts: 1
Re: small army needed - test major patch
« Reply #15 on: September 22, 2006, 08:15:32 AM »
Quote from: mightybrick on September 22, 2006, 05:09:22 AM
I tried testing the rockbox.ipod v19 for the nano, but when rockbox tries to boot, I get a blank screen with the message:
Quote
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!   ;D

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  :D)
Logged

Offline RaeNye

  • Developer
  • Member
  • *
  • Posts: 161
Re: small army needed - test major patch
« Reply #16 on: September 22, 2006, 09:45:34 AM »
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).
Logged

Offline RaeNye

  • Developer
  • Member
  • *
  • Posts: 161
Re: small army needed - test major patch
« Reply #17 on: September 22, 2006, 10:01:03 AM »
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.
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: small army needed - test major patch
« Reply #18 on: September 22, 2006, 10:12:39 AM »
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*.
Logged

Offline soap

  • Member
  • *
  • Posts: 1678
  • Creature of habit.
Re: small army needed - test major patch
« Reply #19 on: September 22, 2006, 05:15:36 PM »
Quote from: BigBambi on September 22, 2006, 03:50:58 AM
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.
Logged
Rockbox Forum Guidelines
The Rockbox Manual
How to Ask Questions the Smart Way

Offline Rincewind

  • Member
  • *
  • Posts: 266
Re: small army needed - test major patch
« Reply #20 on: September 22, 2006, 07:22:07 PM »
Quote from: Llorean on September 22, 2006, 10:12:39 AM
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?
« Last Edit: September 22, 2006, 07:27:54 PM by Rincewind »
Logged
Iriver H120, Sansa e280

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: small army needed - test major patch
« Reply #21 on: September 22, 2006, 09:30:20 PM »
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.
Logged

Offline Rincewind

  • Member
  • *
  • Posts: 266
Re: small army needed - test major patch
« Reply #22 on: September 23, 2006, 08:48:02 AM »
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)
Logged
Iriver H120, Sansa e280

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: small army needed - test major patch
« Reply #23 on: September 24, 2006, 05:17:21 AM »
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 :D )

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?
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline AlexP

  • Global Moderator
  • Member
  • *
  • Posts: 3688
  • ex-BigBambi
Re: small army needed - test major patch
« Reply #24 on: September 24, 2006, 06:19:23 AM »
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!
Logged
H140, F60, S120, e260, c240, Clip, Fuze v2, Connect, MP170, Meizu M3, Nano 1G, Android

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: small army needed - test major patch
« Reply #25 on: September 24, 2006, 06:33:32 AM »
:( thats terrible... ive dropped mine a few times, but fortunatly it still works.... ebay has heaps of 1.8" discs...
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: small army needed - test major patch
« Reply #26 on: September 24, 2006, 07:10:34 AM »
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
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: small army needed - test major patch
« Reply #27 on: September 24, 2006, 07:12:07 AM »
Quote from: mightybrick on September 22, 2006, 05:09:22 AM
I tried testing the rockbox.ipod v19 for the nano, but when rockbox tries to boot, I get a blank screen with the message:
Quote
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!   ;D

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 :'(
« Last Edit: September 24, 2006, 07:21:14 AM by jdgordon »
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: small army needed - test major patch
« Reply #28 on: September 24, 2006, 07:15:49 AM »
Quote from: RaeNye on September 22, 2006, 09:45:34 AM
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
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline Rincewind

  • Member
  • *
  • Posts: 266
Re: small army needed - test major patch
« Reply #29 on: September 24, 2006, 10:12:45 AM »
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.
Logged
Iriver H120, Sansa e280

  • Print
Pages: 1 [2] 3 4
« previous next »
+  Rockbox Technical Forums
|-+  Rockbox General
| |-+  Rockbox General Discussion
| | |-+  small army needed - test major patch
 

  • SMF 2.0.17 | SMF © 2019, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.129 seconds with 22 queries.