Previous day | Jump to hour: 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 | Next day

Seconds: Show Hide | Joins: Show Hide | View raw
Font: Serif Sans-Serif Monospace | Size: Small Medium Large

Click in the nick column to highlight everything a person has said.
The Logo icon identifies that the person is a core developer (has commit access).

#rockbox log for 2014-01-13

00:00:01 Quit rela (Read error: Connection reset by peer)
00:00:40 Join [Saint] [0] (~saint@rockbox/staff/saint)
00:03:19JdGordonPurlingNayuki: hi! so close to being done :)
00:03:24JdGordonI've got 2 comments thouhg
00:05:01 Join saratoga_ [0] (123e1c65@gateway/web/freenode/ip.
00:05:03saratoga_needs a manual entry for the limiter too i think
00:05:58PurlingNayukiJdGordon: Good news;-)one or two more Patchset to merge I think.
00:06:29JdGordonjust updated with comments
00:07:15lebelliuma brand new feature ready for v3.14, that's great :)
00:09:15 Quit Bagder (Ping timeout: 245 seconds)
00:11:36 Join Bagder [241] (~daniel@rockbox/developer/bagder)
00:12:29PurlingNayukiBut sorry haven't really ever used manual. Will take a look on it.
00:13:26 Quit Zarggg (Quit: Zarggg)
00:14:15JdGordonPurlingNayuki: if you're very lucky you might be able to get one of the manual guys/gals to write it up for you :)
00:14:27JdGordonbluebrother^: or pixelma :)
00:14:33saratoga_put a sentence or two for each option in there
00:14:38saratoga_should be self explanatory
00:15:57kugelIIRC AlexP volunteered for writing manual entries in the past
00:20:13PurlingNayukisaratoga_: Thanks for "advice":-P
00:22:02PurlingNayukiBTW any plan to build the manual in more languages?
00:22:42PurlingNayukiJdGordon: What do you think about the splashf?
00:22:51JdGordonkugel: find_setting() in the quickscreen one isnt wrong, it is seting setting to the defaultvalue
00:23:07JdGordonPurlingNayuki: I was never 100% for it
00:23:18saratoga_i don't think having more languages would be feasible unfortunately given the huge amount of text involved
00:23:45kugelJdGordon: the one in qs_set_default()
00:23:51kugelthat call has no effect at all
00:24:12kugeloh wait, I misread find_setting()
00:24:50kugelno, wait
00:25:37*kugel is confuzzled
00:25:51JdGordonwhich file is find_setting() in?
00:25:56JdGordonI'm using the web interface
00:26:08PurlingNayukiJdGordon: Then kill the splashf
00:26:16JdGordonPurlingNayuki: fine by me
00:26:22PurlingNayukiI don't actually like that
00:26:55JdGordonkugel: it iterates the settings list and puts the setting index into the quickscreens int*
00:27:25kugelright, what effect does that have?
00:27:42JdGordonthe quickscreen uses the index to put the item ont he screen
00:27:54PurlingNayukiBut I also came up with an idea that we count how many times users try to increase the volume over the limit
00:27:56JdGordoninstead of storing a pointer and needing to do lookups then
00:28:11JdGordonPurlingNayuki: then keep the splash and tweak it alter :)
00:29:04PurlingNayukiNo use explaining but just see my next patch set:-P
00:29:17kugelJdGordon: ok. fwiw, shouldn't there be an else? it'll still call find_setting with defaultval == NULL
00:29:27kugelPurlingNayuki: so you like the splash or not?
00:29:33kugelI would like to see it removed
00:30:41kugelit's useless because a) the user did explicitly enable the volume limiter so he shouldnt be surprised and b) there are plenty other possible indicators for volume changes (or non-changes)
00:30:44 Quit Bagder (Ping timeout: 248 seconds)
00:31:28kugelblocking splashes are generally annoying and should be avoided
00:31:53saratoga_i kind of like it, but maybe just show it the first time they try to go over max?
00:32:49saratoga_i could see how it would get annoying after a while otherwise
00:33:07kugelsplashes should indicate something exceptional, but that the volume is limited is to be expected if the volume limiter is enabled
00:33:39PurlingNayukikugel: IMO users do forget it sometimes so I need a "smarter" indicator.
00:34:02kugelhow do you know?
00:34:38 Join Bagder [241] (~daniel@rockbox/developer/bagder)
00:34:47PurlingNayukiDiscussing groups and forum
00:34:55JdGordonkugel: I guess so, if you want to add an else go for it
00:35:06JdGordonusers ARE idiots :)
00:36:22kugelno, not all
00:36:56PurlingNayukiJust annoyed with users calling out for help while I have done almost everything for them.
00:38:38JdGordonthe reason to not have the splash is so users can be in the WPS and spin the volume wheel to full volume and not have the splash get in the way of the wps
00:39:10kugelon some targets you can change the volume in lists also
00:39:31PurlingNayukiWe have prebuilts, fonts and even pre packed themes but they just simply "Why my Rockbox can't display correctly". Terrible.
00:39:39kugelanyway, I don't see why to put an annoying splash in a situation that is to be expected
00:40:59PurlingNayukiWe count how many times users try to increase volume over the limit. If larger than 20 times we splashf. Is it OK?
00:41:55PurlingNayukiAnd 10 times for non-scrollwheel targets.
00:41:56 Quit pamaury (Ping timeout: 248 seconds)
00:42:57kugelthis is getting complex. 20 times per second or per minute?
00:43:02JdGordonif you're going to do that you should limit the spalsh to only showing once every 30s or so
00:43:31*kugel maintains his opinion that it shouldn't be shown at all
00:43:35PurlingNayukiper 3 secs should be fine.
00:44:26PurlingNayukiThat's really complicated and a bit redundant.
00:45:11JdGordonPurlingNayuki: how about adding a splash later on if it proves to be a problem?
00:45:15kugelhow about this one: remove the splash for now and add it if we really get reports of confused users?
00:45:30kugelahh, JdGordon was faster :)
00:45:31*PurlingNayuki is fine with removing it but [Saint] seems to be filled with love in a splashf.
00:45:58PurlingNayukikugel: Agree with you;-)
00:46:16PurlingNayukiJdGordon: And also with you;-)
00:47:27kugelPurlingNayuki, JdGordon: the complex custom menu part of that patch is because the volume isn't a compile time constant?
00:47:47kugelor did I miss something else?
00:48:35PurlingNayukiJust because the maximum and minimum volume varies from targets to targets
00:49:16PurlingNayukiBut we want the limiter to be done in apps/
00:49:44 Quit ender` (Quit: PHP is a minor evil perpetrated and created by incompetent amateurs, whereas Perl is a great and insidious evil, perpetrated by skilled but perverted professionals.)
00:51:10fs-bluebotBuild Server message: New build round started. Revision 25b7cff, 249 builds, 32 clients.
00:53:27***Saving seen data "./dancer.seen"
00:54:18fs-bluebotBuild Server message: Build round completed after 189 seconds.
01:03:07fs-bluebotBuild Server message: New build round started. Revision 31dfabf, 249 builds, 32 clients.
01:04:52saratoga_we trigger a build on a manual change? does it actually build the manual?
01:05:00 Join sakax [0] (
01:05:00 Quit sakax (Changing host)
01:05:00 Join sakax [0] (~sakax@unaffiliated/sakax)
01:06:06fs-bluebotBuild Server message: Build round completed after 179 seconds.
01:06:57 Quit lebellium (Ping timeout: 246 seconds)
01:07:00 Quit kugel (Ping timeout: 248 seconds)
01:07:25 Join lebellium_ [0] (
01:07:32 Nick lebellium_ is now known as lebellium (
01:16:07 Quit ter2 (Read error: Connection reset by peer)
01:17:39 Quit sakax (Remote host closed the connection)
01:52:06 Quit dfkt (Quit: -= SysReset 2.55=- Sic gorgiamus allos subjectatos nunc.)
02:11:33[Saint]I only wanted a splash IFF the volume displayed to the user was going to be directly misleading.
02:12:28[Saint]But I believed JdGordon mitigated this by always ensuring the volume displayed to the user for selection is within the bounds of the volume limit?
02:13:07[Saint]The problem I had was that it allowed the user to set a value for the volume that would be ignored with zero confirmation.
02:13:21[Saint]If that no longer exists, no splash needed.
02:14:02[Saint]If it is still possible for the user to set a value that will be immediately overridden, I would rather we inform.
02:16:25chrisjjpamaury: re ZEN BSoD, thanks for the good news. Note this issue is slightly different on ZEN v. ZEN X-Fi. Only the latter BSoDs between CREATIVE logo and bootloader text, IME.
02:17:37 Quit yosafbridge (Ping timeout: 264 seconds)
02:18:40[Saint]I think the problem isn't the splash but the fact it needs to exist at all.
02:18:44chrisjj"I have identified the ZEN black-screen-of-death culprit" I'm guessing not power management :)
02:18:53 Join yosafbridge [0] (
02:19:13[Saint]If the user can't ever select a value that is beyond the volume limiter, no need to splash at all.
02:19:31[Saint]PurlingNayuki: JdGordon: ^
02:20:35[Saint]I was under the impression JdGordon made it so the max volume the user can set is the capped limit, not the real limit.
02:20:56[Saint]As long as that's the case, no need to splash.
02:21:25[Saint]I only wanted to splash when it was possible to set a bullshit value that would be silently ignored.
02:25:33[Saint]To recap: As long as the user is never able to select an "out of bounds" volume (higher than the capped limit), and a volume higher than the capped rate is never shown to the user except for the express purpose of setting the volume limiter, there's no need to splash. That would be intended functionality.
02:30:54 Join advx [0] (~quassel@
02:31:16 Nick advx is now known as advx_ (~quassel@
02:34:30[Saint]Let's brainstorm a bit, please.
02:35:04[Saint]A perfect touchscreen theme for Rockbox would include...?:
02:35:40 Join pedro_angelo [0] (
02:36:41[Saint](no wrong answers here - what are some UIs you like, why? - what features dies a "good" touch UI include? what information is a priority to you?)
02:39:52lebelliuma perfect touchscreen theme needs the theme engine to work properly for SBS and FMS ;)
02:41:59[Saint]And barring limitations I can't overcome...?
02:42:57[Saint]It is non-trivial to add full playback support to all screens via the .sbs
02:43:31[Saint]it is much more complicated than "it just doesn't work". So lets concentrate on things I can change.
02:45:21 Quit advx_ (Remote host closed the connection)
02:46:34lebelliumhere is how mine looks like
02:46:43lebelliumso it shows what information I need
02:47:43JdGordontouch in th SBS is hard :p
02:47:48JdGordonthough, whats not working about it?
02:48:54[Saint]the wps_* tags are nuts.
02:49:10lebelliumJdGordon: then start with FMS :p
02:49:23[Saint]they control the UI instead of skipping/seeking
02:49:47[Saint]that's the major issue.
02:50:26lebelliumand the sbs refresh rate is much slower
02:50:58[Saint]I don't think JdGordon ever pushed dynamic refresh.
02:51:04[Saint]...I wonder why.
02:51:45[Saint]At one point he had it so screens only refreshed as often as the skin engine requested it.
02:52:02JdGordonwasnt there problems with it?
02:52:20[Saint]I can't remember.
02:52:30fs-bluebotGerrit review #191 at : skin_engine: Use a smart consistant skin refresh rate for all screens by Jonathan Gordon (changes/91/191/2)
02:53:30***Saving seen data "./dancer.seen"
02:54:35JdGordon[Saint]: can you put up a gdoc braindump of skin problems?
02:54:55JdGordonyou're almost guarenteed to not be online when im motivated enough to fix anything :(
02:55:03[Saint]As for as touch goes. the big outstanding issue which you've said multiple times is incredibly difficult to overcome, is the wps_next/prev tags scrolling through the lists instead of skipping/seeking.
02:55:15[Saint]Yeah, I can do that.
02:55:31[Saint]Kugel took care of a few biggies earlier.
02:55:57[Saint]I need to check to make sure I can discard some ugly workaround code later tonight.
02:56:12JdGordontouch buttons should be fixable
02:56:18JdGordoni just need time and a list
02:56:29JdGordonand an example sbs is always apreciated :)
02:56:50[Saint]Right. I'll see what I can do this evening.
02:58:14JdGordonput up a list now so you dont forget :)
02:58:38*JdGordon used to have a mind map tingy on his phone for the different patches/bugs he was working on
02:58:45lebellium[Saint] JdGordon I have an example here There is SBS + WPS + FMS so it's a good basis to fix bugs
02:58:56[Saint]lebellium: that SS looks very feature barron.
02:59:15[Saint]It looks like it would force the user back into the lists for too often.
02:59:29[Saint]*far too
03:00:22[Saint]Where is quickscren? context menu? playback control?
03:00:29lebelliumtry my link above ;)
03:00:37lebelliumnow time to go to bed
03:01:27lebelliumthere are some shortcuts depending on what area you touch (id3tags, album art etc) but indeed there are fewer shortcuts than with cabbiev2
03:01:32[Saint]the theme looks nice.
03:01:40[Saint]but it seems quite unusable.
03:01:59[Saint]no obvious control elements except for basic playback.
03:02:26[Saint]but it /looks/ nice.
03:02:27lebelliumonce you know which area you have to type, short or long press, I don't see the need of an explicit icon :)
03:02:48[Saint]thats rather broken, IMO.
03:03:13[Saint]touch elements shouldn't ever be invisible. the function should he obvious.
03:03:20[Saint]but that is difficult.
03:03:34[Saint]anyway, sleep well. o/
03:04:52[Saint]The difficult part is making sure users know what is a button, and what function it performs, and making it intuitive.
03:04:55lebelliumThat's a compromise, sure. Once the theme engine is fixed for SBS and FMS, I'll publish it on the theme website and I'll have to explain carefully in the description what area to touch for every shortcut. That's not the best way but wel...
03:05:39[Saint]I have been reading a lot about UI development, and pretty much all my past work is awful. Despite my best intentions.
03:06:40[Saint]I need to make my next candidate simple, without distracting colors, and obvious.
03:06:48[Saint]Simple and obvious.
03:07:04lebelliumAsk Apple :D
03:07:05[Saint]And flat.
03:07:43[Saint]No silly gradients everywhere, no making everything look 3D and rounded.
03:07:58[Saint]Its no longer 2000 :)
03:08:07 Join M808 [0] (ba6947ee@gateway/web/freenode/ip.
03:08:17lebelliumhehe. Argh 3AM, I really need to sleep. And please don't forget my cabbiv2 progress bar fix :(
03:08:28[Saint]Sleep well.
03:08:46 Quit M808 (Client Quit)
03:09:02 Quit lebellium (Quit: ChatZilla [Firefox 27.0/20140109165205])
03:12:42PurlingNayuki[Saint]: That's true so kill the splashf
03:15:09PurlingNayukiWhere do Rockbox utility read lang strings when building voice?
03:31:00[Saint]langs folder.
03:31:30[Saint]I'm on mobile, cant remeber the exact path. whoops. not very useful.
03:34:06[Saint] /rockbox/apps/lang
03:52:50PurlingNayuki[Saint]: On Windows?
03:53:26[Saint]Oh, sorry. I thought you meant in the source.
03:53:36PurlingNayukiAnd without source
03:54:00PurlingNayukiWill it download from
04:12:42 Quit Raptors (Read error: Operation timed out)
04:13:39saratogai think
04:15:21 Join Raptors [0] (
04:16:46 Join mt [0] (
04:53:32***Saving seen data "./dancer.seen"
04:56:59 Join pixelma_ [0] (pixelma@rockbox/staff/pixelma)
04:57:07 Quit pixelma (Disconnected by services)
04:57:13 Quit amiconn (Disconnected by services)
04:57:13 Join amiconn_ [0] (quassel@rockbox/developer/amiconn)
04:59:42PurlingNayukisaratoga: It's download, not build
05:13:00JdGordonPurlingNayuki: if you take the splash out and fix up the last comments i'll push it
05:16:25JdGordonit really would be nice if there was a compile time MAX_VOLUME define
05:23:15 Join VailAZdoug [0] (~doug@2607:fb90:605:15a8:607b:9848:c02d:a295)
05:26:56VailAZdougI seem to have semi-bricked a sansa zip. The symptoms are: 1.>it will boot into the original sansa software by hitting the volume and power button at the same time. 2.> When just hitting the power button by itself, Rockbox shows the startup screen and does not go any further. I've tried to uninstall rockbox but there still is a boot stub. What do I remove. (will try to re-install rockbox after this mess is fixed) Thanks in advanc
05:34:34 Quit VailAZdoug (Ping timeout: 245 seconds)
05:37:16 Quit saratoga (Ping timeout: 272 seconds)
05:41:35PurlingNayukiJdGordon: Wait till tonight. Sorry but I have classes.
05:45:54PurlingNayukiThis year Illusion be extremely busy with college applyling
05:47:22 Join VailAZdoug [0] (~doug@2607:fb90:604:c431:607b:9848:c02d:a295)
05:51:13PurlingNayukiJdGordon: IMO it shouldn't be diffcult to define one without rewriting tens of head files
05:51:55PurlingNayukiBut is it really of necessity?
05:55:52JdGordonit means we dont need the custom_option() mess
05:56:01JdGordonwhich in turn means it can go in the quickscreen
05:56:11JdGordonthough that isnt necessarily a good thing! :)
05:57:23PurlingNayukiTry this tonight.
05:57:55PurlingNayukiHow to generate needed-by and depends on un gerrit?
05:57:56 Quit [7] (Disconnected by services)
05:58:09 Join TheSeven [0] (~quassel@rockbox/developer/TheSeven)
05:58:58JdGordoni think it does it automatically if you push multiple commits instead of just a single one
06:00:42PurlingNayukiNo wonder that I can't find a way to manually add this.
06:07:13PurlingNayukiWe can use INT_SETTING to setup this volume limiter...yeah it will be nice to have a compile time MAXVOLUME
06:08:08JdGordonit looks like there are only a few files which would need to be touched to add a #define
06:08:11PurlingNayukiAnd it's actually easy to add that.
06:28:02 Join mortalis [0] (~kvirc@
06:34:17 Quit VailAZdoug (Ping timeout: 276 seconds)
06:45:42 Join VailAZdoug [0] (~doug@2607:fb90:604:8db9:607b:9848:c02d:a295)
06:49:04saratoga_VailAZdoug: you can either uninstall (see manual) or check the disk for errors and copy a new .rockbox folder over (which may or may not fix it depending on what the error is)
06:52:06VailAZdougsaratoga, I just copied over a clean copy of the sansa zip firmware file to the zip root. It's thinking right now with the normal sansa zip nonsense, creating a media database, which hopefully it will finish doing soon.
06:53:33***Saving seen data "./dancer.seen"
07:03:54 Nick SuperBrainAK is now known as DormantBrain (~andy@2001:470:8:a61::5f92:59a1)
07:25:49 Quit VailAZdoug (Ping timeout: 245 seconds)
07:31:43 Quit Scromple (Read error: Connection reset by peer)
07:39:53 Join kugel [0] (
07:39:53 Quit kugel (Changing host)
07:39:53 Join kugel [0] (~kugel@rockbox/developer/kugel)
07:40:14 Join Scromple [0] (~Simon@
07:41:48kugelPurlingNayuki, JdGordon: there is VOLUME_{MIN,MAX}. should be defined for all targets
07:48:03 Quit saratoga_ (Quit: Page closed)
07:49:57 Join fragilematter [0] (~fragilema@unaffiliated/fragilematter)
08:01:52 Quit kugel (Quit: Yaaic - Yet another Android IRC client -
08:05:58 Quit mt (Ping timeout: 252 seconds)
08:06:19 Join kugel [0] (
08:06:19 Quit kugel (Changing host)
08:06:20 Join kugel [0] (~kugel@rockbox/developer/kugel)
08:11:30 Quit kugel (Ping timeout: 246 seconds)
08:14:45 Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn)
08:18:09 Join ender` [0] (
08:22:42 Join bertrik [0] (
08:22:42 Quit bertrik (Changing host)
08:22:42 Join bertrik [0] (~quassel@rockbox/developer/bertrik)
08:31:43 Quit pedro_angelo (Remote host closed the connection)
08:53:35***Saving seen data "./dancer.seen"
08:59:35 Join Zagor [0] (~bjst@2a01:2b0:3041:3018:2677:3ff:fea3:3ef4)
08:59:35 Quit Zagor (Changing host)
08:59:35 Join Zagor [242] (~bjst@rockbox/developer/Zagor)
09:06:14 Join petur [0] (~petur@rockbox/developer/petur)
09:08:19 Quit bertrik (Ping timeout: 245 seconds)
09:28:39 Join einhirn [0] (
09:32:39 Quit [Saint] (Remote host closed the connection)
09:32:42 Quit fragilematter (Ping timeout: 246 seconds)
09:34:48 Join [Saint] [0] (~saint@rockbox/staff/saint)
09:36:29 Join fragilematter [0] (~fragilema@unaffiliated/fragilematter)
09:55:44 Join wodz [0] (
09:57:16 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
10:10:05gevaertssaratoga: yes, we do build one or two manuals to catch the worst errors
10:12:47PurlingNayukikugel: For all targets or some?
10:24:22 Join LinusN [0] (
10:25:31 Nick pixelma_ is now known as pixelma (pixelma@rockbox/staff/pixelma)
10:35:17 Quit pamaury (Ping timeout: 252 seconds)
10:36:40copperDB Error: database is locked (5)
10:37:23 Join lebellium [0] (
10:38:26pixelmagevaerts: we do?
10:39:34 Quit einhirn (Ping timeout: 245 seconds)
10:40:59 Join kugel [0] (~kugel@
10:40:59 Quit kugel (Changing host)
10:40:59 Join kugel [0] (~kugel@rockbox/developer/kugel)
10:42:27pixelmaPurlingNayuki: I haven't followed your code just read the discussions here - does the user get feedback about the volume limiter being enabled at all? I mean I could imagine some icon change in the status bar or so (even the built-in one)
10:43:52PurlingNayukipixelma: The limit is by default set to the maximum volume, and users can set this themselves.
10:44:14PurlingNayukiTo this point, they set the limit by themselves so no need to indicate.
10:47:38 Join einhirn [0] (
10:48:29 Quit wodz (Read error: No route to host)
10:52:13PurlingNayukiJdGordon: G#697, hope this to be the final version.
10:52:15fs-bluebotGerrit review #697 at : Implement volume limiter feature by PurlingNayuki (changes/97/697/19)
10:53:37***Saving seen data "./dancer.seen"
10:56:09PurlingNayuki...Oops but actually not. Forget to bring the manual.
10:58:26pixelmaI meant for the case they enabled the feature (set a lower limit than maximum). As I imagine it, it would be nice to see at a first glance that something changed which limits what I can do about the volume. Just an idea though after reading the discussion here
10:59:17[Saint]I think its reasonable to expect the user to remember they enabled it explicitly.
11:00:34lebelliumyou sometimes forget that you changed a setting in Rockbox
11:00:38lebelliumat least, it happens to me
11:01:20[Saint]By that measure, every setting needs a status bar icon.
11:01:20copperdamn it
11:01:25copperI made a mistake
11:02:13pixelmaI thought a simple icon change (somehow "locked volume" instead of the normal volume wasn't to intrusive and is like showing button lock too
11:02:19coppercould someone please replace "UPDATED January 12, 2013" with "UPDATED January 12, 2014" in the theme descriptions of my Googley themes? and
11:03:24[Saint]pixelma: I believe it should be apparent by not being able to set the volume past a level that you _should_ remember setting previously.
11:03:30[Saint]But...maybe you're right.
11:03:49[Saint]It just seems a shame to keep on piling requirements on to this change.
11:03:54PurlingNayukiAny manual volunteer?
11:04:20[Saint]In reality, I suspect only those who actually know what it is will be using it. Like many of our settings.
11:05:25PurlingNayukiThe default is set to the maximum volume which equals to no limit, so I think it's fine not to give basic users any indicator.
11:06:01[Saint]There is a fine line.
11:06:10PurlingNayukicopper: Try upload again with the same name and email address.
11:06:15[Saint]But yes, we shouldn;t have to remind users they enabled an advanced feature.
11:06:45pixelma[Saint]: didn't you say you read about UI design? ;)
11:07:46PurlingNayukiAgain I'll say if users don't need this or don't understand this they will, or can simply ignore its existence.
11:07:48[Saint]I'm currently only reading about touchscreen flows.
11:08:31PurlingNayukiIf they're curious about this or need this, RTFM.
11:08:41copperPurlingNayuki: I'm not going to update my 14 themes all over again
11:08:58copperthe theme update process is excessively painful
11:09:02PurlingNayukicopper: That's a large work :(
11:09:04[Saint]If we really have issues with users setting it accidentally, post commit, then it would be easy to investigate a custom icon for volume in the status bar and cabbie.
11:09:17 Join wodz [0] (
11:09:34[Saint]But at this stage I wouldn;t worry unless we find out it becomes a problem.
11:09:36gevaertspixelma: see e.g.;type=iriverh120manual
11:09:44wodzkugel: Please take a look at upadated g#711
11:09:45fs-bluebotGerrit review #711 at : buflib: Add crc field protecting buflib cookie integrity by Marcin Bukat (changes/11/711/2)
11:10:40 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
11:13:30PurlingNayukiBTW how to build a manual?
11:14:23wodzAnyone feeling like testing g#713 and g#718 on irivers and iaudios?
11:14:24fs-bluebotGerrit review #713 at : by Marcin Bukat (changes/13/713/2)
11:14:25fs-bluebotGerrit review #718 at : coldfire: Implement HAVE_INIT_ATTR magic by Marcin Bukat (changes/18/718/1)
11:15:07wodzpamaury: ping
11:18:06 Quit fragilematter (Quit: Leaving.)
11:19:32pamaurywodz: ^
11:20:26pixelmaPurlingNayuki: should still be ok
11:24:17wodzpamaury: I was thinking again about rk27xx usb driver. We cannot limit dma engine in how many data it will deliver directly but what about of getting data to intermediate buffer of max possible transfer size and memcpy from it later?
11:24:18PurlingNayukiJust read before you told me. Thanks anyway :)
11:26:27pamaurywodz: sure that's an option, it's just very annoying because you need one buffer per EP/dir, but with buflib it should be easy to allocate them
11:27:14wodzpamaury: We have problem with out EPs only, no?
11:28:28wodzbut as you mentioned buflib allocated buffer for lowlevel driver I get scared :-)
11:31:08pamauryyou can allocate them statically but that a waste :-/ But yeah I am scraed too ^^
11:32:04fs-bluebotBuild Server message: New build round started. Revision f0bec02, 249 builds, 31 clients.
11:32:15JdGordonthanks PurlingNayuki !
11:35:33fs-bluebotBuild Server message: Build round completed after 209 seconds.
11:39:43 Join fragilematter [0] (~fragilema@unaffiliated/fragilematter)
11:44:43kugelJdGordon: hm. I mentioned that there are compile time defines for VOLUME_MIN and _MAX. thought you would investigate that before pushing
11:48:26PurlingNayukiJdGordon: Thank you and those who helpped me!
11:48:56PurlingNayukikugel: These is for some targets, but not for all.
11:49:18kugelreally? should be defined for all IIUC
11:50:32PurlingNayukikugel: Unfortunately no.
11:50:53PurlingNayukiI will do the job later when I finish meal
11:51:14kugelwodz: why didn't you add a "core_check_valid"?
11:52:34wodzkugel: didn't think about it honestly
11:52:42gevaertskugel: df1704 and pcm1792 have VOLUME_MIN and _MAX. as3514 has it in the c file, but not in a header. That's it
11:54:57kugelhm, perhaps it used to be there for every target? IIRC it controlled the steps the sim shows
11:55:53 Quit [Saint] (Remote host closed the connection)
11:58:02 Join [Saint] [0] (~saint@rockbox/staff/saint)
12:01:08gevaertskugel: there are ways to get at the numbers if you care about the actual volume setting, but they depend on deep preprocessor magic, and making those expand to a *different* setting is not at all easy
12:01:15*gevaerts did have a look at that a while ago
12:01:43gevaertsHave a look at how AUDIOHW_SETTING() is used
12:02:55gevaertsBasically the bit that "calls" AUDIOHW_SETTING() is included several times, each with a different definition for AUDIOHW_SETTING(), which then expands to various tables
12:03:30kugelsounds awkward
12:03:55gevaertss/awkward/horrible/ :)
12:13:22 Quit kugel (Ping timeout: 272 seconds)
12:20:40 Join kugel [0] (~kugel@
12:20:40 Quit kugel (Changing host)
12:20:40 Join kugel [0] (~kugel@rockbox/developer/kugel)
12:22:02 Join Marex [0] (~Marex@
12:42:30 Quit kugel (Ping timeout: 260 seconds)
12:53:38***Saving seen data "./dancer.seen"
13:08:20 Join kugel [0] (~kugel@rockbox/developer/kugel)
13:23:19 Quit [Saint] (Remote host closed the connection)
13:25:21 Join [Saint] [0] (~saint@rockbox/staff/saint)
13:30:17 Quit kugel (Ping timeout: 276 seconds)
13:56:02 Join mt [0] (
13:59:56lebelliumkugel (logs) [Saint] JdGordon : On Clip Zip USB seems to work better with my theme now. No issue so far, although I'm still not sure sure the USB screen is supposed to look like that: And well, it takes much more time to be recognized by the computer than with cabbiev2 but at least it works.
13:59:58lebelliumRDS issues seem to be gone too.
13:59:59lebelliumBut I have a huge problem with YP-R0. With the latest build, it crashes whenever I want to change the theme or when I go to sound settings.
14:02:04lebelliumwell, more particularly when I cancel the maximum volume setting
14:04:31lebelliumwhen I want to change a theme it displays "loading" then crash
14:04:48lebelliumwhen I'm in the maximum volume setting and want to go back it displays "cancelled" then crash
14:25:01 Join dfkt [0] (OxO29A@unaffiliated/dfkt)
14:32:17toehser1sim head gets segfault
14:35:33funmanlebellium: how is your theme size? smaller than before?
14:37:03lebelliumfunman: hum I did not update my theme
14:37:50 Join maruk1 [0] (
14:38:56 Quit maraz (Quit: "")
14:39:06 Join maraz [0] (
14:40:44 Join amayer [0] (
14:50:26 Quit mt (Ping timeout: 272 seconds)
14:53:14 Quit cmhobbs (Ping timeout: 272 seconds)
14:53:41***Saving seen data "./dancer.seen"
15:02:37toehser1Program received signal SIGSEGV, Segmentation fault.
15:02:37toehser10x0000000000427c76 in volume_limit_is_changed (setting=0x8f5d1c <global_settings+1564>,
15:02:37toehser1 defaultval=0x0) at /home/tom/rockbox/git-tip/rockbox/apps/settings_list.c:681
15:02:37toehser1681 *(int*)defaultval = sound_max(SOUND_VOLUME);
15:02:51toehser1Funny, you guys were just talking about something volume related...
15:03:29toehser1Note, I haven't touched the volume or any settings related to volume...
15:08:09gevaertstoehser1: on boot?
15:08:21*gevaerts notices that the volume limit doesn't survive reboot
15:15:15gevaertsah, at least on shutdown
15:15:42toehser1no, after song skip for a while seeming ok
15:16:38gevaertssimilar. The ata_idle callback then probably
15:16:42gevaertsThat code can't work
15:17:26gevaerts"If this pointer is NULL, then put this value at the address it points to"
15:18:04wodzgevaerts: It will work of course - on arm it will overwrite reset vector most probably :-)
15:18:14gevaertsThere is that :)
15:18:38*gevaerts remembers calling a function pointer on gigabeat F and forgetting to set the pointer
15:19:01 Quit wodz (Quit: Leaving)
15:27:39fs-bluebotBuild Server message: New build round started. Revision 72367f4, 249 builds, 32 clients.
15:27:50gevaertstoehser1: that one should do it
15:32:16fs-bluebotBuild Server message: Build round completed after 278 seconds.
15:32:28toehser1seems ok so far, what was the issue?
15:33:06gevaertsHave a look at the lines I removed, and try to figure out when they could ever have been correct :)
15:33:41gevaertsI suspect there was a distinct lack of coffee when that particular bit was written
15:36:44PurlingNayukiI'm confused with void* defaultval actually.
15:41:57 Quit mortalis (Ping timeout: 246 seconds)
15:47:23*gevaerts gives PurlingNayuki more coffee
15:47:36gevaertsThere, that should fix it :)
15:47:59*PurlingNayuki really appreciates it
15:50:00gevaertsPurlingNayuki: the way I understand it, you can have a pointer to a default value in CUSTOM_SETTING() (fourth argument). You set that to NULL, so you're not using it, but you could have static int somewhere with the default value, in which case the defaultval pointer in volume_limit_is_changed() and volume_limit_set_default() would make sense
15:51:53PurlingNayukiI was not using it because it accepts compile-time constant only :(
15:52:35PurlingNayukiThough I can use sound_max() anytime to get the default value so it seems not to be useful.
15:54:06gevaertsIn this case, indeed
15:54:13gevaertsBut you had some remnants in the code
15:54:28PurlingNayukiSo... remove it.
15:54:36gevaertsI did :)
15:54:51PurlingNayukiI did the bug and you remove the bug :O
15:55:01gevaertsIt happens to everyone!
15:55:10gevaertsI'm claiming it's JdGordon's fault, anyway
15:55:23PurlingNayukiOops... it wasn't.
15:56:13PurlingNayukiI got a brief test on my iPod Nano 2G, Sansa Fuze and Gigabeat S but it did survive. It's a little strange.
15:56:24PurlingNayukiOr it was due to my good luck? :)
15:56:57gevaertsNull-dereferences can be a bit variable on various hardware
15:57:44PurlingNayukiSo the bug varies from targets to targets.
15:58:20gevaertsCould be, yes
16:12:55 Quit pamaury (Read error: Operation timed out)
16:21:52 Join krabador [0] (~krabador@unaffiliated/krabador)
16:22:04 Join ikeboy [0] (
16:53:45***Saving seen data "./dancer.seen"
16:55:28 Join Rower [0] (
17:00:21 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
17:00:27 Quit Zagor (Quit: Clint excited)
17:01:56toehser1I got 1 panic on the real player but I can't reproduce it now.
17:05:31 Quit fragilematter (Quit: Leaving.)
17:11:40 Join MarcAndersen [0] (
17:13:13MarcAndersenHi. I am using the Danish language of Rockbox but some of the voice clips says something completely wrong, and that does not happen in the English voice file. Can I correct this?
17:13:55gevaertsMarcAndersen: that usually means that the language file is out of date
17:14:40MarcAndersenCan I do something here without a build environment?
17:15:20gevaertsSorry, I mean the voice file
17:15:51MarcAndersenI just rebuilt the voice though
17:16:20MarcAndersenfrom the latest dev build for sansa clip zip. It is correct in 3.13.
17:19:25 Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier.
17:36:47toehser1When playing on battery I'm seeing 55% full charge, when plugged into charger it shows only 22% full charge... huh?
17:39:33 Quit petur (Quit: *plop*)
17:50:51 Quit ikeboy (Quit: ikeboy)
18:02:39copperpamaury: the guy with the LCD bug has a st7783 type display
18:02:59copperweird uh?
18:03:30pamauryyou mean the bug in plugins ?
18:05:08copperthe buffer overflow bug that kugel fixed
18:05:12copperthe video I made
18:06:55pamauryah ok, I thought you were talking about another bug
18:07:14copperweird that we both saw it but not you
18:07:16pamaurybut now we know the bug was not in the lcd driver
18:07:33pamauryI guess it depend on the displayed strings
18:07:59pamaurywith memory corruption, things can be pretty random
18:08:47 Join kugel [0] (~kugel@rockbox/developer/kugel)
18:10:41 Join bertrik [0] (
18:10:41 Quit bertrik (Changing host)
18:10:41 Join bertrik [0] (~quassel@rockbox/developer/bertrik)
18:18:10lebelliumkuge: see logs :)
18:18:18lebelliumkugel *
18:19:19 Join pretty_function [0] (~sigBART@
18:22:11 Join y4n [0] (~y4n@unaffiliated/y4ndexx)
18:26:38kugellebellium: I already saw. Good news
18:27:03 Quit Provel (Read error: Connection reset by peer)
18:29:00lebelliumkugel: could you try the latest build on your R0 to see if you can reproduce the crash?
18:29:51kugellebellium: which crash?
18:30:14kugelthe one at the volume limiter setting was fixed, no?
18:30:45lebelliumlet me check
18:31:25lebelliumI thought it was a different issue
18:32:48 Quit maruk1 (Quit: Leaving.)
18:34:58lebelliumoh indeed
18:35:09lebelliumit's fixed
18:37:23lebelliumI mean, the volume limiter setting fix also fixed the crash when switching theme
18:37:34lebelliumI don't understand why but it works :)
18:44:42 Join n1s [0] (
18:44:42 Quit n1s (Changing host)
18:44:42 Join n1s [0] (~n1s@rockbox/developer/n1s)
18:51:01lebelliumkugel JdGordon However I have an issue in FMS (only): the status bar doesn't display properly
18:53:46***Saving seen data "./dancer.seen"
18:55:33kugellebellium: please report a bug and attach your sbs/theme
19:02:30 Join Provel [0] (
19:23:17 Join felipeluna [0] (c81ba7f4@gateway/web/freenode/ip.
19:27:03 Quit pretty_function (Remote host closed the connection)
19:31:33 Quit felipeluna (Quit: Page closed)
19:33:14 Join felipeluna [0] (c81ba7f4@gateway/web/freenode/ip.
19:35:26felipelunahey, guys
19:35:42felipelunaso I'm just gonna ask this, anyone who knows, please answer
19:36:30felipelunathe nsf reader (based on libgme) loops on a nes "song" twice
19:36:39felipelunaI wanna change this to be N times
19:37:01felipelunaThere's a lot of code to understand and I thought I would ask here first
19:37:26felipelunaif it is already a parameter, it'd be great to know how can I change it
19:50:20kugellebellium: so no issues left apart from the USB screen weirdness?
19:51:43lebelliumapparently not
19:52:07lebelliumbut I wouldn't dare to say the USB connection is fixed after about 10-15 try
19:52:19lebelliumI'd like to try more
19:59:03 Quit ender` (Quit: Sometimes it pays to stay in bed on Monday, rather than spending the rest of the week debugging Monday's code. -- Christopher Thompson)
20:01:38 Quit kugel (Ping timeout: 245 seconds)
20:05:08 Join ender` [0] (
20:20:55 Join kiwicam [0] (~quassel@
20:25:24 Quit bluebrother^ (Ping timeout: 248 seconds)
20:25:39 Join bluebrother [0] (~dom@rockbox/developer/bluebrother)
20:39:02 Join fml [0] (
20:41:12fmlIt's known that I'm a TeX maniac :-) In the last commit to wps_tags.tex there is an unneeded space before ".\newline", if I'm not mistaken.
20:47:57 Quit fml (Quit: ChatZilla [Firefox 24.2.0/20131205180928])
20:53:47***Saving seen data "./dancer.seen"
20:58:50 Join mt [0] (
20:58:53 Join ikeboy [0] (
21:00:57 Join rela [0] (~x@pdpc/supporter/active/rela)
21:14:10 Quit y4n (Quit: only amiga makes it possible)
21:19:00 Quit Raptors (Read error: Connection reset by peer)
21:19:17 Join Raptors [0] (
21:28:30 Quit rela (Read error: Connection reset by peer)
21:40:38 Join rixon [0] (~quassel@
21:41:08rixonyo! can someone add me to WikiUsersGroup? username PaulKenyon
21:43:46rixonit's so I can add a model to the SigmaTel page
21:44:27AlexPrixon: Done
21:44:48rixonThanks AlexP
21:44:54AlexPyou're welcome
21:48:07 Join wodz [0] (
21:49:33wodzpamaury: Why do I need separate buffer per out EP actually? I think I need only one.
21:50:59 Join b0hoon [0] (
21:57:28fs-bluebotBuild Server message: New build round started. Revision feff021, 249 builds, 33 clients.
22:02:06fs-bluebotBuild Server message: Build round completed after 278 seconds.
22:03:53pamaurywodz: because several transfer can happen simultanously ?
22:06:03wodzCan they really? I mean on the wire the communication IS serial so how is it possible to have a few simultanously?
22:06:31 Quit felipeluna (Ping timeout: 272 seconds)
22:08:16b0hoonZagor, Bagder: (logs): Could you update the main site, please?
22:16:33 Quit ikeboy (Remote host closed the connection)
22:20:35 Quit amayer (Read error: Connection reset by peer)
22:22:14pamaurywodz: you can have several transfers per frame
22:22:49pamauryand anyway, the host can send data for EP0 (say) then EP1 on the next frame and in between you didn't have time to copy the data
22:23:13pamaurywith one buffer that would oblige you to really allow only one receive transfer at the time
22:23:38pamaurywhich is pretty much impossible if you don't know in which order they will arrive ?!
22:26:47 Join einhirn [0] (
22:28:01pamauryb0hoon: why did you abandon your arm renaming patch ?
22:29:09 Join kugel [0] (~kugel@rockbox/developer/kugel)
22:30:48 Quit fs-bluebot (Ping timeout: 252 seconds)
22:31:44b0hoonpamaury: I don't feel strong enough to face with it :)
22:32:08 Join fs-bluebot [0] (
22:34:04b0hoonHowever wiki should be updated, it's misleading for a new ports.
22:36:28 Join amayer [0] (
22:37:26b0hoonIt will be better if i'll stick to a simpler tasks :P, right now i'm trying to fix the backlight for the hdd1630.
22:39:22kugelb0hoon: nice to see you back anyway!
22:40:04 Join Strife89 [0] (~Strife89@2602:306:250d:18b9:11f1:4f7b:8e17:8279)
22:40:16kugelb0hoon: I didnt want to put you off, but this target tree pattern split ($manuf vs $soc) has been debated a lot of times, and if it was as simple as renaming it would have been done long ago
22:40:28b0hoonkugel: heh thanks, but time will show how much time i'll have to spend here
22:41:05b0hoonkugel: yeah i know...
22:42:53 Quit wodz (Read error: Operation timed out)
22:42:53kugelbasically nothing has been done because we don't know how to handle PP best (it's a very manifold platform), the older coldfire etc should be relatively simple
22:43:16kugeland it's not a high priority task too
22:45:28 Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier.
22:47:07b0hoonyes, i was thinking the same thing, it's not that important, so for now i'll pass.
22:53:50***Saving seen data "./dancer.seen"
22:58:20 Quit dfkt (Remote host closed the connection)
22:59:52 Quit amayer (Quit: Leaving)
23:05:29fs-bluebotBuild Server message: New build round started. Revision b25cd97, 249 builds, 34 clients.
23:09:43fs-bluebotBuild Server message: Build round completed after 253 seconds.
23:26:51 Quit Strife89 (Quit: Leaving)
23:36:54 Quit n1s (Quit: Ex-Chat)
23:44:50 Quit krabador (Quit: Sto andando via)
23:47:47 Quit bertrik (Remote host closed the connection)
23:51:48fs-bluebotBuild Server message: New build round started. Revision 518d9ec, 249 builds, 32 clients.
23:53:52 Quit kugel (Ping timeout: 252 seconds)
23:56:14fs-bluebotBuild Server message: Build round completed after 266 seconds.

Previous day | Next day