00:01:13 | | Join scorche|sh [50] (n=scorche@rockbox/administrator/scorche) |
00:01:39 | | Quit ender` (" We're not lost. We're locationally challenged. -- John M. Ford") |
00:04:09 | | Quit domonoky (Read error: 104 (Connection reset by peer)) |
00:07:01 | | Quit scorche|sh ("leaving") |
00:09:01 | | Join AndyI [0] (n=pasha_in@212.14.205.32) |
00:12:25 | | Nick Ypsy is now known as YPSY (n=ypsy@87.106.45.183) |
00:21:01 | | Quit AndyIL (Read error: 110 (Connection timed out)) |
00:24:06 | CIA-85 | New commit by rasher (r23374): Copy SDL.dll from the SDL dir which was used, rather than relying on PATH. |
00:27:00 | | Quit esperegu (Read error: 60 (Operation timed out)) |
00:31:57 | | Part linuxstb ("Leaving") |
00:34:15 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
00:36:48 | | Part toffe82 |
00:37:19 | | Quit matsl (Read error: 110 (Connection timed out)) |
00:38:09 | | Join webguest07 [0] (n=ce35323b@giant.haxx.se) |
00:39:37 | | Join webgues [0] (n=ce35323b@giant.haxx.se) |
00:40:47 | CIA-85 | New commit by rasher (r23375): Don't rely on {} expansion since not all shells have that. |
00:42:27 | | Quit JdGordon| ("FUCK!") |
00:44:39 | | Quit webgues ("CGI:IRC (Ping timeout)") |
00:44:39 | | Quit webguest07 ("CGI:IRC (Ping timeout)") |
00:47:26 | | Quit Galois ("Leaving") |
00:51:29 | | Quit stripwax ("http://miranda-im.org") |
00:53:14 | | Quit pcc1 (Read error: 110 (Connection timed out)) |
00:53:39 | | Join Galois [0] (i=djao@efnet.math.uwaterloo.ca) |
00:53:59 | | Join linuxstb [0] (n=linuxstb@rockbox/developer/linuxstb) |
00:54:18 | | Part JackWinter ("Konversation terminated!") |
00:56:27 | | Quit dmb ("Leaving") |
00:57:56 | | Join stripwax [0] (n=Miranda@87.194.34.169) |
01:00 |
01:01:51 | | Quit Zarggg () |
01:03:50 | | Quit StealthyXIIGer (Read error: 104 (Connection reset by peer)) |
01:04:22 | | Quit DerPapst ("Leaving.") |
01:04:40 | | Join StealthyXIIGer [0] (n=stealthy@c-68-62-19-6.hsd1.mi.comcast.net) |
01:06:48 | | Join Zarggg [0] (n=zarggg@65-78-69-194.c3-0.eas-ubr6.atw-eas.pa.cable.rcn.com) |
01:19:15 | | Join elinenbe_ [0] (n=elinenbe@207-237-212-81.c3-0.80w-ubr4.nyr-80w.ny.cable.rcn.com) |
01:19:16 | | Quit elinenbe (Read error: 104 (Connection reset by peer)) |
01:19:18 | | Nick elinenbe_ is now known as elinenbe (n=elinenbe@207-237-212-81.c3-0.80w-ubr4.nyr-80w.ny.cable.rcn.com) |
01:22:27 | | Quit Thundercloud (Remote closed the connection) |
01:23:00 | | Quit kugel (Remote closed the connection) |
01:24:11 | | Quit bertrik (Read error: 113 (No route to host)) |
01:26:59 | | Quit MethoS- (Remote closed the connection) |
01:27:43 | | Quit blithe_ (Read error: 110 (Connection timed out)) |
01:47:22 | | Quit Lynx_ (" HydraIRC -> http://www.hydrairc.com <- *I* use it, so it must be good!") |
01:53:19 | *** | Saving seen data "./dancer.seen" |
01:53:36 | | Quit JdGordon ("Leaving.") |
01:57:10 | | Quit jordan` (Read error: 113 (No route to host)) |
01:58:33 | | Join JdGordon [0] (n=Miranda@c-24-22-210-83.hsd1.wa.comcast.net) |
02:00 |
02:04:42 | | Nick fxb is now known as fxb__ (n=felixbru@85.214.97.64) |
02:06:20 | | Join jordan` [0] (n=jordan@78.235.252.137) |
02:08:24 | | Quit amiconn (Nick collision from services.) |
02:08:25 | | Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn) |
02:08:43 | | Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn) |
02:09:04 | | Join killan_ [0] (n=nnscript@c-0efa70d5.06-397-67626721.cust.bredbandsbolaget.se) |
02:09:15 | | Quit killan (Read error: 104 (Connection reset by peer)) |
02:10:45 | | Quit pixelma (Nick collision from services.) |
02:10:46 | | Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma) |
02:11:06 | | Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma) |
02:18:07 | | Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.") |
02:27:22 | | Quit JdGordon (Read error: 104 (Connection reset by peer)) |
02:33:47 | | Quit elinenbe (Read error: 104 (Connection reset by peer)) |
02:39:56 | | Join fejfighter [0] (n=fejfight@C-61-68-20-44.hay.connect.net.au) |
02:42:09 | | Quit NoGare (Read error: 145 (Connection timed out)) |
02:44:33 | | Join JdGordon [0] (n=Miranda@c-24-22-210-83.hsd1.wa.comcast.net) |
02:45:58 | saratoga | wow someone actually asked about TTA support |
02:46:05 | saratoga | i thought i was the only person who noticed we didn't have it |
02:47:45 | | Quit stripwax (Read error: 145 (Connection timed out)) |
02:48:24 | linuxstb | saratoga: Quick, fix it! |
02:48:36 | saratoga | i have a half working decoder if you want it :) |
02:49:25 | linuxstb | You should post it to flyspray. You never know, someone may be bored and finish it... |
02:49:26 | | Join xDonny [0] (i=xDonny@bas3-kingston08-1178012042.dsl.bell.ca) |
02:49:29 | saratoga | linuxstb: while you are here i wanted to ask about the codec api |
02:49:53 | linuxstb | OK, I'll try. |
02:49:55 | saratoga | for test_codec i want to convert it to use the audio buffer as a simple ring buffer that refills when its empy |
02:50:02 | saratoga | so that the clip and such can use test_codec |
02:50:10 | xDonny | can anybody help me with my ipod nano2g? when i try to play a video it zooms through the video, then it pauses, then plays the video with crappy sound quality |
02:50:22 | saratoga | i'm not sure what that would mean for cur_position though |
02:50:36 | saratoga | can i decrease that safely without confusing codecs or do they depend on it to monotonically increase? |
02:50:47 | | Quit GeekShado_ ("The cake is a lie !") |
02:51:17 | linuxstb | saratoga: I wouldn't bother using it as a ring buffer, just stop when you reach a low watermark (32KB should be OK), memmove the remaining data to the start of the buffer, read some more from the file, then continue. |
02:51:30 | saratoga | linuxstb: thats basically what i mean |
02:51:56 | saratoga | but if i do that it means that when i rebuffer i either have to fake the old position values or else decrease current position so it still points to the start of the file |
02:51:59 | saratoga | err start of the buffer |
02:52:09 | xDonny | linuxstb: would my problem be anything to do with the cpu clocking at a different speed? |
02:52:24 | linuxstb | xDonny: Sorry, I've no idea. I haven't been keeping up with recent Nano2G work. |
02:52:35 | xDonny | oh... |
02:52:40 | xDonny | who could help me? |
02:52:40 | saratoga | probably just a bug in whatever timer mpegplayer uses |
02:53:31 | saratoga | maybe it would be safer to just keep a seperate buffer pointer in test_codec and use it to adjust cur_position |
02:54:57 | linuxstb | saratoga: Hmm, looking at test_codec.c I see what you mean. Yes, I think you'll need to keep a separate pointer storing the offset in the file of the first byte in the buffer. |
02:55:35 | saratoga | ok thats what i was afraid of |
02:55:43 | saratoga | should still be easy but not as trivial a fix as i'd hoped |
02:56:15 | saratoga | linuxstb: why do we have two different functions for getting data from the compressed audio buffer anyway? |
02:56:38 | saratoga | request_buffer and read_filebuf seem to do the same thing in slightly different ways |
02:57:29 | linuxstb | No, they do different things. One copies data and advances the pointer immediately, the other gives you a pointer to the data, with up to 32KB guaranteed to be available. |
02:58:01 | saratoga | but can't one always be used in place of the other with minor changes? |
02:58:02 | linuxstb | The latter is obviously more efficient (and the one I would always use), but the first was implemented to make things easier when porting codecs that used that API. |
02:58:11 | saratoga | ah ok |
02:58:24 | linuxstb | saratoga: Yes, I wouldn't object to the read function being deprecated and removed. |
02:58:29 | xDonny | what are you two working on? |
03:00 |
03:00:27 | saratoga | linuxstb: we seem to use request_buffer more often then not so removing it won't be fun |
03:00:40 | linuxstb | xDonny: "test_codec" is a plugin we use to test how fast Rockbox audio codecs are running. We're talking about that. |
03:00:51 | linuxstb | saratoga: That's the one I would keep... |
03:01:01 | | Quit liar (Remote closed the connection) |
03:01:19 | saratoga | ah |
03:01:25 | linuxstb | saratoga: I see my reply changed the order... |
03:01:49 | linuxstb | By "first" I meant read_filebuf. |
03:02:04 | saratoga | well then how come you used read_filebuf so much in wma.c ? :) |
03:02:16 | | Join midgey [0] (n=tjross@rockbox/developer/midgey) |
03:02:35 | linuxstb | There must have been a reason.... |
03:02:38 | * | linuxstb looks |
03:02:53 | saratoga | is the idea that it lets you easily backtrack if needed? |
03:03:46 | linuxstb | Seems I use it for the packet parsing. The actual data is accessed via a request_buffer() |
03:03:51 | | Quit StealthyXIIGer (Read error: 104 (Connection reset by peer)) |
03:03:59 | | Join StealthyXIIGer [0] (n=stealthy@c-68-62-19-6.hsd1.mi.comcast.net) |
03:04:08 | saratoga | if (ci->read_filebuf(data, datalen) == 0) |
03:04:19 | saratoga | thats a possible place to optimize wma then? |
03:05:21 | saratoga | hmm i have to go, will look into this all later |
03:06:05 | xDonny | alright, hopefully theres a fix for my probelm soon, i hate cracky audio |
03:06:09 | xDonny | but i'm goin to bed |
03:06:11 | | Quit xDonny () |
03:06:25 | | Join Rob2223 [0] (n=Miranda@p4FDCD56B.dip.t-dialin.net) |
03:07:57 | linuxstb | Maybe, but I doubt it's very significant. |
03:08:11 | linuxstb | Everything is just reading 1 byte, apart from the line you quoted, and I think that's just a maximum of 18 bytes (that's the size of data[]) |
03:08:17 | | Quit AlexP ("No Ping reply in 180 seconds.") |
03:08:37 | | Join AlexP [0] (n=alex@rockbox/staff/AlexP) |
03:09:27 | linuxstb | saratoga: But vorbis is guilty of copying everything with a read_filebuf() - so that might be a useful optimisation. |
03:09:59 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
03:10:50 | linuxstb | But that would probably mean writing a new Ogg parser, and discarding the one Tremor includes. |
03:11:02 | * | linuxstb leaves for bed |
03:11:49 | | Quit HBK () |
03:11:59 | | Join HBK [0] (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
03:21:58 | | Quit Rob2222 (Read error: 113 (No route to host)) |
03:25:12 | | Quit JdGordon (Read error: 104 (Connection reset by peer)) |
03:30:51 | | Quit Stephen_ ("Leaving") |
03:35:19 | | Join JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
03:36:15 | | Quit sbhsu (Read error: 104 (Connection reset by peer)) |
03:37:48 | | Join rodan [0] (n=mctalley@c-68-47-253-5.hsd1.tn.comcast.net) |
03:41:11 | rodan | Did sansa purposely build Fuze V2 so that rockbox couldn't be used on it? |
03:42:06 | | Join sbhsu [0] (n=a6530466@192.192.120.197) |
03:43:25 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
03:43:47 | scorche | highly unlikely... |
03:44:18 | | Quit cowgarden (Read error: 104 (Connection reset by peer)) |
03:44:46 | rodan | the difference between v1 and v2 is firmware , but is it firmware from the company or from end user updates? |
03:53:22 | *** | Saving seen data "./dancer.seen" |
03:59:31 | | Join angelwolf71885 [0] (n=chatzill@cpe-173-171-133-36.tampabay.res.rr.com) |
04:00 |
04:00:14 | | Quit angelwolf71885 (Client Quit) |
04:07:49 | | Part rodan |
04:07:57 | | Quit Rondom (Nick collision from services.) |
04:08:08 | | Join Rondom [0] (n=Rondom@dslb-084-057-143-069.pools.arcor-ip.net) |
04:16:36 | | Join evilnick_ [0] (n=evilnick@ool-4571af51.dyn.optonline.net) |
04:16:36 | | Quit evilnick (Read error: 104 (Connection reset by peer)) |
04:34:57 | | Join FOAD_ [0] (n=dok@dinah.blub.net) |
04:43:29 | | Join blithe [0] (n=blithe@blakesmith.me) |
04:47:02 | | Quit adiroiban (Read error: 110 (Connection timed out)) |
04:51:03 | | Quit FOAD (Read error: 110 (Connection timed out)) |
04:51:03 | | Nick FOAD_ is now known as FOAD (n=dok@dinah.blub.net) |
04:58:13 | | Quit BHSPitLappy (Remote closed the connection) |
05:00 |
05:27:31 | | Quit midgey () |
05:32:49 | | Quit panni_ ("( www.nnscript.de :: NoNameScript 3.81 :: www.XLhost.de )") |
05:36:00 | | Quit fejfighter (Remote closed the connection) |
05:46:48 | | Quit Horscht ("Verlassend") |
05:53:24 | *** | Saving seen data "./dancer.seen" |
06:00 |
06:07:51 | CIA-85 | New commit by jdgordon (r23376): First go at converting the inbuilt statusbar into a skin, right now its only perfect for 176pixel wide displays (and not liked to the build system so ... |
06:09:25 | JdGordon | apparently the "volume changing" tag doesnt work in the custom statusbars |
06:16:11 | | Join Llorean [0] (n=DarkkOne@adsl-99-4-146-40.dsl.hstntx.sbcglobal.net) |
06:18:39 | | Join shai [0] (n=Shai@l192-117-110-233.cable.actcom.net.il) |
06:37:42 | JdGordon | anyone know if you can simulate charging in the sim? |
06:39:06 | CIA-85 | New commit by jdgordon (r23377): fix the charging icon so it will actually display correctly |
06:51:23 | | Quit Llorean ("Leaving.") |
06:53:41 | | Join fyre^OS [0] (n=nnscript@cpe-69-203-148-25.si.res.rr.com) |
06:58:30 | | Quit sbhsu (Read error: 145 (Connection timed out)) |
07:00 |
07:06:49 | | Quit StealthyXIIGer (Read error: 104 (Connection reset by peer)) |
07:07:10 | | Join StealthyXIIGer [0] (n=stealthy@c-68-62-19-6.hsd1.mi.comcast.net) |
07:11:30 | | Quit fyrestorm (Read error: 110 (Connection timed out)) |
07:15:35 | | Join LinusN [0] (n=linus@rockbox/developer/LinusN) |
07:20:02 | | Join matsl [0] (n=matsl@91.191.140.126) |
07:21:07 | | Quit fyre^OS (Read error: 60 (Operation timed out)) |
07:29:25 | | Join mt_ [0] (n=mtee@41.233.146.27) |
07:31:45 | | Quit StealthyXIIGer (Read error: 60 (Operation timed out)) |
07:32:33 | JdGordon | would anyone madly object to me fixing the viewport parser in the skin thingy so you could have a negative value for the pixel placement/dimensions? (i.e -8 for height) which would mean X pixels away from the edge.. so -8 height would be LCD_HEIGHT-8? |
07:33:37 | JdGordon | to make sharing skins between different displays simpler |
07:36:12 | | Join sbhsu [0] (n=a6530466@Zion.dorm.au.edu.tw) |
07:42:51 | * | JdGordon is slightly surprised how easy that was actually |
07:43:20 | | Quit mt (Read error: 113 (No route to host)) |
07:44:12 | | Join mt [0] (n=mtee@rockbox/developer/mt) |
07:44:38 | CIA-85 | New commit by jdgordon (r23378): Allow the x and y pixel values of viewports to be a negative number.. ... |
07:49:09 | JdGordon | to go along with that commit... it would be nice if we used "u" for unsigned ints in that list parser and "d" for signed ints to make it slightly more type safe |
07:49:19 | | Quit mt_ (Read error: 145 (Connection timed out)) |
07:53:25 | *** | Saving seen data "./dancer.seen" |
08:00 |
08:02:16 | | Quit phanboy_iv ("Leaving") |
08:08:10 | | Join Rob2222 [0] (n=Miranda@p4FDCCD11.dip.t-dialin.net) |
08:08:19 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
08:08:46 | amiconn | saratoga: Why does the clip require you to make test_codec imprecise? Can't you just use small enough test files? |
08:09:17 | kugel | JdGordon: %mp can do recording (and fm radio) status too |
08:09:49 | JdGordon | the CustomWPS page needs to be updated then... |
08:10:27 | kugel | oh, I seem to have forgotten this |
08:12:33 | JdGordon | someone was complaing about this in the forum very recently also |
08:13:36 | | Quit dmb (Connection timed out) |
08:13:44 | JdGordon | arg... putting the values for rec and radio in #ifdefs means the skins cant be shared easily |
08:14:38 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
08:14:45 | kugel | JdGordon: no, it's compatible |
08:15:09 | JdGordon | oh right |
08:15:17 | * | JdGordon isnt entirely awake |
08:15:24 | | Quit Rob2223 (Read error: 60 (Operation timed out)) |
08:15:30 | JdGordon | interesting... |
08:15:46 | kugel | %cs was a poor choice for current screen :/ all RTC tags start with %c |
08:16:29 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
08:16:43 | JdGordon | we dont really have much wiggle room there... very soon we will need to think of how to go to a different label scheme |
08:22:09 | kugel | alright, I updated CustomWPS |
08:23:09 | amiconn | Two letters, small+capital allows 2704 combinations... |
08:24:06 | | Join esperegu [0] (n=quassel@145.116.15.244) |
08:25:36 | JdGordon | yes, but they arnt very understandable |
08:26:38 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
08:27:13 | | Join Zagor [242] (n=bjorn@rockbox/developer/Zagor) |
08:27:13 | JdGordon | its 2756 :) 52 single letter tokens also |
08:37:18 | amiconn | Are there such? |
08:37:43 | * | amiconn doesn't remember, because he didn't change his wps'es in ages |
08:37:54 | kugel | %C and %X |
08:38:30 | | Join flydutch [0] (n=flydutch@host248-201-dynamic.15-87-r.retail.telecomitalia.it) |
08:38:35 | kugel | and %D, %x, %s, %t |
08:42:54 | | Quit bubsy ("Party time, EXCELLENT! *wring wring wring!* Party on, Wayne! Party on, Garth!") |
08:44:46 | pixelma | %D ? |
08:45:38 | pixelma | isn't %d for directory and needs a number for the level, like %d1 etc. |
08:46:00 | pixelma | but there are %V and %T too |
08:46:48 | kugel | %D is %d for the next track, apparently |
08:47:15 | pixelma | yes, but does pure %D exeist without the number? |
08:47:28 | pixelma | or exist |
08:48:40 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
08:48:44 | kugel | probably not. but technically the tags are %d/%D |
08:52:57 | pixelma | ah ok, similar things apply to e.g. %t |
08:53:19 | pixelma | except if you want to use the default 2 seconds |
08:54:03 | pixelma | kugel: any news on the progressbar problem, or did I miss something? |
08:55:58 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
08:56:06 | | Join petur [50] (n=petur@rockbox/developer/petur) |
08:56:30 | kugel | no, I haven't had a chance to look into it |
08:56:47 | | Quit bubsy ("Party time, EXCELLENT! *wring wring wring!* Party on, Wayne! Party on, Garth!") |
08:58:19 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
09:00 |
09:00:49 | | Quit bertrik (Client Quit) |
09:00:49 | kugel | I think I see the problem |
09:01:24 | | Join mt_ [0] (n=mtee@41.233.146.27) |
09:01:50 | | Join Overand [0] (i=overand@crappy.domain.name) |
09:02:07 | Overand | I have an iPod 5th gen (Video, 60 gig) - and the clock keeps getting set back exactly one year. |
09:02:51 | Overand | I'm not sure if it's a rockbox thing, or an ipod-end thing - it does occasionally get plugged into a windows machine (at work) in 'iPod mode.' (rescue mode has much slower USB transfers) - Curious if anyone's noticed anything like this. |
09:03:03 | Overand | (Incidentally, I checked the date on the work machine and it's correct) |
09:03:17 | Overand | (Also I don't think the work machine has any apple software instlaled on it) |
09:03:24 | kugel | %pb is problematic, it's the only tag which demands to know the default vp dimensions at parsing time |
09:04:12 | | Quit HBK (Read error: 104 (Connection reset by peer)) |
09:05:08 | | Join HBK [0] (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
09:05:12 | | Quit mt (Read error: 145 (Connection timed out)) |
09:05:19 | | Nick mt_ is now known as mt (n=mtee@41.233.146.27) |
09:06:04 | | Join adiroiban [0] (n=adiroiba@194.54.129.79) |
09:09:05 | | Quit adiroiban (Client Quit) |
09:09:14 | | Join adiroiban [0] (n=adiroiba@194.54.129.79) |
09:13:22 | | Join maruk [0] (n=papier@titanium.sdv.fr) |
09:18:09 | | Quit Lss (Read error: 131 (Connection reset by peer)) |
09:18:26 | | Join Lss [0] (n=Lss@cm46.delta91.maxonline.com.sg) |
09:19:16 | | Quit bubsy (Read error: 54 (Connection reset by peer)) |
09:20:27 | pixelma | it doesn't only need to know the default viewport dimensions, in case you put it in an own viewport it would need to know that vp's dimensions as I think it should use that viewports width then |
09:20:28 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
09:21:47 | pixelma | I think that was working before but am not sure |
09:21:58 | kugel | the dimensions of the other viewports are known, that's not the problem |
09:22:35 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
09:23:12 | kugel | moved setting up the default viewport to the end of parsing (so that %wd/%we is known, and because the default vp got in the way of sbs) |
09:23:42 | | Quit Grahack ("Leaving.") |
09:27:06 | | Quit shaggy-h (Read error: 104 (Connection reset by peer)) |
09:45:50 | | Quit kugel (Read error: 110 (Connection timed out)) |
09:48:51 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
09:48:57 | | Quit Thundercloud (Remote closed the connection) |
09:49:14 | | Join midijunkie [0] (n=Miranda@131.234.54.163) |
09:53:29 | *** | Saving seen data "./dancer.seen" |
10:00 |
10:05:05 | | Quit mt (Read error: 113 (No route to host)) |
10:19:18 | | Quit midijunkie ("?(???~•~)?") |
10:19:39 | | Join midijunkie [0] (n=Miranda@131.234.54.163) |
10:20:20 | | Quit adiroiban (Read error: 145 (Connection timed out)) |
10:20:44 | | Quit jon-kha (Read error: 148 (No route to host)) |
10:21:19 | | Part midijunkie ("I'm a happy Miranda IM user! Get it here: http://miranda-im.org") |
10:32:39 | | Join shaggy-h [0] (n=kiwi@host-87-74-127-193.dslgb.com) |
10:46:52 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
11:00 |
11:38:58 | | Quit Xerion (Read error: 104 (Connection reset by peer)) |
11:39:13 | | Join funman [0] (n=fun@233.174.86-79.rev.gaoland.net) |
11:39:59 | | Join Xerion_ [0] (i=xerion@82-170-197-160.ip.telfort.nl) |
11:40:24 | | Nick Xerion_ is now known as Xerion (i=xerion@82-170-197-160.ip.telfort.nl) |
11:40:29 | | Quit kugel (Read error: 60 (Operation timed out)) |
11:47:54 | | Join DerPapst [0] (n=DerPapst@wlan-nat-24.fh-friedberg.de) |
11:53:31 | *** | Saving seen data "./dancer.seen" |
12:00 |
12:13:44 | | Nick YPSY is now known as Ypsy (n=ypsy@87.106.45.183) |
12:20:00 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
12:24:57 | | Join adiroiban [0] (n=adiroiba@h194-54-129-79.teleson.ro) |
12:25:47 | | Join MethoS- [0] (n=clemens@134.102.106.250) |
12:57:28 | | Quit alexbobp (Read error: 145 (Connection timed out)) |
13:00 |
13:02:01 | | Quit antil33t (Read error: 104 (Connection reset by peer)) |
13:02:07 | | Join antil33t [0] (n=Mudkips@119.224.12.185) |
13:09:17 | | Quit preglow (kubrick.freenode.net irc.freenode.net) |
13:09:17 | NSplit | kubrick.freenode.net irc.freenode.net |
13:09:17 | | Quit crwl (kubrick.freenode.net irc.freenode.net) |
13:09:43 | NHeal | kubrick.freenode.net irc.freenode.net |
13:09:43 | NJoin | preglow [0] (n=thomj@tvilling2.pvv.ntnu.no) |
13:09:43 | NJoin | crwl [0] (n=crwlll@a91-156-100-168.elisa-laajakaista.fi) |
13:25:32 | | Join teru [0] (n=teru@KD059133112132.ppp.dion.ne.jp) |
13:42:16 | | Quit AB3JU (Read error: 60 (Operation timed out)) |
13:42:19 | | Join dz` [0] (n=dz@alt.dissonance.nl) |
13:42:50 | | Join jon-kha [0] (i=jon-kha@83.150.91.127) |
13:53:34 | *** | Saving seen data "./dancer.seen" |
13:58:26 | | Quit DerPapst ("Leaving.") |
14:00 |
14:08:30 | | Quit GodEater (Read error: 113 (No route to host)) |
14:08:52 | | Quit Bob_C_ (Read error: 104 (Connection reset by peer)) |
14:09:15 | | Join domonoky [0] (n=Domonoky@rockbox/developer/domonoky) |
14:10:27 | | Join itcheg [0] (i=4117734b@gateway/web/freenode/x-ywhxusopvwwrgogw) |
14:10:46 | | Quit MethoS- (Remote closed the connection) |
14:23:26 | | Join evilnick_B [0] (i=0c140464@rockbox/staff/evilnick) |
14:24:16 | | Join _zic [0] (n=user@91-171-83-235.rev.libertysurf.net) |
14:33:21 | | Join panni_ [0] (i=hannes@ip-95-222-21-143.unitymediagroup.de) |
14:37:24 | saratoga | amiconn: the idea was to pause the timer while rebuffering |
14:38:00 | saratoga | the problem with using small files is that 300-400KB is only a couple seconds of lossless files, and it would mean not using the same test files on the AMS targets as existing ones |
14:40:50 | | Quit funman ("free(random());") |
14:54:06 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
15:00 |
15:04:34 | | Join Horscht [0] (n=Horscht2@xbmc/user/horscht) |
15:13:32 | | Quit _zic (Read error: 113 (No route to host)) |
15:18:24 | | Join jasio [0] (n=yann@cpc2-rdng20-2-0-cust902.15-3.cable.virginmedia.com) |
15:20:45 | | Join Crackerizer [0] (n=7d1a8ca7@giant.haxx.se) |
15:20:55 | Crackerizer | Hello, |
15:21:47 | Crackerizer | May i ask a question, is there any math function in rockbox? |
15:21:57 | Crackerizer | such as ceil, log, etc.. |
15:24:08 | Zagor | Crackerizer: we avoid using floating point math |
15:25:14 | | Nick fxb__ is now known as fxb (n=felixbru@85.214.97.64) |
15:25:25 | | Join kkurbjun [0] (n=kkurbjun@rockbox/developer/kkurbjun) |
15:25:33 | Crackerizer | Zagor: so, how does we implement these functions? or we just dont use it? |
15:26:03 | Zagor | Crackerizer: it depends. tell us what you want to do. |
15:27:19 | Crackerizer | Zagor: I'd to find ceil(log2(A)). To me, ceil implementation is easy but not log2. |
15:29:06 | Zagor | yes, but what feature are you trying to implement? |
15:31:02 | | Quit itcheg (Ping timeout: 180 seconds) |
15:31:57 | TheSeven | log2 isn't hard either |
15:32:02 | Crackerizer | I'm working on diacritic character patch. There is a folk here suggests me an algorithm which need these functions. |
15:32:35 | Crackerizer | You can have a look at patch no#10720 |
15:33:12 | TheSeven | int result = 0; while (input) { input >>= 1; result++; } return result; (can be further optimized) |
15:33:24 | TheSeven | that's ceil(log2(input)) |
15:35:44 | Crackerizer | TheSeven: Oh, that's very simple.. |
15:40:59 | Crackerizer | TheSeven: How do you find this code? |
15:41:55 | TheSeven | Crackerizer: i just quickly wrote it? |
15:43:00 | | Join toffe82 [0] (n=chatzill@12.169.218.14) |
15:45:40 | Crackerizer | TheSeven: I know you did :). But what knowledge do you use to construct such a simple algorithm out of a quite complex math function? |
15:46:10 | Crackerizer | This is very interesting to me. |
15:46:26 | Torne | log2 is just the number of digits in a number ehen written in binary |
15:46:52 | Torne | well, if you round it to ceil |
15:47:00 | Torne | same as any other logarithm base |
15:47:38 | Crackerizer | Torne: Oh, I see... |
15:47:40 | saratoga | log2 is just a shift |
15:47:58 | saratoga | ideally you should try to precompute the shift amount though so that the loop is unneeded |
15:48:35 | Torne | er, log2 is not a shift, it's the inverse |
15:48:59 | Crackerizer | Saratoga: Ah, true. |
15:49:23 | saratoga | oh yeah |
15:49:43 | saratoga | i misread what that code was trying to do |
15:50:15 | TheSeven | i just found that function yesterday in an iBSS disassembly, they did it in quite an interesting way that only needed 2 AND instructions and then a maximum of 8 loop iterations |
15:50:16 | Crackerizer | You can have a look at patch#10720 |
15:50:22 | saratoga | but ideally you should try to avoid doing the log2 calculation if you can, many parts of rockbox simply store numbers as the power of 2 needed, so that they don't have to compute logs |
15:50:58 | Crackerizer | I'm working at diacritic character patch. |
15:51:17 | | Join DerPapst [0] (n=DerPapst@asr-nat2.its.fh-giessen.de) |
15:52:05 | Crackerizer | and need suggestions. |
15:53:35 | *** | Saving seen data "./dancer.seen" |
15:54:52 | | Join itcheg [0] (i=4117734b@gateway/web/freenode/session) |
15:57:18 | | Join b0hoon [0] (n=3e57b852@giant.haxx.se) |
15:57:26 | | Join Strife89 [0] (n=nds@168.16.239.253) |
15:57:31 | Crackerizer | Saratoga: Emm, the input is actually constant. I would use the precalculated value instead. |
15:57:50 | Crackerizer | Thank you for your help. |
15:59:37 | | Quit b0hoon (Client Quit) |
16:00 |
16:00:27 | jasio | Hi. I just tried software keylock on an Onda VX777. It doesn't lock the touchscreen. Is that expected in the current status or did I forget to enable something? |
16:08:29 | | Quit itcheg (Ping timeout: 180 seconds) |
16:09:59 | | Join StealthyXIIGer [0] (n=stealthy@c-68-62-19-6.hsd1.mi.comcast.net) |
16:12:45 | | Quit teru ("Quit") |
16:16:34 | | Join jgarvey [0] (n=jgarvey@cpe-174-097-130-131.nc.res.rr.com) |
16:18:38 | | Quit Rob2222 () |
16:22:44 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
16:23:54 | | Part LinusN |
16:25:13 | | Join phanboy4 [0] (n=benji@c-24-98-43-198.hsd1.ga.comcast.net) |
16:35:16 | | Join Rob2222 [0] (n=Miranda@p4FDCCD11.dip.t-dialin.net) |
16:38:33 | jasio | Anybody knows if software hold/lock can lock the touchscreen as well as the keys? |
16:39:03 | jasio | I tried on VX777 and the one key is locked but not the screen. |
16:41:40 | * | domonoky doesnt know about vx777, but as this is a new target (and all touchscreen targets are new and in development) i would suspect this is just not correctly implemented at moment. |
16:42:52 | jasio | i thought so. looking at the source but not sure yet how the soft lock works |
16:43:00 | kugel | domonoky: hey. did you test whether collecting 5 messures from the mini2440 is really needed? |
16:43:13 | pixelma | software lock in its current state is only used on the Archoses, physical button only, targets so I guess locking the touch screen as well hasn't been implemented yet for the touch screen targets there. |
16:43:13 | kugel | mini2440's touchscreen |
16:44:07 | domonoky | kugel: not really, i just copied the code from d2. But i saw variing touchscreen readings in my early experimentation.. so some filtering is probably needed. |
16:44:09 | kugel | also, the test program from friendlyarm uses interrupts to read the touchscreen |
16:45:17 | kugel | domonoky: I can imagine that readings are *slighly* off (a hand can't be entirely steady). IIRC the D2 had more serious wrong readings |
16:45:25 | domonoky | kugel: yes, the example uses interrupts for adc readings. but i wanted to start simple, also the current adc code from gigabeatf doesnt use ints. |
16:46:23 | domonoky | kugel: i saw completly off readings.. like 512 - 500 - 0 - 490 in my early experiments... but that might be better now, i dont know. |
16:47:15 | kugel | oh ok |
16:47:59 | domonoky | kugel: feel free to test and improve if you want. (uart_printf or logf over serial is very good for debugging, if you have a serial cable) :-) |
16:48:13 | kugel | I have one now |
16:48:36 | kugel | I just don't know how to use it tough, never messed with serial ;) |
16:50:29 | domonoky | kugel: then learn it, it isnt hard :-) |
16:51:23 | | Join xnyhps [0] (n=xnyhps@2001:470:1f14:da:219:e3ff:fe36:98e5) |
16:52:05 | | Join esperegu_ [0] (n=quassel@145.116.15.244) |
16:53:47 | jasio | another question: in the button_mapping structure how is pre_button_code used? random example, { ACTION_PS_TOGGLE_MODE, BUTTON_MENU|BUTTON_REPEAT, BUTTON_MENU } what does the last BUTTON_MENU do? |
16:54:56 | | Join Bob_C [0] (n=chatzill@host86-144-201-0.range86-144.btcentralplus.com) |
16:55:19 | | Quit Strife89 () |
16:56:37 | | Quit esperegu (Read error: 113 (No route to host)) |
16:57:53 | | Quit mc2739 (Read error: 131 (Connection reset by peer)) |
16:57:58 | | Join mc2739_ [0] (n=mc2739@rockbox/developer/mc2739) |
17:00 |
17:00:53 | | Join GodEater [0] (n=bibble@bb-87-80-121-64.ukonline.co.uk) |
17:02:20 | kugel | BUTTON_MENU needs to be pressed first |
17:03:23 | | Quit Zagor ("Don't panic") |
17:03:40 | jasio | so press BUTTON_MENU once, release, press and hold ? |
17:04:35 | kugel | no, press BUTTON_MENU, and then press both without releasing BUTTON_MENU |
17:05:46 | jasio | but the other button is also BUTTON_MENU |
17:06:23 | domonoky | kugel: both ? this action triggers just on long press of BUTTON_MENU, the prereq is need in this case to distinguish from other button combinations with BUTTON_MENU i think. |
17:06:41 | kugel | oh right, it reads BUTTON_REPEAT, sorry |
17:07:12 | | Quit shaggy-h (Read error: 110 (Connection timed out)) |
17:07:13 | * | kugel somehow read BUTTON_REWIND |
17:08:06 | kugel | jasio: in that case, it mostly means that this actual will fire only once |
17:08:55 | kugel | if you start holding BUTTON_MENU, there's only a single moment where BUTTON_MENU|BUTTON_REPEAT is true and where the previous button was BUTTON_MENU only |
17:09:09 | | Quit thegeek_ (Read error: 113 (No route to host)) |
17:09:11 | jasio | ok. I'll experiment a bit more. I'm trying to get a decent keymap for the VX777. There is only one button but I can at least have short press and hold |
17:09:49 | jasio | it's starting to make sense ;) |
17:10:55 | | Join thegeek [0] (n=nnscript@s168c.studby.ntnu.no) |
17:11:18 | kugel | the button are reads every 10ms. the action will only be fired off if BUTTON_MENU|BUTTON_REPEAT is read and the previous read (i.e. 10ms ago) was BUTTON_MENU only. that's why it's called prerequisite |
17:11:26 | kugel | s/reads/read/ |
17:15:51 | | Quit DerPapst ("Leaving.") |
17:16:24 | pixelma | jasio: the touchscreen targets also have the generic touchscreen keymap - which uses the screen divided into a grid as buttons and also on some screens already you can touch things directly |
17:17:46 | jasio | pixelma: the touchscreen on the vx777 is already working pretty good both in grid and absolute modes. |
17:19:16 | jasio | one thing I'll have to look at is that the bottom part doesn't work as expected. It's like there is an area that is mapped like a button (does nothing in WPS, sort of Cancel in menus) |
17:23:55 | | Quit StealthyXIIGer (Read error: 110 (Connection timed out)) |
17:25:33 | | Quit shai ("Leaving") |
17:27:38 | | Join funman [0] (n=fun@rockbox/developer/funman) |
17:27:50 | | Join Hillshum [0] (n=hillshum@75-165-232-83.slkc.qwest.net) |
17:29:10 | Bob_C | wahey, mini2440 can now emulate a very loud squirrel ;) |
17:29:21 | | Quit dz` (kubrick.freenode.net irc.freenode.net) |
17:29:21 | NSplit | kubrick.freenode.net irc.freenode.net |
17:29:21 | | Quit Hadaka (kubrick.freenode.net irc.freenode.net) |
17:29:25 | | Join AB3JU [0] (n=dz@alt.dissonance.nl) |
17:29:52 | | Join Naked [0] (n=naked@83.246.72.254) |
17:30:05 | | Nick Naked is now known as Hadaka (n=naked@83.246.72.254) |
17:30:55 | | Quit yawny (Client Quit) |
17:31:47 | funman | dionoea: any progress on Clip ? |
17:32:47 | | Quit Xerion (Read error: 104 (Connection reset by peer)) |
17:33:13 | | Join elcan [0] (i=user36@64.32.20.56) |
17:33:20 | | Join Xerion [0] (i=xerion@82-170-197-160.ip.telfort.nl) |
17:33:47 | | Quit elcan (Client Quit) |
17:34:03 | | Quit phanboy4 ("Leaving") |
17:34:23 | | Join elcan [0] (i=user36@pr0.us) |
17:35:05 | JdGordon | (01:23:15 AM) kugel: moved setting up the default viewport to the end of parsing (so that %wd/%we is known, and because the default vp got in the way of sbs) <- that could cause other issues.... The default viewport *should* be known before parsing starts... the %we/d tag logic are (or can be) made smart enough to fix it correctly |
17:40:06 | kugel | JdGordon: I saw that now too |
17:41:03 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
17:41:47 | JdGordon | ok good :) |
17:42:02 | JdGordon | I had nightmares about what you might do for a possible fix :D |
17:43:13 | * | kugel gets tired of getting told wrongly how his own code works |
17:43:35 | JdGordon | thats what happens when there are no comments in the code |
17:49:16 | kugel | well, I could've added more, but that wouldn't have changed anything (just read your reply on the ml). |
17:50:16 | | Quit maruk ("Leaving.") |
17:50:46 | evilnick_B | JdGordon: RE: your last reply on the ml; does that mean that you're suggesting removing users being able to change fonts in favour of only using .cfg for skins? |
17:51:00 | evilnick_B | s/skins/wpses/ |
17:51:12 | | Join mcuelenaere [0] (n=mcuelena@78-22-101-254.access.telenet.be) |
17:53:31 | JdGordon | yes, although only to illustrate my point... I'm not seriously suggesting that |
17:53:40 | *** | Saving seen data "./dancer.seen" |
17:58:54 | jasio | Got soft lock of the touchscreen by just moving a few lines in action.c :) |
17:59:11 | | Join bubsy_ [0] (n=bubsy@94.139.72.137) |
17:59:16 | jasio | but my keymap is still not quite right |
18:00 |
18:00:25 | | Quit bubsy_ (Client Quit) |
18:02:07 | | Quit bubsy (Remote closed the connection) |
18:02:38 | | Quit kugel (Remote closed the connection) |
18:07:16 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
18:09:07 | domonoky | Bob_C: nice ! |
18:09:09 | NHeal | (timeout) kubrick.freenode.net irc.freenode.net |
18:09:49 | mcuelenaere | domonoky, Bob_C: any reason why you guys didn't use the "generic" touchscreen calibration for the mini2440? |
18:09:50 | | Join phanboy4 [0] (n=benji@gate-22.spsu.edu) |
18:10:49 | domonoky | mcuelenaere: it uses the generic touchscreen calibration.. but i use a extra correction in the driver, because the touchscreen is mounted sideways. |
18:11:07 | * | mcuelenaere checks |
18:11:34 | | Quit petur ("*plop*") |
18:11:40 | domonoky | 0,0 is bottom left on mini2440 touchscreen driver. |
18:12:26 | | Join JdGordon1 [0] (n=jonno@mac0736d0.tmodns.net) |
18:12:29 | domonoky | but anyone should feel free to improve it, i just copied from the d2 touchdriver so that it works :-) |
18:12:32 | mcuelenaere | ok, I just presumed that because you added that code you didn't |
18:12:44 | | Quit jfc^3 (Read error: 104 (Connection reset by peer)) |
18:13:14 | | Join jfc^3 [0] (n=john@dpc6682208002.direcpc.com) |
18:15:26 | Bob_C | I haven't looked at touchscreen so can't comment... |
18:17:03 | | Join shai [0] (n=Shai@l192-117-110-233.cable.actcom.net.il) |
18:27:30 | | Quit Grahack (Client Quit) |
18:28:00 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
18:28:40 | | Join JdGordon2 [0] (n=jonno@mac0736d0.tmodns.net) |
18:28:56 | | Quit JdGordon1 ("Leaving.") |
18:36:18 | | Quit matsl (Read error: 145 (Connection timed out)) |
18:42:56 | * | kugel wonders why parse_list doesn't simply call atoi() |
18:43:28 | | Quit phanboy4 (Read error: 104 (Connection reset by peer)) |
18:44:27 | | Join Strife89 [0] (n=michael@168.16.237.214) |
18:44:46 | kugel | it seems parse_list is reimplementing it. linuxstb ? |
18:47:35 | | Join DerPapst [0] (n=DerPapst@wlan-nat-24.fh-friedberg.de) |
18:52:53 | | Quit TheSeven ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
18:54:33 | | Quit thegeek (Read error: 104 (Connection reset by peer)) |
18:56:02 | CIA-85 | New commit by Domonoky (r23379): make the mini2440 simulator compile. |
18:57:03 | | Nick Ypsy is now known as YPSY (n=ypsy@87.106.45.183) |
18:59:11 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
19:00 |
19:00:33 | | Join JdGordon| [0] (n=Miranda@nat/microsoft/x-thfdwtfxqfysaeuc) |
19:02:51 | | Quit JdGordon2 (Read error: 110 (Connection timed out)) |
19:04:21 | | Quit Horscht ("Verlassend") |
19:07:37 | | Join Horscht [0] (n=Horscht2@xbmc/user/horscht) |
19:09:20 | linuxstb | kugel: Doesn't parse_list increment the pointer past the integer? atoi just returns the integer. |
19:10:34 | | Nick fxb is now known as fxb__ (n=felixbru@85.214.97.64) |
19:11:20 | | Quit Strife89 ("Switching OSes.") |
19:12:49 | kugel | it could do atoi(), and then p += strchr(p, sep), couldn't it? |
19:14:19 | linuxstb | But then it's doing the work twice... And the semantics are not the same - e.g. it would accept something like 10a (I would hope parse_list would reject that) |
19:17:27 | | Join Strife89 [0] (n=michael@168.16.237.233) |
19:19:48 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
19:24:14 | | Quit faemir ("Leaving") |
19:25:02 | kugel | linuxstb: nope |
19:29:41 | | Join cas [0] (n=chatzill@87.196.218.255) |
19:36:30 | | Quit Crackerizer ("CGI:IRC") |
19:44:30 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
19:49:02 | JdGordon| | do any targets not have enough buttons for up/down/left/right/select and maybe a cancel action? |
19:50:52 | bertrik | meizu m3 :P but it's not really a target yet. It has an up/down touch strip, a top and a bottom button and some kind of cancel button, but no left/right |
19:51:06 | kugel | that should be the minimum buttons for all targets. I've been messing with that in my PLA rework |
19:51:50 | GeekShadow | bertrik, I got one if there are tests to do ;) |
19:52:29 | | Join Sajber^ [0] (n=Sajber@c-d73471d5.012-155-73746f22.cust.bredbandsbolaget.se) |
19:53:43 | *** | Saving seen data "./dancer.seen" |
19:59:29 | | Quit Hillshum (Read error: 104 (Connection reset by peer)) |
20:00 |
20:10:33 | | Quit DerPapst ("Leaving.") |
20:10:55 | JdGordon| | has anything been done about the fuzes volume issues? |
20:11:25 | JdGordon| | apart from it being generally quiter than other targets.. is the left ear quieter than the right? or am i going deaf in one ear more than the other? |
20:11:56 | kugel | I think it's you (or your headphones). my fuze has no volume issues |
20:12:43 | | Quit jgarvey ("Leaving") |
20:12:56 | JdGordon| | it used to be noticably quiter than other targets with the same headphones... didnt it? |
20:13:09 | JdGordon| | targets being other ams ones |
20:14:36 | kugel | I can't say that, my clip is pretty much the same as my fuze in that regard |
20:21:17 | cas | domonoky: ping |
20:21:33 | domonoky | cas: pong |
20:22:21 | cas | domonoky: I saw that there is a lot of code now on SVN for Mini2440. The wiki page says audio at 10%... are you working on audio or Bob_C ? |
20:23:24 | domonoky | afaik Bob_C is working on that... (he said earlier, that he already heard some squirrel sound :-) ) |
20:23:50 | cas | domonoky: ok, and you? |
20:25:46 | domonoky | cas: nothing at moment.. and you ? :-) |
20:26:45 | cas | domonoky: nothing :-) |
20:27:28 | cas | domonoky: I am being busy with other things for the Lyre project, not Rockbox directly related. |
20:28:20 | kugel | btw, it's a pitty that there's no proper power off button defined |
20:28:45 | kugel | I didn't find a way to save the settings yet, shutting down would be easiest :) |
20:30:06 | domonoky | kugel: how about idle shutdown ? (but mini2440 cant really shutdown at moment) |
20:31:51 | JdGordon| | I thought the idle notify stuff isnt enabled for flash targets so it just writes immediatly? |
20:32:10 | JdGordon| | so settings should get saved immediatly? |
20:33:27 | kugel | well, it's not like that |
20:35:18 | pixelma | no, settings won't get written immediately on flash targets |
20:39:46 | Bob_C | I ran out of buttons to define a Power btn |
20:39:49 | JdGordon| | can I bribe someone to take 5 min (actually its going to be more than that now :p ) to read the current dev-ml thread and throw in their opinion? |
20:40:50 | JdGordon| | the argument has nothing to do with code.. it is entirely the logic behind it... so you only need to have an idea how the skins should work |
20:40:59 | kugel | Bob_C: it doesn't need to be a separate button. actually, most targets have the power of button on a button that's almost as used as the non-power of ones |
20:42:15 | Bob_C | well, i defined UDLR menu + select,but we can define buttons and keymap however we like I guess |
20:42:23 | | Quit Galois ("Leaving") |
20:42:24 | Bob_C | I built an external keypad anyway |
20:42:36 | kugel | oh really? |
20:42:46 | | Quit TheSeven ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
20:42:47 | kugel | can I see a picture? |
20:44:29 | Bob_C | Sure, I will have to search for my CF card first |
20:45:18 | Bob_C | Its just some stripboard and cheap buttons, nothing special |
20:51:46 | | Quit funman ("free(random());") |
20:52:18 | | Quit GodEater ("leaving") |
20:52:35 | | Join GodEater [0] (n=bibble@bb-87-80-121-64.ukonline.co.uk) |
20:52:53 | bertrik | has anyone ever tried to make splash have rounded corners? |
20:53:14 | gevaerts | We need WSS! |
20:53:45 | n1s | JdGordon|: i can tell you this; i have no idea how all this vieport stuff interacts with different parts of the ui or how it affects the drawing, but i sure would love to see some documentation of it like the documentation of the graphics api i the wiki |
20:53:56 | kugel | bertrik: I don't think so, but I doubt it's worth the effort |
20:54:13 | JdGordon| | n1s: documentation is for nancy boys! :D |
20:54:25 | bertrik | I think it could look slightly cooler that way, I'm just asking if anyone ever tried it |
20:55:04 | n1s | JdGordon: yes, well it's also generally good for understanding the *intended* function of the code :) |
20:55:11 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
20:55:19 | kugel | it's basically only 3 simple drawing function calls right now (border, filling and text). round edges need additional logic |
20:55:37 | | Join fyrestorm [0] (n=nnscript@cpe-69-203-148-25.si.res.rr.com) |
20:56:01 | JdGordon| | gevaerts: while splash screen? its definetly doable... |
20:56:03 | n1s | bertrik: never heard of any attempts at that |
20:56:36 | | Join Galois [0] (i=djao@efnet.math.uwaterloo.ca) |
20:56:57 | JdGordon| | actually... a splash skin could be alot of fun :) |
20:58:10 | * | n1s would like the list selection bar to be skinnable beyond gradients too, imo the whole gradient thing is misplaced and should be replaced by more flexible bitmap stuff |
20:58:29 | * | bluebrother would like to have inline settings :) |
20:58:40 | bluebrother | s/have/see/ |
20:58:57 | JdGordon| | I probably still have my old screenshots of inline settings |
20:58:57 | bertrik | inline settings, what is that? |
20:59:29 | JdGordon| | having the setting in the menu so you dont have to go into another screen to change then |
20:59:30 | JdGordon| | them |
20:59:41 | JdGordon| | and so you can see the value without entring it also |
21:00 |
21:00:16 | bertrik | ok |
21:00:23 | domonoky | kugel: idle shutdown seems to correctly save settings on mini2440 :-) |
21:01:55 | bertrik | indeed just being able to see settings when the selection bar is over an item would be nice already |
21:02:22 | n1s | I also think that too much of the current gui is fullscreen based, if you enter a yes/no setting for example a smaller window to display the coice would look better on large displays |
21:02:46 | bluebrother | nah, popups are annoying. Just make the setting inline :) |
21:02:48 | | Quit Utchybann (Read error: 113 (No route to host)) |
21:03:05 | bluebrother | that original patch was really nice. |
21:03:10 | kugel | with a ballon or roll effect please :) |
21:03:15 | n1s | bluebrother: inline has disadvantages |
21:03:22 | JdGordon| | SKINS EVERYWHERE!!!! |
21:03:35 | * | bluebrother slaps kugel with a ballon or roll effect |
21:03:42 | n1s | ... with settings that have many values for example |
21:03:51 | bluebrother | n1s: true, but you don't need to use it everywhere. |
21:04:11 | pixelma | just prepend the actual value behind the setting name |
21:04:22 | | Join DerPapst [0] (n=DerPapst@79.232.246.163) |
21:04:26 | JdGordon| | the hardest thing about inline settings, is for the settings which to the user look like regular ones, but in the code are actually a different beast |
21:04:49 | bluebrother | huh? What's a "regular" setting? |
21:04:51 | n1s | well, non-fullscreen windows can be useful in other contexts than settings, such as theonplay menu |
21:05:14 | * | kugel thinks making the simple on/off ones inline for the start would be a good idea |
21:05:17 | JdGordon| | n1s: I agree... thats easily fixed though.. just pass a viewport to its do_menu() call |
21:05:55 | n1s | JdGordon: i would expect stuff in the background to keep updating, will it? :) |
21:05:57 | JdGordon| | bluebrother: regular being the ones which have all the options in a list... except there are at least one setting which look like that but arnt |
21:06:01 | pixelma | I don't like that you can't see the amount of values easily (and e.g. if it would be easier to use wrap and go backwards - and that it wastes space the way it was done and either looked ugly on really small and really large screens |
21:06:23 | | Join daggett [0] (n=denis@96.20.53.158) |
21:06:45 | JdGordon| | you can see the difference with the icons in the menus.. any which have the setting icon can work, any that have the one for a submenu or "function" cant |
21:06:46 | * | gitster asks pixelma to re-review his comment on FS #10680 when he has time. |
21:07:04 | kugel | s/he/she/ :) |
21:07:04 | n1s | s/it/they |
21:07:18 | amiconn | TheSeven: log2 for 32 bits is easier than that. Pure C needs a maximum of 5 if() {shift, add} blocks |
21:07:55 | TheSeven | yes, you can of course divide and conquer it... |
21:08:03 | JdGordon| | the 3 bottom peak meter settings cant work for example |
21:08:06 | amiconn | And we already have a number of implementations for that. On ARMv5+ there's the clz instruction, which just needs single correction afterwards |
21:09:22 | pixelma | gitster: you were right, I confused the two. It's not an undocumented use of %St though as it is described on the CustomWPS wiki (which is very often more up to date than the manual because it seems to be easier to update) |
21:09:38 | pixelma | more of a manual bug then |
21:10:02 | gitster | pixelma: thanks; then there is nothing that blocks FS #10680 and FS #10681, I guess. |
21:12:55 | bluebrother | the original patch which I really liked: http://www.rockbox.org/tracker/task/5833 |
21:13:57 | linuxstb | kugel: What did you mean by "nope" earlier? |
21:14:05 | kugel | gitster: I think so too |
21:14:09 | | Quit TheSeven ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
21:14:22 | kugel | linuxstb: Looking at the code, it doesn't look like it would reject 10a |
21:14:52 | kugel | but maybe I didn't look hard enough |
21:15:23 | n1s | amiconn: isn't our ffs (!) function basically a log2? |
21:15:23 | | Quit bluebrother (Read error: 104 (Connection reset by peer)) |
21:15:52 | linuxstb | kugel: line 959 of misc.c |
21:15:56 | JdGordon| | why should parse_list reject 10a anyway? |
21:16:01 | kugel | ah yea, I didn't see that before |
21:16:05 | amiconn | n1s: no |
21:16:10 | linuxstb | JdGordon|: Because "10a" isn't an integer... |
21:16:11 | kugel | I only looked at the 'd' case |
21:16:38 | JdGordon| | 10 is though... |
21:16:39 | n1s | amiconn: ah, right, that is first == lsb? |
21:16:49 | linuxstb | JdGordon|: Yes, and "10" would be accepted as such... |
21:16:55 | amiconn | ffs finds the least significant "1" bit. log2 finds the most significant "1" bit |
21:17:04 | JdGordon| | and then it should skip past a untill the next seperator |
21:17:45 | JdGordon| | any objections to using u for unsigned and d for signed ints in that parser? we wont always want the ability to get bac negative values for ints |
21:18:19 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
21:18:35 | | Join liar [0] (n=liar@83.175.83.185) |
21:18:45 | n1s | "first" is ambigous |
21:20:07 | | Join NoGare [0] (i=NoGare@216.8.173.127) |
21:21:14 | linuxstb | JdGordon|: Why would you want to accept "10a" as an integer? Shouldn't the wps parser be as strict as possible? |
21:21:16 | | Join bluebrother [0] (n=dom@f053154046.adsl.alicedsl.de) |
21:22:22 | bertrik | hm, I could have a look at the meizu NAND contents with the samsung DFU tool, to learn more about the FTL |
21:24:08 | JdGordon| | linuxstb: two different things.. that list parser is used by more than just the skin parser, and yes, 10a isnt an integer but I dont really see why the 10 couldnt be accepted.. Its not something I really care about either way.. im just saying |
21:24:38 | | Quit TheSeven (Connection reset by peer) |
21:29:05 | | Part xnyhps |
21:29:51 | | Join tomers [0] (n=chatzill@bzq-84-109-85-100.red.bezeqint.net) |
21:30:22 | tomers | New update to "FS #10728 - Cowon D2: Add support for D2 in rbutil" - hope this is the last one |
21:31:46 | * | tomers Got to go. Will read your comments (if there will be any) from the log tommorrow |
21:31:58 | | Quit tomers (Client Quit) |
21:32:11 | | Join domonoky1 [0] (n=Domonoky@g229178191.adsl.alicedsl.de) |
21:32:47 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
21:38:47 | bluebrother | tomers: IMO linuxstb has a point about splitting changes from moving the file. At least posting two consecutive patches (with one only moving the file) would make it much easier to review the changes. |
21:39:50 | | Quit mcuelenaere (Read error: 60 (Operation timed out)) |
21:40:00 | * | n1s very much agrees with that |
21:40:20 | linuxstb | I don't think the file move is controversial - that can just be done... |
21:40:38 | n1s | i even think svn refuses to commit a move and change unless you force it |
21:40:39 | bluebrother | tomers: also, have you thought about how to handle the two-drives issue for the D2? In the current state it's something I'd call definitely broken. Users expect it to "work". |
21:40:50 | | Join mcuelenaere [0] (n=mcuelena@78-22-101-254.access.telenet.be) |
21:41:36 | CIA-85 | New commit by nls (r23380): Use array for quickscreen item settings to simplify some logic |
21:41:45 | bluebrother | n1s: afaik you can do that even without forcing it. And doing move + change in the same commit isn't something I'd consider problematic either −− it's the diff format that makes it hard to review such a patch as it doesn't know about the move operation. |
21:43:03 | n1s | bluebrother: i think such commits are harder to review, and an additional commit to move is not a big cost |
21:43:58 | bluebrother | n1s: I do agree with that, but I don't think it's that important for the final commit −− it should have been reviewd that time anyway :) |
21:44:36 | bluebrother | but there's no harm in doing it in two steps so I don't see a reason to force one commit |
21:44:49 | n1s | bluebrother: yes, unfortunately you sometimes see some code and wonder when it got in and why |
21:45:09 | linuxstb | Why don't one of us just move it now? |
21:45:30 | bluebrother | sure, would be a ... solution ;-) |
21:45:37 | * | n1s has no idea about what specific file we are discussing |
21:45:57 | linuxstb | tools/mktccboot.c - move it under rbutil/ with the other mk* tools |
21:46:28 | | Join phanboy4 [0] (n=benji@gate-20.spsu.edu) |
21:46:28 | linuxstb | (as it's not needed by the build system, it's not a problem IMO) |
21:46:43 | n1s | i'd say go for it |
21:47:34 | | Quit domonoky (Read error: 110 (Connection timed out)) |
21:48:06 | * | linuxstb wonders why tomers's Makefile is so complex though, compared to the single rule in tools/Makefile |
21:48:37 | * | linuxstb will just do a trivial move, and tomers can then sync his patch, so we can see what he's actually changing |
21:49:08 | bluebrother | I guess it's his attempt to merge it with the mkamsboot Makefile. Though I'd prefer that to be a separate change too. |
21:49:55 | | Quit parafin (Nick collision from services.) |
21:50:03 | | Join parafin|away [0] (i=parafin@paraf.in) |
21:50:19 | | Nick parafin|away is now known as parafin (i=parafin@paraf.in) |
21:50:32 | | Quit NoGare ("ChatZilla 0.9.85 [Firefox 3.5.4/20091016092926]") |
21:50:59 | bluebrother | I'm wondering if it would make sense to move all tools that are not required by the build process from tools/ to utils/ |
21:51:44 | CIA-85 | New commit by bluebrother (r23381): Update bootloader download location and add a note about building with VS2005. |
21:51:51 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
21:53:44 | *** | Saving seen data "./dancer.seen" |
21:55:40 | jasio | I've just submitted my first patch. It simply adds support to the VX777 keymap for the only button on the device. |
21:56:59 | linuxstb | bluebrother: Yes, it would be nice to clean that up. Although most things seem to be assimilated into rbutil eventually.... |
21:57:13 | JdGordon| | jasio: morse code input? :D |
21:57:27 | bluebrother | yeah, that tools/ vs utils/ tends to confuse me every now and then. |
21:58:00 | bluebrother | another way I could think of would be to move it the other way round −− and assimilated stuff goes to rbutil/ |
21:58:03 | jasio | JdGordon|: that's an idea. One button is still useful as cancel or exit. |
21:59:14 | | Quit bluebrother (Nick collision from services.) |
21:59:17 | | Join bluebroth3r [0] (n=dom@rockbox/developer/bluebrother) |
21:59:36 | | Nick bluebroth3r is now known as bluebrother (n=dom@rockbox/developer/bluebrother) |
22:00 |
22:02:21 | | Join petur [50] (n=petur@rockbox/developer/petur) |
22:02:28 | | Join mitk [0] (n=594d9564@giant.haxx.se) |
22:02:34 | | Join robin0800 [0] (n=quassel@cpc3-brig8-0-0-cust436.brig.cable.ntl.com) |
22:02:55 | | Join jhulst [0] (n=jhulst@unaffiliated/jhulst) |
22:03:39 | mitk | JdGordon|: ping |
22:03:47 | kugel | hm, I just remember that I named it viewport_set_fullscreen |
22:04:07 | JdGordon| | mitk: pong |
22:04:47 | mitk | JdGordon|: Just FS #10717 reminder |
22:08:35 | JdGordon| | kugel: set_fullscreen() shuold return the whole screen... there is a big difference between the whole screen minus 8 pixels height, and the whole screen minus the area left behind from the sbs |
22:09:49 | JdGordon| | mitk: yeah, i forgot about it... tnoight maybe |
22:09:54 | JdGordon| | or get someone else to do it :) |
22:09:55 | kugel | the point of set_fullscreen is easily get a viewport to draw stuff in, but when you don't want to deactive the statusbar |
22:10:13 | JdGordon| | thats why get_defaultS() is |
22:10:34 | kugel | is what? |
22:11:41 | JdGordon| | thats what get defaults shoudl return |
22:11:44 | CIA-85 | New commit by dave (r23382): Move mktccboot from tools/ to rbutil/mktccboot/ - inspired by FS #10728 but with no functional changes to code. |
22:14:43 | mitk | Ok, I'll try. Dear Commiters! Is there anyone who can commit FS #10717. It's insert_last_shuffled functionality to dynamic playlists. Details in FS. |
22:16:29 | jasio | I have added a patch for touchscreen targets (FS #10733) that lets software hold lock both buttons and touchscreen. No real code change, just moving a few lines. |
22:16:47 | CIA-85 | New commit by mcuelenaere (r23383): The Onda VX777 has only one button, add it to the keymap. ... |
22:17:02 | jasio | Tested on VX777. Don't know if there are many touchscreen targets that don't have a hard hold button |
22:17:10 | mcuelenaere | jasio: see SVN ;) |
22:17:51 | jasio | thanks :) |
22:18:12 | mcuelenaere | oh you weren't talking about FS #10732 |
22:18:38 | jasio | it's a new one |
22:19:05 | * | linuxstb hopes the build system is just on a go-slow... |
22:19:35 | | Join barrywardell [0] (n=barrywar@rockbox/developer/barrywardell) |
22:19:35 | mcuelenaere | jasio: RE FS #10733: I don't see that behaviour on my VX747 |
22:19:55 | | Join barrywardell_ [0] (n=barrywar@rockbox/developer/barrywardell) |
22:20:00 | mcuelenaere | jasio: ah wait, the VX777 doesn't have a hardware lock button? |
22:20:15 | stripwax | mitk - does regular "insert shuffled" insert the files shuffled into the entire playlist; or only insert the selected files in a shuffled order at the current insertion point? wondering how we can make the logic consistent for "insert shuffled last" (or whether we want an equivalent "insert shuffled next") |
22:20:17 | jasio | mcuelenaere: no hardware button other than power! |
22:20:19 | * | mcuelenaere didn't know Rockbox had software lock button |
22:21:16 | gevaerts | linuxstb: I get "svn: Can't connect to host 'svn.rockbox.org': Connection refused" on one of my clients. If there are some more, it will be slow... |
22:21:18 | mcuelenaere | jasio: also that patch seems to have added a space after #include "lang.h" |
22:21:29 | mcuelenaere | gevaerts: I got that too just know, retrying helped |
22:21:40 | rasher | gevaerts: I got that once as well |
22:21:53 | rasher | Nearing completion now. |
22:21:54 | gevaerts | it's bound to delay some builds |
22:22:14 | jasio | mcuelenaere: ok, got to be careful with spaces. i'll have a look |
22:22:24 | JdGordon| | stripwax: yep, insert shuffled just puts it somewhere in the playlist... insert shuffled last is so you can add a folder and have the folder shuffled, but not shuffled in with other folders |
22:22:28 | mitk | stripwax: Regular insert_shuffled inserts to entire playlist when no music is played. When music is played inesrt_shuffled inserts between currently played track and end of playlist |
22:23:20 | mitk | Isnsert_last_shuffled is doing things like JdGordon| sad |
22:23:43 | stripwax | yeah - does it make sense to have an equivalent 'insert_next_shuffled' (equivalent to 'insert_last_shuffled' that is) |
22:24:16 | JdGordon| | na.. too many options then |
22:24:25 | gevaerts | rasher: is the binsize server running? There seems to be a backlog building up |
22:24:41 | stripwax | reason i'm thinking is that "insert" puts the track after the last insert, but "insert shuffled" doesn't right now (insert_shuffled is really "insert_shuffled_everywhere") |
22:25:02 | rasher | gevaerts: oh ah, it's not.. and hasn't been running very much lately.. I'll let it run tomorrow |
22:25:06 | stripwax | and i'd quite like an 'insert shuffled' that was like regular 'insert', but .. shuffled .. |
22:26:09 | | Quit Strife89 ("Going home.") |
22:26:36 | mitk | There is no way now to insert shuffled folder to playlist shuffled to the end of playlist. Inesrt_last_ shufled can do that. |
22:26:46 | stripwax | i.e. each of "insert", "insert next", "insert last" have an equiavent shuffled version; then we also have "insert everywhere" which behaves like the current insert shuffled |
22:27:01 | Torne | mitk: but there's also no way to do all kinds of other combinations of these things, is the point ehre ;) |
22:27:07 | Torne | isn't there also insert vs queue? :) |
22:27:09 | stripwax | There is no way currently to insert a shuffled folder to playlist shuffled after the current playing track either |
22:27:10 | Torne | just to make it worse? |
22:27:14 | stripwax | right queue vs insert too |
22:27:27 | stripwax | Trying to work out why adding just one of those combinations makes more sense than adding the others too.... |
22:27:35 | * | stripwax is playing devil's advocate as usual |
22:27:43 | Torne | queue vs insert, next vs last vs as-previous, shuffled vs not |
22:27:52 | stripwax | exactly |
22:28:00 | Torne | and then the current "insert shuffled" as well |
22:28:12 | Torne | that's, what, 13 combinations? :) |
22:28:19 | stripwax | Maybe, we have a config option to let you select which insert vs queue vs shuffled combos appear on that menu? |
22:28:23 | * | stripwax is only half joking |
22:28:24 | | Quit barrywardell (Connection timed out) |
22:28:39 | * | kugel thinks the insert submenu should be divided, with just insert and queue at the top additionally |
22:28:48 | stripwax | oooh. like. |
22:28:59 | stripwax | Insert -> Shuffled -> Last ? |
22:29:04 | stripwax | too many clicks? |
22:29:22 | stripwax | Torne - yep. |
22:29:23 | kugel | that's why insert and queue would be at the top |
22:29:23 | JdGordon| | not if we can map the spare button to an playlist insert action |
22:29:55 | stripwax | JdGordon : - ? you'd still need to pick the kind of insert you want, no? |
22:30:01 | kugel | because those basically repeat your last action |
22:30:43 | JdGordon| | stripwax: if you could assign rec to "insert shuffled last" then its one click... or "queue next" thats also one click |
22:30:55 | jasio | mcuelenaere: new patch without the space |
22:31:00 | stripwax | JdGordon : - er, which 'rec' button are you referring to? |
22:31:10 | JdGordon| | the spare one on e200/h300 |
22:31:11 | mcuelenaere | ok |
22:31:18 | * | stripwax has both an H120 and an ipod 5g .. with no rec button .. |
22:31:19 | | Quit barrywardell_ (Read error: 104 (Connection reset by peer)) |
22:31:49 | * | JdGordon| also has a 5g with no rec button :D |
22:32:16 | JdGordon| | stripwax: actually.. it it were split up there shuold be another option along side insert and queue which is "repeat the last insert method" |
22:32:24 | stripwax | ok so context menu could have its own 'repeat last insert/queue action' |
22:32:33 | stripwax | so all other targets could benefit from a one-click operation too. |
22:32:43 | stripwax | Jinx |
22:32:45 | kugel | isn't that what insert does? |
22:32:53 | stripwax | does insert repeat the last Queue operation? |
22:33:07 | Torne | iirc there's only one "last position" |
22:33:12 | stripwax | and will it take into account the shuffled vs not |
22:33:22 | stripwax | ^will it^would/should/could it |
22:33:24 | Torne | so insert inserts at the last position used by anything and queue queues at hte last position used by anything |
22:33:29 | JdGordon| | isnt there an open bug which says that the "insert" action makes no sense? |
22:33:44 | stripwax | oh, is there? in that case, probably ignore most of this convo :) |
22:33:55 | mitk | This is why insert_last_shuffled I made |
22:34:17 | | Quit Lss (Read error: 104 (Connection reset by peer)) |
22:34:18 | stripwax | we ought to fix the insert action bug at the same time |
22:34:31 | kugel | what bug? that it makes no sense? |
22:34:58 | JdGordon| | maybe not... cant find it |
22:35:02 | mitk | I mean this is not bug. It's strange feature |
22:35:07 | stripwax | mitk - oh hang on - insert_last_shuffled inserts shuffled at the 'last insert point you used' or at the 'last part of the playlist i.e. the end' ? |
22:35:09 | | Join Lss [0] (n=Lss@cm46.delta91.maxonline.com.sg) |
22:35:25 | mitk | At the end of the playlist |
22:35:35 | mitk | Always |
22:35:41 | JdGordon| | I think it went something like you could insert (nex maybe) so the playlist order becomes "1ABC23" if you inserted 123 then ABC |
22:35:56 | kugel | JdGordon|: I think I fixed that one ages ago |
22:35:59 | * | JdGordon| isnt cool enough to use such fancy playlist insert options |
22:36:07 | * | kugel now remembers that one too |
22:36:26 | CIA-85 | New commit by mcuelenaere (r23384): Touchscreen targets: fix software hold not locking touchscreen ... |
22:36:28 | stripwax | yeah, that makes no sense |
22:36:46 | stripwax | so there should also be 'insert before' and 'insert after' to be explicit |
22:36:58 | stripwax | so 123ABC or ABC123 as appropriate |
22:37:04 | JdGordon| | how about a "just play the damn track" option? |
22:37:05 | kugel | the problem was that ABC wasn't added after 123 |
22:37:08 | stripwax | instead of just 'Insert' |
22:37:18 | | Quit J-23 (Read error: 104 (Connection reset by peer)) |
22:37:21 | kugel | the insertion point was properly updated |
22:37:31 | kugel | insert before/after wasn't the issue |
22:37:37 | stripwax | kugel - understood, the insertion point was updated to one track after the last insert |
22:37:55 | stripwax | rather than after the end of all the tracks inserted |
22:37:58 | kugel | I think it wasn't updated at all in some cases |
22:38:08 | mitk | But you are inserting rather something you are intended to listen.Insert last, no insert first |
22:38:48 | stripwax | if I insert ABC then 'insert before' 123 I get 123ABC, all of which is after the current playing track |
22:39:02 | stripwax | if I insert ABC then 'insert after' I get ABC123 |
22:39:09 | * | JdGordon| would like it if it was 3 options, insert, queue, and shuffle, where for the first two you would get a playlist viewer and you can actually choose where to do the insert |
22:39:29 | stripwax | and separately fix 'Insert' to be consistent, deterministic, and less cryptically named |
22:39:35 | kugel | stripwax: that would be a new feature then. in your case doing the same as insert next actually |
22:39:47 | stripwax | kugel - no, not necessarily |
22:39:55 | stripwax | e.g. Insert Last followed by Insert Before, right? |
22:40:06 | kugel | stripwax: it's totally deterministic now, isn't it? always after the insertion point? |
22:40:13 | stripwax | Choosing where to insert is a killer feature.. |
22:40:30 | stripwax | kugel - I thought you just said it had a bug ... that's what I meant about 'fixing Insert' |
22:40:35 | kugel | btw: r20247 |
22:40:57 | kugel | stripwax: I also said I fixed that ages ago :) (see the rev I just posted) |
22:41:05 | mitk | What is insertion point after reshuffling playlist? |
22:41:06 | stripwax | oh ok. |
22:41:16 | stripwax | mitk -eeep |
22:41:51 | stripwax | what is insertion point when loading a new playlist from scratch? Possibly, the same logic should apply |
22:42:19 | kugel | oh, r20241 too |
22:42:30 | | Join J-23 [0] (n=zelazko@unix.net.pl) |
22:42:53 | n1s | JdGordon: i've been thinking that too, this interface we have is pretty good but awfully cofusing to new users |
22:42:57 | kugel | it defaults to after current track IIRC, but it's long ago since I messed with that |
22:43:14 | stripwax | kugel - fwiw, that default makes sense to me |
22:43:23 | n1s | oh and we want multi-select in the pl viewre |
22:43:39 | stripwax | n1s - for multiple-deletes or something else? |
22:43:45 | * | stripwax tries to visualise multiple-moves .. |
22:44:03 | n1s | stripwax: yeah, multi delete, multi move |
22:44:16 | stripwax | there's also that 'moving a track while track changes deselects the move' bug that I'd promised to fix .. will take a look at that |
22:44:18 | kugel | isn't moving stuff around in the playlist viewer broken enough? |
22:44:27 | stripwax | kugel - no, it is fixed, perfectly |
22:44:28 | * | stripwax smiles |
22:44:40 | stripwax | kugel - seriously though, broken how? |
22:44:44 | kugel | ah right, you did some work on it |
22:44:47 | stripwax | aye |
22:44:55 | kugel | I can't remember if I last tried it before or after that |
22:45:28 | | Join froggyman [0] (n=sopgenor@pool-72-69-220-194.chi01.dsl-w.verizon.net) |
22:45:39 | | Nick fxb__ is now known as fxb (n=felixbru@85.214.97.64) |
22:45:40 | stripwax | n1s - still trying to visualise multi-move .. which track do you see moving around, the first of your multi-selects or the last? |
22:45:46 | | Quit phanboy4 (Read error: 54 (Connection reset by peer)) |
22:46:05 | stripwax | I think it could work reasonably well if it's a contiguous block you're moving, but if it's discrete items being moved at the same time .. <brain explodes> |
22:46:51 | gevaerts | stripwax: easy. They keep the same number of tracks between them, so if you move e.g. tracks 17 and 19 to before 2, you get tracks 2 and 4, with ex-2 now being 3 |
22:47:24 | | Quit jhulst (Remote closed the connection) |
22:47:36 | n1s | stripwax: i was thinking of a continous block |
22:47:47 | stripwax | gevaerts - oh no, I realise that. I'm just wondering what you see while you're moving. say you selected tracks 6 and 66, .. what point has the green array, the entry point for track 6, or the entry point for track 66 .. ? |
22:47:51 | * | stripwax thinks probably track 6 |
22:47:59 | * | stripwax thinks contiguous is much easier in any case |
22:48:06 | gevaerts | stripwax: weighted average |
22:48:09 | stripwax | hehehe |
22:48:22 | stripwax | oops, meant 'arrow' not 'array'. but yeah. |
22:51:15 | | Quit TheSeven ("ChatZilla 0.9.85 [Firefox 3.5.3/20090824101458]") |
22:52:01 | | Part froggyman |
22:52:02 | stripwax | anyway where were we? I have no objections to an insert_shuffled_last function. |
22:52:24 | mitk | Me too :) |
22:54:00 | mcuelenaere | do the MTP people still hang out here regularly? Looks like the last commit in the MTP tree was at 17/10 |
22:54:09 | stripwax | anyone else - any objections? if not, I'll commit it .. |
22:54:14 | JdGordon| | gogogog |
22:54:19 | stripwax | but we should continue the discussion of rationalising all of this.. |
22:54:25 | JdGordon| | I onder if it should be called "append shuffled" though |
22:54:50 | stripwax | only if we rename 'insert last' to 'append' ... |
22:54:57 | mitk | Append is new action! |
22:55:19 | Torne | append doesn't distinguish queue/insert.. |
22:55:43 | stripwax | queue last could be renamed to 'queue append', or somesuch |
22:55:55 | stripwax | blah. no. |
22:56:16 | * | Torne doesn't like queue/insert anyway tbh |
22:56:41 | Torne | (entirely aside from the fact tht i've never wanted to use queue at all, the names don't really explain the difference) |
22:57:02 | * | stripwax agrees (although does use queue..) |
22:57:09 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
22:57:35 | | Join phanboy4 [0] (n=benji@168.28.180.20) |
22:57:51 | * | kugel agrees (although hardly uses any isnert/queue) |
22:57:54 | mitk | But append will be much confusing. For all users because doesn't distinguis insert/queue |
22:58:12 | * | gevaerts thinks that they should be merged, with a "play N times" option, with "infinite" being equivalent to "insert" now, "1" being what "queue" does, and the rest being new and improved features |
22:58:47 | kugel | and we offer all values from 1 to infinite? :) |
22:59:00 | * | kugel also wants NaN then! |
22:59:02 | gevaerts | yes :) |
23:00 |
23:00:39 | n1s | i think those two dirty things are floating point constructions, the max number of playtimes should be INT_MAX, should be enough for anyone really ;) |
23:02:12 | JdGordon| | what about quappend? |
23:02:20 | JdGordon| | and quinsert? |
23:03:36 | kugel | and quincy |
23:03:36 | mitk | Maybe rand( queue or append or insert)? |
23:03:44 | n1s | eh so quinsert is "insert queued"? |
23:08:08 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
23:10:11 | | Join saLOUt [0] (n=rriemann@dslb-088-073-093-254.pools.arcor-ip.net) |
23:10:31 | | Quit n1s ("Lämnar") |
23:16:17 | | Quit daggett ("Ex-Chat") |
23:17:03 | saLOUt | hi everyone. i have the recent build on my e240 v2. I get everytime i want to play a video a "data abort" + restart. the same videos worked for some month.. any ideas? |
23:19:43 | JdGordon| | did you update the whole zip? |
23:20:16 | saLOUt | JdGordon|: i just copied the whole zip over the existing installation |
23:20:23 | stripwax | mitk - so looking at that patch, what should the behaviour be if audio is not currently playing? looks like insert shuffled last will do nothing, is that right? |
23:20:40 | saLOUt | do you think it it better to start from a clean .rockbox ? |
23:20:45 | saLOUt | *is |
23:22:18 | | Quit domonoky1 (Read error: 104 (Connection reset by peer)) |
23:22:34 | Torne | it should be fine to just overwrite the existing files |
23:22:43 | Torne | as long as you extract the entire zip |
23:23:23 | saLOUt | Torne: i used Ark -> extract all here |
23:23:45 | mitk | stripwax: Yep. When audio is not curently playing rockbox creates new playlist from scratch |
23:24:52 | mitk | stripwax: Then insert_last_shuffled doesn't make sense. Just insert_shuffled |
23:25:05 | mcuelenaere | hmm I got make working but I'm getting '/home/mcuelenaere/rockbox_git/rockbox/apps/plugins/plugins.make:46: target `/home/mcuelenaere/rockbox_git/build/apps/plugins/boomshine.lua' doesn't match the target pattern' |
23:25:14 | stripwax | mitk - fair enough |
23:26:29 | kugel | mcuelenaere: it doesn't need to be preprocessed/compiled right? you could simply copy it in make install/zip/7z/etc |
23:26:48 | mcuelenaere | kugel: well it needs #ifdef TOUCHSCREEN #endif in SOURCES |
23:27:13 | kugel | oh, well |
23:27:33 | mcuelenaere | is make install/zip/7z/etc handled by one script? |
23:27:40 | CIA-85 | New commit by stripwax (r23385): Committing FS #10717 by Tomasz Kowalyczyk: add PLAYLIST_INSERT_LAST_SHUFFLED (appends the inserted folder into shuffled order to end of playlist) |
23:27:59 | | Nick mc2739_ is now known as mc2739 (n=mc2739@rockbox/developer/mc2739) |
23:28:51 | mitk | stripwax: Thanks. My life will be much easier. I'm using it hardly :) |
23:29:36 | | Quit parafin ("So long and thanks for all the fish") |
23:29:39 | stripwax | hardly? as in 'using it very hard' or 'barely using it' ? |
23:29:57 | | Join parafin [0] (i=parafin@paraf.in) |
23:31:10 | mitk | I mean I'm using it very often so life without it was hard:) |
23:31:43 | | Join TopyMobile__ [0] (n=topy@g227178069.adsl.alicedsl.de) |
23:32:09 | stripwax | mitk :) |
23:32:25 | | Join Topy [0] (n=Topy44@g227178069.adsl.alicedsl.de) |
23:33:07 | saLOUt | can you give me the link to the newest bootloader for e200 v2 ? |
23:33:08 | stripwax | JdGordon: , kugel etc, shall we continue that discussion on ml ? |
23:33:15 | | Quit TopyMobile_ (Read error: 60 (Operation timed out)) |
23:33:47 | JdGordon| | stripwax: hrm? |
23:33:49 | mitk | All green. Going to bed. cya |
23:34:18 | | Part toffe82 |
23:34:23 | stripwax | JdGordon : - you know, inqueue versus quinsert .. |
23:34:34 | | Quit mitk ("CGI:IRC") |
23:34:45 | JdGordon| | oh please no :) |
23:35:18 | kugel | maybe tomorrow, I'm soon getting bloody fingers |
23:35:21 | stripwax | well, at least the serious parts .. :) submenus rather than one giant list of all permutations ; playlist browser for insert points ; multiple select etc... |
23:35:32 | stripwax | kugel - i'll start a thread anyway .. |
23:35:57 | saLOUt | i solved the problem with "data abot on ........." by delete and copy .rockbox |
23:36:01 | JdGordon| | oh, sure.. serious parts are fine |
23:36:07 | JdGordon| | but lets finish the current fight first! |
23:36:14 | stripwax | hehe |
23:36:23 | | Quit matsl (Read error: 110 (Connection timed out)) |
23:37:09 | * | kugel would like a playlist viewer item somewhere in the quinsert forest |
23:38:25 | kugel | and possibly a preview where after which song the first song of the current selection would be inserted (unless you insert shufflled that is) |
23:38:47 | kugel | but otoh I'm hardly using this insert features anyway ;) |
23:42:58 | | Join Hillshum [0] (n=hillshum@75-165-232-83.slkc.qwest.net) |
23:43:02 | | Quit evilnick_B ("Page closed") |
23:45:11 | stripwax | kugel - there's already a playlist viewer there, in the context menu. or do you mean 'to view a different playlist than the current one' ? |
23:45:43 | stripwax | kugel - oh you mean the thing where you can browse to the insertion point. yep. agreed. i want that too. |
23:46:34 | kugel | I mean "View current playlist" next to insert,queue |
23:47:52 | stripwax | that's already there, no? |
23:48:24 | CIA-85 | New commit by mcuelenaere (r23386): Add Lua support to plugins Makefile |
23:48:25 | kugel | not here |
23:48:27 | CIA-85 | New commit by mcuelenaere (r23387): Add new Lua touchscreen-only game: Boomshine ... |
23:49:10 | | Quit T44 (Connection timed out) |
23:49:36 | kugel | bertrik: any idea why your latest patch causes causes blue pixels and shifts? |
23:49:36 | | Join uflops [0] (n=yogurt@90-231-195-226-no112.tbcn.telia.com) |
23:49:50 | kugel | I mean, it's basically what I expected to happen, but you said it shouldn't happen in theory |
23:50:48 | bertrik | I don't know, we wait for FIFO empty before switching from write to read, but maybe there is still a write in progress when it indicates FIFO-empty |
23:51:23 | kugel | well, the obvious thing is that we're writing to DBOP_DOUT possibly during and lcd update |
23:51:55 | kugel | I suspected it would confuse the controller, but you said it should be fine |
23:51:57 | | Quit ender` (" All general statements are false. -- The Ultimate Law") |
23:53:08 | stripwax | kugel - hm, indeed. might as well merge a 'view playlist' option with an 'insert dynamically somewhere into the playlist' option, since you can always invoke the latter, view the playlist, and cancel |
23:53:47 | *** | Saving seen data "./dancer.seen" |
23:54:30 | | Join froggyman [0] (n=sopgenor@pool-72-69-220-194.chi01.dsl-w.verizon.net) |
23:54:57 | kugel | bertrik: maybe if we don't set enw? |
23:55:22 | | Part froggyman |
23:55:25 | | Join thegeek [0] (n=nnscript@s168c.studby.ntnu.no) |
23:57:34 | bertrik | I don't know, I won't look into it tonight anymore |