| 00:00:06 | saratoga | bah i've got a vorbis file that plays in the sim, but fails on the clip |
| 00:06:21 | saratoga | oh never mind, win32 sim works, but on linux crashes |
| 00:10:08 | saratoga | woah this Ogg mallocs over 750KB |
| 00:10:26 | | Join LambdaCalculus37 [0] (~rmenes@rockbox/staff/LambdaCalculus37) |
| 00:24:40 | | Join Strife89-iBook [0] (~Strife19@168.16.226.187) |
| 00:25:47 | | Quit wodz (Quit: Leaving) |
| 00:34:29 | | Quit DerPapst1 (Quit: Leaving.) |
| 00:34:40 | | Quit liar (Ping timeout: 258 seconds) |
| 00:38:19 | saratoga | whats the tool to see how much memory a codec statically allocates? |
| 00:47:24 | | Quit ChickeNES (Quit: Computer has gone to sleep.) |
| 00:49:45 | saratoga | jhMikeS: ping |
| 00:55:40 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 01:00 |
| 01:00:52 | | Join sideral [0] (~sideral@213.165.85.248) |
| 01:00:52 | | Quit sideral (Changing host) |
| 01:00:52 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 01:09:54 | | Quit silbo (Remote host closed the connection) |
| 01:10:11 | | Quit sideral (Ping timeout: 240 seconds) |
| 01:25:37 | | Quit bertrik (Quit: :tiuQ) |
| 01:27:59 | | Join Evilnick_ [0] (~Evilnick@ool-18bcf602.dyn.optonline.net) |
| 01:30:26 | | Quit dfkt (Quit: -= SysReset 2.55=- Sic gorgiamus allos subjectatos nunc.) |
| 01:31:44 | | Quit Evilnick__ (Ping timeout: 240 seconds) |
| 01:32:55 | | Quit simonlnu (Quit: bbialw) |
| 01:34:15 | | Join Topy [0] (~Topy44@f048107014.adsl.alicedsl.de) |
| 01:36:03 | | Quit T44 (Read error: Connection reset by peer) |
| 01:39:03 | | Quit LambdaCalculus37 (Quit: time to make dinner) |
| 01:58:38 | *** | Saving seen data "./dancer.seen" |
| 02:00 |
| 02:13:36 | | Join jhMikeS [0] (~jethead71@adsl-99-150-163-230.dsl.sfldmi.sbcglobal.net) |
| 02:13:36 | | Quit jhMikeS (Changing host) |
| 02:13:36 | | Join jhMikeS [0] (~jethead71@rockbox/developer/jhMikeS) |
| 02:18:39 | * | jhMikeS witnesses pingyness in his general direction from saratoga |
| 02:19:08 | saratoga | i want to make the malloc functions kill decode if they run out of memory |
| 02:19:21 | saratoga | is there some easy way to do that short of checking in every possible location for a failed malloc? |
| 02:20:42 | jhMikeS | put it in right in the mallocer, maybe? |
| 02:21:03 | jhMikeS | setjmp is there for vorbis |
| 02:24:58 | | Quit pamaury (Remote host closed the connection) |
| 02:25:58 | saratoga | ah yeah looks like thats already done then |
| 02:28:28 | jhMikeS | at least natively...why not for the rest? |
| 02:29:18 | | Quit ender` (Quit: If the experiment works, you must be using the wrong equipment. -- Patrick's Theorem) |
| 02:32:14 | saratoga | rest of what? |
| 02:33:33 | jhMikeS | app builds |
| 02:34:33 | saratoga | ah |
| 02:35:30 | | Quit n17ikh (Ping timeout: 240 seconds) |
| 02:48:41 | * | jhMikeS now needs to know why speex appears mildly borked |
| 02:50:18 | | Join simonlnu [0] (simon@unaffiliated/simonrvn) |
| 02:51:41 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 03:00 |
| 03:02:50 | | Quit factor (Read error: Connection reset by peer) |
| 03:06:43 | | Join factor [0] (~factor@r74-195-188-223.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
| 03:14:47 | | Join merbanan [0] (~banan@c-94-255-221-202.cust.bredband2.com) |
| 03:26:08 | | Join Peddy [0] (~apc@unaffiliated/peddy) |
| 03:31:52 | | Quit bluefoxx (Quit: Can we, should we, will we?) |
| 03:43:27 | | Quit Keripo (Quit: Leaving.) |
| 03:58:42 | *** | Saving seen data "./dancer.seen" |
| 04:00 |
| 04:06:54 | | Quit [7] (Disconnected by services) |
| 04:07:01 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
| 04:15:02 | | Quit pixelma (Disconnected by services) |
| 04:15:02 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
| 04:15:03 | | Quit amiconn (Disconnected by services) |
| 04:15:04 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
| 04:15:21 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
| 04:15:24 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
| 04:33:06 | | Join kugel [0] (~kugel@g231107097.adsl.alicedsl.de) |
| 04:33:07 | | Quit kugel (Changing host) |
| 04:33:07 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
| 04:36:42 | | Quit kugelp (Ping timeout: 260 seconds) |
| 04:37:54 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
| 04:49:31 | | Quit GeekShadow (Quit: The cake is a lie !) |
| 04:50:53 | | Quit Peddy (Quit: Leaving) |
| 05:00 |
| 05:00:38 | | Join ChickeNES [0] (~ChickeNES@128.135.100.102) |
| 05:01:32 | | Quit ChickeNES (Client Quit) |
| 05:02:27 | | Join ChickeNES [0] (~ChickeNES@128.135.100.102) |
| 05:03:28 | | Quit factor (Read error: Connection reset by peer) |
| 05:04:34 | | Join factor [0] (~factor@r74-195-188-223.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
| 05:17:53 | | Join Rob2223 [0] (~Miranda@p4FFF3B64.dip.t-dialin.net) |
| 05:17:54 | | Join Topy44 [0] (~Topy44@f048107014.adsl.alicedsl.de) |
| 05:19:05 | | Quit Topy (Read error: Connection reset by peer) |
| 05:21:56 | | Quit Rob2222 (Ping timeout: 250 seconds) |
| 05:24:24 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 05:29:13 | | Join bluefoxx [0] (fuzzylomba@S0106485b3917092d.vs.shawcable.net) |
| 05:40:02 | | Join timccc [0] (~lisa@112.166.15.141) |
| 05:43:05 | jhMikeS | it seems speex is in fact quite troubled in the same way in its old form too, like when resuming it always plays the first second or two of a track and then jumps to the correct point :\ |
| 05:48:16 | | Quit bluefoxx (Quit: Can we, should we, will we?) |
| 05:56:35 | | Join bluefoxx [0] (fuzzylomba@S0106485b3917092d.vs.shawcable.net) |
| 05:58:45 | *** | Saving seen data "./dancer.seen" |
| 06:00 |
| 06:11:31 | | Part Strife89-iBook ("Leaving") |
| 06:23:34 | CIA-87 | New commit by jethead71 (r29804): Clean up speex a bit after the playback changes in r29785. Haven't found any idiosyncracies not present before then. Fix it always returning ... |
| 06:25:11 | | Quit bluefoxx (Ping timeout: 260 seconds) |
| 06:27:19 | CIA-87 | r29804 build result: 18 errors, 72 warnings (jethead71 committed) |
| 06:29:54 | | Join bluefoxx [0] (fuzzylomba@S0106485b3917092d.vs.shawcable.net) |
| 06:42:17 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 06:46:39 | | Quit bzed (Ping timeout: 240 seconds) |
| 06:49:00 | | Join bzed [0] (~bzed@devel.recluse.de) |
| 06:52:56 | | Quit jhMikeS (Ping timeout: 240 seconds) |
| 06:58:42 | | Join Horschti [0] (~Horscht@xbmc/user/horscht) |
| 07:00 |
| 07:02:19 | | Quit Horscht (Ping timeout: 258 seconds) |
| 07:58:48 | *** | Saving seen data "./dancer.seen" |
| 07:58:51 | | Join Panther21 [0] (~45f48d09@giant.haxx.se) |
| 07:59:05 | Panther21 | Hey hey |
| 07:59:34 | Panther21 | I need to get unbanned from the forums, I was banned over 2 years ago, because I asked to be, can I get unbanned? |
| 08:00 |
| 08:00:24 | Panther21 | hello? |
| 08:01:13 | | Quit Panther21 (Client Quit) |
| 08:10:30 | Strife89 | You have to give us more than two minutes to respond .... |
| 08:25:40 | | Join stoffel [0] (~quassel@p57B4C22F.dip.t-dialin.net) |
| 08:33:43 | | Join n1s [0] (~quassel@rockbox/developer/n1s) |
| 08:59:10 | | Join ender` [0] (krneki@foo.eternallybored.org) |
| 09:00 |
| 09:06:31 | | Quit stoffel (Ping timeout: 240 seconds) |
| 09:25:18 | | Join stoffel [0] (~quassel@p57B4C22F.dip.t-dialin.net) |
| 09:28:31 | | Join sideral [0] (~sideral@213.165.85.248) |
| 09:28:31 | | Quit sideral (Changing host) |
| 09:28:31 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 09:34:30 | | Quit BHSPitMonkey (Quit: Ex-Chat) |
| 09:40:31 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
| 09:47:27 | | Quit timccc (Quit: Leaving.) |
| 09:55:41 | n1s | TheSeven: are you ok with me committing the usb delay patch? |
| 09:56:11 | n1s | hmm, maybe someone with a nano2g should test it too |
| 09:58:52 | *** | Saving seen data "./dancer.seen" |
| 10:00 |
| 10:04:34 | | Join lovasoa [0] (~lovasoa@cac94-9-88-162-232-8.fbx.proxad.net) |
| 10:05:55 | | Join robin0800 [0] (~robin0800@cpc3-brig8-0-0-cust703.3-3.cable.virginmedia.com) |
| 10:07:51 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
| 10:11:08 | | Quit lovasoa (Ping timeout: 240 seconds) |
| 10:13:01 | | Quit robin0800 (Quit: Leaving) |
| 10:13:21 | | Join robin0800 [0] (~robin0800@cpc3-brig8-0-0-cust703.3-3.cable.virginmedia.com) |
| 10:14:16 | | Join bertrik [0] (~bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 10:14:21 | | Quit bertrik (Changing host) |
| 10:14:21 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 10:22:56 | | Join silbo [0] (~quassel@193.40.244.204) |
| 10:27:55 | | Quit sideral (Quit: Leaving.) |
| 10:31:08 | | Quit robin0800 (Quit: Leaving) |
| 10:39:49 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 10:40:50 | | Join Stummi [0] (~Stummi@rockbox/developer/Stummi) |
| 10:41:26 | | Quit pamaury (Remote host closed the connection) |
| 10:42:04 | | Join pamaury [0] (~quassel@cez63-2-88-164-98-172.fbx.proxad.net) |
| 10:42:04 | | Quit pamaury (Changing host) |
| 10:42:04 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 10:45:30 | | Join slooopy [0] (~sloo@95-90-30-123-dynip.superkabel.de) |
| 10:58:49 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 10:59:15 | Llorean | Do we not have "Pause on Headphone Unplug" on RaaA? |
| 11:00 |
| 11:10:31 | n1s | hmm, dependancy bug, make doesn't notice when ffmpeg_get_bits.h in the codeclib changes |
| 11:39:21 | | Quit stoffel (Ping timeout: 248 seconds) |
| 11:53:19 | | Join Buschel [0] (~chatzilla@p54A3AB72.dip.t-dialin.net) |
| 11:57:12 | Buschel | how do we proceed with the newly introduced warnings/errors through amiconn's build machine? |
| 11:58:55 | *** | Saving seen data "./dancer.seen" |
| 12:00 |
| 12:00:16 | amiconn | The errors are pseudo-errors; the colouring script needs fixing |
| 12:01:16 | Buschel | amiconn: that's the one thing. the other is: shall we fix those issues? your machine uses a newer gcc, right? |
| 12:02:49 | amiconn | Imo the warnings should be fixed if possible. The other otpion would be blocking more warnings, but I don't like that solution |
| 12:03:08 | amiconn | Such blocks tend to become permanent (iirc we already have some) |
| 12:03:21 | bertrik | Well currently, it's a bit annoying to get build errors reported on every build |
| 12:03:44 | amiconn | The errors need fixing elsewhere. Who's in charge of that script? |
| 12:04:18 | amiconn | This is gcc 4.6.1 as the build log says, which is now default in debian unstable |
| 12:04:28 | | Join DerPapst [0] (~Alexander@p57954670.dip.t-dialin.net) |
| 12:04:30 | Buschel | I have got some fixes for those warnings -> FS #12088. would be good to have some reviews... |
| 12:05:15 | Buschel | fixing the warnings in ffmpeg_get_bits.h is a pain... maybe we should switch block the warning there... |
| 12:06:17 | bertrik | I don't care about debian unstable, I do care about getting bogus warning/errors on every build |
| 12:06:39 | amiconn | The warnings aren't bogus |
| 12:08:11 | bertrik | amiconn, do you want to force this issue by keeping the build table red? |
| 12:08:55 | bertrik | I find it annoying to have to double-check the build table whenever I commit something. |
| 12:10:43 | n1s | Buschel: yeah the ffmpeg thing is stupif |
| 12:10:45 | n1s | d |
| 12:11:11 | n1s | maybe we can add some attribute or just turn it off as you say |
| 12:11:24 | Buschel | n1s: yes, a macro-hell |
| 12:11:36 | amiconn | I'd rather keep the warnings instead of covering them up |
| 12:12:06 | amiconn | As I already said, cover-up actions tend to become permanent. This way we have a reminder that something needs fixing |
| 12:12:14 | amiconn | I don't see that as 'forcing' |
| 12:14:37 | n1s | most of them are easy to get rid of |
| 12:14:37 | Buschel | well, in the case of this very special warning ('set but unused') I think it it less dangerous to just switch it off instead of reworking weird macro stuff which works since ancient times... |
| 12:14:59 | Buschel | (for ffmpeg_get_bits.h that is) |
| 12:17:57 | | Join Jaykay [0] (~chatzilla@p5DC573BF.dip.t-dialin.net) |
| 12:18:17 | n1s | apparently the unused attribute should work |
| 12:19:04 | bertrik | I'd like enabling of these warnings like these to be decided on consciously, not because someone uses <random distro> which has them on by default. |
| 12:19:29 | n1s | bertrik: gcc has them on by default in 4.6 |
| 12:19:41 | n1s | it's not a debian thing |
| 12:21:30 | | Quit Jaykay (Client Quit) |
| 12:21:39 | bertrik | whatever, distro or gcc version was not my point |
| 12:22:17 | | Join Jaykay [0] (~chatzilla@p5DC573BF.dip.t-dialin.net) |
| 12:22:54 | | Quit Jaykay (Client Quit) |
| 12:23:17 | | Join benedikt93 [0] (~benedikt9@unaffiliated/benedikt93) |
| 12:23:56 | TheSeven | n1s: what would be the final delay values? |
| 12:24:36 | n1s | TheSeven: the second patch in FS #12083 |
| 12:25:23 | n1s | i just added the delays that the bootrom has and slightly changed the order to be more similar to the bootrom init |
| 12:26:00 | TheSeven | i'm still curious what's causing usb being connected on bootup to fail |
| 12:26:07 | TheSeven | does that happen with emcore as well? |
| 12:26:31 | n1s | no |
| 12:27:04 | n1s | in fact i have iloader now i think since i mistakenly used an old installer |
| 12:27:19 | n1s | but it boots fine with usb connected |
| 12:27:44 | TheSeven | so you might want to compare, as the lowlevel USB driver is essentially copy&paste |
| 12:27:53 | n1s | but it doesn't really do anything with the usb unless you run an app, right? |
| 12:28:08 | TheSeven | there's the debugging interface :) |
| 12:30:16 | n1s | i should try that then, it's described on the freemyipod wiki somewhere? |
| 12:30:37 | TheSeven | i think it's actually sufficient to check if the device enumerates |
| 12:30:51 | n1s | Buschel: i made a nasty preprocessor thing that works around the warning in the ffmpeg file |
| 12:31:12 | Buschel | I am curious ;) |
| 12:31:15 | pamaury | must the lcd_update function clock until all the data has be sent to the lcd or not ? |
| 12:32:11 | TheSeven | pamaury: that depends on what your definition of "clock" is, how the hardware works, and how the lcd code works :) |
| 12:32:14 | TheSeven | which target? |
| 12:32:36 | n1s | Buschel: http://pastebin.com/P3a9rJjV |
| 12:32:44 | pamaury | s/clock/block :) |
| 12:33:30 | n1s | TheSeven: noting happens if i just start iloader with usb connected, dmesg sees nothing |
| 12:33:46 | n1s | or should i run the emBIOS console thing? |
| 12:33:49 | TheSeven | i don't think so, if dma works on a copy of the framebuffer. if it works on the original source, it needs to |
| 12:33:59 | TheSeven | n1s: no, you should see the device enumerate immediately |
| 12:34:22 | pamaury | the dma is able to work the framebuffer directly, I don't want to make a copy |
| 12:34:26 | TheSeven | now try booting unplugged, and plugging after the boot menu came up |
| 12:34:48 | TheSeven | pamaury: then you need to block |
| 12:35:03 | n1s | TheSeven: still nothing |
| 12:35:07 | TheSeven | weird |
| 12:35:29 | TheSeven | hm, you'd probably need to add your delays to that as well :) |
| 12:35:53 | TheSeven | pamaury: this bugs me as well, as i'm running into the same thing on nano2g/classic, but rockbox is incapable of doing sane double buffering, both on the LCD and audio side |
| 12:36:20 | TheSeven | and while it's possible to hack your way around that for audio, it isn't possible for the LCD |
| 12:36:21 | n1s | it's interesting that it doesn't hang like rockbox did |
| 12:36:41 | TheSeven | it probably just hung some background thread |
| 12:36:57 | pamaury | ok |
| 12:37:21 | pamaury | that's a shame |
| 12:37:51 | TheSeven | fix it :) |
| 12:38:41 | pamaury | when I'm done with the port :) |
| 12:39:22 | n1s | hmm, theres one more experiment i should do |
| 12:41:26 | | Join GeekShadow [0] (~Antoine@93.21.160.159) |
| 12:41:26 | | Quit GeekShadow (Changing host) |
| 12:41:26 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
| 12:42:30 | pixelma | the reminder thing hasn't worked most of the times I remember if the fixes weren't within a short time frame |
| 12:45:14 | pixelma | just giving another argument, I don't have a strong opinion on this particular case |
| 12:46:15 | bluebrother | the thing is that people get used to the build system showing errors / warnings and start to ignore them. Nevertheless they should get fixed. |
| 12:47:26 | Buschel | I would like to have consistent settings for all build machines though. it is disturbing to see the warnings/errors move from build to build |
| 12:48:15 | bluebrother | that's true but how should this work for new compiler options? Does gcc ignore warning options it doesn't understand? |
| 12:48:26 | | Quit sideral (Quit: Leaving.) |
| 12:49:42 | pixelma | I understood Buschel that he thinks all machines should use the same gcc, not sure though |
| 12:52:42 | bluebrother | well, using different compilers has the advantage that we have a way to catch warnings different versions of gcc produce |
| 12:53:20 | bluebrother | using the same compiler version on all machines means we only get the warnings of that version, and getting more warnings is usually a good thing |
| 12:53:58 | bluebrother | (as in "the compiler warns in more suspicious cases", not as in "the code produces more warning" of course ;-) |
| 12:54:05 | n1s | configure already disables some warnings based on gcc version so it isn't that tricky |
| 12:54:30 | | Join DerPapst1 [0] (~Alexander@p57916634.dip.t-dialin.net) |
| 12:56:24 | | Quit DerPapst (Ping timeout: 246 seconds) |
| 12:58:48 | Buschel | how should the warnings in apps/codecs/speex.c be fixed? commenting page_nb_packets in both places it occurs (like done in FS #12088)? or adding __attribute((unused))__ to the declaration? |
| 12:59:51 | Buschel | several of the other warnings make unneeded code visible that can be deleted. |
| 13:00 |
| 13:00:51 | n1s | the asf things look weird to me |
| 13:01:17 | Buschel | n1s: that's the same story like speex.c |
| 13:01:46 | Buschel | the attribute thing would be better to read imho |
| 13:02:07 | | Quit bmbl (Quit: Verlassend) |
| 13:02:27 | n1s | i mean that the various get functions don't really do anything sionce the return value is discarded |
| 13:02:44 | n1s | i'd rather just comment out the whole lines |
| 13:05:44 | Buschel | you're right. I |
| 13:06:10 | Buschel | *I missed that the bit reader macro does not increment anything. will change it |
| 13:11:25 | Buschel | seems to be the same for speex.c. spx_ogg_page_packets() also does only return a value, but not change anything else |
| 13:12:00 | | Quit bertrik (Quit: :tiuQ) |
| 13:13:22 | n1s | TheSeven: the two hangs look different , the one that the delay patch fixes happens earlier, rb doesn't even enter the usb screen while the start up hang is later |
| 13:13:56 | n1s | the startup hang also doesn't go away after a reboot after sucessful init |
| 13:17:49 | | Join Evilnick [0] (~Evilnick@ool-18bcf602.dyn.optonline.net) |
| 13:20:00 | | Quit Evilnick_ (Ping timeout: 240 seconds) |
| 13:23:59 | Buschel | n1s: does this patch look better now? -> http://pastie.org/1853115 |
| 13:28:13 | * | gevaerts thinks the build output parser should be fixed to not show spurious red |
| 13:28:45 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 13:28:56 | n1s | Buschel: yeah, looks fine to me |
| 13:31:34 | n1s | TheSeven: the startup hang is definitely in the while (GRSTCTL & 1) loop, if i add a timeout to that it gets past it but usb doesn't actually work. |
| 13:36:54 | bluebrother | the script for counting warnings / errors is checklog.pl, correct? |
| 13:37:41 | TheSeven | gevaerts: any idea where kugel's reds are coming from? |
| 13:37:56 | TheSeven | isn't that file called arm-support.S rather than support-arm.S? |
| 13:38:22 | gevaerts | bluebrother: yes |
| 13:39:00 | gevaerts | TheSeven: kugel's reds? |
| 13:39:18 | TheSeven | hm, no, I must have renamed that while copying it to emcore... |
| 13:39:22 | TheSeven | http://build.rockbox.org/shownewlog.cgi?rev=29800;type=cowond2 |
| 13:39:35 | TheSeven | yet another race condition? |
| 13:40:15 | gevaerts | hm, right |
| 13:40:21 | TheSeven | hm, but the compilation calls for both incarnations of that file are way before the error |
| 13:40:28 | TheSeven | so that must be some other weirdness |
| 13:40:41 | gevaerts | I think I've seen that sort of red before, but I've never understood it |
| 13:42:46 | CIA-87 | New commit by Buschel (r29805): Fix several 'variable set but not used' warnings reported from GCC 4.6.0. |
| 13:43:10 | Buschel | let's see |
| 13:45:56 | | Quit bertrik (Ping timeout: 258 seconds) |
| 13:46:09 | n1s | should i commit that nasty attribute thing? |
| 13:46:46 | CIA-87 | r29805 build result: All green |
| 13:47:40 | TheSeven | do we even have non-__GNUC__ compilers? |
| 13:47:41 | bluebrother | hmm, shouldn't be too hard to fix. Just filter out those "In file included from" lines |
| 13:47:57 | bluebrother | like http://pastebin.com/Tz5k8Jte (diff is reversed −− d'oh!) |
| 13:48:44 | n1s | TheSeven: no |
| 13:49:25 | Buschel | this "all green" is a bit misleading now... |
| 13:51:15 | n1s | this is not a delay thing at least, adding 1 sec delays between every step in the init doesn't help |
| 13:52:08 | TheSeven | yeah you should probably fix the bug in emcore as well, and then check if that one hangs on boot as well |
| 13:53:00 | n1s | yeah, but i haven't set up the build stuff for that |
| 13:53:31 | TheSeven | hm, maybe it's a too rapid succession between disabling and enabling the OTG/PHY clocks? |
| 13:53:44 | TheSeven | if it's connected on boot, there will be a enable-disable-enable sequence |
| 13:54:53 | pamaury | TheSeven: any progress with the bootrom usb disassembly of the clip+ |
| 13:55:13 | * | n1s tests adding a delay after usb_drv_exit(); |
| 13:55:30 | TheSeven | pamaury: nope |
| 13:55:32 | bluebrother | gevaerts: does it make sense committing this change? Or does someone need to push the change to go live anyway (in which case committing it doesn't make much sense, perhaps someone knows a better way to fix this)? |
| 13:57:05 | gevaerts | bluebrother: as far as I know one of the Swedes has to update on the server |
| 13:57:14 | gevaerts | But I think this is the best way |
| 13:57:22 | | Join dfkt [0] (~dfkt@unaffiliated/dfkt) |
| 13:57:43 | bluebrother | ok. I'll leave it for now and hopefully tonight someone of the swedes can have a look |
| 13:58:04 | bluebrother | but IMO removing those false errors should be done soonish. |
| 13:58:06 | | Join stoffel [0] (~quassel@p57B4C22F.dip.t-dialin.net) |
| 13:58:59 | *** | Saving seen data "./dancer.seen" |
| 13:59:32 | | Join n17ikh [0] (~n17ikh@c-68-59-25-51.hsd1.sc.comcast.net) |
| 14:00 |
| 14:02:33 | n1s | interesting, if i break out of the loop after a timeout, a subsequent usb connection doesn't work either but times out again so there seems to be some weird state thing |
| 14:03:38 | pamaury | arf, I can't get the colors right on the fuze+ in 16-bit mode ! |
| 14:13:46 | | Quit ChickeNES (Quit: Computer has gone to sleep.) |
| 14:14:44 | bluebrother | hmpf, updating mingw-cross-env didn't fix the NVDA not speaking issue. Not that I'm surprised but still :( |
| 14:26:43 | pamaury | ah, finally a nice rockbox logo |
| 14:30:33 | linuxstb | Buschel: Have you looked at that ffmpeg_get_bits.h code in current ffmpeg/libav? |
| 14:33:13 | | Quit stoffel (Ping timeout: 276 seconds) |
| 14:38:48 | * | pamaury hesistate to commit his fuze+ code |
| 14:38:53 | pamaury | *hesitate |
| 14:45:00 | CIA-87 | New commit by pamaury (r29806): sbtools: do not rely on the ELF flags and always assume the entry point is valid |
| 14:48:01 | | Quit user890104 () |
| 14:48:02 | | Quit n1s (Ping timeout: 258 seconds) |
| 14:48:18 | CIA-87 | r29806 build result: 17 errors, 51 warnings (pamaury committed) |
| 14:49:43 | pamaury | huh ? |
| 14:50:48 | pamaury | wtf, the build table shows no error |
| 14:51:13 | pamaury | hum, these are not my errors ! |
| 14:51:34 | pixelma | it does and those are still the gcc errors that were talked about earlier |
| 14:51:54 | pixelma | a few fixed though |
| 14:53:32 | CIA-87 | New commit by pamaury (r29807): Add imx_hid_recovery_tool to send a sb file to a imx chip using the HID recovery mode |
| 14:54:04 | | Quit factor (Ping timeout: 252 seconds) |
| 14:57:09 | CIA-87 | r29807 build result: All green |
| 14:58:11 | pamaury | is there something special I should commit with the initial commit of the fuze+ ? |
| 15:00 |
| 15:02:52 | CIA-87 | New commit by pamaury (r29808): Sansa Fuze+: initial commit (bootloader only, LCD basically working) |
| 15:08:15 | CIA-87 | r29808 build result: 0 errors, 9 warnings (pamaury committed) |
| 15:12:03 | Buschel | linuxstb: asking because of the warnings? |
| 15:12:29 | | Join stoffel [0] (~quassel@p57B4C37D.dip.t-dialin.net) |
| 15:17:11 | linuxstb | Buschel: Yes, I was wondering if they had been fixed there. |
| 15:18:02 | Buschel | linuxstb: n1s made a patch for this −− he did not submit it yet -> http://pastebin.com/P3a9rJjV |
| 15:34:57 | | Quit knittl (Read error: Operation timed out) |
| 15:34:57 | | Quit simabeis_ (Read error: Operation timed out) |
| 15:35:02 | | Join knittl [0] (~knittl@thehappy.de) |
| 15:35:02 | | Quit knittl (Changing host) |
| 15:35:02 | | Join knittl [0] (~knittl@unaffiliated/knittl) |
| 15:35:05 | | Join simabeis [0] (~simabeis@lobmenschen.de) |
| 15:39:06 | | Join Evilnick_ [0] (~Evilnick@ool-18bcf602.dyn.optonline.net) |
| 15:40:30 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 15:41:06 | | Quit Evilnick (Ping timeout: 250 seconds) |
| 15:47:14 | | Quit liar (Quit: hallowed are the ori!) |
| 15:48:31 | CIA-87 | New commit by Buschel (r29809): Fix further 'variable set but not used' warnings reported from GCC 4.6.0. |
| 15:50:23 | | Join user890104 [0] (~Venci@6bez10.info) |
| 15:52:56 | CIA-87 | r29809 build result: All green |
| 15:58:27 | Buschel | saratoga: you there? |
| 15:59:00 | *** | Saving seen data "./dancer.seen" |
| 16:00 |
| 16:00:30 | Buschel | libwmapro\wmaprocodec.c, line 1384-1397 looks strange... |
| 16:02:49 | | Quit antil33t (Read error: Connection reset by peer) |
| 16:03:02 | | Join antil33t [0] (antil33t@124-197-51-80.callplus.net.nz) |
| 16:05:22 | | Join Jerom1 [0] (~jerome@95.171.148.84) |
| 16:05:33 | Buschel | hmm, forget about it... it's fine |
| 16:10:33 | | Quit bertrik (Ping timeout: 240 seconds) |
| 16:13:04 | | Join factor [0] (~factor@r74-195-188-223.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
| 16:16:41 | | Quit factor (Client Quit) |
| 16:17:14 | | Join factor [0] (~factor@r74-195-188-223.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
| 16:19:42 | | Join jhMikeS [0] (~jethead71@rockbox/developer/jhMikeS) |
| 16:21:22 | | Quit jhMikeS (Read error: Connection reset by peer) |
| 16:21:36 | | Join jhMikeS [0] (~jethead71@adsl-99-150-163-230.dsl.sfldmi.sbcglobal.net) |
| 16:21:36 | | Quit jhMikeS (Changing host) |
| 16:21:36 | | Join jhMikeS [0] (~jethead71@rockbox/developer/jhMikeS) |
| 16:23:14 | | Quit Jerom1 (Quit: Leaving.) |
| 16:40:33 | | Join Lear [0] (chatzilla@141.191.216.81.static.g-hn.siw.siwnet.net) |
| 16:41:40 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 16:44:26 | CIA-87 | New commit by Buschel (r29810): Fix nearly all residual 'variable set but not used' warnings reported from GCC 4.6.0. |
| 16:46:43 | Buschel | does anybody have a free spc file available? |
| 16:48:00 | CIA-87 | r29810 build result: All green |
| 16:48:33 | | Quit factor (Read error: Connection reset by peer) |
| 16:50:19 | | Join factor [0] (~factor@r74-195-188-223.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
| 16:54:03 | | Join u42p [0] (~v35b@d096169.adsl.hansenet.de) |
| 17:00 |
| 17:01:54 | | Quit Evilnick_ (Quit: Leaving) |
| 17:04:07 | | Join MethoS- [0] (~clemens@134.102.106.250) |
| 17:12:59 | | Nick kugel is now known as kugelp (~kugel@rockbox/developer/kugel) |
| 17:25:58 | | Quit bertrik (Ping timeout: 250 seconds) |
| 17:30:17 | | Quit slooopy (Ping timeout: 250 seconds) |
| 17:31:08 | | Join slooopy [0] (~sloo@95-90-30-123-dynip.superkabel.de) |
| 17:38:49 | | Join bertrik [0] (~bertrik@82-171-28-167.ip.telfort.nl) |
| 17:38:49 | | Quit bertrik (Changing host) |
| 17:38:49 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 17:50:42 | | Quit Buschel (Ping timeout: 260 seconds) |
| 17:59:01 | *** | Saving seen data "./dancer.seen" |
| 18:00 |
| 18:02:21 | | Quit stoffel (Ping timeout: 250 seconds) |
| 18:04:41 | | Quit bertrik (Read error: Connection timed out) |
| 18:06:17 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 18:07:43 | | Join kevku [0] (~kevku@2001:470:28:773:babe:feed:dead:bee) |
| 18:10:00 | | Quit pamaury (Remote host closed the connection) |
| 18:22:43 | | Quit bertrik (Ping timeout: 250 seconds) |
| 18:24:32 | | Quit kevku (Quit: KVIrc 4.0.4 Insomnia http://www.kvirc.net/) |
| 18:24:53 | | Join Jerom1 [0] (~jerome@95.171.148.84) |
| 18:31:10 | | Quit Lear (Quit: ChatZilla 0.9.86.1 [Firefox 4.0.1/20110413222027]) |
| 18:40:40 | | Quit mc2739 (Ping timeout: 240 seconds) |
| 18:42:57 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
| 18:44:18 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 18:47:52 | | Join n1s [0] (~quassel@rockbox/developer/n1s) |
| 18:49:00 | | Join ChickeNES [0] (~ChickeNES@wireless-199-162.uchicago.edu) |
| 18:50:50 | | Join Buschel [0] (~chatzilla@p54B67094.dip.t-dialin.net) |
| 18:53:25 | CIA-87 | New commit by nls (r29811): Work around another 'set but not used' warning. |
| 18:53:46 | * | n1s went for the simple but non compiler portable version |
| 18:57:02 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
| 18:57:50 | CIA-87 | r29811 build result: All green |
| 19:00 |
| 19:02:31 | | Join domonoky [0] (~Domonoky@rockbox/developer/domonoky) |
| 19:03:26 | | Join sideral [0] (~sideral@213.165.85.248) |
| 19:03:26 | | Quit sideral (Changing host) |
| 19:03:26 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 19:11:01 | | Join L-Strife89 [0] (~Strife89@168.16.226.187) |
| 19:11:32 | | Join mudd1 [0] (~cmertes@ip-78-94-202-227.unitymediagroup.de) |
| 19:29:52 | n1s | hmm, someone has reported the same usb issues i have on the classic for nano2g |
| 19:33:46 | | Join t0rc [0] (~t0rc@unaffiliated/t0rc/x-5233201) |
| 19:44:17 | | Quit silbo (Ping timeout: 260 seconds) |
| 19:58:01 | | Join stoffel [0] (~quassel@p57B4C37D.dip.t-dialin.net) |
| 19:59:05 | *** | Saving seen data "./dancer.seen" |
| 20:00 |
| 20:06:28 | | Quit stoffel (Remote host closed the connection) |
| 20:07:09 | | Join silbo [0] (~quassel@193.40.244.204) |
| 20:14:51 | | Quit t0rc (Quit: WeeChat 0.3.4) |
| 20:18:03 | | Quit dfkt (Quit: -= SysReset 2.55=- Sic gorgiamus allos subjectatos nunc.) |
| 20:20:17 | | Quit tmzt (Ping timeout: 276 seconds) |
| 20:25:56 | | Join TheLemonMan [0] (~lem0n@ppp-38-128.98-62.inwind.it) |
| 20:26:32 | | Join Banana-B [0] (~Bec@ip72-213-54-196.om.om.cox.net) |
| 20:27:01 | | Join tmzt [0] (~tmzt@adsl-99-164-58-59.dsl.akrnoh.sbcglobal.net) |
| 20:27:22 | | Quit GeekShadow (Quit: The cake is a lie !) |
| 20:28:41 | | Part Banana-B |
| 20:29:56 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 20:33:13 | n1s | hmm |
| 20:36:10 | * | n1s seems to have fixed the startup hang too but what change did it and why? |
| 20:39:31 | | Quit jordan` (Quit: Coyote finally caught me) |
| 20:41:10 | | Join otheros [0] (~02ca6ae5@giant.haxx.se) |
| 20:43:59 | | Quit otheros (Client Quit) |
| 20:48:16 | | Join t0rc [0] (~t0rc@unaffiliated/t0rc/x-5233201) |
| 20:57:16 | | Quit t0rc (Read error: Connection reset by peer) |
| 21:00 |
| 21:03:39 | n1s | yay, fix seems reliable |
| 21:03:57 | n1s | anyone with a classic or nano2g who want to try FS #12083? |
| 21:08:09 | | Join t0rc [0] (~t0rc@unaffiliated/t0rc/x-5233201) |
| 21:08:29 | * | n1s found a bug report for nano2g |
| 21:20:26 | bluebrother | n1s: I have a nano2g but am a bit busy right now. I can give it a try later tonight |
| 21:20:52 | n1s | great |
| 21:27:36 | | Quit domonoky (Read error: Connection reset by peer) |
| 21:28:01 | | Quit TheLemonMan (Remote host closed the connection) |
| 21:32:53 | | Join jordan` [0] (~gromit@ALagny-154-1-26-60.w83-112.abo.wanadoo.fr) |
| 21:47:18 | | Join kevku [0] (~kevku@2001:470:28:773:babe:feed:dead:bee) |
| 21:53:14 | | Join lovasoa [0] (~lovasoa@2a01:e35:8a2e:8080:226:bbff:fe15:33b0) |
| 21:53:28 | lovasoa | Hi |
| 21:54:05 | lovasoa | Does anyone knows why the "Database" menu is not localized ? |
| 21:59:08 | *** | Saving seen data "./dancer.seen" |
| 22:00 |
| 22:00:11 | | Join bertrik [0] (~bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 22:00:11 | | Quit bertrik (Changing host) |
| 22:00:11 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 22:08:30 | | Quit lovasoa (Remote host closed the connection) |
| 22:11:55 | | Join TheLemonMan [0] (~lem0n@ppp-38-128.98-62.inwind.it) |
| 22:15:14 | | Nick kugelp is now known as kugel (~kugel@rockbox/developer/kugel) |
| 22:16:02 | kugel | bertrik: ping |
| 22:17:37 | | Quit TheLemonMan (Remote host closed the connection) |
| 22:17:49 | CIA-87 | New commit by Buschel (r29812): Correct manual: rockbox support APEv2 metadata tags. |
| 22:18:05 | | Quit factor (Ping timeout: 246 seconds) |
| 22:18:46 | kugel | bertrik: I think IAP could be converted to use the multimedia button mechanism, to allow playback controls (from the docs) anywhere not just the wps |
| 22:21:46 | CIA-87 | r29812 build result: All green |
| 22:28:05 | | Join TheLemonMan [0] (~lem0n@ppp-38-128.98-62.inwind.it) |
| 22:28:18 | | Quit benedikt93 (Quit: HIP-HOP sounds best when you listen to METAL instead.) |
| 22:31:06 | | Quit mystica555_ (Ping timeout: 260 seconds) |
| 22:32:22 | bertrik | kugel, possibly, I don't know the multimedia button mechanism |
| 22:33:36 | CIA-87 | New commit by Buschel (r29813): Remove tabs. |
| 22:33:52 | kugel | bertrik: add #define HAVE_MULTIMEDIA_BUTTONS, then queue_post(&button_queue, MULTIMEDIA_PLAYPAUSE) etc. at the appropriate places should be it |
| 22:35:20 | | Quit u42p (Quit: Leaving) |
| 22:36:55 | CIA-87 | r29813 build result: All green |
| 22:38:56 | bertrik | argh, we have many more tabs in the code, even in firmware\ |
| 22:40:28 | amiconn | Buschel: Are you sure this is correct: http://svn.rockbox.org/viewvc.cgi/trunk/apps/gui/option_select.c?r1=29808;r2=29809;pathrev=29809 |
| 22:40:41 | Llorean | Where do they come from? Maybe someone hasn't noticed a misconfigured editor or something? |
| 22:40:52 | bertrik | kugel, indeed it looks nicer to me to make a remote act like a bunch of buttons, rather than controlling playback directly |
| 22:41:41 | Llorean | I'd personally rather a remote offer me the full functionality of Rockbox (that is, if voice is on, browse the menus, set up playlists, etc) |
| 22:41:58 | Llorean | Especially since I can control my iPod from my car steering wheel, making driving much safer if I want to adjust anything. |
| 22:43:04 | bluebrother | sounds like we need an option ;-) |
| 22:43:25 | | Nick Horschti is now known as Horscht (~Horscht@xbmc/user/horscht) |
| 22:46:33 | Buschel | amiconn: I cannot see anything wrong in there. |
| 22:46:46 | | Quit TheLemonMan (Quit: Destructor called) |
| 22:47:33 | bertrik | Llorean, many tabs are in apps/plugins, which is often external code adapted to rockbox. Tabs in firmware seem to be limited to just 6 files. |
| 22:48:48 | linuxstb | bertrik: The trouble with cleaning tabs is that it pollutes the svn history |
| 22:49:11 | bertrik | Llorean, with respect to buttons and iap: I don't have a good overview yet |
| 22:49:19 | | Join CaptainKewl [0] (~captainke@207-38-215-126.c3-0.nyr-ubr1.nyr.ny.cable.rcn.com) |
| 22:50:05 | bertrik | linuxstb, it doesn't bother me *that* much, so I won't rush to fix them |
| 22:50:48 | amiconn | Buschel: The minimum is not necessarily zero, and there's also the ASCENDING_INT_SETTINGS case, where min and max are switched |
| 22:51:09 | | Join T44 [0] (~Topy44@f048107014.adsl.alicedsl.de) |
| 22:51:22 | | Quit Topy44 (Ping timeout: 250 seconds) |
| 22:51:52 | amiconn | Hmm, weird. |
| 22:52:07 | Buschel | amiconn: the local variable "mín" is not used after setting any of those values. |
| 22:52:09 | amiconn | 'min' seems to be unused in general - hwo is this supposed to work? |
| 22:52:53 | amiconn | Looks like it's relying on 'steps' |
| 22:53:50 | * | amiconn still thinks the logic is weird |
| 22:54:38 | amiconn | Actually it's backwards. Imo it should be switched to rely on 'min' and 'steps' instead of 'max' and 'steps' |
| 22:55:43 | | Quit n1s (Remote host closed the connection) |
| 22:57:08 | amiconn | It's interesting that these changes actually reduce binsize a bit on many targets. That means earlier gcc didn't optimize those unused values away |
| 23:00 |
| 23:01:20 | | Quit bluebrother (Disconnected by services) |
| 23:01:21 | | Join bluebroth3r [0] (~dom@rockbox/developer/bluebrother) |
| 23:07:50 | | Quit L-Strife89 (Quit: Leaving) |
| 23:09:12 | | Join webguest93 [0] (~02caba51@giant.haxx.se) |
| 23:12:44 | CIA-87 | New commit by Buschel (r29814): Display the correct sign of replaygain for levels from -0.99 to -0.01 dB. |
| 23:13:36 | | Quit webguest93 (Client Quit) |
| 23:16:59 | CIA-87 | r29814 build result: All green |
| 23:18:05 | | Quit Buschel (Quit: ChatZilla 0.9.86.1 [Firefox 3.6.16/20110319135224]) |
| 23:18:26 | kugel | Llorean: it depends how the dock is designed I guess; it may offer navigation buttons and playback control buttons |
| 23:19:11 | Llorean | Are there such docks? |
| 23:20:00 | kugel | I don't know, you can still navigate with the ipod itself |
| 23:20:22 | Llorean | Which completely negates the benefits to safety I suggested it could have. |
| 23:20:28 | kugel | but you don't usually put into a dock to browse through menus or play games from the sofa, do you? |
| 23:20:41 | Llorean | Maybe find out if there are such docks before restricting functionality from the dock? |
| 23:21:13 | kugel | I guess making it worth with multimedia buttons is how it works with the OF, now? |
| 23:21:17 | kugel | no* |
| 23:22:33 | Llorean | Why do we care what the OF does? |
| 23:22:45 | Llorean | Should we really limit what functions of the player can be accessed via the dock to match the OF's restrictions? |
| 23:23:20 | | Join Keripo [0] (~Keripo@eng319.wireless-resnet.upenn.edu) |
| 23:24:21 | | Quit Stummi (Quit: Bye!) |
| 23:24:26 | kugel | I wouldn't call it adding limitations |
| 23:24:31 | bluebroth3r | I have a dock thats remote has buttons for up, down, ffwd, rew, play/pause, vol up, vol down and a few more |
| 23:24:49 | bluebroth3r | so what "multimedia" purpose is an "up" button? |
| 23:24:49 | kugel | bertrik: did you mean to disagree with me? |
| 23:25:20 | Llorean | kugel: "not allowing the user to access the menus from the dock" isn't an added limitation? |
| 23:25:26 | kugel | bluebroth3r: none, still act as up :) |
| 23:25:46 | bluebroth3r | and what does "up" in "multimedia mode"? |
| 23:25:58 | kugel | there's no multimedia mode |
| 23:26:01 | bluebroth3r | btw, that remote also has a button labelled "enter" |
| 23:26:17 | bluebroth3r | you're talking about multimedia buttons so there has to be some kind of multimedia mode |
| 23:26:22 | bluebroth3r | at least button-wise |
| 23:26:39 | kugel | so your dock has extra buttons for navigation, those wouldn't change |
| 23:27:22 | kugel | bluebroth3r: no mode involved, the multimedia buttons are additional buttons which work along with normal ones |
| 23:27:51 | bluebroth3r | and why are they multimedia buttons? Isn't Rockbox a multimedia player anyway? |
| 23:28:53 | bertrik | kugel, no I didn't mean to disagree |
| 23:29:41 | kugel | you said it sounds nicer to act as remote buttons rather than controlling playback directly |
| 23:29:53 | Llorean | I'd just like a mode where, whatever the minimum set of buttons a remote can have as per Apple's rules, as much of Rockbox as possible can still be reached. |
| 23:29:54 | kugel | which is status quo and not what I propose :) |
| 23:30:50 | kugel | bluebroth3r: like on a computer keyboard with such such buttons; they're labelled with play, prev, next etc |
| 23:31:06 | * | bluebroth3r thinks that naming is stupid |
| 23:31:50 | kugel | I didn't name them, that's what the whole world uses |
| 23:32:46 | kugel | up, down, enter and what's else on your remote aren't multimedia buttons and can be used for navigation just fine |
| 23:33:19 | Llorean | Do we know if there's a minimum set of required buttons on remotes for these? |
| 23:33:51 | kugel | I don't |
| 23:34:19 | bluebroth3r | that's the thing btw: http://www.alice-dsl.net/dominik.riebeling/rockbox/IMAG0144.jpg |
| 23:35:21 | bluebroth3r | though I'm not completely sure if the volume buttons are sent to the Ipod at all. My first tests rather looked like this only affects the line volume as passed by the dock |
| 23:37:30 | Llorean | I mean shouldn't that be where this discussion happens? Figure out whether we're okay with saying "some people aren't allowed to go to the menu if their remote only has multimedia buttons" |
| 23:37:38 | Llorean | But also find out if there are remotes that only have these buttons. |
| 23:40:51 | bertrik | I guess we just try to support each one as well as possible |
| 23:42:59 | Llorean | If you can detect what it has, sure. |
| 23:48:16 | bluebroth3r | well, those "multimedia" buttons on a pc create different keycodes so they don't interfere with the normal buttons |
| 23:49:02 | bluebroth3r | which leads to the question if iap has such "multimedia" buttons defined at all |
| 23:49:39 | bluebroth3r | if not, how could we identify such buttons? |
| 23:49:40 | Llorean | bluebroth3r: I've seen very simple iPod remotes that only have a Play/Stop, Vol+, Vol-, Next, and Prev button |
| 23:50:07 | Llorean | Even if those created different keycodes from up/down/left/right, you wouldn't be able to do anything outside the WPS from them if we treated them as dedicated multimedia in that situation. |
| 23:50:08 | bluebroth3r | no menu? |
| 23:50:28 | Llorean | Even if it had menu, if "next" was always next track, and Vol+ was always volume up, what would you do in the menu? |
| 23:50:43 | Llorean | But you could hold play-pause to stop, then select a new playlist through navigation, for example |
| 23:50:48 | Llorean | Or if the playlist ended, you could select a new one. |
| 23:50:52 | bluebroth3r | nothing useful :) |
| 23:51:33 | Llorean | But if Next/Prev and Vol+/Vol- worked as right/left and up/down (in situations where the remote didn't have buttons dedicated for such), you could navigate menus and access much more of the player via the remote. |
| 23:52:41 | Llorean | I mean, that's basically what we did with the GIgabeat F remote. I guess we *could* have the iPod less functional while using a remote than our other players, but it wouldn't make much sense. |
| 23:52:59 | bluebroth3r | my point is that the usual buttons are mapped in Rockbox. So now treating them as "multimedia" buttons means creating a different keymap for device and dock |
| 23:53:25 | Llorean | bluebroth3r: We have 'remote' keymaps on the H100, so I don't think that's too big a deal |
| 23:53:45 | bluebroth3r | which doesn't sound like a good idea to me |
| 23:53:55 | Llorean | To me the key is making sure that as long as there's a minimum set of buttons available, the user doesn't need to handle the player to access things |
| 23:54:14 | bluebroth3r | true, but the h100 has different buttons on the remote |
| 23:54:37 | Llorean | True. |
| 23:55:11 | kugel | bluebroth3r: the iap code makes BUTTON_RC_* keycodes, so they're separate already |
| 23:55:22 | bluebroth3r | plus those have even labels that don't exist on the player, so things are somewhat different |
| 23:56:00 | bluebroth3r | kugel: I wad talking about the wire side, not code |
| 23:56:03 | Llorean | kugel: SO what happens to people who have a remote that only has the buttons on the iPod? Or less buttons? Are they stuck because these are now "multimedia" buttons, and can't do anything if playback isn't happening? |
| 23:56:40 | kugel | they can use the controls on ipod to navigate |
| 23:56:57 | kugel | if you navigate you need to be at the doc anyway, due to the tiny screen |
| 23:57:10 | Llorean | So we should require them to actually get up and go to the iPod to make life *very* marginally easier for people with remotes with extra buttons? |
| 23:57:11 | bluebroth3r | shouldn't those have at least play/pause in any case? |
| 23:57:20 | kugel | with multimedia buttons you can use a remote control (if exists) to control playback from the sofa |
| 23:57:24 | Llorean | kugel: You're completely unaware of the voice UI then? And ignoring the "in car" use I described earlier? |
| 23:57:49 | kugel | navigation buttons on the dock and the ipod (which is on the dock) is redundant anyway |
| 23:57:53 | Llorean | If you map the buttons to what they do on the hardware, someone can access *everything* in Rockbox from the couch anyway |
| 23:58:34 | Llorean | kugel: What's the benefit of locking *some* remotes out of functionality, just so other remotes can access it in more ways? |
| 23:58:41 | bluebroth3r | *if* extra buttons create extra iap commands (on the wire that is) then thosr can get mapped |
| 23:58:45 | Llorean | Why is that better than "all (or most) remotes can access all functionality"? |
icon identifies that the person is a core developer (has commit access).