| 00:09:19 | | Quit bertrik (Quit: :tiuQ) |
| 00:09:24 | | Quit n1s (Quit: Ex-Chat) |
| 00:29:05 | | Quit Zagor_ (Quit: Clint excited) |
| 00:31:38 | *** | Saving seen data "./dancer.seen" |
| 00:41:02 | | Quit sirrozha () |
| 00:42:09 | | Quit froggyman (Quit: Ex-Chat) |
| 00:42:14 | | Join milk [0] (~milk@94-193-93-226.zone7.bethere.co.uk) |
| 00:45:18 | | Quit sideral (Quit: Leaving.) |
| 00:49:18 | * | pixelma wonders what the volume keys now do on Android |
| 00:52:25 | JdGordon | apparently change the volume |
| 00:54:14 | pixelma | the discussion before was about the volume range |
| 00:54:26 | JdGordon | Zagor: thats not really fixing fade-out :) why the heck isnt fade done in the playback engine or dsp? it is really out of place in wps.c |
| 00:56:06 | | Join FoH [0] (~foh@adsl-71-69-118.bhm.bellsouth.net) |
| 00:56:25 | | Quit Jerom (Quit: Leaving.) |
| 00:59:53 | | Quit kadoban (Ping timeout: 255 seconds) |
| 01:00 |
| 01:02:26 | | Quit Elfish (Ping timeout: 248 seconds) |
| 01:03:16 | | Quit ranmachan (Read error: Operation timed out) |
| 01:03:23 | | Join ranmachan [0] (ranma@yumi.tdiedrich.de) |
| 01:05:57 | jhMikeS | JdGordon: that's a deep rabbit hole :) |
| 01:06:07 | | Join kadoban [0] (~kadoban@ip98-165-177-158.ph.ph.cox.net) |
| 01:07:17 | | Join Elfish [0] (amba@2a01:4f8:100:90a1:abc:abc:abc:abc) |
| 01:17:50 | | Join froggyman [0] (~seth@205.213.134.249) |
| 01:17:51 | | Quit froggyman (Changing host) |
| 01:17:51 | | Join froggyman [0] (~seth@unaffiliated/froggyman) |
| 01:23:29 | scorche | B4gder: ping |
| 01:27:47 | | Quit kevku (Ping timeout: 260 seconds) |
| 01:31:37 | | Quit dfkt (Quit: -= SysReset 2.53=- Sic gorgiamus allos subjectatos nunc.) |
| 02:00 |
| 02:02:29 | pixelma | scorche: it looks like the akismet caught a "this topic has been moved to" post of mine |
| 02:03:02 | pixelma | my impression is that it doesn't like links in posts in general and is a bit overcautious |
| 02:04:43 | | Quit robin0800 (Quit: Leaving) |
| 02:05:11 | pixelma | damnit, I hope the poster gets an email notification - or doesn't look for it in the old place :\ |
| 02:05:21 | pixelma | until sorted |
| 02:07:10 | | Quit timccc (Read error: Connection reset by peer) |
| 02:08:48 | JdGordon | yeah, akismet is either setup slightly wrongly or just sucks on SMF... |
| 02:08:57 | jhMikeS | why can't I find that really old pre-mpegplayer plugin patch - I seem to be mentally challenged over and over about locating it :\ |
| 02:09:28 | | Join timccc [0] (~timccc@112.166.15.141) |
| 02:10:06 | JdGordon | scorche: I see you're editing the /GSoCOrgApp2011 page... changing the title is probably a good idea :) |
| 02:10:54 | scorche | there isnt really a "set up" for akismet on SMF |
| 02:11:05 | scorche | it is just over-cautious |
| 02:11:39 | scorche | though it should tell the poster that they were flagged and will be sorted out when an admin clears their post - i need to work on that |
| 02:12:04 | scorche | i can always disable akismet, though it has been catching plenty of spam |
| 02:12:12 | JdGordon | well, ideally it shouldnt be sending reply notifications out until it gets through akismet |
| 02:12:20 | pixelma | it would be cool if it could "learn" about trustworthy members or at least user groups |
| 02:12:28 | JdGordon | and seeing the flagged reply in the reply window is also funny |
| 02:13:00 | scorche | pixelma: well, each time i hit "not spam" it *should* learn something |
| 02:13:06 | saratoga | how does it decide what to flag? |
| 02:13:15 | pixelma | or allowed links - everything rockox.org or so |
| 02:13:16 | scorche | JdGordon: yeah - that is likely part of its crappy implementaton in SMF |
| 02:13:27 | saratoga | like why does it flag all of my replies to one thread, but not replies in other threads |
| 02:13:40 | scorche | pixelma: i cant really specify such things since akismet is a "cloud" thing |
| 02:13:49 | scorche | saratoga: because it doesnt like you |
| 02:13:56 | pixelma | the flagging of redirection posts surprised me now |
| 02:15:11 | pixelma | thanks for taking care of that so quickly |
| 02:17:29 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 02:31:05 | | Quit mshathlonxp (Quit: Leaving) |
| 02:31:39 | *** | Saving seen data "./dancer.seen" |
| 02:34:07 | | Quit simonrvn (Read error: Operation timed out) |
| 02:34:11 | | Join robin0800 [0] (~robin0800@cpc2-brig8-0-0-cust964.3-3.cable.virginmedia.com) |
| 02:38:05 | | Quit robin0800 (Client Quit) |
| 02:40:04 | | Join robin0800 [0] (~robin0800@cpc2-brig8-0-0-cust964.3-3.cable.virginmedia.com) |
| 02:40:34 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 02:41:58 | | Quit cjcopi (Read error: Operation timed out) |
| 02:50:33 | | Quit MethoS- (Remote host closed the connection) |
| 02:54:01 | | Join mystica555_ [0] (~mike@m4f2636d0.tmodns.net) |
| 02:54:43 | | Quit komputes (Remote host closed the connection) |
| 03:00 |
| 03:00:06 | | Join cjcopi [0] (~craig@adsl-76-241-72-119.dsl.bcvloh.sbcglobal.net) |
| 03:02:40 | | Join GodEater_ [0] (~bibble@5ac83eac.bb.sky.com) |
| 03:02:40 | | Quit GodEater_ (Changing host) |
| 03:02:40 | | Join GodEater_ [0] (~bibble@rockbox/staff/GodEater) |
| 03:04:57 | | Quit GodEater (Ping timeout: 240 seconds) |
| 03:05:09 | scorche | as a reminder, if you are planning to be involved in GSoC (as a mentor capacity - or just want some say in it) - join #rockbox-gsoc |
| 03:05:32 | | Quit simonrvn (Read error: Operation timed out) |
| 03:06:53 | | Quit krabador (Quit: Sto andando via) |
| 03:10:08 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 03:13:57 | | Quit froggyman (Ping timeout: 240 seconds) |
| 03:20:18 | | Join audio-i [0] (~ba1d7485@giant.haxx.se) |
| 03:23:57 | | Quit cjcopi (Ping timeout: 240 seconds) |
| 03:26:47 | | Join froggyman [0] (~seth@205.213.134.249) |
| 03:26:47 | | Quit froggyman (Changing host) |
| 03:26:47 | | Join froggyman [0] (~seth@unaffiliated/froggyman) |
| 03:27:11 | audio-i | hi! if I was to use %?if to display a different viewport when current track time %pc is below 60 minutes, what is the time format I should use for the operand? |
| 03:28:05 | JdGordon | something like %?if(%pc, <, 3600)<> |
| 03:28:36 | JdGordon | time is in seconds im pretty sure |
| 03:28:36 | audio-i | I think I tried that and it didn't work... trying again |
| 03:28:50 | JdGordon | i cant check the code now though |
| 03:29:38 | JdGordon | actually... its prob a string so your shit out of luck |
| 03:30:16 | audio-i | yeah it seems so |
| 03:30:59 | audio-i | only > O seems to return something usable |
| 03:31:11 | audio-i | what does it mean it's a string |
| 03:32:06 | audio-i | just characters? not a number I guess therefore no > < etc would work |
| 03:32:30 | | Join pjm0616 [0] (~user@sigfpe-1-pt.tunnel.tserv15.lax1.ipv6.he.net) |
| 03:32:31 | | Quit kadoban (Quit: bye) |
| 03:32:49 | | Join kadoban [0] (~kadoban@ip98-165-177-158.ph.ph.cox.net) |
| 03:32:56 | JdGordon | it just displays text |
| 03:33:08 | JdGordon | file a bug that %pc isnt useable with %if |
| 03:33:37 | audio-i | ok will do ;) |
| 03:33:55 | audio-i | not excatly requesting it :) but in this like other cases |
| 03:34:14 | audio-i | I find a "character length" tag would be very useful |
| 03:34:40 | JdGordon | how? |
| 03:35:32 | audio-i | like %cl on "rockbox" would return 7 |
| 03:35:48 | audio-i | i think %cl is taken though |
| 03:36:03 | JdGordon | yeah, but how would that be useful? |
| 03:36:20 | | Quit milk (Read error: Connection reset by peer) |
| 03:36:36 | audio-i | In this case, if %pc is greater than 5, I would use a different viewport |
| 03:37:11 | audio-i | laso useful in "controlled scrolling" routines |
| 03:37:51 | audio-i | or I could even use a bigger font if the number of characters allowes me to |
| 03:38:16 | audio-i | actually the ipod classic has that kind of effect |
| 03:38:34 | audio-i | when the title is long, it uses smaller fonts |
| 03:39:02 | | Quit froggyman (Ping timeout: 260 seconds) |
| 03:39:21 | audio-i | avoids scrolling |
| 03:45:05 | | Quit DerPapst (Quit: Leaving.) |
| 03:50:22 | | Join cjcopi [0] (~craig@adsl-76-241-72-119.dsl.bcvloh.sbcglobal.net) |
| 03:50:39 | | Quit robin0800 (Quit: Leaving) |
| 03:55:47 | | Quit balintx (Remote host closed the connection) |
| 03:56:08 | | Join balintx [0] (~quassel@szerver1.gulyasp-koll.sulinet.hu) |
| 03:58:22 | audio-i | damn! I thought I'd found another way of making it but it didn't work |
| 03:59:29 | audio-i | JdGordon: I was trying to use %pS with 3600 but it seems there is something like a 320 limit? |
| 03:59:43 | JdGordon | i doubt that |
| 04:00 |
| 04:00:22 | audio-i | well, i set it to 3600 but it goes false at 5 min 20 sec |
| 04:00:42 | JdGordon | the track is how long? |
| 04:00:55 | audio-i | 2 hours something |
| 04:01:18 | JdGordon | hmm.... this could be 32bit overflowing maybe... |
| 04:01:43 | JdGordon | 5:20 is how many milliseconds? |
| 04:02:37 | audio-i | 32000 |
| 04:02:42 | JdGordon | unsigned long time = token->value.i * (HZ/TIMEOUT_UNIT); <- /me cant do maths |
| 04:04:13 | audio-i | seems like you are right... and I'm certainly shit out of luck :/ |
| 04:04:35 | JdGordon | making %pc do what you want should be relativly easy |
| 04:04:58 | JdGordon | is that track length? or remaining time? |
| 04:05:11 | audio-i | current track time |
| 04:05:32 | audio-i | elapsed time |
| 04:06:19 | JdGordon | http://pastebin.com/N4774ymg |
| 04:07:00 | | Quit JdGordon (Quit: leaving) |
| 04:09:18 | | Join JdGordon| [0] (~jonno@124-168-180-95.dyn.iinet.net.au) |
| 04:09:19 | | Quit JdGordon| (Changing host) |
| 04:09:19 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
| 04:10:12 | | Quit z35 (Quit: Leaving) |
| 04:10:48 | audio-i | thanks! will try it later ;) somebody asked something of retroTape on the forums that could be resolved with this |
| 04:13:17 | | Quit amiconn (Disconnected by services) |
| 04:13:17 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
| 04:13:18 | | Join Llorean1 [0] (~DarkkOne@99-68-45-56.lightspeed.hstntx.sbcglobal.net) |
| 04:13:20 | | Quit pixelma (Disconnected by services) |
| 04:13:22 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
| 04:13:24 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
| 04:13:37 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
| 04:15:36 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 04:16:25 | | Quit Llorean (Ping timeout: 246 seconds) |
| 04:18:35 | | Quit audio-i (Quit: CGI:IRC (EOF)) |
| 04:21:52 | | Join froggyman [0] (~seth@unaffiliated/froggyman) |
| 04:22:09 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 04:23:28 | | Join notaguest [0] (~3a601b9e@giant.haxx.se) |
| 04:24:10 | | Quit notaguest (Client Quit) |
| 04:28:49 | | Quit Dreamxtreme (Ping timeout: 255 seconds) |
| 04:31:41 | *** | Saving seen data "./dancer.seen" |
| 04:33:24 | | Join tails___ [0] (~tails@host-109-171-18-177.bbcustomer.zsttk.net) |
| 04:34:42 | | Join kugel_ [0] (~kugel@rockbox/developer/kugel) |
| 04:35:57 | | Quit tails__ (Ping timeout: 250 seconds) |
| 04:37:49 | | Quit kugel (Ping timeout: 255 seconds) |
| 04:39:30 | | Quit mystica555_ (Read error: No route to host) |
| 04:41:55 | | Quit TheSeven (Ping timeout: 276 seconds) |
| 04:46:05 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
| 04:47:58 | | Join mystica555_ [0] (~mike@71-33-147-209.hlrn.qwest.net) |
| 04:54:14 | | Join Barahir_ [0] (~jonathan@frnk-590f69c6.pool.mediaWays.net) |
| 04:57:38 | | Quit Barahir (Ping timeout: 276 seconds) |
| 05:00 |
| 05:03:10 | | Quit ps-auxw (Ping timeout: 260 seconds) |
| 05:05:26 | | Join Rob2223 [0] (~Miranda@p4FFF3157.dip.t-dialin.net) |
| 05:08:56 | | Quit Rob2222 (Ping timeout: 252 seconds) |
| 05:12:10 | | Nick Llorean1 is now known as Llorean (~DarkkOne@99-68-45-56.lightspeed.hstntx.sbcglobal.net) |
| 05:12:16 | | Quit Llorean (Changing host) |
| 05:12:16 | | Join Llorean [0] (~DarkkOne@rockbox/user/Llorean) |
| 05:13:36 | | Join Dreamxtreme [0] (~Dre@92.18.110.105) |
| 05:14:09 | | Join ps-auxw [0] (~arneb@p4FF7FF22.dip.t-dialin.net) |
| 05:14:46 | | Quit ps-auxw (Client Quit) |
| 05:26:00 | | Quit simonrvn (Read error: Operation timed out) |
| 05:30:21 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 05:37:30 | | Join ps-auxw [0] (~arneb@p4FF7EC7F.dip.t-dialin.net) |
| 05:41:42 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 05:50:00 | | Quit simonrvn (Read error: Operation timed out) |
| 05:50:17 | | Join JdGord [0] (~3a601b9e@giant.haxx.se) |
| 05:50:38 | | Quit JdGord (Changing host) |
| 05:50:38 | | Join JdGord [0] (~3a601b9e@rockbox/developer/JdGordon) |
| 05:53:49 | | Quit JdGord (Client Quit) |
| 05:58:49 | | Join JdGord [0] (~3a601b9e@giant.haxx.se) |
| 05:58:58 | | Quit JdGord (Client Quit) |
| 06:00 |
| 06:00:08 | | Join JdGord [0] (~3a601b9e@giant.haxx.se) |
| 06:00:16 | | Quit JdGord (Client Quit) |
| 06:00:19 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 06:16:00 | | Quit kadoban (Read error: Connection reset by peer) |
| 06:17:48 | | Quit simonrvn (Quit: see ya - n'multes) |
| 06:21:03 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 06:31:44 | *** | Saving seen data "./dancer.seen" |
| 06:33:09 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 06:33:50 | | Quit Judas_PhD (Remote host closed the connection) |
| 06:35:43 | | Quit simonrvn (Quit: see ya - n'multes) |
| 06:40:05 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 06:40:17 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 06:41:28 | | Join leavittx_ [0] (~lev@89.221.199.187) |
| 06:50:55 | | Quit liar (Read error: No route to host) |
| 06:53:45 | | Join Horscht [0] (~Horscht@xbmc/user/horscht) |
| 06:56:21 | | Quit Horschti (Ping timeout: 246 seconds) |
| 06:57:05 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
| 07:00 |
| 07:00:30 | | Quit panni_ (Quit: ( www.nnscript.de :: NoNameScript 3.81 :: www.XLhost.de )) |
| 07:03:35 | | Quit T44 (Read error: Connection reset by peer) |
| 07:03:36 | | Join Topy [0] (~Topy44@g228173019.adsl.alicedsl.de) |
| 07:14:40 | | Quit liar (Ping timeout: 276 seconds) |
| 07:34:41 | | Join MagusG [0] (magusg@c-71-59-57-46.hsd1.ga.comcast.net) |
| 07:35:24 | | Quit bluefoxx_ (Quit: Can we, should we, will we?) |
| 07:39:05 | | Join bluefoxx [0] (fuzzylomba@S0106485b3917092d.vs.shawcable.net) |
| 07:44:05 | | Quit bluefoxx (Quit: Can we, should we, will we?) |
| 07:44:43 | | Join bluefoxx [0] (fuzzylomba@S0106485b3917092d.vs.shawcable.net) |
| 07:46:13 | | Join bthomson [0] (~bthomson@pool-71-114-64-197.washdc.dsl-w.verizon.net) |
| 07:50:37 | | Join kevku [0] (~kevku@2001:7d0:0:f9af:feed:feed:feed:feed) |
| 08:00 |
| 08:04:30 | S_a_i_n_t | JdGordon|: Is there a way to split lines in a .wps/.sbs/.fms/etc so you don't end up with some lines that are ~200 columns long? |
| 08:05:15 | S_a_i_n_t | I reduced my ASCII header to 80 char, but...it kinda seems pointless to do so with truly massive lines in the skin code. |
| 08:07:45 | S_a_i_n_t | I was just wondering if there ws some magic tag for newline that's smart enough to stick the next line on the end of the previous line when it gets parsed or something, so you could have nice 80 char wide themes so everything looks nice and neat. |
| 08:07:52 | S_a_i_n_t | s/ws/was/ |
| 08:12:14 | JdGordon| | # should do it |
| 08:19:32 | | Quit simonrvn (Read error: Operation timed out) |
| 08:21:19 | | Quit bluebrother (Read error: Operation timed out) |
| 08:21:34 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 08:21:52 | S_a_i_n_t | Wha? To split a line of actual code? so: |
| 08:22:17 | S_a_i_n_t | %?C<foo|# |
| 08:22:23 | S_a_i_n_t | bar> |
| 08:23:37 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
| 08:24:18 | JdGordon| | S_a_i_n_t: yeah |
| 08:24:31 | | Join mudd1 [0] (~cmertes@ip-78-94-203-49.unitymediagroup.de) |
| 08:25:05 | JdGordon| | well... there has been plenty of debate over weather # should eat the \n or not... |
| 08:25:10 | JdGordon| | iirc svn has it eating it |
| 08:26:55 | | Quit sasquatch (Ping timeout: 250 seconds) |
| 08:27:56 | | Join LinusN [0] (~linus@rockbox/developer/LinusN) |
| 08:31:46 | *** | Saving seen data "./dancer.seen" |
| 08:35:21 | | Join ender` [0] (krneki@foo.eternallybored.org) |
| 08:37:04 | | Part LinusN |
| 08:37:33 | | Join LinusN [0] (~linus@rockbox/developer/LinusN) |
| 08:39:01 | | Part LinusN |
| 08:42:11 | | Join LinusN [0] (~linus@rockbox/developer/LinusN) |
| 08:45:41 | | Quit simonrvn (Read error: Operation timed out) |
| 08:49:36 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 08:51:12 | | Join sasquatch [0] (~username@46.115.150.24) |
| 09:00 |
| 09:04:10 | | Join _Zagor [0] (~bjst@rockbox/developer/Zagor) |
| 09:04:22 | | Nick _Zagor is now known as Zagor (~bjst@rockbox/developer/Zagor) |
| 09:05:25 | | Quit sasquatch (Ping timeout: 252 seconds) |
| 09:06:47 | | Join sasquatch [0] (~username@p4FF2D55C.dip.t-dialin.net) |
| 09:26:16 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 09:27:55 | | Join einhirn [0] (~Miranda@p54850B0E.dip0.t-ipconnect.de) |
| 09:29:45 | | Quit leavittx_ (Ping timeout: 240 seconds) |
| 09:29:47 | | Join esperegu [0] (~quassel@145.116.15.244) |
| 09:32:06 | | Join pamaury [0] (~quassel@vit94-1-82-67-248-70.fbx.proxad.net) |
| 09:32:06 | | Quit pamaury (Changing host) |
| 09:32:06 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 09:33:30 | | Quit Elfish (Ping timeout: 248 seconds) |
| 09:35:35 | | Quit ranmachan (Ping timeout: 260 seconds) |
| 09:35:49 | | Quit literal (Ping timeout: 276 seconds) |
| 09:35:54 | | Join ranmachan [0] (~ranma@yumi.tdiedrich.de) |
| 09:37:16 | | Join literal [0] (hinrik@w.nix.is) |
| 09:44:08 | | Join Elfish [0] (~amba@2a01:4f8:100:90a1:abc:abc:abc:abc) |
| 09:46:11 | | Quit mudd1 (Ping timeout: 250 seconds) |
| 09:47:39 | | Quit mystica555_ (Ping timeout: 264 seconds) |
| 09:48:32 | | Join n1s [0] (~n1s@sb-fw.bmc.uu.se) |
| 09:48:33 | | Quit n1s (Changing host) |
| 09:48:33 | | Join n1s [0] (~n1s@rockbox/developer/n1s) |
| 09:52:34 | | Join mystica555_ [0] (~mike@m4f2636d0.tmodns.net) |
| 10:00 |
| 10:01:55 | | Quit sasquatch (Quit: WeeChat 0.3.2) |
| 10:02:07 | | Join sasquatch [0] (~username@p4FF2D55C.dip.t-dialin.net) |
| 10:02:21 | | Quit S_a_i_n_t (Ping timeout: 276 seconds) |
| 10:03:24 | | Join [Saint] [0] (S_a_i_n_t@203.184.0.69) |
| 10:15:14 | | Quit Slasheri_ (Ping timeout: 250 seconds) |
| 10:29:36 | | Quit jordan` (Quit: Coyote finally caught me) |
| 10:29:56 | | Join jordan` [0] (~jordan@jem75-13-78-235-252-137.fbx.proxad.net) |
| 10:31:49 | *** | Saving seen data "./dancer.seen" |
| 10:35:30 | | Quit JdGordon| (Ping timeout: 276 seconds) |
| 10:35:41 | | Nick kugel_ is now known as kugel (~kugel@rockbox/developer/kugel) |
| 10:36:01 | | Join JdGordon| [0] (~jonno@124-168-180-95.dyn.iinet.net.au) |
| 10:36:01 | | Quit JdGordon| (Changing host) |
| 10:36:01 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
| 10:42:25 | | Join robin0800 [0] (~robin0800@cpc2-brig8-0-0-cust964.3-3.cable.virginmedia.com) |
| 10:43:39 | | Quit JackWinter (Remote host closed the connection) |
| 10:46:32 | | Join Slasheri [0] (miipekk@xen.ihme.org) |
| 10:46:33 | | Quit Slasheri (Changing host) |
| 10:46:33 | | Join Slasheri [0] (miipekk@rockbox/developer/Slasheri) |
| 10:47:34 | pixelma | Zagor: what will the volume keys now do on Android (regarding the volume steps and range)? |
| 10:49:00 | Zagor | it adjusts the rockbox volume |
| 10:49:54 | Zagor | before my commit, the volume buttons adjusted the *ringtone* volume so it's not like we're losing anything |
| 10:50:34 | Zagor | though we should add proper android media volume adjustment asap |
| 10:50:36 | kugel | ugh, apparently you need to add something to debian/changelog to bump the debian version of a package |
| 10:50:55 | kugel | ffs, sorry |
| 10:51:27 | pixelma | so I set the global volume (which affects the maximum available) outside Rockbox and then the percentage (or whatever it is) in Rockbox with the difference to now being that you can use the volume keys too instead of touchscreen things or the menu? |
| 10:51:54 | kugel | Zagor: no, they didn't |
| 10:51:55 | | Join JackWinter [0] (~jack@vodsl-9173.vo.lu) |
| 10:52:18 | kugel | they changed the volume of the music if music was playing. that's just how it works on android |
| 10:53:21 | Zagor | oh, my mistake. though I do find it quite weird that "rockbox" adjusts ringtone volume just because I happened to be in pause |
| 10:53:23 | pixelma | but the global volume (and I seem to remember that it showed the "ringtone volume" splash too) |
| 10:53:45 | Zagor | I suppose that is due to our disabling the service in pause? |
| 10:54:06 | kugel | yes, the global "media volume". I now need to go out of rockbox to change the global volume with the volume keys? |
| 10:54:57 | Zagor | at the moment, yes. we have to fix proper volume handling. |
| 10:55:06 | kugel | then I object this change |
| 10:55:06 | Zagor | I'll look into that |
| 10:55:22 | Zagor | we don't want two volume controls, it's just silly |
| 10:56:03 | kugel | I don't want different behvaior just because rockbox is in the foreground |
| 10:56:10 | [Saint] | glargh....I liked it :/ |
| 10:56:31 | Zagor | kugel: so you don't want the rockbox high resolution volume? |
| 10:56:47 | kugel | I do, but that's not what you have committed |
| 10:56:50 | Zagor | we had a long discussion about this yesterday, and several people felt the 16-step android volume is not sufficient |
| 10:57:00 | pixelma | it's confusing indeed and I'm looking forward to a better solution, if android would at least offer more fine grained control at least for media (I can even see the reason to have quicker controls for call volume) |
| 10:57:12 | kugel | I was part of that discussion, yes |
| 10:57:36 | pixelma | [Saint]: "it"? |
| 10:58:00 | kugel | IMO you just made it worse, there's still 2 volumes, but now there's inconsistent behavior |
| 10:58:31 | [Saint] | pixelma: it == having independant RB/Android volume control. |
| 10:58:32 | Zagor | that's one way to view it. I'll try to improve it asap. |
| 10:59:23 | kugel | can we get back to the old behavior until you improved it? |
| 10:59:34 | pixelma | [Saint]: you still have, just not as "easily" accessible |
| 10:59:55 | [Saint] | yeah. |
| 11:00 |
| 11:00:29 | Zagor | kugel: why? it's not like we are distributing builds. just cut it out of your build if you can't stand it for a few hours/days |
| 11:00:39 | Zagor | it's a work in progress |
| 11:00:40 | | Quit kevku (Read error: Operation timed out) |
| 11:00:54 | kugel | I will |
| 11:02:23 | kugel | I still think your commit was premature. "commit early, commit often" is fine but not if the intermediate commits make things just worse |
| 11:02:59 | Zagor | that's not objective. I made android behave more like other targets, in that the volume buttons adjust the rockbox volume. |
| 11:03:11 | Zagor | if that is better or worse is a matter of opinion |
| 11:03:58 | | Quit robin0800 (Quit: Leaving) |
| 11:04:46 | kugel | volume handling just isn't the same thing on RaaA |
| 11:05:19 | Zagor | I know. but it should be. |
| 11:06:13 | kugel | that's debatable |
| 11:06:51 | Zagor | yes, we debated it yesterday. and the consensus was that a single high-resolution volume control (optional high-resolution, even) is the desired solution |
| 11:07:49 | [Saint] | I seem to recall it being about 50/50 personally. |
| 11:07:56 | | Join mudd1 [0] (~cmertes@2001:638:504:20e0:221:70ff:fe83:655e) |
| 11:07:59 | kugel | alright, we'll see what you come up with. but if it means rockbox can't change the global volume anymore then I will probably object |
| 11:08:23 | Zagor | kugel: so would I |
| 11:09:53 | * | kugel doesn't know if the global volume steps are the same on all android devices |
| 11:10:10 | | Quit mudd1 (Client Quit) |
| 11:10:53 | Zagor | from what I have read, android offers 16 steps on all models. but the levels of each step might be different. we'll have to see what is possible. |
| 11:13:01 | pixelma | 16 here for media and 8 for ringtone so they already offered additional steps |
| 11:13:18 | pixelma | even if it's not many |
| 11:13:45 | | Join casainho [0] (~chatzilla@pal-213-228-181-14.netvisao.pt) |
| 11:13:49 | kugel | i have 9 levels (1 of them is mute) |
| 11:15:09 | Zagor | kugel: for ringtone or media? |
| 11:15:44 | | Quit Horscht (Quit: Verlassend) |
| 11:15:47 | kugel | ringtone is 8+vibrate+mute, media is 8+mute |
| 11:16:12 | Zagor | wow only 8! which phone is that? |
| 11:16:18 | kugel | htc legend |
| 11:16:30 | Zagor | horrible |
| 11:16:39 | pixelma | maybe asking for Android version is needed too |
| 11:16:53 | Zagor | pixelma: if it is linked to android version |
| 11:16:54 | kugel | 2.3.3 here |
| 11:17:01 | Zagor | then it's not |
| 11:17:08 | kugel | but I'm not surprised if vendors handle it differently |
| 11:17:36 | pixelma | Zagor: yeah, I know. But I thought running custom ROMs *could* have an influence |
| 11:17:48 | Zagor | pixelma: right, it could |
| 11:19:08 | kugel | well, it means you can't rely on the exact volume steps, no? |
| 11:19:10 | Zagor | though I can't see why a custom rom would make volume resolution worse than stock |
| 11:19:45 | Zagor | kugel: unless we can someone find out |
| 11:19:48 | Zagor | somehow |
| 11:22:35 | Zagor | getStreamMaxVolume() |
| 11:39:42 | linuxstb | My HTC Desire HD has 16 for media, 8 for ringer. |
| 11:42:19 | Zagor | it looks like we could "just" expand RockboxPCM.java to also adjust the stream volume |
| 11:49:48 | | Join robin0800 [0] (~robin0800@cpc2-brig8-0-0-cust964.3-3.cable.virginmedia.com) |
| 11:50:50 | | Join mcuelMobile [0] (b274a083@gateway/web/freenode/ip.178.116.160.131) |
| 11:52:30 | mcuelMobile | Zagor: take a look at FS #11914 patch8 |
| 11:53:08 | Zagor | lookie there! |
| 11:54:54 | | Join mcuelMobile2 [0] (b274a083@gateway/web/freenode/ip.178.116.160.131) |
| 11:55:05 | mcuelMobile2 | stupid crashin webclient |
| 11:55:12 | | Quit robin0800 (Ping timeout: 255 seconds) |
| 11:55:14 | Zagor | great, I'll cut and paste a bit to see if I can make what I want |
| 11:55:56 | mcuelMobile2 | great |
| 11:56:38 | mcuelMobile2 | I guess the hardest problem is to fix the fade on pause |
| 11:56:48 | | Quit mcuelMobile (Ping timeout: 245 seconds) |
| 11:57:28 | Zagor | mcuelMobile2: no, that's easy. we do that using pcm volume only, not affecting the stream. |
| 11:58:05 | Zagor | or, "easy". :-) it needs to make sense in the code too. |
| 11:59:05 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 11:59:40 | mcuelMobile2 | but how will the set_volume function which volume to set? |
| 11:59:55 | mcuelMobile2 | know* |
| 12:00 |
| 12:00:21 | Zagor | the idea is to use pcm attenuation as small steps between the bigger stream volume steps |
| 12:00:30 | | Quit JackWinter (Remote host closed the connection) |
| 12:01:15 | Zagor | so the function sometimes sets only pcm, and sometimes both |
| 12:01:56 | mcuelMobile2 | well gtg, will look at it later |
| 12:02:06 | | Quit mcuelMobile2 (Quit: Page closed) |
| 12:03:15 | | Join JackWinter [0] (~jack@vodsl-9173.vo.lu) |
| 12:03:17 | mshathlonxp | can anyone tell me why in rockbox "keyboard" aren't signs of baltic languages - lack of demand? :) |
| 12:03:38 | gevaerts | You know you can change the layout? |
| 12:03:50 | mshathlonxp | not yet ;> |
| 12:03:51 | n1s | mshathlonxp: you can have whatever you want on the keyboard |
| 12:03:57 | gevaerts | It's not possible to make one keyboard layout that suits all languages |
| 12:04:22 | mshathlonxp | well, looks like default suits all, except for russian, asian and baltic :D |
| 12:05:05 | gevaerts | Right. It doesn't suit most of the world |
| 12:05:24 | mshathlonxp | ah, I forgot arabs as well |
| 12:05:37 | n1s | its easy to suite different languages that use mostly the same characters |
| 12:06:07 | mshathlonxp | gevaerts it does |
| 12:06:18 | gevaerts | It does? |
| 12:06:50 | mshathlonxp | chinese is only 1/6 of all world and other unsupported languages even less ;D |
| 12:07:28 | gevaerts | So? |
| 12:07:49 | gevaerts | Are you aware of the fact that Chinese is *not* the only Asian written language? |
| 12:08:41 | | Join HaimN [0] (~HaimN@95.86.98.36) |
| 12:25:12 | | Quit timccc (Read error: Connection reset by peer) |
| 12:25:31 | | Join timccc [0] (~timccc@112.166.15.141) |
| 12:26:58 | pixelma | hmm, I just found two MP3s that don't show the title tag on my Ondio even though it is there (it uses the filename fallback I have coded in my WPS). The difference to other working tracks is that they also have those itunesgapless tags set which the MAS probably can't make use of anyway. I'll try if removing these helps |
| 12:31:45 | pixelma | it does |
| 12:31:51 | *** | Saving seen data "./dancer.seen" |
| 12:33:50 | n1s | pixelma: sounds like a bug still |
| 12:34:09 | pixelma | yes |
| 12:34:14 | n1s | did this happen before the semi-recent changes to id3 code? |
| 12:35:25 | | Quit pamaury (Remote host closed the connection) |
| 12:39:54 | pixelma | can't tell for sure, I don't think I noticed before and I only recently started keeping those tags, I don't think I have many of such files |
| 12:40:50 | pixelma | and I don't use all my targets regularly |
| 12:43:46 | pixelma | it seems though that the grouping tag support now suddenly started working which I think it didn't before on hwcodec. Some weirdness going on with the different tagging fields |
| 12:48:58 | mshathlonxp | gevaerts well, the others don't have that much speakers :) |
| 13:00 |
| 13:02:32 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 13:07:48 | n1s | mshathlonxp: how about hindi? |
| 13:09:51 | pixelma | it's getting a bit offtopic-y I think ;) |
| 13:10:48 | n1s | yes |
| 13:15:29 | | Quit user890104 () |
| 13:28:16 | | Join MethoS- [0] (~clemens@134.102.106.250) |
| 13:31:43 | mshathlonxp | n1s I dunno how hindi is being written, however I know that second official language in india is english anyway :> |
| 13:32:45 | n1s | mshathlonxp: please take this to -community if you want to discuss it further |
| 13:35:22 | HaimN | Hi, i need some help, my Sansa Fuze not read my SD card, the original OS read it bun rockbox not, its happened It only happens on one card, another SD card worked fine for me |
| 13:35:27 | HaimN | thanks! |
| 13:40:11 | HaimN | i found a patch that correct it http://www.rockbox.org/tracker/task/11870 but I don't know how to work with patches (I know compile scorce code but I don't know how put patches) |
| 13:41:17 | HaimN | I'm sorry for my bad english |
| 13:44:34 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
| 13:47:22 | kugel | JdGordon|: why do you edit someone else's comment instead of just adding one? |
| 13:48:40 | kugel | (FS #12005) |
| 13:49:12 | | Quit mshathlonxp (Ping timeout: 276 seconds) |
| 13:50:21 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 13:50:53 | | Join Horscht [0] (~Horscht@p5DD577BC.dip.t-dialin.net) |
| 13:50:53 | | Quit Horscht (Changing host) |
| 13:50:53 | | Join Horscht [0] (~Horscht@xbmc/user/horscht) |
| 13:53:47 | pixelma | not the first time he did that, it was one of mine years ago and the bug report lost quite some info through the editing in my case |
| 13:57:27 | kugel | I don't understand the reason to do that |
| 13:57:51 | pixelma | me neither |
| 13:58:21 | kugel | it even requires more clicks |
| 13:59:00 | pixelma | and is misleading - FS #8998 was mine |
| 14:00 |
| 14:02:33 | sideral | HaimN: http://www.rockbox.org/wiki/WorkingWithPatches |
| 14:03:53 | HaimN | sideral: I tried to work with it, but I got an error |
| 14:04:28 | | Quit antil33t (Read error: Connection reset by peer) |
| 14:04:38 | | Join antil33t [0] (antil33t@124-197-51-80.callplus.net.nz) |
| 14:04:43 | sideral | what's the error message? (use pastebin.com for copy&paste) |
| 14:06:05 | HaimN | http://pastebin.com/KcQwZt5H |
| 14:06:42 | | Quit mshathlonxp (Read error: Connection timed out) |
| 14:07:27 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 14:08:38 | sideral | "hunk failed" that means (a part of) the patch doesn't apply to your source tree. possibly the source tree has evolved since the patch was cut. |
| 14:09:15 | sideral | which of the patches did you want to apply? you should only need sd-init-fix.patch |
| 14:10:04 | HaimN | sideral: I tried only sd-init-fix.patch |
| 14:10:32 | sideral | and which version of the source code do you use? |
| 14:11:09 | HaimN | revision 29559 |
| 14:12:06 | sideral | The patch applies perfectly for me to this revision |
| 14:12:38 | sideral | maybe a DOS CRLF issue? http://www.rockbox.org/wiki/WorkingWithPatches#Newlines |
| 14:13:23 | HaimN | I guess I do not have installed correctly the patch |
| 14:13:35 | HaimN | i work in ubuntu |
| 14:15:14 | sideral | try downloading the patch again. Or apply the change manually; the patch is rather trivial |
| 14:16:57 | sideral | wget -O sd-init-fix.patch "http://www.rockbox.org/tracker/task/11870?getfile=23201" |
| 14:17:57 | | Join milk_ [0] (~milk@94-193-93-226.zone7.bethere.co.uk) |
| 14:18:02 | HaimN | How can I download it? When I click it it opens it in a browser, (what I did before was to copy the text file with the same name) |
| 14:18:17 | HaimN | OK, thanks! |
| 14:18:22 | | Quit Robdgreat (Ping timeout: 246 seconds) |
| 14:18:58 | sideral | Browsers typically have a "Save as..." function that saves the page source verbatim |
| 14:20:07 | HaimN | its save it as a HTML file |
| 14:20:23 | | Join Robdgreat [0] (~rob@unaffiliated/robdgreat) |
| 14:21:13 | sideral | then either your browser is broken, or you're using an evil web proxy |
| 14:21:28 | HaimN | Where should I put the patch? |
| 14:22:27 | sideral | where you want. you only need it once, for the patch command |
| 14:23:38 | HaimN | sideral: This is probably my ISP, I have content filtering (like porn sites or violent) |
| 14:25:06 | gevaerts | This patch isn't very violent |
| 14:25:19 | sideral | did the wget command manage to download the patch in plain text? |
| 14:25:47 | sideral | gevaerts: sure it is! have you ever been "deselected"? |
| 14:26:01 | Zagor | yay, this works pretty good |
| 14:26:17 | gevaerts | sideral: never after a 100ms nap :) |
| 14:26:29 | | Quit einhirn (Read error: Connection reset by peer) |
| 14:26:43 | sideral | gevaerts: that sounds quite cruel too |
| 14:29:37 | | Join Highlander [0] (~Highlande@mek33-4-82-236-45-205.fbx.proxad.net) |
| 14:30:24 | HaimN | sideral: yes, but i'm still get the error massege |
| 14:31:53 | *** | Saving seen data "./dancer.seen" |
| 14:32:02 | sideral | HaimN: Try doing the change the patch wants to apply by hand (find the right line, insert "udelay(100000);") |
| 14:32:17 | HaimN | ok |
| 14:36:16 | HaimN | I should delete these rows?: |
| 14:36:16 | HaimN | http://pastebin.com/7qDqcxLH |
| 14:36:44 | HaimN | or just add udelay(100000); |
| 14:36:46 | | Join user890104 [0] (~Venci@6bez10.info) |
| 14:38:51 | gevaerts | Just add the udelay |
| 14:40:46 | HaimN | ok, thanks! now I compile it |
| 14:47:05 | sideral | HaimN: if you want to read up on the patch file format, check this out: http://en.wikipedia.org/wiki/Diff#Unified_format |
| 14:47:48 | | Join TheLemonMan [0] (~lem0n@ppp-143-150.98-62.inwind.it) |
| 14:50:04 | HaimN | sideral: thanks |
| 14:56:07 | HaimN | its work! Thank you all, (Especially you sideral) |
| 14:56:41 | sideral | No worries, and thanks for trying Rockbox! |
| 14:58:29 | HaimN | I'm using Rockbox for almost a year and I enjoyed every minute, programmer already Rockbox on 30 more players of my friends! |
| 14:59:27 | HaimN | *I installed Rockbox already on another 30 musicians of my friends! |
| 14:59:45 | | Join dfkt [0] (dfkt@unaffiliated/dfkt) |
| 15:00 |
| 15:00:38 | sideral | OK, then thanks for spreading the gospel! :) |
| 15:01:13 | HaimN | :) |
| 15:02:35 | | Quit Highlander (Quit: Quitte) |
| 15:07:23 | | Quit krazykit (Ping timeout: 246 seconds) |
| 15:07:35 | | Join kevku [0] (~kevku@2001:7d0:0:f9af:feed:feed:feed:feed) |
| 15:09:39 | | Join krazykit [0] (~krazykit@99-126-205-52.lightspeed.cicril.sbcglobal.net) |
| 15:10:27 | | Join sirrozha [0] (~sirrozha@89.23.217.205) |
| 15:11:25 | | Quit sirrozha (Remote host closed the connection) |
| 15:13:10 | | Quit TheLemonMan (Quit: Destructor called) |
| 15:14:03 | | Join giovannizilli [0] (~5d25fc3c@giant.haxx.se) |
| 15:14:14 | | Quit antil33t (Read error: Connection reset by peer) |
| 15:14:22 | | Join antil33t [0] (antil33t@124-197-51-80.callplus.net.nz) |
| 15:15:30 | giovannizilli | I'm trying to use the convttf tool to convert anti alised fonts, but got some mitakes |
| 15:16:17 | giovannizilli | cd /rockbox/tools; make; and get an error |
| 15:16:18 | giovannizilli | mkzenboot.c:29: fatal error: zlib.h: File o directory non esistente compilation terminated. make: *** [mkzenboot.o] Errore 1 |
| 15:17:28 | gevaerts | Try make convttf |
| 15:17:56 | giovannizilli | convttf.c:21: fatal error: ft2build.h: File o directory non esistente compilation terminated. make: *** [convttf] Errore 1 |
| 15:18:23 | gevaerts | Install the freetype development package |
| 15:18:58 | | Join benedikt93 [0] (~benedikt9@p5B0C4AC5.dip.t-dialin.net) |
| 15:19:02 | | Quit benedikt93 (Changing host) |
| 15:19:02 | | Join benedikt93 [0] (~benedikt9@unaffiliated/benedikt93) |
| 15:19:59 | | Quit simonrvn (Quit: see ya - n'multes) |
| 15:20:00 | giovannizilli | freetype6-dev? |
| 15:20:26 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 15:22:05 | giovannizilli | ok, done. Thank you very much. |
| 15:22:20 | | Join robin0800 [0] (~robin0800@cpc2-brig8-0-0-cust964.3-3.cable.virginmedia.com) |
| 15:24:16 | robin0800 | just uploaded a new MaximumArt Theme for the Sansa C200 could someone please delete the old one? many thanks |
| 15:24:27 | | Quit krazykit (Ping timeout: 252 seconds) |
| 15:26:26 | | Join krazykit [0] (~krazykit@207-229-161-95.c3-0.frg-ubr1.chi-frg.il.cable.rcn.com) |
| 15:28:02 | | Quit giovannizilli (Quit: CGI:IRC) |
| 15:28:43 | gevaerts | robin0800: done |
| 15:30:16 | | Join facelessloser [0] (~5ad6cd24@giant.haxx.se) |
| 15:30:54 | facelessloser | could someone help me out, i need to install an older version of rockbox |
| 15:34:44 | robin0800 | gevaerts, thanks again .....managed to use %?if(%sC, <=, 35)<> as suggested last night by JdGordon |
| 15:38:02 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 15:38:41 | | Join panni_ [0] (hannes@ip-178-203-73-7.unitymediagroup.de) |
| 15:39:15 | | Quit panni_ (Read error: Connection reset by peer) |
| 15:39:31 | | Join panni_ [0] (hannes@ip-178-203-73-7.unitymediagroup.de) |
| 15:41:49 | | Quit facelessloser (Quit: CGI:IRC (EOF)) |
| 15:53:10 | | Join mcuelenaere [0] (~mcuelenae@rockbox/developer/mcuelenaere) |
| 15:53:51 | mcuelenaere | has anyone run RB Android on a (recent version of) an emulator succesfully lately? |
| 15:58:29 | Zagor | good news: the code works. getting the algorithm right is a bit of a challenge though. |
| 15:59:28 | Zagor | i.e. how much should I lower the pcm volume when I bump the stream volume a step? |
| 15:59:41 | | Part LinusN |
| 15:59:56 | dionoea | mcuelenaere: the audio output doesn't always work (I get it working like 10% of the time) |
| 16:00 |
| 16:00:02 | dionoea | is that the problem you were having ? |
| 16:00:49 | mcuelenaere | no, some crash in native code |
| 16:00:50 | dionoea | kugel had a pcm driver rewrite patch which I was supposed to try... maybe it fixes it |
| 16:01:07 | mcuelenaere | but I was trying to pinpoint whether that was due to local changes or a stricter NDK |
| 16:01:40 | dionoea | ah hum ... never had that (but then I didn't try it in the last 2 weeks on an emulator) |
| 16:02:02 | mcuelenaere | dionoea: I did have trouble with the audio driver in the past though (emulator and my old G1) |
| 16:02:08 | mcuelenaere | so a rewrite sounds nice :) |
| 16:03:43 | Zagor | I have nice algorithms for "too little" and "too much" but not "just right" :) |
| 16:04:23 | dionoea | mcuelenaere: http://pastie.org/1606043 |
| 16:05:31 | mcuelenaere | dionoea: thanks, will look at it later |
| 16:05:37 | kugel | Zagor: how much steps does the stream have via code? |
| 16:05:50 | | Quit HaimN (Ping timeout: 255 seconds) |
| 16:06:45 | kugel | mcuelenaere: audio in the emulator just stopped working for me at some point (during early development) |
| 16:06:58 | kugel | I suspected it was a qemu issue since there's no problem on my phone |
| 16:07:18 | | Quit [Saint] (Ping timeout: 240 seconds) |
| 16:07:24 | kugel | but when it worked it was so utterly slow that it wasn't useful anyway |
| 16:07:32 | Zagor | kugel: stream is 0 to getStreamMaxVolume(). which is 15 on my phone and 8 on yours. |
| 16:07:41 | mcuelenaere | Zagor: how about aligning it to the nearest 990/getStreamMaxVolume()? |
| 16:07:41 | Zagor | pcm is, as you know, 0 to 1. |
| 16:07:51 | mcuelenaere | kugel: did you look at logcat? |
| 16:07:57 | mcuelenaere | (at that point in time) |
| 16:08:12 | kugel | can't remember |
| 16:08:13 | Zagor | mcuelenaere: how do you mean? |
| 16:09:18 | mcuelenaere | I remember having some thread issues in the audio driver (e.g. invoking methods loaded from Java thread 1 on Java thread 2) |
| 16:09:21 | mcuelenaere | @kugel |
| 16:09:37 | mcuelenaere | Zagor: that was RE your how much should I lower the PCM volume when I bump the stream volume a step |
| 16:10:22 | Zagor | yes. but I don't understand your suggestion. getStreamMaxVolume() is a constant. |
| 16:11:01 | mcuelenaere | Rockbox PCM volume is -990..0, so going to the nearest 990/16=61 (getStreamMaxVolume on some devices) should fix that, shouldn't it? |
| 16:11:50 | kugel | we want more than 16 steps |
| 16:11:52 | Zagor | no, the problem is I have to control two different volume controls: the music stream volume and the pcm volume. the stream volume is 0-15 (on my phone) and the pcm volume is 0.0 to 1.0. |
| 16:12:07 | | Quit Barahir_ (Read error: Operation timed out) |
| 16:12:13 | mcuelenaere | so if you're on -500, and you bump up the stream volume you'd go to -~488 |
| 16:12:33 | mcuelenaere | hmm ok, different problem |
| 16:13:07 | Zagor | when I go from -930 to -920 I go from stream volume 1 to 2. then I also lower the pcm volume to make it a smooth step. |
| 16:13:22 | | Join [Saint] [0] (S_a_i_n_t@203.184.0.69) |
| 16:13:44 | Zagor | the challenge is getting the audible difference the same when going from -930 to -920 as when going -920 to -910. |
| 16:14:23 | mcuelenaere | can't you leave the PCM volume at 1 and only touch the stream volume? (I'm guessing I'm not familiar with the full problem) |
| 16:14:47 | Zagor | mcuelenaere: I can. then we get the standard android low-resolution volume. |
| 16:15:13 | Zagor | the goal of the excercise is to add high resolution volume control by also using the pcm volume. |
| 16:15:38 | Zagor | because the difference between 1 and 2 in stream volume is really big |
| 16:15:38 | mcuelenaere | ah yes, I somehow thought Rockbox did some additional volume mixing |
| 16:15:44 | mcuelenaere | ok |
| 16:16:22 | | Join Barahir [0] (~jonathan@frnk-4d0082fc.pool.mediaWays.net) |
| 16:16:23 | Zagor | on android we leave that to the AudioTrack class |
| 16:19:12 | mcuelenaere | do you know the relation between PCM volume & stream volume, e.g. does PCM=1.0, stream=1 equal to PCM=0.5, stream=2? |
| 16:19:17 | | Quit sideral (Ping timeout: 252 seconds) |
| 16:19:18 | | Quit antil33t (Read error: Connection reset by peer) |
| 16:19:30 | | Join antil33t [0] (antil33t@124-197-51-80.callplus.net.nz) |
| 16:19:54 | Zagor | mcuelenaere: my testing indiciates it is unfortunately not that simple, no |
| 16:20:12 | | Quit mystica555_ (Read error: Connection reset by peer) |
| 16:20:17 | mcuelenaere | and I fear that it won't be the same across all devices.. |
| 16:20:30 | Zagor | probably not |
| 16:22:41 | | Quit tails___ (Ping timeout: 250 seconds) |
| 16:22:57 | Zagor | one workaround is to only add high resolution volume to stream volume 1. that's where it is most needed, and fully deterministic. |
| 16:24:17 | mcuelenaere | but then the user can't change the volume the same way when he/she is out of the Rockbox activity |
| 16:25:59 | Zagor | rockbox can listen to volume changes and remove the pcm attenuation when the user bumps volume from outside rockbox |
| 16:26:20 | Zagor | but no, pcm volume can only be controlled from rockbox |
| 16:26:53 | | Join u42p [0] (~v35b@d001237.adsl.hansenet.de) |
| 16:27:14 | Zagor | well actually if rockbox listens to the volume changes, it can still do pcm volume adjustments |
| 16:27:15 | mcuelenaere | kugel: ever got "JNI WARNING: ref 0x... should not be visible to native code"? |
| 16:27:44 | kugel | never seen that one |
| 16:27:46 | Zagor | obviously, the pcm attenuation only applies to rockbox but that will always be the case |
| 16:27:59 | mcuelenaere | right |
| 16:28:38 | | Quit panni_ (Read error: Connection reset by peer) |
| 16:28:39 | mcuelenaere | anyone willing to test an Android-related patch? (just to eliminate my environment as a problem) |
| 16:29:47 | | Join LinusN [0] (~linus@rockbox/developer/LinusN) |
| 16:29:51 | Zagor | I can do it later tonight. I have to go soon. |
| 16:31:36 | | Part LinusN |
| 16:31:55 | *** | Saving seen data "./dancer.seen" |
| 16:32:44 | | Join tails_ [0] (~tails@host-94-251-111-208.bbcustomer.zsttk.net) |
| 16:32:54 | mcuelenaere | right, seems like the emulator has somehow gone stricter (or it hasn't been tested for some time) |
| 16:33:06 | mshathlonxp | n1s btw how can I change keyboard layout? |
| 16:33:40 | * | mcuelenaere really wants native backtraces on Android |
| 16:36:04 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 16:41:20 | pixelma | mshathlonxp: read on, follow links... http://download.rockbox.org/daily/manual/rockbox-ipodnano1g/rockbox-buildch4.html#x7-430004.1.3 |
| 16:45:28 | CIA-2 | New commit by mcuelenaere (r29560): Android port: generate initial config when none is present with dynamically ... |
| 16:45:38 | CIA-2 | New commit by mcuelenaere (r29561): Android: update project to SDKv11 |
| 16:45:42 | CIA-2 | New commit by mcuelenaere (r29562): Android: future-proof the RunForegroundManager code to Honeycomb |
| 16:45:48 | CIA-2 | New commit by mcuelenaere (r29563): RaaA: move Android apps-code to separate dir under apps/hosted |
| 16:45:54 | CIA-2 | New commit by mcuelenaere (r29564): Android: remove rockbox.apk when doing 'make clean' |
| 16:46:01 | CIA-2 | New commit by mcuelenaere (r29565): Android: replace installApk.sh "launching Rockbox activity"-behaviour with ... |
| 16:46:06 | CIA-2 | New commit by mcuelenaere (r29566): Android: get rid of warnings (no functional changes) |
| 16:46:48 | Zagor | whammo! :) |
| 16:47:03 | dionoea | mcuelenaere: are you runing honeycomb in the emulator ? |
| 16:47:18 | mcuelenaere | nope, my poor laptop can't handle it :) |
| 16:47:58 | mcuelenaere | but this code was tested on Android 2.2.1 (and based on reference code from the SDK) |
| 16:48:13 | dionoea | the crashes you get are on a froyo emulator then ? |
| 16:48:41 | mcuelenaere | currently, a 1.6 one |
| 16:48:45 | mcuelenaere | (donut?) |
| 16:49:05 | CIA-2 | r29560 build result: All green |
| 16:49:08 | dionoea | ah. I was runing 2.2 last time i tired |
| 16:49:13 | linuxstb | mcuelenaere: Shouldn't you use appextra instead of this? http://svn.rockbox.org/viewvc.cgi/trunk/apps/apps.make?r1=29562;r2=29563;pathrev=29563 |
| 16:49:13 | mcuelenaere | but I read somewhere that the emulator has some checking enabled not in release builds |
| 16:49:29 | linuxstb | mcuelenaere: (I removed that code recently...) |
| 16:49:30 | * | mcuelenaere greps appextra |
| 16:49:49 | pixelma | for what it's worth - I also get the occasional "audio stops working" on my phone. Considering that I haven't listened for a long time and just do a bit of testing, it happens quite often |
| 16:50:26 | linuxstb | mcuelenaere: It's used in apps.make just about that code you added. It's defined in tools/configure for each target |
| 16:50:28 | mcuelenaere | linuxstb: this was based on some "old" patches from FS #11914 |
| 16:50:32 | scorche | B4gder: Zagor: ping |
| 16:50:41 | kugel | mcuelenaere: what changed with honeycomb w.r.t. foreground running? |
| 16:50:45 | linuxstb | mcuelenaere: Then surely they conflicted? |
| 16:50:50 | mcuelenaere | kugel: it removed setForeground |
| 16:51:00 | Zagor | scorche: yes? |
| 16:51:11 | mcuelenaere | linuxstb: it did, but I didn't see anything appextra related |
| 16:51:16 | mcuelenaere | linuxstb: will change it now |
| 16:51:27 | kugel | mcuelenaere: that's removed long ago, isn't it? |
| 16:51:32 | scorche | Zagor: I am getting ready to submit the GSoC org app - have you had a look and can give it your final blessing? |
| 16:51:43 | kugel | mcuelenaere: IIRC 2.2 already removed it, and we handled that already |
| 16:51:44 | * | Zagor takes a look |
| 16:52:04 | kugel | ah, wait I understand. you mean the function is removed entirely (instead of being a no-op)? |
| 16:52:06 | mcuelenaere | kugel: perhaps it deprecated it, but in Honeycomb the actual call is removed |
| 16:52:07 | mcuelenaere | yes |
| 16:52:39 | CIA-2 | r29565 build result: All green |
| 16:52:57 | kugel | mcuelenaere: why didn't you just change the other api. your change seems larger than needed |
| 16:54:05 | linuxstb | mcuelenaere: A question unreleated to your commits today - why the #ifdef APPLICATION around target/hosted/android/app/button-application.c in firmware/SOURCES ? |
| 16:54:55 | mcuelenaere | kugel: hmm I didn't really look at it much, due to setForeground not being available anymore I just looked at the Android reference and found this code and went with it (after fixing it) |
| 16:55:37 | mcuelenaere | linuxstb: hmm did I add that? |
| 16:55:48 | linuxstb | svn blame says you did ;) |
| 16:56:25 | mcuelenaere | git blame says kugel ;) |
| 16:56:47 | linuxstb | It was this commit, no? |
| 16:56:49 | linuxstb | http://svn.rockbox.org/viewvc.cgi?view=rev;revision=27832 |
| 16:57:13 | scorche | also, if you havent yet and have interest, look over http://www.rockbox.org/wiki/GSoCOrgApp2011 http://www.rockbox.org/wiki/GSoCApplicationTemplate2011 and http://www.rockbox.org/wiki/SummerOfCode2011 |
| 16:57:34 | scorche | (also, join #rockbox-gsoc |
| 16:57:53 | mcuelenaere | linuxstb: hmm I have this http://svn.rockbox.org/viewvc.cgi?view=rev;revision=27310 |
| 16:58:25 | * | kugel sees no #ifdef APPLICATION in either commit |
| 16:58:38 | mcuelenaere | ah sorry, wrong grep |
| 16:59:05 | * | kugel wonders if people look at the revisions they post here :p |
| 16:59:09 | mcuelenaere | linuxstb: I guess I based myself on the SDL port? |
| 16:59:14 | mcuelenaere | kugel: I did :) |
| 16:59:33 | linuxstb | kugel: You can't see it in the link I posted? |
| 16:59:38 | mcuelenaere | kugel: http://svn.rockbox.org/viewvc.cgi/trunk/firmware/SOURCES?r1=27832&r2=27831&pathrev=27832 and http://svn.rockbox.org/viewvc.cgi/trunk/firmware/SOURCES?r1=27310&r2=27309&pathrev=27310 |
| 17:00 |
| 17:00:04 | | Part u42p ("Leaving") |
| 17:00:23 | kugel | linuxstb: I can't see it |
| 17:01:07 | pixelma | it's very clear from linuxstb's link |
| 17:01:07 | kugel | ah you talked about SOURCES |
| 17:01:10 | mcuelenaere | linuxstb: so do I add hosted/android to APPEXTRA now or ..? (doesn't seem very clean) |
| 17:02:47 | | Quit Farthen (Ping timeout: 240 seconds) |
| 17:03:06 | linuxstb | mcuelenaere: That's what the APPEXTRA mechanism is for - extra, target-specific subdirs in apps/. IMO it's cleaner than putting target-specific #ifs in apps/Makefile |
| 17:03:11 | linuxstb | I mean apps/apps.make |
| 17:03:46 | linuxstb | mcuelenaere: But going back to firmware/SOURCES, it does just look like a copy/paste from the SDL code (which IIUC is used for both the sim and SDL apps), so it's needed there, but not for Android. |
| 17:03:59 | | Join Farthen [0] (~Farthen@static.225.178.40.188.clients.your-server.de) |
| 17:04:03 | mcuelenaere | linuxstb: so the Makefile for Android should containt APPEXTRA=recorder:gui:radio:hosted:hosted/android? |
| 17:04:10 | mcuelenaere | contain* |
| 17:04:18 | linuxstb | mcuelenaere: No need for ":hosted:" any more I think. |
| 17:04:19 | mcuelenaere | linuxstb: yea, probably |
| 17:04:24 | mcuelenaere | @SOURCES |
| 17:04:32 | linuxstb | (unless you left something there) |
| 17:04:41 | mcuelenaere | currently, no |
| 17:05:31 | scorche | last call before i submit the application... |
| 17:05:53 | mcuelenaere | linuxstb: you're sure hosted isn't needed anywhere else? |
| 17:06:16 | mcuelenaere | oh it's only in apps/ ok |
| 17:07:12 | | Part Zagor |
| 17:07:59 | mcuelenaere | kugel: RE foreground, you want me to revert to the old way and do it that way? |
| 17:08:43 | CIA-2 | New commit by mcuelenaere (r29567): Android: use APPEXTRA instead of makefile hack |
| 17:11:50 | mshathlonxp | or maybe http://www.rockbox.org/wiki/LoadableKeyboardLayouts ? :) |
| 17:12:47 | CIA-2 | r29567 build result: All green |
| 17:12:56 | kugel | mcuelenaere: I don't feel strong, but I guess I made the two separate interfaces for exactly this case |
| 17:14:49 | scorche | "Organization Application submitted." |
| 17:16:52 | linuxstb | scorche: When are the successful ones announced? |
| 17:17:04 | scorche | http://www.google-melange.com/document/show/gsoc_program/google/gsoc2011/timeline |
| 17:17:35 | * | linuxstb wanted a date, not a link ;) |
| 17:18:36 | linuxstb | Next Friday,18th March at 19;00 UTC |
| 17:18:42 | scorche | the date is in the link! ;) |
| 17:22:29 | | Quit Judas_PhD (Quit: This is a quitting message) |
| 17:25:38 | | Quit mshathlonxp (Ping timeout: 276 seconds) |
| 17:26:30 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 17:28:05 | | Quit timccc (Ping timeout: 250 seconds) |
| 17:28:13 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
| 17:28:55 | | Quit Judas_PhD (Client Quit) |
| 17:30:18 | | Quit [Saint] (Quit: I'm only going to Heaven if it feels like Hell, I'm only going to Heaven if it tastes like caramel...) |
| 17:30:47 | | Quit tails_ (Ping timeout: 252 seconds) |
| 17:31:43 | | Join tails_ [0] (~tails@host-94-251-111-208.bbcustomer.zsttk.net) |
| 17:36:22 | | Join [Saint] [0] (S_a_i_n_t@203.184.0.12) |
| 17:40:39 | | Quit [Saint] (Ping timeout: 250 seconds) |
| 17:46:31 | | Join [Saint] [0] (S_a_i_n_t@203.184.0.12) |
| 17:47:55 | | Join sideral [0] (~sideral@213.165.85.248) |
| 17:47:55 | | Quit sideral (Changing host) |
| 17:47:55 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 17:49:37 | | Join bmbl [0] (~bmbl@dsl-217-160-5.pool.bitel.net) |
| 17:49:37 | | Quit bmbl (Changing host) |
| 17:49:37 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
| 17:50:56 | | Join tails__ [0] (~tails@host-94-251-111-208.bbcustomer.zsttk.net) |
| 17:52:08 | | Quit jhMikeS (Ping timeout: 240 seconds) |
| 17:53:44 | * | gevaerts looks for people who know a bit about playlists and bookmarks to finish those gsoc ideas. JdGordon, pondlife, Llorean: you were most active on the mailing list about this |
| 17:54:29 | gevaerts | kugel: can you write a bit about the Android native widgets thing? |
| 17:55:18 | kugel | shouldn't people that actually want that do it? |
| 17:55:32 | | Quit tails_ (Ping timeout: 276 seconds) |
| 17:55:32 | gevaerts | possibly |
| 17:55:51 | kugel | I don't need/want native widgets so I haven't even thought about it any further. but others keep popping that up |
| 17:55:58 | gevaerts | ok |
| 17:56:42 | kugel | iirc linuxstb is a strong advocate of it |
| 17:57:21 | gevaerts | Can you write up something else then? ;) |
| 17:57:31 | * | gevaerts is always hopeful! |
| 17:59:39 | | Join stoffel [0] (~quassel@p57B4B067.dip.t-dialin.net) |
| 18:00 |
| 18:04:36 | | Join toffe82_ [0] (~chatzilla@maf.wirelesstcp.net) |
| 18:05:29 | | Quit mshathlonxp (Quit: Leaving) |
| 18:08:06 | | Quit sideral (Quit: Leaving.) |
| 18:09:26 | | Join sideral [0] (~sideral@rockbox/developer/sideral) |
| 18:09:52 | | Join komputes [0] (~komputes@ubuntu/member/komputes) |
| 18:20:33 | | Quit sideral (Ping timeout: 246 seconds) |
| 18:25:51 | | Quit literal (Quit: leaving) |
| 18:25:57 | | Join literal [0] (hinrik@w.nix.is) |
| 18:26:31 | | Quit n1s (Quit: Lämnar) |
| 18:29:46 | | Quit Xerion (Quit: ) |
| 18:31:58 | *** | Saving seen data "./dancer.seen" |
| 18:36:29 | | Join Xerion [0] (~xerion@5419A4D7.cm-5-2c.dynamic.ziggo.nl) |
| 18:43:00 | | Quit sasquatch (Quit: WeeChat 0.3.2) |
| 18:43:24 | | Join sasquatch [0] (~username@p4FF2CDED.dip.t-dialin.net) |
| 18:50:48 | | Join LambdaCalculus37 [0] (~3f74f70d@rockbox/staff/LambdaCalculus37) |
| 18:52:35 | | Quit pamaury (Read error: Operation timed out) |
| 18:55:57 | * | B4gder added himself to the gsoc page |
| 18:57:45 | rasher | Not that it matters much, but shouldn't the third tier here: |
| 18:57:55 | rasher | http://www.rockbox.org/wiki/GSoCOrgApp2011#What_is_your_plan_for_dealing_with_disappearing_mentors_63 be the other mentors? |
| 19:00 |
| 19:00:26 | | Join {phoenix} [0] (~dirk@p57AA31CF.dip.t-dialin.net) |
| 19:03:16 | | Join kronflux [0] (~kronflux@blk-89-250-30.eastlink.ca) |
| 19:06:54 | | Join leavittx [0] (~lev@89.221.199.187) |
| 19:11:35 | | Join bertrik [0] (~bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 19:11:35 | | Quit bertrik (Changing host) |
| 19:11:35 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
| 19:27:26 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 19:29:44 | | Quit LambdaCalculus37 (Quit: CGI:IRC 0.5.9 (2006/06/06)) |
| 19:32:26 | | Quit user890104 () |
| 19:34:35 | CIA-2 | New commit by mcuelenaere (r29568): Android: use NewGlobalRef for references that are used globally |
| 19:34:40 | CIA-2 | New commit by mcuelenaere (r29569): Android: Don't share the JNI environment across threads, but obtain it the ... |
| 19:36:23 | mcuelenaere | (r29568 fixes the emulator) |
| 19:38:40 | CIA-2 | r29568 build result: All green |
| 19:40:22 | kugel | mcuelenaere: huh? |
| 19:40:36 | kugel | the environment was not shared acress threads. that wouldn't work anyway |
| 19:40:55 | kugel | I think your commit is not needed |
| 19:41:19 | | Join Stummi [0] (~Stummi@rockbox/developer/Stummi) |
| 19:41:26 | mcuelenaere | kugel: I had some warnings that there was an access from the wrong thread to an jobject/jclass/whatever a long time ago |
| 19:41:33 | mcuelenaere | this should fix that |
| 19:41:40 | kugel | "a long time ago"? |
| 19:42:06 | CIA-2 | r29569 build result: All green |
| 19:42:09 | mcuelenaere | yes, can't remember when but it was using the emulator and when the CPU was very busy |
| 19:42:16 | mcuelenaere | + playing audio |
| 19:42:19 | kugel | the other commit is also completely unneeded |
| 19:42:24 | mcuelenaere | no it's not |
| 19:42:30 | kugel | you don't need global refs |
| 19:42:31 | mcuelenaere | that fixes the emulator |
| 19:42:49 | mcuelenaere | kugel: http://android.git.kernel.org/?p=platform/dalvik.git;a=blob_plain;f=docs/jni-tips.html;hb=HEAD#local_vs_global_references |
| 19:43:00 | kugel | I read that yes |
| 19:43:25 | kugel | I explicitely avoided global refs so that the VM can do it's cleanup when main() returns |
| 19:43:26 | mcuelenaere | kugel: more precisely, making RockboxFramebuffer_instance a global ref fixed it |
| 19:43:57 | kugel | ah right, that one was probably buggy |
| 19:44:00 | kugel | but the others not |
| 19:44:49 | kugel | objects created in native code are GC'd when the entry point function returns. you add unneeded complexity |
| 19:45:21 | kugel | mcuelenaere: I wonder why you put up rather trivial patches on FS but this ones not |
| 19:45:40 | mcuelenaere | true, I had forgotten that main() doesn't return |
| 19:45:55 | mcuelenaere | kugel: I've made these two just today |
| 19:46:04 | mcuelenaere | the others were from the FS item |
| 19:46:07 | kugel | IMO both are unneeded |
| 19:46:35 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
| 19:46:46 | mcuelenaere | perhaps so, but the problem with RockboxFramebuffer_instance remains |
| 19:46:51 | kugel | I have actually work that lets main return (to shut down the service), in which case normal references would be much more convinient |
| 19:48:00 | kugel | right, the RockboxFramebuffer_instance is different because it's not created by native code since recently so it may need other handling (albeit its reference shouldn't change anyway) |
| 19:49:06 | | Quit mshathlonxp (Read error: Connection timed out) |
| 19:49:51 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 19:53:50 | | Quit MethoS- (Read error: Connection reset by peer) |
| 19:54:15 | | Quit kronflux (Quit: Leaving) |
| 19:55:55 | | Quit B4gder (Read error: Operation timed out) |
| 19:56:59 | kugel | mcuelenaere: I think I'd rather see those reverted and fix the issues in another way |
| 19:57:16 | kugel | (until now I wasn't even aware there are issues) |
| 19:57:16 | mcuelenaere | kugel: I'm working on partly reverting r29568 |
| 19:57:33 | mcuelenaere | I don't think r2959 is wrong |
| 19:57:40 | mcuelenaere | 29569* |
| 19:57:55 | kugel | can you use the env pointer from the Surface callbacks instead of the global one? |
| 19:58:08 | | Quit kkit|sh (Read error: Operation timed out) |
| 19:58:27 | kugel | I don't say r29569 is wrong, it's just unneeded |
| 19:59:47 | mcuelenaere | well, even apart from the functional change, it does clean up the files a bit, there's no need for the external env_ptr anymore |
| 19:59:47 | | Join B4gder [0] (~daniel@rockbox/developer/bagder) |
| 20:00 |
| 20:01:55 | mcuelenaere | kugel: doesn't it already use the env pointer from the Surface callbacks? |
| 20:02:00 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 20:02:13 | mcuelenaere | and I don't see how that's connected to the global reference problem |
| 20:02:26 | kugel | not for lcd_update/_rect |
| 20:05:00 | kugel | I thought you need to use the env of the calling thread, but perhaps you need the one of the thread that created the object |
| 20:05:20 | kugel | or the same which you used for GetMethodID |
| 20:05:49 | mcuelenaere | you need the env associated with the current thread |
| 20:06:51 | | Quit pamaury (Ping timeout: 252 seconds) |
| 20:07:17 | mcuelenaere | kugel: http://pastebin.com/XTkXm56q is the revert, and doesn't revert the fix |
| 20:07:56 | | Join Barahir_ [0] (~jonathan@frnk-590fd3fb.pool.mediaWays.net) |
| 20:08:04 | kugel | mcuelenaere: so so everything should've been alright before your commits |
| 20:08:28 | mcuelenaere | no, when extended JNI checks are enabled it triggers that JNI warning I mentioned earlier |
| 20:08:31 | mcuelenaere | (which is on emulators) |
| 20:08:48 | | Join user890104 [0] (~Venci@6bez10.info) |
| 20:09:59 | | Quit Barahir (Read error: Operation timed out) |
| 20:10:03 | mcuelenaere | (this is all about r29568) |
| 20:10:04 | kugel | but can you tell which access was wrong? |
| 20:10:40 | mcuelenaere | when making RockboxFramebuffer_instance a global ref, it is fixed so probably in lcd_update(_rect) |
| 20:10:57 | kugel | but they use the env of the current (calling) thread |
| 20:11:32 | kugel | I suspect it wants the same env used for GetMethodID (or the warning is just wrong) |
| 20:13:03 | mcuelenaere | kugel: this is what I get when I don't make it global: http://pastebin.com/R30YV012 |
| 20:13:36 | mcuelenaere | method ID's can be shared across environments I think |
| 20:13:50 | mcuelenaere | it's only jobject and it derivates that can be free'd |
| 20:14:19 | mcuelenaere | "Note: method and field IDs are just 32-bit identifiers, not object references, and should not be passed to NewGlobalRef." |
| 20:14:32 | kugel | okay |
| 20:14:39 | mcuelenaere | oops, mixing up separate things |
| 20:14:58 | kugel | the activity creates the framebuffer, from the UI thread. does that help any? |
| 20:16:02 | mcuelenaere | I don't think the warning is related to referencing it from the wrong thread, it's probably because we shouldn't dereference a local reference |
| 20:17:23 | mcuelenaere | the RB fb instance is passed to surfaceCreated as a local reference, thus when that method returns it could potentially be freed |
| 20:17:35 | kugel | doesn't local/global reference just mean something about object lifetime? |
| 20:18:10 | kugel | before the surface is freed SurfaceDestroyed will be called |
| 20:18:40 | mcuelenaere | sure, but that doesn't have anything to do with the rules wrt local/global reference |
| 20:19:26 | kugel | the reference is valid until it's garbage collected |
| 20:19:38 | kugel | global reference just menas "don't garbage collec this" |
| 20:19:45 | mcuelenaere | no, it's valid until the function returns |
| 20:20:17 | mcuelenaere | "Every object that JNI returns is a "local reference". This means that it's valid for the duration of the current native method in the current thread." |
| 20:20:18 | | Quit casainho (Quit: ChatZilla 0.9.86 [Firefox 3.6.15/20110303171539]) |
| 20:20:29 | mcuelenaere | "Even if the object itself continues to live on after the native method returns, the reference is not valid." |
| 20:20:42 | mcuelenaere | "(Dalvik VM will warn you about most reference mis-uses when extended JNI checks are enabled.)" |
| 20:20:51 | kugel | ah, interesting. I missed that sentence |
| 20:20:52 | mcuelenaere | which I think is enabled on emulators |
| 20:21:15 | kugel | so yes, we need a global reference for the fb instance |
| 20:23:03 | CIA-2 | New commit by mcuelenaere (r29570): Partly revert "Android: use NewGlobalRef for references that are used globally" ... |
| 20:23:32 | | Join enthdegree [0] (~enthdegre@cpe-174-099-058-029.nc.res.rr.com) |
| 20:23:46 | mcuelenaere | ugh, forgot one part |
| 20:25:11 | CIA-2 | New commit by mcuelenaere (r29571): Android: no need to keep RockboxPCM_class around |
| 20:25:18 | kugel | mcuelenaere: still not entirely happy with lcd-android.c |
| 20:26:05 | kugel | I think you can unconditionally create the global reference in SurfaceCreated and unconditionally delete it in SurfaceDestryoed (without need for lcd_deinit()), can't you? |
| 20:26:20 | CIA-2 | r29570 build result: All green |
| 20:26:54 | mcuelenaere | hmm yes, probably, that could make lcd_deinit go away |
| 20:29:28 | CIA-2 | r29571 build result: All green |
| 20:31:59 | *** | Saving seen data "./dancer.seen" |
| 20:35:40 | | Quit stoffel (Remote host closed the connection) |
| 20:37:32 | | Quit kevku (Quit: KVIrc 4.0.2 Insomnia http://www.kvirc.net/) |
| 20:37:41 | | Join stoffel [0] (~quassel@p57B4B067.dip.t-dialin.net) |
| 20:38:33 | CIA-2 | New commit by mcuelenaere (r29572): Android: delete the global reference in surfaceDestroyed ... |
| 20:41:47 | CIA-2 | r29572 build result: All green |
| 20:42:08 | | Quit mshathlonxp (Quit: reb00t) |
| 20:45:23 | | Join mtstickney [0] (~mts@cpe-67-253-94-131.maine.res.rr.com) |
| 20:45:35 | | Join msh_ [0] (~msh@5acba089.bb.sky.com) |
| 20:45:41 | | Nick msh_ is now known as mshathlonxp (~msh@5acba089.bb.sky.com) |
| 20:48:34 | kugel | mcuelenaere: I think you also want to create the global ref uncontitionally, otherwise you possibly delete a non-existent ref? |
| 20:49:43 | | Quit mc2739 (Ping timeout: 248 seconds) |
| 20:50:05 | mcuelenaere | kugel: when's that? the only time RockboxFramebuffer_instance gets set is by a NewGlobalRef() call (and when it's released, it's set to NULL) |
| 20:50:48 | kugel | ah ok |
| 20:51:06 | kugel | I still find conditional "alloc" and unconditional "free" very confusing |
| 20:51:32 | | Quit simonrvn (Quit: see ya - n'multes) |
| 20:51:56 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
| 20:52:05 | mcuelenaere | well, I'm not sure what would happen when we globally reference something more than once, but doing it conditionnaly seems cleaner |
| 20:52:09 | mcuelenaere | conditionally* |
| 20:52:27 | | Join simonrvn [0] (simon@2001:470:8c85:11fe::c0a8:195) |
| 20:52:38 | mcuelenaere | it's basically: if(cur != new) cur = new; |
| 20:54:28 | | Join Zagor [0] (~bjst@rockbox/developer/Zagor) |
| 20:59:30 | mtstickney | is there a way to bypass an initial filesystem check in rockbox? |
| 21:00 |
| 21:00:28 | | Join gbl08ma [0] (~gbl08ma@195-23-182-162.net.novis.pt) |
| 21:00:30 | | Quit {phoenix} (Remote host closed the connection) |
| 21:00:32 | linuxstb | mtstickney: What do you mean? What initial filesystem check? |
| 21:00:37 | Zagor | mcuelenaere, kugel: what's the easiest way to get a debug printout/dialog/whatever in android? |
| 21:00:50 | mcuelenaere | Zagor: you know of logcat? |
| 21:00:57 | Zagor | no |
| 21:01:01 | mcuelenaere | enable logf, and sprinkle your code with logf codes |
| 21:01:09 | mcuelenaere | ah, do you use the emulator or an actual device? |
| 21:01:14 | Zagor | actual device |
| 21:01:17 | mtstickney | linuxstb: I'm assuming that's what the "scanning disk" message refers to |
| 21:01:36 | mcuelenaere | Zagor: enable adb on the device and just run "adb logcat" on your pc |
| 21:01:50 | mtstickney | linuxstb: the trouble is I've got a corrupt fs, but I get a panic during the scan, so I can't repair the fs with another computer |
| 21:01:51 | mcuelenaere | and don't forget to connect your device to your pc :) |
| 21:02:33 | mcuelenaere | enabling adb can be done in settings->apps->development->USB-debug (loosely translated from dutch) |
| 21:02:43 | | Quit enthdegree (Ping timeout: 276 seconds) |
| 21:04:05 | kugel | mcuelenaere, Zagor: DEBUGF/debugf works as well (in debug builds) |
| 21:04:06 | mcuelenaere | hmm, pausing and pressing play again makes the audio stop on my SGS |
| 21:04:19 | Zagor | and what do I call from java? |
| 21:04:25 | linuxstb | mtstickney: OK, that's the Rockbox "directory cache" (dircache) feature. Which device are you running Rockbox on? |
| 21:04:25 | | Quit leavittx (Ping timeout: 252 seconds) |
| 21:04:29 | mcuelenaere | kugel: is debugf available all the time? (without adding LOGF_ENABLE) |
| 21:04:36 | mcuelenaere | Zagor: Log.d("Rockbox", "...."); |
| 21:04:45 | mcuelenaere | or Log.w if you want warnings etc |
| 21:04:45 | kugel | LOG (if it's there, but that's just a wrapper for Log.d("Rockbox", ...) |
| 21:05:00 | kugel | mcuelenaere: in debug builds |
| 21:05:06 | mcuelenaere | ok |
| 21:05:11 | mtstickney | linuxstb: it's a sansa e200v2 |
| 21:05:19 | kugel | but yea, without adding LOGF_ENABLE |
| 21:07:34 | mcuelenaere | does Rockbox already handle .nomedia? |
| 21:07:52 | kugel | no, should we? |
| 21:08:11 | mcuelenaere | I think that's the Android equivalent to .nodb (or whatever that is on Rockbox) |
| 21:08:27 | saratoga | you can disable dircache, but its probably easier to just fix your file system |
| 21:08:32 | kugel | yes, but for the android's database |
| 21:08:46 | kugel | and it includes pictures |
| 21:09:03 | mtstickney | saratoga: I can't do either, because the initial dircache scan crashes rockbox (so it doesn't function as a drive when plugged in via usb) |
| 21:09:03 | kugel | I actually have .nomedia in my rockbox folder so that the photo viewers don't pick up the album art |
| 21:09:33 | saratoga | mtstickney: you'd fix the file system from USB mode, not rockbox, we don't support USB on your player yet |
| 21:09:40 | mcuelenaere | well some apps seem to create some .nomedia files, so why not also ignore those folders upon scanning? |
| 21:09:45 | gevaerts | saratoga: yes we do |
| 21:10:01 | gevaerts | mtstickney: boot to the original firmware, and fix it from there |
| 21:10:15 | saratoga | do we? |
| 21:10:41 | kugel | yes |
| 21:10:55 | saratoga | ah yes, i'm mixing up my AS chips again |
| 21:12:12 | saratoga | for some reason i read e200v2 as as3525v2 |
| 21:12:13 | mtstickney | gevaerts: good call, I forgot about the OF's usb mode |
| 21:12:38 | linuxstb | But that sounds like a bug to me - Rockbox should be able to enter USB mode even if the disk is corrupted. |
| 21:13:16 | gevaerts | linuxstb: I'd say we need bootloader USB for this sort of thing |
| 21:14:05 | saratoga | or failing that at least go to USB mode before doing DB and disk caching if you boot up with it inserted |
| 21:14:43 | linuxstb | gevaerts: Yes, I guess that makes more sense on most targets. |
| 21:14:54 | gevaerts | saratoga: I'm not against that, but *I*'m not touching that code :) |
| 21:15:25 | saratoga | yeah i'm not going near that one either |
| 21:15:32 | saratoga | USB bootloader is probably a lot easier |
| 21:15:36 | gevaerts | yes :) |
| 21:15:57 | kugel | mcuelenaere: http://pastie.org/1660887 this way the code makes more sense to me (as a reader) |
| 21:15:59 | gevaerts | I guess we'd hide bootloader USB behind a button though |
| 21:16:17 | saratoga | on most targets don't we always use it if theres a USB cable connected? |
| 21:16:21 | saratoga | my gigabeat F does I think |
| 21:16:29 | gevaerts | Yes, and that's *annoying* |
| 21:16:32 | saratoga | unless thats the OF USB mode with a similar font |
| 21:16:52 | gevaerts | Is your gigabeat flashed? |
| 21:16:57 | saratoga | no stock |
| 21:17:03 | gevaerts | Then it's the OF |
| 21:17:11 | gevaerts | hm |
| 21:17:15 | gevaerts | Actually, I'm not sure |
| 21:17:22 | saratoga | it looks like sysfont IIRC |
| 21:17:53 | gevaerts | Bootloader USB tends to break car adapter mode |
| 21:17:56 | saratoga | ideally it would boot back into rockbox after USB disconect so you didn't realize the bootloader handled the transfer |
| 21:18:13 | saratoga | on AMS we can actually tell the difference between a charger and a USB cable |
| 21:18:30 | gevaerts | Only if you wait |
| 21:18:59 | Zagor | yes, got it! |
| 21:20:27 | Zagor | it actually is linear, my code was just wrong |
| 21:21:17 | gevaerts | saratoga: the detection is done by waiting for USB bus activity. I guess the bootloader could wait a second or so (I don't know what the best time is) if it sees power |
| 21:21:49 | saratoga | that seems sensible |
| 21:21:59 | saratoga | if USB host isn't detected, just keep booting, rather then crash |
| 21:22:09 | Topic | "Rockbox 3.8 is released | Please read before speaking: http://www.rockbox.org/wiki/IrcGuidelines | Please direct offtopic/social chat to #rockbox-community | This channel is logged at http://www.rockbox.org/irc | An upgrade caused issues with the wiki. We are on the case | Please finish GSoC ideas from -dev and put them on http://www.rockbox.org/wiki/SummerOfCode2011" by ChanServ (ChanServ@services.) |
| 21:22:34 | gevaerts | I'd still add a button. Some hosts may be slow, and we don't want to wait for a minute |
| 21:22:40 | | Join pamaury [0] (~quassel@vit94-1-82-67-248-70.fbx.proxad.net) |
| 21:22:40 | | Quit pamaury (Changing host) |
| 21:22:40 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
| 21:23:20 | gevaerts | That only works for software stack targets of course |
| 21:26:28 | mcuelenaere | kugel: it is it guaranteed that for every surfaceCreated call, there's an equivalent surfaceDestroyed call? |
| 21:26:39 | mcuelenaere | if so, then it would simplify it yes |
| 21:26:40 | kugel | yes |
| 21:26:47 | mcuelenaere | then, commit away :) |
| 21:26:59 | kugel | with various SurfaceChanged calls inbetween |
| 21:27:27 | mcuelenaere | but we don't handle that in native code, do we? |
| 21:27:57 | mcuelenaere | oh, it isn't even handled in java code I see |
| 21:28:34 | mcuelenaere | perhaps that should emulator surfacedestroyed+surfacecreated? |
| 21:28:41 | mcuelenaere | if the size changes |
| 21:29:00 | mcuelenaere | but I don't see what would trigger that at runtime |
| 21:33:20 | kugel | I couldn't parse that |
| 21:40:23 | CIA-2 | New commit by zagor (r29573): Use the pcm volume to add finer steps between every android stream volume step. |
| 21:42:03 | | Quit esperegu (Remote host closed the connection) |
| 21:42:20 | Zagor | kugel: when you have a minute, I would be interested to see how/if that works with your 9-step music stream |
| 21:43:43 | kugel | Zagor: I'll have a look |
| 21:43:51 | CIA-2 | r29573 build result: All green |
| 21:44:36 | Zagor | now we need to add volume event listening so we adapt to changes done outside rockbox |
| 21:45:09 | kugel | didn't mcuelenaere patch have that? |
| 21:46:33 | Zagor | yes |
| 21:46:57 | | Quit factor (Read error: Connection reset by peer) |
| 21:49:14 | bluebrother | Zagor: your volume button changes for Android have an ugly side effect: the volume steps of the buttons are different depending on if Rockbox UI is showing or not |
| 21:49:35 | Zagor | side effect? that is the whole point! |
| 21:50:10 | bluebrother | seriously? I find it strange to have the buttons behave differently depending on if Rockbox is showing or not (or even the screen is locked) |
| 21:50:11 | Zagor | inside rockbox, you have higher volume resolution than android has |
| 21:50:51 | Zagor | well, the only "solution" to that is to disable the high-resolution volume control |
| 21:51:21 | bluebrother | sure, but this makes the button behaviour inconsistend. |
| 21:51:40 | Zagor | yes. I suggest we make it optional. |
| 21:52:18 | bluebrother | sounds like an option :) |
| 21:52:56 | AlexP | I don't like the idea of an option, can we have an option to show the option? |
| 21:53:07 | Zagor | yes! |
| 21:53:12 | AlexP | excellent! |
| 21:53:22 | Zagor | configurable menus |
| 21:53:26 | B4gder | we can have the app guess if you want to see the option! |
| 21:53:50 | AlexP | Can I enable curmudgeonly old bugger mode? :) |
| 21:54:07 | AlexP | Configurable menus done well would be nice |
| 21:54:15 | | Quit stoffel (Remote host closed the connection) |
| 21:54:19 | * | gevaerts wants someone to implement an option to increase interest in gsoc |
| 21:54:35 | AlexP | gevaerts: To harass people you really have to say something :) |
| 21:54:55 | gevaerts | AlexP: I said things, and I changed the topic! |
| 21:55:00 | AlexP | And if there is something that you feel I can contribute to, then do prod :) |
| 21:55:11 | AlexP | I didn't notice either :) |
| 21:55:20 | kugel | I actually think in-app vs out-of-app is understandable by the user, but perhaps screen-on vs screen-off not |
| 21:55:30 | gevaerts | And that shouldn't even have been necessary, there have been several calls on the mailing list |
| 21:55:35 | kugel | but it's probably less confusing than 2 entirely separate levels |
| 21:55:40 | AlexP | gevaerts: indeed there have |
| 21:56:27 | Zagor | kugel: is screen-off the same as out-of-app in this case? |
| 21:56:38 | kugel | I expect so, I haven't tried yet |
| 21:56:41 | * | AlexP is looking forward to a bit of an android bashing-things-out session at devcon |
| 21:56:58 | kugel | but IIRC screen-off is handled by the lockscreen app (if that exists) |
| 21:57:00 | bluebrother | it at least behaves so :) |
| 21:57:02 | B4gder | june 3rd is looking more and more as the date now |
| 21:57:07 | AlexP | indeed |
| 21:57:13 | gevaerts | AlexP: can we do the bashing without Android? |
| 21:57:18 | bluebrother | yeps |
| 21:57:22 | Zagor | kugel: yes, tested now. biiig volume steps :-) |
| 21:57:43 | AlexP | B4gder: I'd say it is the 3rd even |
| 21:58:00 | AlexP | It'd take a late run of 3 or 4 people to even think about changing it |
| 21:58:11 | B4gder | yes, unlikely to happen |
| 21:58:26 | AlexP | Which is a bugger - holiday weekend (everywhere except UK) = more expensive eurostar |
| 21:58:41 | AlexP | But we shall cope! |
| 21:58:47 | B4gder | I'm sure the fund can sponsor |
| 21:58:51 | AlexP | heh :) |
| 21:59:00 | gevaerts | Can the fund buy a train? :) |
| 21:59:17 | bluebrother | personally I would prefer the 17th. But I guess doesn't make much of a difference given the numbers :) |
| 21:59:31 | bluebrother | gevaerts: Rockboxers on a train? |
| 21:59:51 | AlexP | gevaerts: We could mount a hostile take over of Deutsche Bahn - they arwe bidding for a eurostar service |
| 21:59:56 | kugel | I would really prefer 3rd june since (as I mentioned) my internship starts mid- or end-june |
| 22:00 |
| 22:00:07 | * | gevaerts considers the idea of building a Tower on a train |
| 22:00:10 | AlexP | bluebrother: can you make the 3rd? |
| 22:00:19 | kugel | I don't want to take a free friday early on |
| 22:00:26 | bluebrother | AlexP: yep, it's just a bit inconvenient for me |
| 22:00:31 | AlexP | ah, OK |
| 22:00:33 | bluebrother | but I'll manage that :) |
| 22:00:50 | AlexP | cool :) |
| 22:01:16 | bluebrother | I just need to figure how I want to handle extending my stay. If I'm around I really have to do that :) |
| 22:04:43 | | Join DJ_PUNK2011 [0] (~skinheadb@94-43-112-72.dsl.utg.ge) |
| 22:04:56 | | Join factor [0] (~factor@75.108.68.114) |
| 22:05:54 | DJ_PUNK2011 | Hello |
| 22:06:05 | DJ_PUNK2011 | Does anybody know the band name of this music or maybe song name |
| 22:06:05 | DJ_PUNK2011 | http://www.youtube.com/watch?v=aG2cUjEzYJI&playnext=1&list=PLE2C2280B0170FF35 |
| 22:06:20 | | Join Mir_ppc [0] (opera@114.sub-69-99-69.myvzw.com) |
| 22:06:28 | Mir_ppc | hello |
| 22:06:39 | bertrik | I haven't planned ahead to june, I'll probably be able to make nearly any date |
| 22:06:49 | Mir_ppc | i just installed roxkbox on a Clip (v1) 2gb |
| 22:06:50 | bluebrother | DJ_PUNK2011: the "This video is not available in your country" band? |
| 22:06:53 | Mir_ppc | its crashed |
| 22:07:06 | Mir_ppc | this is the error message |
| 22:07:06 | Mir_ppc | Mir_ppcPrefetchabort at 60000016 |
| 22:07:07 | Mir_ppc | Mir_ppcfsr 0x8 |
| 22:07:07 | DBUG | Enqueued KICK Mir_ppc |
| 22:07:07 | Mir_ppc | Mir_ppcdomain 0, fault 8) |
| 22:07:12 | DJ_PUNK2011 | Hey u guys |
| 22:07:14 | DJ_PUNK2011 | how are you |
| 22:07:28 | DJ_PUNK2011 | I have a question 2 you maybe you know this link |
| 22:07:30 | AlexP | DJ_PUNK2011: Please stay on topic |
| 22:07:31 | DJ_PUNK2011 | I found a music |
| 22:07:32 | bluebrother | DJ_PUNK2011: this is a technical channel. |
| 22:07:42 | DJ_PUNK2011 | sorry |
| 22:07:54 | DJ_PUNK2011 | blue brother just need to write a question which might know you |
| 22:08:08 | AlexP | DJ_PUNK2011: If it is not to do with Rockbox, then don't |
| 22:08:12 | DJ_PUNK2011 | does anybody from you know the singer or a band of this music? Please find youtube link below: |
| 22:08:12 | DJ_PUNK2011 | http://www.youtube.com/watch?v=aG2cUjEzYJI&playnext=1&list=PLE2C2280B0170FF35 |
| 22:08:18 | AlexP | DJ_PUNK2011: Stop it |
| 22:08:49 | | Quit ej0rge (Read error: Operation timed out) |
| 22:08:52 | DJ_PUNK2011 | ok I stopped |
| 22:09:03 | DJ_PUNK2011 | just wrote a question |
| 22:09:22 | AlexP | This channel is for support and development of Rockbox only |
| 22:09:27 | AlexP | Not random quetions |
| 22:09:34 | DJ_PUNK2011 | a I c |
| 22:09:38 | DJ_PUNK2011 | Sorry for that Alex |
| 22:09:50 | DJ_PUNK2011 | I thought this is just normal chat conversations |
| 22:09:53 | Mode | "#rockbox +o B4gder" by ChanServ (ChanServ@services.) |
| 22:11:57 | Zagor | mcuelenaere: which API is internal in your fs#11914 patch8? |
| 22:12:43 | | Quit DJ_PUNK2011 (Quit: DJ_PUNK2011) |
| 22:13:20 | Mir_ppc | bbiab |
| 22:14:24 | | Join kevku [0] (~kevku@2001:7d0:0:f9af:feed:feed:feed:feed) |
| 22:15:18 | Mode | "#rockbox -o B4gder" by ChanServ (ChanServ@services.) |
| 22:17:55 | mcuelenaere | Zagor: android.media.VOLUME_CHANGED_ACTION |
| 22:18:16 | mcuelenaere | it isn't documented, but you can find more about it in AOSP |
| 22:19:05 | | Part mtstickney |
| 22:21:23 | Zagor | ok. do you want to rework and commit that patch, or do you want me to? |
| 22:21:41 | mcuelenaere | Zagor: RE r29573, the RockboxService variable was made final because that's a requirement when accessing it from an anonymous class, so strictly speaking it isn't needed in your case |
| 22:22:50 | Zagor | ok. my java fu is very limited. |
| 22:23:49 | mcuelenaere | yeah, I'm not sure why it's needed either, Eclipse just suggested that :) |
| 22:24:22 | | Join ej0rge [0] (~alhaz@alhaz.fttp.xmission.com) |
| 22:27:44 | kugel | Zagor: android's volume popup is gone :( |
| 22:31:55 | Zagor | yes, disabled since we use our own volume display. |
| 22:32:01 | *** | Saving seen data "./dancer.seen" |
| 22:32:12 | kugel | can we enable it? |
| 22:32:48 | Zagor | of course we can, but it doesn't show the actual volume. only the low-res android steps. |
| 22:32:55 | Zagor | so it is rather misleading |
| 22:35:10 | kugel | I think I would still like it |
| 22:35:41 | Zagor | I think it only makes sense if you disable high resolution volume |
| 22:36:35 | Zagor | otherwise the popup will show the same "value" several times, indicating volume adjustment is broken |
| 22:36:45 | kugel | I realize that |
| 22:45:41 | kugel | but I don't think it'd be perceived as borken, but just as "different" |
| 22:46:07 | Zagor | I disagree. the user presses the button repeatedly, sees no effect and hears no effect. ergo, broken. |
| 22:46:20 | kugel | he hears the effect? |
| 22:47:14 | Zagor | no. the steps are so small it is very easy to miss one or two. especially if the user expects the normal big steps. |
| 22:48:23 | kugel | we could have larger steps anyway |
| 22:48:28 | kugel | 100 are a bit too much |
| 22:48:39 | saratoga | how many does android have |
| 22:49:02 | Zagor | saratoga: different for different models. nexus one has 16 steps |
| 22:49:04 | kugel | mcuelenaere: I'm still not a fan of r29569 |
| 22:49:20 | saratoga | wow not very many |
| 22:49:38 | Zagor | no. kugels has as few as 9. |
| 22:49:46 | Zagor | hence the need for this fix |
| 22:49:59 | saratoga | why does it vary so widely by device? |
| 22:50:06 | Zagor | I don't know |
| 22:50:09 | AlexP | mine has 8 I think |
| 22:50:15 | kugel | Zagor: anyway it seems to work on my phone |
| 22:50:22 | | Join bimbel [0] (~bmbl@unaffiliated/bmbl) |
| 22:50:22 | Zagor | kugel: great! |
| 22:50:23 | kugel | steps are smooth as far as i can tell |
| 22:50:36 | | Quit bmbl (Ping timeout: 241 seconds) |
| 22:50:50 | AlexP | Oh, no mine has 16 |
| 22:50:58 | AlexP | 8 is the ringer volume :) |
| 22:51:08 | Zagor | right, it is different for different stream types |
| 22:51:21 | saratoga | huh mine has 15 as well |
| 22:51:50 | kugel | wait, it seems the volume goes lower for a fraction of second when pressing the key |
| 22:51:59 | mcuelenaere | kugel: it could be reverted, but then we need to ensure that we don't yield when a native function is called with a different JNI environment (which isn't guaranteed in the PCM code IIRC) |
| 22:53:13 | kugel | PCM uses it's own env in the callback |
| 22:53:13 | saratoga | FWIW i don't see the graphic not updating ever step as that big a deal, i'm sure i've seen various devices that do that, and of course a lot of rockbox skins that use a volume icon |
| 22:53:13 | kugel | what do you mean by yield? |
| 22:53:13 | saratoga | although making sure the steps were 1.5 to 2dB would be nice |
| 22:53:13 | saratoga | so you can hear them easily |
| 22:53:13 | kugel | a rockbox yield? all rockbox threads have the same env because it's within 1 pthread |
| 22:54:08 | Zagor | c200 has 80 steps |
| 22:54:18 | kugel | saratoga: on cabbie it was never a problem that the graphic doesn't change with every step |
| 22:54:36 | mcuelenaere | kugel: yes, a rockbox yield |
| 22:54:59 | kugel | I don't understand why it is a problem that the android display doesn't change, while cabbie also doesn't update |
| 22:55:08 | kugel | cabbie volume display has even less states I think |
| 22:55:26 | Zagor | kugel: because users expect the volume widget to change when the press the button. it always does. |
| 22:55:58 | AlexP | We really have to start to think a bit differently for touchscreen vs standard devices IMO |
| 22:56:02 | Zagor | with rockbox-specific things, nobody has any expectations and we can do more whatever we like. |
| 22:56:26 | kugel | mcuelenaere: a rockbox yield is safe |
| 22:56:28 | Zagor | with host OS things, we cannot be too confusing |
| 22:57:04 | Ctcp | Ignored 1 channel CTCP requests in 0 seconds at the last flood |
| 22:57:04 | * | mcuelenaere tries to remember |
| 22:57:13 | mcuelenaere | kugel: does the PCM java code run in a separate (p)thread? |
| 22:57:24 | kugel | only the callback |
| 22:57:27 | mcuelenaere | ok |
| 22:58:01 | mcuelenaere | IIRC the problem I had that "long time ago", was that in the callback there was a yield() somehow, which triggered an lcd_update() in a different rockbox thread |
| 22:58:23 | mcuelenaere | and as the JNI environment in the LCD subsystem belongs to a different thread, it crashed |
| 22:58:28 | mcuelenaere | or it gave a warning at least |
| 22:58:40 | kugel | I thought I ensured the env handling is done safely, but I didn't add comments regarding that (my mistake). and if there's something I overlooked please point me at it |
| 22:59:06 | kugel | in that long time the lcd subsystem was rewritten several times :) |
| 22:59:36 | kugel | we know update from within the rockbox threads, early we posted to the UI thread |
| 23:00 |
| 23:00:03 | Zagor | heh, ak4537 has 128 steps |
| 23:00:29 | kugel | I'd be happy with just 50 steps, I mostly only use the top 20 steps anyway |
| 23:00:46 | Zagor | yeah 50 is probably enough |
| 23:01:00 | Zagor | configurable! ;-) |
| 23:01:03 | | Quit Stummi (Quit: Bye!) |
| 23:01:24 | kugel | and we should either make it a real db scale or change the volume display in rockbox to be percentage based |
| 23:01:33 | kugel | -99 to 0 is missleading too |
| 23:01:53 | Zagor | we don't have db information on android |
| 23:02:10 | | Join Buschel [0] (~chatzilla@p54A3ADE2.dip.t-dialin.net) |
| 23:02:20 | Zagor | I don't think it's a big problem though. it is attenuation, the unit is not terribly important |
| 23:03:16 | Buschel | JdGordon|: did you see FS #11997 ? It seems this is connected to some WPS stuff when displaying album art |
| 23:03:39 | mcuelenaere | kugel: http://www.rockbox.org/irc/rockbox-20101106.txt at 00.57.30 and following |
| 23:03:45 | mcuelenaere | http://pastebin.com/raw.php?i=Jza2XBS4 |
| 23:04:13 | kugel | that can't be :) |
| 23:04:14 | mcuelenaere | long time ago ~= 5 monhts ;) |
| 23:04:18 | mcuelenaere | months* |
| 23:04:28 | kugel | I think I talked about this with someone else recently |
| 23:04:57 | kugel | it should be possible to sleep() from the callback. that would even deadlock on target |
| 23:05:13 | kugel | because it's most often called from a DMA isr |
| 23:05:41 | kugel | it should *not* |
| 23:05:50 | mcuelenaere | I know :) |
| 23:06:00 | | Quit FoH (Quit: process management issues) |
| 23:06:01 | Zagor | surely sleep has an interrupt check? |
| 23:06:29 | mcuelenaere | I couldn't figure out any other way to explain it at that time |
| 23:06:45 | mcuelenaere | problem was that it's hard to trigger |
| 23:07:17 | kugel | it's hard to prove without backtrace but it really would cause weird things to happen on a native taret |
| 23:08:58 | mcuelenaere | I'm currently trying to get native backtracing working :) |
| 23:09:13 | mcuelenaere | s/trying/looking into/ |
| 23:09:34 | kugel | Zagor: no it hasnt |
| 23:10:04 | kugel | there isn't even an easy way to tell if you're in interrupt context |
| 23:11:12 | mcuelenaere | setting a variable to true when entering the interrupt handler? ;) |
| 23:11:22 | Zagor | the kernel knows |
| 23:11:46 | kugel | Zagor: I'm pretty sure it doesn't |
| 23:11:54 | mcuelenaere | Zagor: rockbox kernel? |
| 23:12:05 | kugel | the new semaphores have a semi-check. it's only an indicator though |
| 23:12:11 | Zagor | kugel: uh, yes it does |
| 23:12:20 | kugel | where? |
| 23:12:31 | Zagor | mcuelenaere: linux. I thought you were discussing android? |
| 23:12:46 | mcuelenaere | Zagor: yes, but this is Rockbox internals? |
| 23:13:03 | mcuelenaere | I mean, Rockbox interrupts aren't *real* interrupts on hosted targets |
| 23:13:05 | kugel | we talk about the rockbox kernel and our sleep() |
| 23:13:08 | | Quit gbl08ma (Ping timeout: 246 seconds) |
| 23:16:06 | | Quit mshathlonxp (Quit: Leaving) |
| 23:16:14 | kugel | mcuelenaere: you didn't by chance debug what threadid=10 back then did you? |
| 23:16:17 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 23:16:32 | mcuelenaere | kugel: I'm afraid the IRC logs are all we can rely on |
| 23:16:57 | mcuelenaere | kugel: what do you mean with what threadid=10 is? I think it just means the thread with ID=10 :) |
| 23:17:11 | kugel | yea, but what thread :) |
| 23:17:16 | mcuelenaere | ah no, sorry |
| 23:17:24 | | Join gbl08ma [0] (~gbl08ma@195-23-182-25.net.novis.pt) |
| 23:18:03 | gbl08ma | [this IRC client is driving me nuts] |
| 23:18:39 | kugel | I think our env handling is pretty safe and we shouldn't need to add additional bloat to it |
| 23:20:54 | mcuelenaere | perhaps I could eliminate some getJavaEnvironment() calls, by passing the env around |
| 23:22:12 | * | kugel maintains that getJavaEnvironment() isn't needed at all |
| 23:22:14 | | Quit mshathlonxp (Quit: Leaving) |
| 23:22:20 | kugel | have you actually been able to repro it sense then? |
| 23:22:37 | mcuelenaere | no.. |
| 23:23:25 | mcuelenaere | let me try that now |
| 23:23:32 | kugel | Zagor: do you also hear what I described? |
| 23:24:03 | | Join timccc [0] (~timccc@112.166.15.141) |
| 23:24:07 | Zagor | kugel: a dip in volume when pressing the button? I can't hear anything like that |
| 23:24:16 | kugel | yes |
| 23:24:22 | kugel | it's only for a very short moment |
| 23:24:46 | Zagor | every step, or only some? |
| 23:24:50 | | Join mshathlonxp [0] (~msh@5acba089.bb.sky.com) |
| 23:25:09 | kugel | it goes the other direction as I press actually |
| 23:25:23 | Zagor | is it more audible if you hold the button to repeat it? |
| 23:25:55 | * | kugel starts thinking this could be a CM issue |
| 23:26:12 | Zagor | maybe |
| 23:26:33 | kugel | perhaps there are spurious volume button presses. that |
| 23:27:19 | kugel | that'd explain my 8 steps and the fact that there's always also a volume bump when I use long-vol press to switch tracks |
| 23:28:30 | kugel | Zagor: it's very audible with short presses, not so much with long presses |
| 23:28:54 | kugel | if I press vol down the volume is bumped very shortly (and vice versa with vol up) |
| 23:29:56 | | Quit tails__ (Ping timeout: 246 seconds) |
| 23:30:05 | | Join kronflux [0] (~kronflux@blk-89-250-30.eastlink.ca) |
| 23:30:08 | kugel | it doesn't seem to happen when I choose the volume in the list with the trackball |
| 23:30:09 | kronflux | #harmonymc |
| 23:30:12 | kronflux | oops |
| 23:33:21 | | Quit Buschel (Quit: ChatZilla 0.9.86 [Firefox 3.6.13/20101203075014]) |
| 23:33:58 | | Part toffe82_ |
| 23:35:23 | mcuelenaere | kugel: btw, threadid=10 was the "audio thread": '"audio thread" prio=5 tid=10 NATIVE' (http://pastebin.com/raw.php?i=Jza2XBS4) |
| 23:35:31 | kugel | Zagor: btw, thanks for that fade on stop commit :) |
| 23:35:48 | Zagor | you're welcome! :) |
| 23:36:03 | kugel | mcuelenaere: oh, I thought tid=7 was that one |
| 23:36:28 | mcuelenaere | kugel: I'm not sure, but I think that log is pretty right :) |
| 23:36:58 | kugel | there's only one audio thread, and that calls pcmSamplesToByteArray |
| 23:37:51 | mcuelenaere | wow :) I just played an MP3 in the emulator and it just crashed fully :) |
| 23:38:09 | kugel | that's my last experience with it also from a few month back |
| 23:38:19 | mcuelenaere | haven't had that one before |
| 23:38:37 | kugel | I generally stopped using the emulator because of that |
| 23:41:45 | gbl08ma | what emulator? I have been playing mp3 to take screenshots of my theme ports on the sim of nano1g and it plays without problems (don't know about sound quality, but that's not the goal of the sim) |
| 23:42:01 | mcuelenaere | crap, that's reproducable.. |
| 23:42:28 | Zagor | the volume works nicely on my old 1.6 acer liquid too |
| 23:43:14 | AlexP | gbl08ma: android emulator, not Rockbox simulator |
| 23:43:19 | gbl08ma | ah |
| 23:43:35 | gbl08ma | it's been the whole day developing for android, hasn't it? |
| 23:43:44 | * | mcuelenaere hopes this is due to his Makefile changes |
| 23:44:00 | mcuelenaere | still, QEMU shouldn't crash.. |
| 23:47:07 | mcuelenaere | ugh |
| 23:48:16 | CIA-2 | New commit by kugel (r29574): Create GlobalRef in surfaceCreated, in accordance to r29572. |
| 23:52:24 | CIA-2 | r29574 build result: All green |
| 23:53:37 | | Quit ej0rge (Ping timeout: 260 seconds) |
| 23:54:17 | mcuelenaere | perhaps this is also one of those "extended" JNI checks which are disabled on real targets? |
| 23:56:13 | kugel | perhaps. can you enable them on real? |
| 23:56:32 | mcuelenaere | yes and yes |
| 23:56:40 | mcuelenaere | adb shell setprop dalvik.vm.checkjni true |
| 23:56:42 | mcuelenaere | requires root |
| 23:56:46 | * | mcuelenaere is trying right now |
icon identifies that the person is a core developer (has commit access).