00:00:16 | | Quit matsl (Quit: Riece/3.1.2 XEmacs/21.5-b29 (garbanzo, linux)) |
00:00:39 | * | gevaerts likes the idea of "Sure, you can have five fonts on your WPS, but you'll have fewer images then!" :) |
00:01:14 | amiconn | Given the same total buffer size, a unified buffer makes better use of the available space |
00:01:36 | gevaerts | amiconn: http://pastebin.com/f7855b788 |
00:02:27 | | Quit Torne (Ping timeout: 256 seconds) |
00:02:30 | | Join Torne [0] (torne@rockbox/developer/Torne) |
00:03:20 | amiconn | Hmm, very interesting... |
00:03:31 | amiconn | The higher levels are seeing less speedup |
00:03:32 | *** | Saving seen data "./dancer.seen" |
00:03:47 | * | amiconn wonders what's going on there |
00:06:29 | amiconn | Hmm, D2 uses no IRAM? |
00:06:37 | * | amiconn suspects caching issues then |
00:07:06 | amiconn | How large is the TCC7801 cache? |
00:08:15 | | Quit Galois (Ping timeout: 260 seconds) |
00:08:20 | | Join piroko [0] (~jeremy@pohl.ececs.uc.edu) |
00:09:02 | piroko | It looks like the wiki section for ipod battery runtime hasn't been updated in a while. Anyone out there have a current approximate runtime for the 4G greyscale ipod? |
00:09:15 | piroko | (I guess I could just see how long mine lasts) :) |
00:10:57 | gevaerts | amiconn: from what I can see, 16K instruction and 16K data |
00:11:55 | gevaerts | piroko: that sounds like the best way. 2006 is indeed long ago... |
00:15:37 | amiconn | Hmm, that should at least be large enough to not slow down -c4000 |
00:16:05 | | Quit bertrik (Quit: De groeten) |
00:16:14 | amiconn | Anyway, it's an overall speedup (and -c5000 isn't really important on that cpu) |
00:19:20 | | Join Kitr88 [0] (Kitr88@BSN-182-136-96.dial-up.dsl.siol.net) |
00:20:24 | | Join toffe82 [0] (~chatzilla@ppp-69-238-93-63.dsl.frs2ca.pacbell.net) |
00:20:42 | | Join checker [0] (~62134578@giant.haxx.se) |
00:21:16 | checker | is there a file size minimum on .sbs? |
00:21:24 | | Quit Kitar|st (Ping timeout: 264 seconds) |
00:23:21 | CIA-88 | New commit by amiconn (r24590): APE: Fused vector math for the filters on ARMv5te. Speedup on Cowon D2 is ~4% for -c2000..-c4000 (less for -c5000). Thanks to Frank Gevaerts for ... |
00:23:40 | amiconn | gevaerts: Do you want to update the wiki, or shall I? |
00:23:51 | gevaerts | go ahead |
00:24:23 | checker | do "|" characters mess up the .sbs for some reason? |
00:24:46 | | Quit Frampis (Read error: Operation timed out) |
00:24:51 | JdGordon_ | checker: yes |
00:25:01 | JdGordon_ | use %| if you want a | to be displayed |
00:25:20 | JdGordon_ | that should be on the wiki page? |
00:26:02 | checker | ...... yes |
00:26:06 | checker | there it is :D |
00:27:00 | | Quit Guest83017 (Read error: Operation timed out) |
00:27:32 | checker | thanks |
00:28:51 | | Join feisar [0] (jljhook@irkki.fi) |
00:29:23 | | Nick feisar is now known as Guest38641 (jljhook@irkki.fi) |
00:29:27 | | Quit Sajber^ (Quit: Leaving.) |
00:30:01 | | Join Sajber^ [0] (~Sajber@h-65-75.A213.priv.bahnhof.se) |
00:30:44 | | Join Frampis [0] (famas@noppakerho.com) |
00:31:14 | checker | would a progress bar on the .sbs make file browsing much slower? |
00:31:43 | JdGordon_ | it shuoldnt |
00:31:56 | checker | scrolling text seems to take a toll |
00:32:27 | | Quit Guest38641 (Read error: Operation timed out) |
00:32:55 | JdGordon_ | scrolling text updates more frequently |
00:39:20 | | Quit bmbl (Quit: Bye!) |
00:39:24 | | Quit Frampis (Ping timeout: 264 seconds) |
00:45:41 | | Join feisar_ [0] (jljhook@irkki.fi) |
00:47:01 | | Quit jgarvey (Quit: Leaving) |
00:49:24 | | Join Frampis [0] (famas@noppakerho.com) |
00:52:09 | | Join togetic [0] (~togetic@unaffiliated/ibuffy) |
00:54:34 | | Quit soap (Remote host closed the connection) |
01:00 |
01:01:15 | | Join soap [0] (~soap@rockbox/staff/soap) |
01:04:41 | | Join Strife1989 [0] (~michael@adsl-154-22-173.mcn.bellsouth.net) |
01:07:25 | | Quit Strife89 (Ping timeout: 240 seconds) |
01:07:31 | | Nick fxb__ is now known as fxb (~felixbrun@h1252615.stratoserver.net) |
01:13:35 | | Quit avacore (Ping timeout: 260 seconds) |
01:13:37 | * | amiconn is puzzled |
01:14:37 | amiconn | Why does -c2000 (+19%) and -c3000 (+28%) become significantly faster when using fused vector math in the sim, but significantly slower for -c4000 (-13%) and -c5000 (-24%) ? |
01:19:21 | | Join Casainho [0] (~chatzilla@87.196.59.58) |
01:21:02 | | Nick Strife1989 is now known as Strife89 (~michael@adsl-154-22-173.mcn.bellsouth.net) |
01:30:17 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
01:39:15 | | Quit thegeek_ (Read error: Connection reset by peer) |
01:41:52 | | Join akur [0] (~akur@bl9-152-242.dsl.telepac.pt) |
01:42:30 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
01:42:40 | | Quit LambdaCalculus37 (Ping timeout: 245 seconds) |
01:43:59 | | Join kramer3d_ [0] (~kramer@unaffiliated/kramer3d) |
01:44:40 | | Join LambdaCalculus37 [0] (~rmenes@rockbox/staff/LambdaCalculus37) |
01:44:47 | | Quit linuxstb (Read error: Operation timed out) |
01:46:10 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
01:48:40 | | Quit petur (Quit: Zzzz) |
01:49:33 | | Quit BHSPitMonkey (Ping timeout: 256 seconds) |
01:49:33 | | Quit Casainho (Ping timeout: 256 seconds) |
01:49:47 | | Nick kramer3d_ is now known as kramer3d (~kramer@unaffiliated/kramer3d) |
01:50:35 | | Join Galois [0] (djao@efnet.math.uwaterloo.ca) |
01:54:11 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
02:00 |
02:02:20 | | Quit BlakeJohnson86 (Quit: Leaving.) |
02:03:05 | Adnyxo | kramer mod? |
02:03:34 | *** | Saving seen data "./dancer.seen" |
02:03:47 | | Join BlakeJohnson86 [0] (~bjohnson@2002:1876:a27b:0:227:13ff:fe65:1262) |
02:05:10 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
02:16:21 | | Quit LambdaCalculus37 (Quit: Fwump) |
02:19:04 | | Quit kaniini (Ping timeout: 240 seconds) |
02:25:25 | | Nick fxb is now known as fxb__ (~felixbrun@h1252615.stratoserver.net) |
02:26:00 | | Quit dfkt (Quit: -= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.) |
02:29:25 | | Quit Sajber^ (Read error: Connection reset by peer) |
02:33:35 | | Quit linuxstb (Read error: Operation timed out) |
02:37:58 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
02:44:42 | | Quit komputes (Quit: I haven't slept for ten days, because that would be too long.) |
02:57:43 | | Quit Rob2222 (Quit: Rob2222) |
03:00 |
03:01:25 | | Join drf|away [0] (~Your_Mom@markley-135168.reshall.umich.edu) |
03:01:51 | drf|away | So guys. I managed to get a v1 Sansa Fuze. However, I installed the bootloader, it said it installed correctly, and it's still booting the OF. |
03:02:09 | | Quit akur (Quit: Leaving.) |
03:04:12 | drf|away | oh never mind, I selected the wrong drive... I'm stupid |
03:04:48 | | Join iq [0] (~chatzilla@unaffiliated/iq) |
03:04:50 | iq | Hi |
03:08:18 | | Part froggyman |
03:12:36 | | Part drf|away |
03:14:22 | | Quit kramer3d (Ping timeout: 252 seconds) |
03:15:24 | | Quit Kitr88 (Read error: Connection reset by peer) |
03:15:55 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
03:16:41 | | Quit moos (Remote host closed the connection) |
03:18:51 | | Quit Adnyxo (Ping timeout: 265 seconds) |
03:22:11 | | Join Rob2222 [0] (~Miranda@p4FDC9E7D.dip.t-dialin.net) |
03:27:01 | | Quit linuxstb (Ping timeout: 276 seconds) |
03:32:42 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
03:40:52 | | Join docgnome [0] (~docgnome@174.120.42.4) |
03:42:11 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
03:44:12 | docgnome | someone recommend an cheap music player that can run rockbox? |
03:51:00 | krazykit` | basically every supported sandisk player |
03:52:08 | docgnome | k cool |
03:52:29 | | Quit DerPapst (Quit: Leaving.) |
03:53:26 | | Join CaptainKewl [0] (jds@207-237-117-89.c3-0.80w-ubr2.nyr-80w.ny.cable.rcn.com) |
03:54:19 | | Quit linuxstb (Ping timeout: 245 seconds) |
03:56:26 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
03:59:11 | checker | #progress bar %V|45|302|150|1|0|003031|C6CFD6|-| %pb|rockitlrspb.bmp|44|301|152|7| is there a problem with this? or i guess, what is the problem lol |
03:59:23 | checker | for .sbs |
03:59:30 | | Part docgnome |
04:00 |
04:02:56 | | Join dude187 [0] (~chris@cpe-75-187-48-206.columbus.res.rr.com) |
04:03:05 | checker | wait i see the image is now outside of the viewport |
04:03:37 | *** | Saving seen data "./dancer.seen" |
04:07:03 | checker | #progress bar %V|44|301|150|7|0|003031|C6CFD6|-| %pb|rockitlrspb.bmp|45|300|150|5| ok this now... |
04:07:07 | checker | still doesnt work |
04:11:17 | | Quit panni_ (Read error: Connection reset by peer) |
04:12:29 | checker | x and y values are measured from within the viewport... duh. so 1,1 centers it and fixes it. solved. |
04:14:04 | | Quit iq (Quit: ChatZilla 0.9.86 [Firefox 3.5.5/20091102152451]) |
04:20:02 | | Quit linuxstb (Read error: Operation timed out) |
04:20:02 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
04:21:42 | | Quit TheSeven (Disconnected by services) |
04:21:42 | | Join The_Seven [0] (~theseven@rockbox/developer/TheSeven) |
04:21:42 | | Nick The_Seven is now known as TheSeven (~theseven@rockbox/developer/TheSeven) |
04:22:34 | | Join earHurts [0] (~earHurts@47.sub-75-197-250.myvzw.com) |
04:23:04 | earHurts | The ipod video sumulator: how do I "reboot" it |
04:23:36 | | Nick dmb is now known as d (~Dmb@unaffiliated/dmb) |
04:24:04 | | Nick d is now known as ______dmb (~Dmb@unaffiliated/dmb) |
04:28:12 | | Quit antil33t (Read error: Connection reset by peer) |
04:30:53 | | Quit linuxstb (Ping timeout: 258 seconds) |
04:31:40 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
04:33:46 | | Join S_a_i_n_t [0] (S_a_i_n_t@203.184.0.254) |
04:34:10 | S_a_i_n_t | JdGordon: ping |
04:34:33 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
04:34:38 | | Quit dude187 (Ping timeout: 276 seconds) |
04:35:24 | | Join dude187 [0] (~chris@cpe-75-187-48-206.columbus.res.rr.com) |
04:37:29 | S_a_i_n_t | r24581, Nano 1g, context menyu option "set as backdrop" fails to set the backdrop with the message "Backdrop Failed"...is the a known issue by now? |
04:39:14 | S_a_i_n_t | also, r24581, Nano1g, plugin "mandlebrot" hangs on a solid black screen and causes the player to need a hard-restart. |
04:41:45 | | Quit antil33t (Read error: Connection reset by peer) |
04:44:06 | JdGordon | S_a_i_n_t: burp |
04:44:53 | S_a_i_n_t | did you know about the backdrop failing to set via the context menu already? |
04:45:17 | JdGordon | no? |
04:45:26 | S_a_i_n_t | well, now you do. |
04:45:31 | S_a_i_n_t | 24581 |
04:45:34 | JdGordon | file it |
04:45:42 | S_a_i_n_t | fails with message "backdrop failed" |
04:45:50 | S_a_i_n_t | will do. |
04:45:51 | | Quit Barahir (Ping timeout: 265 seconds) |
04:46:49 | checker | anyone know anything about a section of album art disappearing after key movement on the .wps on the gigabeat f20? |
04:47:30 | | Join Barahir [0] (~jonathan@gssn-5f755c8d.pool.mediaWays.net) |
04:47:56 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
04:53:11 | S_a_i_n_t | checker: whick theme are you using? |
04:53:15 | S_a_i_n_t | *which |
04:53:48 | S_a_i_n_t | theme/WPS |
04:53:51 | checker | one in the making :) |
04:54:20 | S_a_i_n_t | are you sure that it's not an overlapping viewport then? |
04:54:29 | S_a_i_n_t | seems quite possible to me |
04:54:33 | earHurts | in the simulator, is there an easy way to print to stdout/stderr? |
04:55:04 | checker | they appear in random spots, its full screen width and about 15px high |
04:55:25 | checker | random spots in the album art area* |
04:55:30 | | Join webguest98 [0] (~4bc5fa2f@giant.haxx.se) |
04:55:46 | earHurts | xxx |
04:56:21 | checker | i dont see any over lapping |
04:56:21 | | Quit webguest98 (Client Quit) |
04:57:03 | checker | it only happens which the volume indicator switchs to the next part of the strip bump, and when i fast forward or rewind |
04:57:06 | S_a_i_n_t | checker: can you veriffy that it happens with Cabbiev2? |
04:57:13 | S_a_i_n_t | *the default WPS |
04:57:16 | checker | let me see |
04:57:23 | S_a_i_n_t | if not, I'd be looking at your code |
04:57:44 | checker | nope its good |
04:58:20 | checker | it goes away after switching to the menu and back |
04:58:27 | checker | on mine* |
04:59:20 | S_a_i_n_t | well, I'd be wanting to look at your WPS code then, as I suspect you've not done something correctly. |
04:59:33 | S_a_i_n_t | I could have a look at it, but can't promise anything. |
04:59:42 | checker | k let me upload it somewhere |
04:59:45 | S_a_i_n_t | pastebin the WPS code |
04:59:48 | | Join dtube [0] (~dtube@adsl-75-4-215-131.dsl.irvnca.sbcglobal.net) |
05:00 |
05:00:37 | checker | pastebin? |
05:00:48 | * | fleebailey33 points at desowin |
05:00:51 | * | fleebailey33 points at dtube |
05:00:56 | S_a_i_n_t | www.pastebin.com |
05:01:54 | checker | k done, but how do i link that to you? |
05:02:03 | S_a_i_n_t | Backdrop issue filed as FS #10996 |
05:02:14 | checker | http://fr.pastebin.ca/1793460 |
05:02:50 | S_a_i_n_t | k, I'll have a look now...be patient tho :P |
05:02:54 | S_a_i_n_t | *though |
05:03:05 | | Quit anewuser (Ping timeout: 240 seconds) |
05:09:28 | | Quit linuxstb (Ping timeout: 246 seconds) |
05:10:42 | S_a_i_n_t | checker: Hmmmm, without knowing your image dimensions...I can't tell if there are any overlaps, the rest of the code *seems* sound (at a first glance) |
05:11:21 | checker | there shouldn't be any overlaps, i dont think thats the problem |
05:11:57 | checker | but its got to be theme related because cabbie doesnt do it... |
05:12:00 | S_a_i_n_t | have you run it in a SIM, or just on the device? |
05:12:12 | checker | just device |
05:12:36 | | Quit MethoS- (Remote host closed the connection) |
05:12:39 | S_a_i_n_t | Aha....I'll try the SIM myself, and see if that gives me any output about what's wrong. |
05:13:08 | checker | you want the images? |
05:13:42 | S_a_i_n_t | please...upload the whole WPS (datafilehost.com is good) |
05:13:59 | checker | ok |
05:14:29 | checker | you want it in a zip |
05:14:49 | S_a_i_n_t | zip, 7z...whatever's good for you |
05:15:07 | checker | ok, 1sec |
05:18:32 | checker | hope i got it all, http://www.datafilehost.com/download-c8626d75.html |
05:19:40 | S_a_i_n_t | sweet, thanks...I'm pottering between jobs, so just stay online and I'll PM you shortly(-ish) |
05:19:51 | checker | i g2g in 5min |
05:20:06 | S_a_i_n_t | PM me your email |
05:20:12 | checker | how do i pm |
05:20:14 | checker | :P |
05:23:11 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
05:23:30 | | Quit n17ikh (Ping timeout: 245 seconds) |
05:24:17 | | Quit checker (Quit: CGI:IRC (EOF)) |
05:28:22 | | Join n17ikh [0] (~n17ikh@host-69-59-126-212.nctv.com) |
05:33:29 | | Quit kramer3d (Ping timeout: 240 seconds) |
05:35:09 | | Quit earHurts (Ping timeout: 265 seconds) |
05:38:53 | | Quit Strife89 (Quit: Bed.) |
05:40:04 | | Quit Horscht (Quit: Verlassend) |
05:42:04 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
05:49:29 | | Quit vegtoruci (Ping timeout: 240 seconds) |
06:00 |
06:01:11 | | Join kaniini [0] (~kaniini65@dyn75-70.yok.fi) |
06:03:40 | *** | Saving seen data "./dancer.seen" |
06:08:11 | | Quit kramer3d (Ping timeout: 252 seconds) |
06:10:03 | | Quit Xerion (Read error: Connection reset by peer) |
06:10:48 | | Join Xerion [0] (~xerion@82-170-197-160.ip.telfort.nl) |
06:22:31 | | Quit tha (Ping timeout: 265 seconds) |
06:23:04 | | Join tha [0] (1038@ccc2.rbg.informatik.tu-darmstadt.de) |
06:30:18 | | Join framo4 [0] (~chatzilla@ip-87-82-152-129.easynet.co.uk) |
06:30:24 | | Quit FlynDice (Remote host closed the connection) |
06:38:11 | | Quit framo4 (Quit: muuh) |
06:38:23 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
06:39:21 | | Quit FlynDice (Remote host closed the connection) |
06:39:25 | | Quit nimak (Read error: Connection reset by peer) |
06:41:08 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
06:41:23 | | Quit linuxstb (Ping timeout: 256 seconds) |
06:42:09 | | Join nima [0] (~nima@adsl-75-45-226-145.dsl.sfldmi.sbcglobal.net) |
06:57:29 | | Quit FlynDice (Remote host closed the connection) |
06:58:48 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
06:59:20 | | Quit FlynDice (Remote host closed the connection) |
07:00 |
07:04:17 | | Quit desowin (Ping timeout: 240 seconds) |
07:04:25 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
07:04:56 | | Join desowin [0] (~desowin@atheme/member/desowin) |
07:13:22 | | Join midgey [0] (~tjross@rockbox/developer/midgey) |
07:15:26 | midgey | saratoga: (for the logs) ipod 4g test_codec results for mdctexp branch -> http://rockbox.pastebin.ca/1793531 |
07:16:36 | | Quit togetic (Quit: WeeChat 0.3.0) |
07:17:16 | | Join togetic [0] (~togetic@unaffiliated/ibuffy) |
07:20:19 | | Quit midgey (Quit: midgey) |
07:20:28 | | Quit antil33t (Read error: Connection reset by peer) |
07:22:11 | | Quit BHSPitMonkey (Ping timeout: 252 seconds) |
07:25:18 | pixelma | S_a_i_n_t: mandelbrot has been renamed to fractals, if you still have mandelbrot on your player (and you mean that) it will be outdated and you should remove it. I'm not sure why it should show a black screen but maybe there was a bug in the version you installed last |
07:26:18 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
07:27:42 | | Join BHSPitMonkey [0] (~stephen@unaffiliated/bhspitmonkey) |
07:29:31 | S_a_i_n_t | pixelma: I built it from current SVN r24581 I believe |
07:29:41 | S_a_i_n_t | that's why I thought it was odd... |
07:30:55 | S_a_i_n_t | yes, 24581...that's how it compiled, *perhaps* something went wrong on my end, but I wanted to check first. |
07:31:14 | pixelma | your new version does not contain a mandelbrot anymore and the one you have there is from before... |
07:31:37 | pixelma | before the rename I mean |
07:34:03 | S_a_i_n_t | what I mean was, I'm pretty damn sure it compiled with both mandlebrot and fractals, and I deleted my .rockbox from my DAP before moving on the new build...it *must* of messed up somewhere in the process of colpiling I guess. I think I'll do a complete new checkout. |
07:34:35 | S_a_i_n_t | s/colpiling/compiling/ |
07:35:46 | JdGordon | anyone game enough to try multifont on target? |
07:36:32 | S_a_i_n_t | yes, I haven;t checked the Forums yet however... |
07:36:46 | S_a_i_n_t | I was unsure how the process went. |
07:40:18 | | Join teru [0] (~teru@KD059133108225.ppp.dion.ne.jp) |
07:40:39 | S_a_i_n_t | Hmmm..so after the font is declared to a slot, the viewport can then read as "%Vx|y|x|y|2|-|-|" "%Vx|y|x|y|3|-|-|" etc. up to 5? |
07:40:41 | S_a_i_n_t | correct? |
07:40:57 | S_a_i_n_t | If so, I'll quickly hack up one of my existing WPS's |
07:40:58 | JdGordon | yeah |
07:41:18 | S_a_i_n_t | cool, I won't get cocky..I'll just try 2 user for now |
07:41:26 | JdGordon | 1 is always th UI font for the screen, 2-X is custom fonts |
07:41:35 | JdGordon | X being unknown just yet |
07:42:34 | JdGordon | the font must obviously be loaded before the %V line |
07:42:42 | S_a_i_n_t | sweet...it'll take some time to compile etc. but you'll know in 30mins or so. |
07:43:11 | JdGordon | comment out firmware/powermanagements.c line 747 or it wont compile |
07:43:24 | S_a_i_n_t | sweet, thanks. |
07:43:38 | JdGordon | and dont use the version i uploaded earier, use the one from yesterday |
07:43:42 | JdGordon | thats one is broken :p |
07:44:23 | S_a_i_n_t | err...so the *first* post of the patch? |
07:45:20 | S_a_i_n_t | or second rather? |
07:45:40 | S_a_i_n_t | Wednesday, 10 February 2010, 09:35 GMT+1 |
07:45:46 | S_a_i_n_t | that one? |
07:45:47 | JdGordon | neither :) the one I'm about to upload |
07:45:55 | S_a_i_n_t | ah.....right :D |
07:48:04 | JdGordon | ok |
07:48:08 | JdGordon | go for it |
07:48:45 | S_a_i_n_t | #ifdef HAVE_LCD_BITMAP |
07:48:45 | S_a_i_n_t | #endif ? |
07:49:02 | JdGordon | dont touch anything. just compile it |
07:49:02 | S_a_i_n_t | ooops, /* glyph_cache_save();*/ |
07:49:17 | S_a_i_n_t | ah...ok, :P |
07:49:20 | | Quit FlynDice (Remote host closed the connection) |
07:50:06 | S_a_i_n_t | so...don;t comment out that line in firmware/powermanagement.c? |
07:50:18 | S_a_i_n_t | that's what I was trying to confirm. |
07:51:48 | S_a_i_n_t | ....sorry, do or don't edit 747 of firmware/powermanagement.c? |
07:52:48 | | Join Zagor [0] (~bjst@46.35.227.87.static.tab.siw.siwnet.net) |
07:52:48 | | Quit Zagor (Changing host) |
07:52:48 | | Join Zagor [0] (~bjst@rockbox/developer/Zagor) |
07:53:09 | JdGordon | dont touch it |
07:53:18 | JdGordon | serisouly... it will rape your cat if you do! |
07:56:20 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
07:56:34 | S_a_i_n_t | well, compiling now....I'll let you know, or screendump a WPS for you whem I've finished |
07:56:42 | S_a_i_n_t | but it's CygWin remember :P |
07:56:47 | S_a_i_n_t | *sloooooow |
07:59:40 | JdGordon | that patch will give you 7 fonts to play with or 6 if you have a target with a remote lcd |
07:59:43 | | Part toffe82 |
08:00 |
08:01:00 | S_a_i_n_t | Hmmmmm, well, I'll try for two...just to be safe, and to stop me from writing a new WPS (even if it is a basic one :P ), I can just add 2 lines to my existing one, and change 2 numbers |
08:01:45 | S_a_i_n_t | actually, I can do 4 at once |
08:02:02 | S_a_i_n_t | so, bah...we'll se anyway. (fingers crossed) |
08:02:18 | pixelma | S_a_i_n_t: could you check file dates of your mandelbrot and fractals.rock? |
08:03:08 | | Join einhirn [0] (~Miranda@bsod.rz.tu-clausthal.de) |
08:03:41 | *** | Saving seen data "./dancer.seen" |
08:04:57 | JdGordon | S_a_i_n_t: let me know if you find it doing more disk access then usual |
08:05:27 | JdGordon | pixelma: any chance you can test the fm patch today? if not then I'll just go ahead and commit it and fix any bugs which get found |
08:05:28 | | Quit nima (Read error: Connection reset by peer) |
08:06:13 | | Join nima [0] (~nima@75.45.226.145) |
08:06:19 | S_a_i_n_t | 11/02/2010 4:11am |
08:06:39 | | Join stoffel [0] (~quassel@p57B4E5B2.dip.t-dialin.net) |
08:06:52 | S_a_i_n_t | for both, I doubt it's a code fault...I'm guessing something messed up while compiling. |
08:07:32 | S_a_i_n_t | I've checked out a new/clean source, if it doesn;t happen again...I'll put it down to a dirty SVN |
08:22:01 | JdGordon | Unhelpful: ping? |
08:22:50 | S_a_i_n_t | JdGordon: after specifying "%Fl|2|12-fontname|" then the following viewport can use that font, but if a viewport is declared after that which just specifies "1" -user, I wouldn't have to use the %Fl line to load that right? I assume o and 1 are always loaded? |
08:23:11 | JdGordon | yes |
08:23:18 | S_a_i_n_t | *0 and 1 |
08:23:39 | JdGordon | 0 is always sysfont, 1 is always the ui font (for that screen), 2-9 are the available skin fonts |
08:24:45 | S_a_i_n_t | sweet, and any viewport that isn't 0 or 1 needs the %Fl line? or they all do now? |
08:26:12 | JdGordon | the %Fl is to load a font, its not linked to a viewport |
08:26:24 | JdGordon | if you want a use a font other than 0,1 you need a Fl |
08:26:26 | JdGordon | otherwise you dont |
08:26:48 | S_a_i_n_t | right, I get it now...they all load at once, not per viewport. |
08:27:14 | JdGordon | yes, and before you ask, each skin needs those lines, and the numbers can be different |
08:27:26 | JdGordon | each font will only be loaded once, even if both screens want it |
08:27:28 | S_a_i_n_t | coolies. |
08:27:35 | JdGordon | s/screens/skins/ |
08:28:12 | | Join stooo [0] (~sto@g228017144.adsl.alicedsl.de) |
08:32:42 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
08:35:15 | | Quit stooo (Quit: Leaving.) |
08:49:28 | | Join petur [0] (~petur@rockbox/developer/petur) |
08:52:58 | pixelma | I find this way of setting more fonts very complicated |
08:53:32 | pixelma | JdGordon: I could in the evening if you remind me |
08:54:26 | JdGordon | which way of setting fonts? |
08:54:46 | | Quit amiconn (Disconnected by services) |
08:54:49 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
08:55:10 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
08:55:26 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
08:55:26 | | Quit pixelma (Disconnected by services) |
08:55:35 | | Quit BHSPitMonkey (*.net *.split) |
08:55:36 | | Quit togetic (*.net *.split) |
08:55:36 | | Quit n17ikh (*.net *.split) |
08:55:36 | | Quit Topy44 (*.net *.split) |
08:55:36 | | Quit mc2739 (*.net *.split) |
08:55:36 | | Quit Zambezi (*.net *.split) |
08:55:36 | | Quit sinuc (*.net *.split) |
08:55:36 | | Quit crashd (*.net *.split) |
08:55:36 | | Quit xavieran (*.net *.split) |
08:55:36 | | Quit jvd (*.net *.split) |
08:55:36 | | Quit Kopfgeldjaeger (*.net *.split) |
08:55:41 | | Join Zambezi [0] (Zulu@80.67.9.2) |
08:55:44 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
08:57:11 | | Quit linuxstb (Ping timeout: 265 seconds) |
08:57:41 | | Join TopyMobile [0] (~topy@f054227083.adsl.alicedsl.de) |
08:58:52 | | Join crashd [0] (foobar@lostnode.org) |
08:58:58 | | Join sinuc [0] (~sinuc@apokryph.org) |
08:59:19 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
08:59:30 | | Join jvd [0] (~syscrash@poipu/developer/syscrash) |
09:00 |
09:01:07 | | Join flydutch [0] (~flydutch@host66-209-dynamic.15-87-r.retail.telecomitalia.it) |
09:02:44 | | Join n17ikh [0] (~n17ikh@host-69-59-126-212.nctv.com) |
09:04:03 | | Join togetic [0] (~togetic@unaffiliated/ibuffy) |
09:04:56 | | Join xavieran [0] (~xavieran@ppp118-209-153-106.lns20.mel6.internode.on.net) |
09:05:01 | JdGordon | pixelma: whats a better way to set the fonts |
09:05:03 | JdGordon | ? |
09:06:03 | * | S_a_i_n_t thinks it's fine....uploading "proof of concept" now. 5 fonts onscreen :D |
09:07:23 | | Join ender` [0] (krneki@foo.eternallybored.org) |
09:08:17 | S_a_i_n_t | JdGordon: Anyone: http://imgur.com/cMQyM.png |
09:08:25 | S_a_i_n_t | 5 user fonts onscreen. |
09:08:38 | * | JdGordon sees garbage text |
09:08:52 | JdGordon | or was that just scrolling? |
09:09:33 | | Join BHSPitMonkey [0] (~stephen@pool-71-170-176-30.dllstx.fios.verizon.net) |
09:09:34 | | Join Topy44 [0] (~topy@my.fastsh.it) |
09:09:37 | * | S_a_i_n_t just noticed that too....it *was* scrolling while I screendumped |
09:09:45 | S_a_i_n_t | weird it's always that line though |
09:10:31 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
09:10:34 | S_a_i_n_t | I'll upload a better pic, it was just because it's scrolling |
09:10:36 | pixelma | JdGordon: maybe with a font browser or a "set as" context menu, not sure. My main problem with this is that it looks like it could be quite complicated if you want to use e.g. Clip themes as a rwps for the Iriver remote and vice versa which should be possible with multifont |
09:10:38 | S_a_i_n_t | *phew* |
09:11:01 | JdGordon | pixelma: wouldnt that make it harder |
09:11:02 | JdGordon | ? |
09:11:21 | S_a_i_n_t | pixelma: it's not complicated at all...just set all the fonts you want to use right at the start of the WPS...that's what I did. |
09:11:24 | S_a_i_n_t | works fine |
09:11:25 | JdGordon | this way the fonts are directly linked to the sckin |
09:12:23 | pixelma | well, can you have two "font #1"s? |
09:12:41 | pixelma | or #2 etc. |
09:12:57 | S_a_i_n_t | works fine for me, I can't imagine a better system...a context menu (esp for the WPS) would be chaos...and not sure how you'd do it either. |
09:13:05 | JdGordon | font 3 in the wps and font 3 in the sbs are not necessarily the same font |
09:13:19 | JdGordon | this way there is no guessing. You use the font number at the top of this skin file |
09:14:02 | JdGordon | the numbers in the skins are just identifiers. I could change it to letters to make it more obvious, but thats a bit complicated |
09:14:12 | S_a_i_n_t | http://imgur.com/AOM6L.png <- *BETTER* proof of concept.... JdGordon, start gloating now :-D |
09:16:00 | JdGordon | maybe the font id's should be letters like image id's. Just to make it hugely obvious what they are |
09:16:05 | pixelma | so for a target with an lcd remote you could use - I don't know - 10 different fonts? (assuming 5 different ones for each screen)? |
09:16:16 | JdGordon | sure |
09:16:19 | pixelma | at the same time that would be |
09:16:36 | JdGordon | font 1 will always be the ui font from the setting for that screen though |
09:16:49 | S_a_i_n_t | JdGordon: I think numbers are more obvious...as then it fits with the current 0 and 1 font scheme |
09:17:15 | S_a_i_n_t | it's easy to remember that "next user font is 2, then 3 and so on" etc. |
09:17:18 | S_a_i_n_t | IMHO |
09:17:41 | JdGordon | not easier than a,b,c |
09:17:54 | JdGordon | or s,m,l |
09:18:14 | pixelma | JdGordon: ok, then there's "only" one problem - you can't set a user font for the remote independently yet |
09:18:22 | S_a_i_n_t | but it *does* fit with the current scheme of the fonts |
09:18:53 | JdGordon | pixelma: yes, and I have no idea how to do that. I'm tihnking a context menu option on .fnt files, but thats not really nice |
09:23:40 | pixelma | every Clip theme would still have problems with viewports selecting the user font. I would expect a remote user font setting on targets with a remote (much like other remote lcd settings). And then an rwps would use that |
09:24:46 | S_a_i_n_t | this is still early days... |
09:25:22 | JdGordon | pixelma: if you manually set the remote font using the "remote font" cfg setting, then that will just work |
09:25:33 | pixelma | as long as it isn't set explicitly it could be the same as the main target user font |
09:25:43 | JdGordon | yes |
09:26:02 | | Quit CaptainKewl (Quit: ( www.nnscript.com :: NoNameScript 4.22 :: www.esnation.com )) |
09:26:04 | pixelma | there is no "remote font" yet |
09:26:23 | JdGordon | but even then. if the skin works on the clip it will work on the h300 remote |
09:26:27 | JdGordon | there is in the patch |
09:27:13 | pixelma | cfg only settings :\ |
09:27:40 | | Quit liar (Ping timeout: 245 seconds) |
09:27:48 | JdGordon | pixelma: yes, and I have no idea how to do that. I'm tihnking a context menu option on .fnt files, but thats not really nice |
09:28:42 | JdGordon | the font browser right now uses the file browser to load the .fnt directly into the setting. I need to somehow make it smarter |
09:29:05 | pixelma | I was thinking a "Remote font" browser |
09:29:35 | JdGordon | yes, but thats not possible right now |
09:29:37 | pixelma | you have a "Remote backlight timeout" setting too |
09:30:10 | JdGordon | we can easily use the file browser for the skins because the main and remote skins have different extensions |
09:30:21 | | Join avacore [0] (nobody@1008ds1-rdo.0.fullrate.dk) |
09:32:28 | pixelma | if you introduced the remote font setting, what is the problem making a second font browser for it? Really just out of curiosity |
09:34:42 | | Quit stoffel (Remote host closed the connection) |
09:34:53 | JdGordon | the equivilant of windows opening .xls in excell happens. .fnt is hardcoded to always be loaded into the main screen font setting. I need to break that assosiation somehow, and keep it simple |
09:35:20 | JdGordon | it's not a matter of not wanting the browser, its a matter of not being able to use it |
09:38:08 | pixelma | and how does your cfg remote font setting work? I'm sorry, I can't imagine this now |
09:39:09 | JdGordon | its identical to the font setting, just with the word remote on the front |
09:39:34 | JdGordon | remote font: /.rockbox/fonts/16-GNU-Unifont.fnt |
09:39:43 | pixelma | I meant if this works for .fnt why would a browser not work? |
09:40:45 | JdGordon | I'm not sure how else to explain it |
09:41:11 | JdGordon | the font browser is literally just opening up the file browser in the font directory |
09:41:32 | JdGordon | pressing select tells the browser to do the select action on the filetype, which is .fnt |
09:44:32 | pixelma | aha |
09:48:42 | rasher | There's some context for the file browser |
09:48:53 | rasher | It could know that it's browsing for remote fonts |
09:49:10 | rasher | and not just call the open function |
09:50:42 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
09:52:33 | * | JdGordon needs to think about it for a bit |
09:52:46 | | Quit Zambezi (Changing host) |
09:52:47 | | Join Zambezi [0] (Zulu@unaffiliated/zambezi) |
09:59:28 | | Quit dtube (Quit: Leaving) |
10:00 |
10:01:40 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
10:03:44 | *** | Saving seen data "./dancer.seen" |
10:12:08 | | Join _zic [0] (~user@91-165-236-145.rev.libertysurf.net) |
10:12:33 | | Quit AndyI (Ping timeout: 248 seconds) |
10:13:36 | | Quit linuxstb (Ping timeout: 264 seconds) |
10:17:46 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
10:30:02 | | Join FOAD_ [0] (~dok@dinah.blub.net) |
10:31:21 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
10:33:03 | | Quit linuxstb (Read error: Operation timed out) |
10:33:30 | | Quit FOAD (Ping timeout: 245 seconds) |
10:33:30 | | Nick FOAD_ is now known as FOAD (~dok@dinah.blub.net) |
10:39:06 | | Quit BHSPitMonkey (Remote host closed the connection) |
10:49:00 | | Quit sinuc (Changing host) |
10:49:01 | | Join sinuc [0] (~sinuc@unaffiliated/sinuc) |
11:00 |
11:01:30 | | Join ranmachan [0] (ranma@yumi.tdiedrich.de) |
11:03:11 | amiconn | JdGordon: You could make the .fnt association show a mini-menu on multiscreen targets |
11:03:13 | ranmachan | Hi there. Any sansa c200v2 experts in here? |
11:03:41 | ranmachan | I followed the instructions on http://www.rockbox.org/wiki/SansaAMS for "Installation for Unusable (other AMS Sansas)" |
11:05:01 | ranmachan | The dualboot would still always load the OF, so I removed the 'boot of if USB is plugged in' check and recompiled mkamsboot |
11:05:38 | ranmachan | Now it tries to boot rockbox, but apparently panics, blinking out an error code with the two leds. |
11:05:47 | ranmachan | If I poweron while holding right it boots the OF |
11:06:01 | ranmachan | Are those panic codes documented somewhere? |
11:06:29 | ranmachan | I assume it's bootloade-c200v2.sansa that is panicing here? |
11:22:20 | | Join Casainho [0] (~chatzilla@87.196.59.58) |
11:23:57 | | Join FOAD__ [0] (~dok@dinah.blub.net) |
11:27:36 | | Quit FOAD (Ping timeout: 256 seconds) |
11:27:36 | | Nick FOAD__ is now known as FOAD (~dok@dinah.blub.net) |
11:41:08 | CIA-88 | New commit by teru (r24591): fix text scrolling handling in do_menu, set_time_screen and time_screen. ... |
11:47:14 | * | _zic really thinks that the usb stack should be enabled by default in the next stable release, as it works just *fine* |
11:47:25 | _zic | why is it still not in 3.5 ? |
11:47:41 | topik | where does it work *fine* ? |
11:47:42 | gevaerts | it is, unless you say which target you're talking about |
11:48:02 | _zic | ipod nano 1st gen |
11:48:18 | Torne | _zic: because the ipods do not charge properly in Rockbox |
11:48:31 | Torne | and it's possible for them to end up *discharging* if you use them to transfer files over USB for an extended period |
11:49:25 | Torne | also it's generally believed to be measurably slower than the original firmware |
11:49:33 | Torne | though this might not actually be true for all models :) |
11:49:42 | _zic | yes, that used to be something like several weeks ago, but yesterday my device charged well while connected to the computer |
11:49:54 | Torne | _zic: i can assure you it *doens't* charge properly |
11:50:10 | Torne | at best it will charge at something like one eighth the speed of the original firmware |
11:50:25 | Torne | it depends what you are doing and what your settings are |
11:50:49 | Torne | the hard disk ipods can easily discharge at quite a rapid rate while plugged in if you are copying files; that's probably not likely on the flash-based ones but they still charge very slowly |
11:52:27 | _zic | ok |
11:52:37 | Torne | if you leave it on charge overnight you won't notice :) |
11:52:53 | Torne | but in the original firmware you can generally recharge any of the ipods in well under two hours, or even under one |
11:52:56 | Torne | from flat to full |
11:53:14 | Torne | (well, ok, probably from 10-90) |
11:53:41 | gevaerts | Torne: while the rockbox USB stack is indeed measurably slower than the OF on some ipods, I claim that it's only 10% to 20% slower. The disk drivers however... |
11:53:46 | * | gevaerts hides |
11:53:59 | Torne | gevaerts: you mean the part that's blame-able on USB? :) |
11:54:06 | Torne | rather than the actual end user observable peformance? :) |
11:54:11 | Torne | that's very helpful. *g* |
11:54:19 | gevaerts | Torne: well yes :) |
11:54:29 | Torne | i tried to make the disk faster! |
11:54:35 | Torne | but it doesn't bloody work :) |
11:54:44 | gevaerts | We should do some new measurements. I don't know the full impact of your changes |
11:54:54 | Torne | yes, we probably should |
11:55:01 | Torne | the WRITE MULTIPLE change might've helped |
11:55:06 | Torne | should've helped :) |
11:55:32 | Torne | DMA definately made read performance much better over USB, possibly it's now comparable to the OF even |
11:55:43 | Torne | but people are generally more concerned about USB write speed on DAPs |
11:56:11 | Torne | if you can suggest a good benchmarking methodology i will experiment :) |
11:56:36 | gevaerts | I usually just use dd |
11:57:56 | gevaerts | "dd if=/dev/sdipod of=/tmp/ipod.img bs=64k count=100 && dd if=/tmp/ipod.img of=/dev/sdipod bs=64k count=100" |
11:58:20 | gevaerts | Always do a quick read before writing to spin the disk |
11:58:45 | * | Torne notes, incidentally, that the disk performance isn't quite linear :) |
11:59:03 | gevaerts | I know, I don't care :) |
11:59:10 | Torne | the beginning is the fastest bit |
11:59:20 | Torne | i guess that's probably what we do want to compare, though |
11:59:35 | Torne | the slowdown over the rest of the surface is uninteresting because that's a fact of mechanics |
12:00 |
12:00:00 | Torne | it limits all disk access impartially :) |
12:00:29 | gevaerts | It might be useful to measure at the end of the disk to see what happens if speed isn't USB-bound anymore |
12:00:39 | Torne | maybe. |
12:00:54 | Torne | i have a test_disk patch that measures read speed across the entire surface (sampling at regular intervals) |
12:01:18 | Torne | i haven't actually put it in because it's ugly and relies on exporting storage_read_sectors to pluginapi |
12:01:30 | Torne | also it has an option to do the same for writes which destroys your data :) |
12:02:03 | Torne | .oO( is there a valid case for adding storage_(read|write)_sectors to pluginapi? ) |
12:03:48 | *** | Saving seen data "./dancer.seen" |
12:04:02 | gevaerts | I'd say yes |
12:04:13 | gevaerts | A partitioning and formatting plugin has its uses |
12:04:23 | | Join Sajber^ [0] (~Sajber@h-65-75.A213.priv.bahnhof.se) |
12:04:34 | gevaerts | possibly even an fsck plugin |
12:04:43 | gevaerts | mostly on archoses though |
12:05:09 | | Nick fxb__ is now known as fxb (~felixbrun@h1252615.stratoserver.net) |
12:05:44 | gevaerts | The standard answer of "it's a lot faster from your PC" isn't valid if you can't access the full disk over USB |
12:05:45 | | Join watto [0] (~watto@193.203.81.165) |
12:11:17 | Torne | i might tidy it up at some point then |
12:13:25 | | Join FOAD__ [0] (~dok@dinah.blub.net) |
12:16:59 | | Quit FOAD (Ping timeout: 252 seconds) |
12:16:59 | | Nick FOAD__ is now known as FOAD (~dok@dinah.blub.net) |
12:18:58 | | Quit JdGordon_ (Ping timeout: 248 seconds) |
12:19:34 | | Quit perfectdrug_ (Quit: perfectdrug_) |
12:32:29 | | Join MethoS- [0] (~clemens@134.102.106.250) |
12:34:36 | Zagor | C64 mp3 decoder: http://www.youtube.com/watch?v=0mF9kXZAjsI |
12:36:17 | ranmachan | FWIW I've reported this on the rockbox sansa subforum now: http://forums.rockbox.org/index.php?topic=23876.0 |
12:42:39 | | Part watto |
12:51:38 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
12:52:27 | | Join matsl [0] (~matsl@dhcp126.contactor.se) |
12:52:52 | | Nick fxb is now known as fxb__ (~felixbrun@h1252615.stratoserver.net) |
12:54:00 | | Join Kitar|st [0] (Kitr88@BSN-142-59-200.dial-up.dsl.siol.net) |
13:00 |
13:05:02 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
13:05:40 | kugel | ranmachan: don't we explicitely say the c200v2 is *unusable* and unsupported? |
13:07:27 | kugel | that means we know that there are problems and don't want them reported on the tracker (and not on the forum anyway) |
13:09:47 | ranmachan | kugel: Well there are other reports on the forum and the official devel thread is also there: http://forums.rockbox.org/index.php?topic=14064.0 |
13:10:10 | ranmachan | I can repost it there if you think that's more appropriate |
13:10:13 | kugel | the development thread is different |
13:13:35 | ranmachan | Hmm, looks like you or someone deleted http://forums.rockbox.org/index.php?topic=23876.0, so I'll make a new post in the devel thread if I may |
13:13:38 | | Join watto [0] (~watto@193.203.81.165) |
13:14:30 | | Join Adnyxo [0] (~aaron@adsl-065-013-002-216.sip.asm.bellsouth.net) |
13:16:02 | | Nick YPSY is now known as Ypsy (~ypsy@geekpadawan.de) |
13:26:37 | ranmachan | Done: http://forums.rockbox.org/index.php?topic=14064.msg162032#msg162032 |
13:50:08 | S_a_i_n_t | when someone tals about the process "syncing a patch against current svn"...what exactly does this involve? If it's relatively simple, I'd really like to get FS #5111 - Ipod piezo driver to apply cleanly again. |
13:50:16 | S_a_i_n_t | stals/talks/ |
13:52:48 | kugel | S_a_i_n_t: open the patch file, go to the hunk(s) that failed (each file to be patched has hunks which start with the @@ x,y a,b @@ line), open the original file and compare what changed |
13:53:52 | S_a_i_n_t | thanks....*seems* relatively simple at least...in theory anyway. |
13:54:27 | kugel | alternatively, you can checkout a revision where you know it works, apply, and then svn up to latest. then svn handles syncing but that may fail (but it's usually easier to resolve) |
13:54:29 | S_a_i_n_t | I'm, debating whether or not I need to get some sleep first to tackle this with minimal frustration however :-p |
13:55:23 | S_a_i_n_t | kugle...I actually wondered if it would cheaply work that way...thanks for that actually. I'll try that method first I believe :P |
13:57:58 | S_a_i_n_t | If I revert my current tree to a previous revision, will it revert my previously applied patches as well? Or will it try to be smart about those also? |
13:58:18 | S_a_i_n_t | Do I need to revert my patches first is what I'm asking I guess. |
13:58:55 | gevaerts | S_a_i_n_t: short answer to all your questions: "it depends" |
14:00 |
14:00:22 | S_a_i_n_t | ah....right. One of those...so playing it safe and reverting patches beforehand could potentially avoid some grief? |
14:00:32 | S_a_i_n_t | I mean, I can always re-apply them. |
14:00:45 | gevaerts | if they still apply properly, yes :) |
14:00:57 | gevaerts | Which gets you back to the "syncing a patch" question |
14:01:06 | S_a_i_n_t | well...there is that. :P |
14:01:33 | S_a_i_n_t | I'm sure I'll cross that bridge when I get to it. |
14:02:31 | S_a_i_n_t | I only need to revert back to 241XX-ish....<sarcasm>surely not much has changed since then </sarcasm> |
14:03:49 | *** | Saving seen data "./dancer.seen" |
14:05:09 | CIA-88 | New commit by gevaerts (r24592): Bump plugin API version. This should have been done in r24587. Also, because the new function is added halfway. the minimum api has to be bumped too/ ... |
14:08:34 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
14:10:52 | | Part watto |
14:12:35 | | Join Casainho_ [0] (~chatzilla@87-196-74-13.net.novis.pt) |
14:15:29 | | Quit Casainho (Ping timeout: 252 seconds) |
14:15:32 | | Nick Casainho_ is now known as Casainho (~chatzilla@87-196-74-13.net.novis.pt) |
14:18:07 | | Quit MethoS- (Ping timeout: 256 seconds) |
14:18:49 | | Join Psirus [0] (~sam-ragga@91-64-114-41-dynip.superkabel.de) |
14:18:49 | | Part Psirus |
14:22:45 | S_a_i_n_t | ok...first snag I've hit. I'm getting "Conflict discovered in 'apps/lang/english.lang'. Select: (p) postpone, (df) diff-full, (e) edit, )mc) mine-conflict, (tc) theirs conflict, (s) show all options. |
14:22:53 | S_a_i_n_t | what should I do? |
14:23:18 | | Join TopyMobile_ [0] (~topy@g228215196.adsl.alicedsl.de) |
14:26:20 | dionoea | e |
14:26:35 | dionoea | and try to fix it |
14:26:36 | CIA-88 | New commit by mc2739 (r24593): Fix FS #10997 - Clipv1 Quickscreen - left button cancels instead of toggling shuffle |
14:26:41 | | Quit TopyMobile (Ping timeout: 240 seconds) |
14:27:15 | | Quit Casainho (Ping timeout: 245 seconds) |
14:33:06 | | Quit Zarggg (Read error: Connection reset by peer) |
14:33:23 | | Join Zarggg [0] (~zarggg@2001:0:4137:9e74:0:4388:beb1:ba3d) |
14:34:13 | S_a_i_n_t | Hmmmm...I believe I may of found an easier option...choosing (df) and then (r) seemed to go smoothly for all conflicts...it all just depends on if it compiles or not. |
14:34:44 | dionoea | what does the r bit do ? |
14:35:05 | | Join kugel_ [0] (~kugel@e178065168.adsl.alicedsl.de) |
14:35:15 | | Quit kugel (Disconnected by services) |
14:35:21 | | Nick kugel_ is now known as kugel (~kugel@e178065168.adsl.alicedsl.de) |
14:35:22 | S_a_i_n_t | resolve |
14:35:25 | | Quit kugel (Changing host) |
14:35:25 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
14:35:30 | S_a_i_n_t | atte,pts to merge |
14:35:36 | S_a_i_n_t | *attempts |
14:35:51 | | Quit mikroflops (Ping timeout: 260 seconds) |
14:36:17 | | Join archivator [0] (~archivato@stu0279.keble.ox.ac.uk) |
14:39:07 | CIA-88 | New commit by mc2739 (r24594): Correct Clip manual quickscreen keymap |
14:43:14 | | Join efyx_ [0] (~efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
14:46:39 | | Join MethoS- [0] (~clemens@134.102.106.250) |
14:52:00 | S_a_i_n_t | Hmmmm....well, that didn't work. |
14:52:26 | | Join DerPapst [0] (~DerPapst@p4FE8F3E1.dip.t-dialin.net) |
14:52:26 | Zagor | before anyone asks, yes there appears to be some network issues to the server |
14:52:26 | | Join Will___ [0] (~chatzilla@adsl-76-235-49-202.dsl.dytnoh.sbcglobal.net) |
14:52:26 | pixelma | maybe add it to the topic? |
14:52:26 | Will___ | is the site down? |
14:52:26 | Topic | "Yes, there are network issues | 3.5 has been 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" by ChanServ (ChanServ@services.) |
14:52:26 | pixelma | Zagor just mentioned some network issues |
14:52:26 | | Quit logbot (Ping timeout: 265 seconds) |
14:52:26 | *** | ERROR: (Closing Link: giant.haxx.se (Ping timeout: 265 seconds)) from niven.freenode.net |
14:52:26 | *** | Cleanup |
14:52:26 | *** | Cleanup |
14:52:26 | *** | Saving seen data "./dancer.seen" |
14:52:26 | *** | Exit |
14:52:28 | *** | Started Dancer V4.16 |
14:52:28 | *** | Connected to irc.freenode.net on port 6667 |
14:52:28 | *** | Logfile for #rockbox started |
14:52:29 | Mode | "logbot :+i" by logbot |
14:52:30 | Ctcp | Version from frigg!~frigg@freenode/utility-bot/frigg |
14:52:31 | *** | Server message 501: 'logbot :Unknown MODE flag' |
14:52:31 | | Join logbot [0] (~rockbox@giant.haxx.se) |
14:52:31 | | Join Will___ [0] (~chatzilla@adsl-76-235-49-202.dsl.dytnoh.sbcglobal.net) |
14:52:31 | | Join DerPapst [0] (~DerPapst@p4FE8F3E1.dip.t-dialin.net) |
14:52:31 | | Join MethoS- [0] (~clemens@134.102.106.250) |
14:52:31 | | Join efyx_ [0] (~efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
14:52:31 | | Join archivator [0] (~archivato@stu0279.keble.ox.ac.uk) |
14:52:31 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
14:52:31 | | Join Zarggg [0] (~zarggg@2001:0:4137:9e74:0:4388:beb1:ba3d) |
14:52:31 | | Join TopyMobile_ [0] (~topy@g228215196.adsl.alicedsl.de) |
14:52:31 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
14:52:31 | | Join Adnyxo [0] (~aaron@adsl-065-013-002-216.sip.asm.bellsouth.net) |
14:52:31 | | Join Kitar|st [0] (Kitr88@BSN-142-59-200.dial-up.dsl.siol.net) |
14:52:31 | | Join matsl [0] (~matsl@dhcp126.contactor.se) |
14:52:31 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
14:52:31 | | Join FOAD [0] (~dok@dinah.blub.net) |
14:52:31 | | Join Sajber^ [0] (~Sajber@h-65-75.A213.priv.bahnhof.se) |
14:52:31 | | Join ranmachan [0] (ranma@yumi.tdiedrich.de) |
14:52:31 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
14:52:31 | | Join _zic [0] (~user@91-165-236-145.rev.libertysurf.net) |
14:52:31 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
14:52:31 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
14:52:31 | | Join avacore [0] (nobody@1008ds1-rdo.0.fullrate.dk) |
14:52:31 | | Join Topy44 [0] (~topy@my.fastsh.it) |
14:52:31 | | Join ender` [0] (krneki@foo.eternallybored.org) |
14:52:31 | | Join xavieran [0] (~xavieran@ppp118-209-153-106.lns20.mel6.internode.on.net) |
14:52:31 | | Join togetic [0] (~togetic@unaffiliated/ibuffy) |
14:52:31 | | Join n17ikh [0] (~n17ikh@host-69-59-126-212.nctv.com) |
14:52:31 | | Join flydutch [0] (~flydutch@host66-209-dynamic.15-87-r.retail.telecomitalia.it) |
14:52:31 | | Join jvd [0] (~syscrash@poipu/developer/syscrash) |
14:52:31 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
14:52:31 | | Join sinuc [0] (~sinuc@unaffiliated/sinuc) |
14:52:31 | | Join crashd [0] (foobar@lostnode.org) |
14:52:31 | | Join Zambezi [0] (Zulu@unaffiliated/zambezi) |
14:52:31 | | Join pixelma [0] (quassel@rockbox/staff/pixelma) |
14:52:31 | | Join amiconn [0] (quassel@rockbox/developer/amiconn) |
14:52:31 | | Join petur [0] (~petur@rockbox/developer/petur) |
14:52:31 | | Join nima [0] (~nima@75.45.226.145) |
14:52:31 | | Join einhirn [0] (~Miranda@bsod.rz.tu-clausthal.de) |
14:52:31 | | Join FlynDice [0] (~FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
14:52:31 | | Join Zagor [0] (~bjst@rockbox/developer/Zagor) |
14:52:31 | | Join teru [0] (~teru@KD059133108225.ppp.dion.ne.jp) |
14:52:31 | | Join antil33t [0] (~Mudkips@203-184-54-232.callplus.net.nz) |
14:52:31 | | Join desowin [0] (~desowin@atheme/member/desowin) |
14:52:31 | | Join tha [0] (1038@ccc2.rbg.informatik.tu-darmstadt.de) |
14:52:31 | | Join Xerion [0] (~xerion@82-170-197-160.ip.telfort.nl) |
14:52:31 | | Join kaniini [0] (~kaniini65@dyn75-70.yok.fi) |
14:52:31 | | Join Barahir [0] (~jonathan@gssn-5f755c8d.pool.mediaWays.net) |
14:52:31 | | Join dude187 [0] (~chris@cpe-75-187-48-206.columbus.res.rr.com) |
14:52:31 | | Join S_a_i_n_t [0] (S_a_i_n_t@203.184.0.254) |
14:52:31 | | Join TheSeven [0] (~theseven@rockbox/developer/TheSeven) |
14:52:31 | | Join Rob2222 [0] (~Miranda@p4FDC9E7D.dip.t-dialin.net) |
14:52:31 | | Join BlakeJohnson86 [0] (~bjohnson@2002:1876:a27b:0:227:13ff:fe65:1262) |
14:52:31 | | Join Galois [0] (djao@efnet.math.uwaterloo.ca) |
14:52:31 | | Join soap [0] (~soap@rockbox/staff/soap) |
14:52:31 | | Join Frampis [0] (famas@noppakerho.com) |
14:52:31 | | Join feisar_ [0] (jljhook@irkki.fi) |
14:52:31 | | Join piroko [0] (~jeremy@pohl.ececs.uc.edu) |
14:52:31 | | Join Torne [0] (torne@rockbox/developer/Torne) |
14:52:31 | | Join fleebailey33 [0] (fleebailey@unaffiliated/fleebailey33) |
14:52:31 | | Join bluebroth3r [0] (~dom@rockbox/developer/bluebrother) |
14:52:31 | | Join HBK [0] (~hbk@HBK.broker.freenet6.net) |
14:52:31 | | Join saratoga [0] (~9803c6dd@gateway/web/freenode/x-uxeftoofekqjpbkk) |
14:52:31 | | Join Tomis [0] (~Tomis@70.134.83.103) |
14:52:31 | | Join Tuplis [0] (~jani@adsl-77-109-221-158.kymp.net) |
14:52:31 | | Join shaggy-h [0] (~kiwi@78-86-164-31.zone2.bethere.co.uk) |
14:52:31 | | Join Hadaka [0] (~naked@naked.iki.fi) |
14:52:31 | | Join evilnick [0] (~evilnick@rockbox/staff/evilnick) |
14:52:31 | | Join dmb_ [0] (~Dmb@unaffiliated/dmb) |
14:52:31 | | Join JdGordon [0] (~jonno@rockbox/developer/JdGordon) |
14:52:31 | | Join ______dmb [0] (~Dmb@unaffiliated/dmb) |
14:52:31 | | Join gevaerts [0] (~fg@rockbox/developer/gevaerts) |
14:52:31 | | Join jd [0] (~jd@Wikipedia/HellDragon) |
14:52:31 | | Join flyback [0] (~teac@c-98-219-129-239.hsd1.pa.comcast.net) |
14:52:31 | | Join maraz [0] (maraz@kapsi.fi) |
14:52:31 | | Join krazykit` [0] (~kkit@ppp-70-236-33-106.dsl.ipltin.ameritech.net) |
14:52:31 | | Join zumbi [0] (~zumbi@77.230.237.25) |
14:52:31 | | Join jordan` [0] (~jordan@78.235.252.137) |
14:52:31 | | Join killan [0] (~nnscript@c-f8fe70d5.06-397-67626721.cust.bredbandsbolaget.se) |
14:52:31 | | Join parafin [0] (parafin@paraf.in) |
14:52:31 | | Join jfc [0] (~john@dpc6682208002.direcpc.com) |
14:52:31 | | Join tmzt [0] (~tmzt@adsl-99-164-34-42.dsl.akrnoh.sbcglobal.net) |
14:52:31 | | Join martian67 [0] (~martian67@about/linux/regular/martian67) |
14:52:31 | | Join Res1 [0] (~Res@adsl-067-034-129-222.sip.mco.bellsouth.net) |
14:52:31 | | Join ThomasAH [0] (~thomas@aktaia.intevation.org) |
14:52:31 | | Join @Llorean [0] (~DarkkOne@rockbox/user/Llorean) |
14:52:31 | | Join bzed [0] (~bzed@devel.recluse.de) |
14:52:31 | | Join linuxguy3 [0] (~timj@adsl-76-203-23-66.dsl.emhril.sbcglobal.net) |
14:52:31 | | Join tchan [0] (~tchan@lunar-linux/developer/tchan) |
14:52:31 | | Join elcan [0] (~loaa@pr0.us) |
14:52:31 | | Join rasher [0] (~rasher@rockbox/developer/rasher) |
14:52:31 | | Join pippijn [0] (~pippijn@unixcorps/staff/pippijn) |
14:52:31 | | Join lyngaas [0] (~staale@19.81-167-149.customer.lyse.net) |
14:52:31 | | Join lostlogic [0] (~lostlogic@rockbox/developer/lostlogic) |
14:52:31 | | Join yosafbridge [0] (~yosafbrid@adsl-71-142-225-118.dsl.scrm01.pacbell.net) |
14:52:31 | | Join CIA-88 [0] (~CIA@208.69.182.149) |
14:52:31 | | Join jhulst [0] (~jhulst@jhulst.com) |
14:52:31 | | Join GodEater [0] (~bibble@rockbox/staff/GodEater) |
14:52:31 | | Join beta2k [0] (1000@d24-36-68-97.home1.cgocable.net) |
14:52:31 | | Join rvvs89 [0] (~ivo@pdpc/supporter/base/rvvs89) |
14:52:31 | | Join Ypsy [0] (~ypsy@geekpadawan.de) |
14:52:31 | | Join koniiiik [0] (johnny64@phoenix.wheel.sk) |
14:52:31 | | Join AlexP [0] (~ap@rockbox/staff/AlexP) |
14:52:31 | | Join advcomp2019_ [0] (~advcomp20@unaffiliated/advcomp2019) |
14:52:31 | | Join Kohlrabi [0] (~Kohlrabi@frustrum.nosebud.de) |
14:52:31 | | Join topik [0] (awesome@213.203.214.114) |
14:52:31 | | Join simabeis [0] (~simabeis@78.46.82.173) |
14:52:31 | | Join Unhelpful [0] (~quassel@rockbox/developer/Unhelpful) |
14:52:31 | | Join aevin [0] (eivindsy@unaffiliated/aevin) |
14:52:31 | | Join BeFalou [0] (~mamutoi@unaffiliated/befalou) |
14:52:31 | | Join zu [0] (~zu@88.191.93.109) |
14:52:31 | | Join alexbobp [0] (~alex@66.112.249.238) |
14:52:31 | | Join ps-auxw [0] (~arneb@2001:470:c807:0:1532:4e5f:2ad3:4123) |
14:52:31 | | Join blithe [0] (~blithe@72.14.176.144) |
14:52:31 | | Join MagusG [0] (magusg@c-98-192-69-229.hsd1.ga.comcast.net) |
14:52:31 | | Join rphillips [0] (~rphillips@66-90-184-168.dyn.grandenetworks.net) |
14:52:31 | | Join scorche [0] (~scorche@rockbox/administrator/scorche) |
14:52:31 | | Join jobec [0] (paulus@viherharakka.cs.tut.fi) |
14:52:31 | | Join kadoban [0] (~mud@cpe-24-93-17-195.rochester.res.rr.com) |
14:52:31 | | Join chaos [0] (~chaos@gentoo/user/ch4os) |
14:52:31 | | Join crwl [0] (~crwlll@a91-156-100-168.elisa-laajakaista.fi) |
14:52:31 | | Join @ChanServ [0] (ChanServ@services.) |
14:52:31 | | Join fxb__ [0] (~felixbrun@h1252615.stratoserver.net) |
14:52:31 | | Join preglow [0] (thomj@tvilling2.pvv.ntnu.no) |
14:52:31 | | Join dionoea [0] (~dionoea@videolan/developer/dionoea) |
14:52:31 | | Join pjm0616 [0] (~user@61.250.113.98) |
14:52:31 | | Join ehntoo [0] (~ehntoo@lug.mtu.edu) |
14:52:31 | | Join scorche|sh [0] (~scorche@squisch.net) |
14:52:31 | | Join rb_voicebot [0] (~rb_voiceb@0x5550f5a3.adsl.cybercity.dk) |
14:52:31 | | Join B4gder [0] (~daniel@rockbox/developer/bagder) |
14:52:33 | Zagor | back now |
14:53:13 | | Join vegtoruci [0] (~vegtoruci@pool-96-246-120-217.nycmny.east.verizon.net) |
14:56:06 | Will___ | (Sansa e200v1) Is it normal for RB to display "maintenance mode" when I connect USB? |
14:58:00 | mc2739 | Will___: does it say "maintenance mode" or "Multimedia Mode"? |
14:58:13 | Will___ | mainenance |
14:58:40 | Will___ | I'm asking because I just fell asleep with my player connected, and when I woke up (7 hours later), it is still powered-on. I thought RB powered-down when charging was done...? |
14:59:18 | mc2739 | no, Rockbox will not turn off when connected to USB |
15:00 |
15:00:18 | Will___ | can 7 hours of connection damage my player? |
15:00:26 | mc2739 | no |
15:00:44 | | Join perfectdrug [0] (~marko@p5B0ECECA.dip.t-dialin.net) |
15:00:54 | * | S_a_i_n_t gives up (the attempt at...) syncong FS #5111 for now.... |
15:01:07 | S_a_i_n_t | *syncing rather |
15:03:10 | | Quit Will___ (Quit: ChatZilla 0.9.86 [Firefox 3.5.7/20091221164558]) |
15:04:31 | | Join dfkt [0] (dfkt@unaffiliated/dfkt) |
15:05:35 | | Join dfkt_ [0] (dfkt@unaffiliated/dfkt) |
15:06:17 | | Quit kramer3d (Quit: Leaving) |
15:08:57 | | Quit matsl (Quit: Riece/3.1.2 XEmacs/21.5-b29 (garbanzo, linux)) |
15:09:19 | | Quit Sajber^ (Quit: Leaving.) |
15:09:42 | | Join Sajber^ [0] (~Sajber@h-65-75.A213.priv.bahnhof.se) |
15:11:32 | | Join watto [0] (~watto@193.203.81.165) |
15:11:50 | | Quit dfkt_ (Quit: -= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.) |
15:14:59 | kugel | S_a_i_n_t: you can't just ignore conflicts |
15:15:21 | S_a_i_n_t | I realised that...now |
15:15:42 | kugel | and even successful compilation doesn't imply correctness |
15:15:46 | Torne | resyncing a language file is basically the most simple and trivial thing you can do, though :) |
15:15:55 | Torne | far easier than resyncing actual code |
15:15:57 | kugel | that's true |
15:16:13 | Torne | take the added language strings from the patch |
15:16:19 | S_a_i_n_t | I'm evidently far too tired for this right now...I'll have another look at it in the morning. |
15:16:22 | Torne | paste them on the end of the latest lang file from svn |
15:16:22 | Torne | done |
15:18:26 | S_a_i_n_t | IIRC, there more conflicts than just the lang file...but there's a rather large % of chance that I read it incorrectly also. |
15:18:33 | CIA-88 | New commit by teru (r24595): color_picker: a bit of rework for color_picker. ... |
15:18:43 | S_a_i_n_t | s/there/there were/ |
15:20:23 | Torne | well, that may be, and that would be harder, yes |
15:20:30 | Torne | but you only mentioned english.lang so hey :) |
15:20:37 | | Join Casainho [0] (~chatzilla@87-196-74-13.net.novis.pt) |
15:21:14 | perfectdrug | would someone mind committing this small thing: http://www.rockbox.org/tracker/task/10948 |
15:21:22 | S_a_i_n_t | that was just the first one I hit...there were5 scattered around apps and firmware |
15:21:23 | | Quit vegtoruci (Ping timeout: 276 seconds) |
15:22:09 | | Join mikroflops [0] (~yogurt@217-208-157-242-no112.tbcn.telia.com) |
15:22:46 | | Join evilnick_B [0] (~0c140464@gateway/web/freenode/x-ifvdaffvyzhtgtua) |
15:32:48 | * | kugel ports his favorite theme to the fuze |
15:34:02 | | Quit Tuplis (Read error: Operation timed out) |
15:34:35 | rasher | Is it DancePuffDuo? |
15:35:01 | kugel | no, this was the one I ported from the fuze to the e200 ;) |
15:35:44 | kugel | look at the e200 themes sorted by download count; the first one :p |
15:36:08 | | Join Tuplis [0] (~jani@adsl-77-109-221-158.kymp.net) |
15:36:37 | Unhelpful | JdGordon: what was it? i'm not going to be around for a long stretch until tonight, but i read logs... and quassel makes sure i know if somebody tries to talk to me. :P |
15:39:14 | kugel | perfectdrug: do you think your png is better than the existing one (except for the black screen)? |
15:39:56 | kugel | I don't care much afterall |
15:40:19 | | Join funman [0] (~fun@rockbox/developer/funman) |
15:40:24 | kugel | funman: hi :) |
15:40:30 | funman | hi! |
15:40:42 | perfectdrug | yes i think so, it is just minor but when you created the svg you look differently at things :D |
15:41:02 | funman | ranmachan: ping |
15:41:23 | | Quit Adnyxo (Ping timeout: 240 seconds) |
15:41:58 | ranmachan | funman: Yes? |
15:43:09 | ranmachan | funma: IIRC With the check in it would always boot into OF |
15:43:28 | funman | can you try something on your c200v2 : add ascodec_write(AS3514_DCDC15,1); in the lcd_enable() function of lcd-c200_c200v2.c like it is done in lcd-ssd1303.c for the clip |
15:44:38 | perfectdrug | bluebroth3r, domonky: is it possible for rbutil to check filesystems for e.g. the irivers too? i talked to someone who used it h140 as external drive and formated it NTFS, rbutil install went fine but the firmwareupgrade does obviously not work, it may be nice if the util told him that there is something wrong. |
15:44:39 | ranmachan | Let's see... |
15:45:09 | kugel | perfectdrug: that's for www/ right? |
15:45:23 | perfectdrug | i think so yes |
15:45:24 | funman | lcd not turning on is a different issue from dualboot, thanks for finding what was wrong! |
15:45:50 | ranmachan | You're welcome :) |
15:46:19 | ranmachan | I just looked at dualboot and thougt that looks like the most likely place to cause it to always boot into OF |
15:46:34 | kugel | perfectdrug: I don't have a checkout right now, I'll try to do it later if I don't forget it. I want to finish my theme first |
15:47:14 | perfectdrug | kugel: ok thanks:) no rush |
15:47:20 | ranmachan | funman: Should I inserted it before the lcd_send_command(R_STANDBY_OFF... ? |
15:47:33 | funman | ranmachan: it's there as a safety in case the button check doesn't work for some reason |
15:47:58 | funman | ranmachan: before |
15:48:13 | ranmachan | Ok, I was assuming it was rather for convenience since the USB stuff is not yet working :) |
15:48:48 | kugel | funman: after my exams I'll try to commit some fuzev2 code |
15:48:54 | funman | cool |
15:49:14 | funman | i'll try to get a Clip+ sometimes soon to replace my bricked Clipv2 |
15:49:33 | ranmachan | BTW, do you know if the 7 test pads on http://www.rockbox.org/wiki/pub/Main/SansaC200v2/sansa_c200v2_back.jpg provide JTAG? |
15:49:51 | dionoea | funman: did you keep track of the number of clips you bricked ? |
15:49:56 | ranmachan | I was wondering about that before, but never got around to test it. |
15:50:18 | kugel | funman: I have another fuzev2 if you want it |
15:50:19 | funman | ranmachan: nope i think it was only tried on m200v4 and e200v2 (you can find info in the forum thread, perhaps around pages 30) |
15:50:26 | funman | dionoea: 4 |
15:50:34 | funman | but only 3 while rockboxing |
15:51:16 | ranmachan | Unfortunately while I'd have the time to test right now, the equipment is about 10000Km away :) |
15:51:23 | funman | kugel: if you can write lcd code we'll be at the same point on all targets so i don't need one |
15:51:47 | | Join piotrekm [0] (~pm@adej195.neoplus.adsl.tpnet.pl) |
15:51:56 | | Quit piotrekm (Changing host) |
15:51:56 | | Join piotrekm [0] (~pm@unaffiliated/piotrekm) |
15:52:12 | kugel | the lcd code is mostly the same as on fuzev1 |
15:52:32 | kugel | I've disassembled it, but haven't written code from it yet |
15:52:53 | ranmachan | funman: Should be enough to rebuild the bootloader, right? |
15:53:03 | funman | ranmachan: i remember my c200v2 lcd didn't work sometimes in the bootloader, but always in the loaded binary, but backlight always worked |
15:53:20 | funman | ranmachan: according to your post it seems to load the binary just fine so i would rebuild only rockbox.sansa |
15:53:24 | dionoea | funman: and did you get free replacements? |
15:53:34 | funman | dionoea: twice |
15:53:55 | funman | i didn't try to replace the clips i had opened |
15:54:30 | dionoea | just use some glue and some paint before sending them back :D |
15:58:17 | funman | ^^ |
15:58:21 | kugel | I've successfully sent an opened fuze back :) |
15:59:26 | ranmachan | funman: No, that doesn't seem to help. |
16:00 |
16:00:02 | funman | if you use ascodec_write(...,15) ? |
16:02:52 | CIA-88 | New commit by mc2739 (r24596): Updated Russian Translation ... |
16:02:53 | funman | you said the button light is on ? |
16:03:31 | ranmachan | No change. |
16:04:05 | ranmachan | Yes, first only the center button (second stage loader?), goes off, then center + menu (rockbox.sansa?) |
16:04:22 | funman | i don't remember there were 2 lights |
16:05:29 | funman | the backlight is off, or on but not showing any pixels ? |
16:05:42 | ranmachan | Backlight looks off, even in a totally dark room. |
16:06:42 | funman | in the OF code i only see GPIOA_PIN5 being modified |
16:06:44 | ranmachan | ranma/S6000696.AVI">http://uguu.de/~ranma/S6000696.AVI (13MB) |
16:08:44 | funman | definitely the backligtjh |
16:09:49 | funman | could it be related to what bertrik and FlynDice were saying the other day, about current over a certain voltage |
16:09:54 | ranmachan | Yeah, I just tried shining a strong flashlight on the LCD. |
16:10:05 | ranmachan | I can see the splash that way. |
16:10:18 | funman | cool :) |
16:10:19 | ranmachan | It's still rebooting in a loop for some reason |
16:10:31 | funman | hm that's another bug |
16:10:33 | | Quit Casainho (Quit: ChatZilla 0.9.86 [Firefox 3.5.3/20091020122601]) |
16:10:42 | funman | if you plug it on usb, can you see the splash as well ? |
16:11:14 | ranmachan | Since I removed the USB check that should make a difference, right? |
16:12:20 | ranmachan | While only the center light is on I can see 4 Lines, the last one 'loading rockbox.sansa' |
16:12:35 | ranmachan | Then the splash and with revision and after that reboot |
16:13:13 | | Join jgarvey [0] (~jgarvey@cpe-071-070-228-143.nc.res.rr.com) |
16:14:01 | ranmachan | It's the same when I power it on by plugin into USB |
16:14:12 | funman | in ascodec-target.h : change (0<<6) in AS3514_CP_DCDC3_SETTING to (1<<6) |
16:14:15 | ranmachan | (Though the button blinking is slightly differnte then) |
16:14:20 | | Quit teru (Quit: Quit) |
16:14:55 | ranmachan | Which one? The one in arm/as3525 I presume? |
16:14:59 | funman | yes |
16:15:51 | ranmachan | Leave the write at ,15 or go back to ,1? |
16:16:29 | funman | remove this one, i don't think it's needed at all |
16:16:52 | funman | and hmm it was buggy, the other bits should be preserved |
16:17:38 | funman | hm no it's not the same register, there are only 5 bits in DCDC15 |
16:19:47 | ranmachan | No change |
16:20:56 | funman | then i don't know, you should disassemble the c200v2 OF and look for what we miss |
16:21:46 | ranmachan | Hmm |
16:22:33 | ranmachan | Any particular offset where I should start looking? |
16:23:06 | | Quit S_a_i_n_t (Quit: [St.] has exited mIRC™) |
16:24:00 | funman | on version 03.02.05, the void backlight(int onoff) function i know is at 0x4128 |
16:26:00 | | Quit n17ikh (Ping timeout: 245 seconds) |
16:28:50 | ranmachan | In the main firmware block according to http://www.rockbox.org/wiki/SansaAMSFirmware? |
16:30:53 | funman | yes |
16:31:14 | | Join n17ikh [0] (~n17ikh@host-69-59-126-212.nctv.com) |
16:31:16 | funman | you can use utils/AMS/hacking to extract it to "firmware" (be careful it will dump around 20 files in the current directory) |
16:33:53 | | Quit funman (Quit: free(random());) |
16:35:53 | | Quit Zambezi (Read error: Connection timed out) |
16:44:04 | | Quit Zagor (Quit: Clint excited) |
16:51:55 | | Join thegeek [0] (~nnscript@s080b.studby.ntnu.no) |
16:52:30 | *** | Saving seen data "./dancer.seen" |
16:56:10 | | Nick evilnick_B is now known as weevilnick (~0c140464@gateway/web/freenode/x-ifvdaffvyzhtgtua) |
16:56:27 | | Join fyrestorm [0] (~nnscript@cpe-24-90-81-175.nyc.res.rr.com) |
17:00 |
17:10:54 | | Join pamaury [0] (~pamaury@ALyon-551-1-36-230.w80-9.abo.wanadoo.fr) |
17:17:32 | | Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
17:19:33 | | Join panni_ [0] (hannes@ip-95-222-52-93.unitymediagroup.de) |
17:23:19 | saratoga | ranmachan: if you are interested in the JTAG, it might be pretty useful to look at |
17:30:40 | ranmachan | saratoga: Right now I'm more interested in looking at the GPIOs from within rockbox :) |
17:31:10 | ranmachan | I managed to stop the rebootloop by disabling USB detection in the usb driver |
17:31:24 | ranmachan | Looks like the GPIOs might be different on my c200v2 |
17:32:00 | ranmachan | I'd like to look at the GPIO state to see if something changes when I plug in usb... |
17:33:04 | ranmachan | mp3 playback: check |
17:39:09 | | Join Zambezi [0] (Zulu@80.67.9.2) |
17:40:05 | ranmachan | Hey, debug->view io ports looks promising :) |
17:43:39 | ranmachan | Hmm, no change when plugging in USB though |
17:43:49 | | Join toffe82 [0] (~chatzilla@12.169.218.14) |
17:44:42 | kugel | ranmachan: you should try writing to that pin |
17:45:03 | kugel | i.e. if 1 would be inserted, try writing 0 to it before reading |
17:49:03 | ranmachan | First I'll try writing to A0,A1,A4 in addition to A5 and look if the backlight turns on :) |
17:49:40 | bluebroth3r | perfectdrug: should be possible. |
17:55:01 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
17:58:27 | | Quit Zambezi (Changing host) |
17:58:28 | | Join Zambezi [0] (Zulu@unaffiliated/zambezi) |
18:00 |
18:00:03 | | Quit petur (Quit: work->home) |
18:01:16 | | Nick weevilnick is now known as evilnick_B (~0c140464@gateway/web/freenode/x-ifvdaffvyzhtgtua) |
18:01:28 | | Quit evilnick_B (Changing host) |
18:01:28 | | Join evilnick_B [0] (~0c140464@rockbox/staff/evilnick) |
18:04:31 | ranmachan | Suddenly: Backlight! Now which GPIO is the question.... |
18:06:32 | kugel | ranmachan: maybe you need to only write to that gpio at init; for example the fuze and e200 have identical buttons basically, except that the fuze needs A1 set for the scrollwheel to work at all |
18:06:40 | | Join komputes [0] (~komputes@ubuntu/member/komputes) |
18:07:29 | | Quit GeekShadow (Read error: Connection reset by peer) |
18:07:39 | ranmachan | kugel: Likely not, the backlight stayed on even when rockbox disabled the lcd (I only modified _on, not _off) |
18:07:59 | | Join GeekShadow [0] (~Antoine@reactos/tester/GeekShadow) |
18:08:20 | | Join bzed_ [0] (~bzed@devel.recluse.de) |
18:08:36 | kugel | so does your c200v2 have a completely other pin for backlight or is A5 also needed |
18:08:38 | kugel | ? |
18:09:01 | ranmachan | That's what I'm trying to find out right now :) |
18:10:51 | | Quit bzed_ (Remote host closed the connection) |
18:10:52 | | Quit bzed (Ping timeout: 258 seconds) |
18:10:54 | ranmachan | Looks like it's controlled by A7, A5 is not needed. |
18:12:12 | | Join bzed [0] (~bzed@devel.recluse.de) |
18:16:17 | ranmachan | Interesting: If I plug/unplug USB GPIOD changes from 1F to 9F for a short period |
18:17:26 | | Join GeekShado_ [0] (~Antoine@195.178.204-77.rev.gaoland.net) |
18:17:34 | ranmachan | Ok, that's just because buttonlight is turned on (menu light) |
18:17:46 | kugel | bertrik has a c200v2, you should talk to him so he can verify if you want it fixed in svn |
18:17:57 | ranmachan | However even though I've disabled the usb detect function it reacts to usb |
18:18:38 | | Quit GeekShadow (Read error: Connection reset by peer) |
18:22:58 | ranmachan | For now I've reported my findings on the devel forum thread: http://forums.rockbox.org/index.php?topic=14064.msg162044#msg162044 |
18:23:11 | ranmachan | And I'm going to sleep now, it's past 2am :) |
18:25:49 | kugel | night |
18:26:48 | ranmachan | ãŠä¼‘ã¿ãªã•ã„ |
18:27:09 | | Join moos [0] (moos@rockbox/staff/moos) |
18:29:21 | pamaury | does anyone here have a simulator build so he can try a 5sec manipulation ? |
18:29:56 | pamaury | It's just a matter of taking a random file and displaying its properties. On my computer it crashes the sim |
18:30:56 | JdGordon | amiconn: yeah, a mini menu could maybe work. "set as main screen font" and "set as remote screen font:? |
18:31:44 | JdGordon | pixelma: reminder to please test the radio skin patch this evening |
18:32:34 | Torne | JdGordon: you mentioned hte multifont thing working for plugins, but not being sure if this was useful |
18:32:35 | | Join moos_ [0] (moos@85-171-102-158.rev.numericable.fr) |
18:32:39 | Torne | i have a use for multifont in plugins :) |
18:32:46 | Torne | frotz requires text styles |
18:33:00 | | Quit togetic (Quit: WeeChat 0.3.0) |
18:33:17 | JdGordon | I didnt say it wouldnt be useful. I just said I thught that plugins shouldnt require fonts |
18:33:25 | Torne | hm |
18:33:41 | | Join togetic [0] (~togetic@unaffiliated/ibuffy) |
18:33:42 | Torne | tbh frotz is going ot have quite picky requirements wrt. fonts |
18:33:56 | Torne | ideally frotz wants to be able to do regular text, bold, italic, and monospace |
18:34:02 | kugel | bugger |
18:34:02 | Torne | well, minimally, in fact |
18:34:09 | Torne | ideally it wants any arbitrary combination of those :) |
18:34:22 | kugel | who put the colon into the "Next Track:" lang string? |
18:34:24 | Torne | at the moment i'm intending to use the user font as regular text, and sysfont as monospace |
18:34:35 | Torne | but tha tmight look weird for user fonts over a certain size :) |
18:34:43 | | Quit moos (Ping timeout: 260 seconds) |
18:34:44 | Torne | and doesn't give bold/italics. |
18:34:46 | | Nick moos_ is now known as moos (moos@85-171-102-158.rev.numericable.fr) |
18:35:00 | kugel | pamaury: seems to work here |
18:35:22 | kugel | my build is pretty old though |
18:35:30 | kugel | JdGordon: did you? :( |
18:35:58 | pamaury | kugel: strange, I'm at svn HEAD (or nearly), DEBUG, logf, e20 sim. what is your config ? |
18:35:58 | Torne | JdGordon: so i guess in an ideal world i'd want to be able to work out a set of fonts that could do some of that. dunno how plausible tha tis, though |
18:36:08 | kugel | tomers did |
18:36:13 | Torne | at the moment only the wps can load fonts, yes? |
18:36:30 | kugel | pamaury: no idea actually :) |
18:37:01 | kugel | no logf, fuze sim is what I know. I can't tell if I have patches applied |
18:37:35 | | Quit zu (Ping timeout: 260 seconds) |
18:37:36 | kugel | I just run the sim for porting my theme without recompiling, i.e. it's the state where I left it off at last |
18:38:31 | pamaury | Hum, it crashes in the properties plugin, on a strcmp, strange. Can someone else try ? (Torne ? JdGordon ?) |
18:38:56 | JdGordon | what am I trying? |
18:39:21 | | Quit moos (Ping timeout: 256 seconds) |
18:39:45 | pamaury | If you have a simulator build, could you try to display the properties of a file or a folder and see if it crashes, please |
18:40:06 | | Join domonoky [0] (~Domonoky@rockbox/developer/domonoky) |
18:40:50 | | Join phanboy4 [0] (~benji@c-24-98-43-198.hsd1.ga.comcast.net) |
18:40:52 | * | kugel ponders making an sbs version |
18:41:56 | | Join bertrik [0] (~bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
18:42:10 | JdGordon | no crash here |
18:42:18 | pamaury | Hum, perhaps my build just got corrupted... |
18:42:31 | JdGordon | svn + multifont patch |
18:43:16 | kugel | domonoky: ping |
18:43:29 | domonoky | pong |
18:43:44 | | Join moos [0] (moos@rockbox/staff/moos) |
18:43:46 | kugel | what the test environment for the theme site again? |
18:44:18 | domonoky | kugel: i have a (hackish) vmware image to test theme site changes. |
18:44:32 | kugel | :( |
18:44:53 | domonoky | i could upload the current image somewhere if you want. |
18:45:18 | kugel | I'm not sure I want it. I don't have vmware installed (and didn't plan to) |
18:45:40 | gevaerts | kugel: virtualbox should be able to handle it |
18:45:44 | JdGordon | you uploaded that to my server somewhere right? I can make it avilable |
18:46:07 | pamaury | Ok, it works, something went wrong during the compilation it seems |
18:46:13 | domonoky | JdGordon: yes i did, but that copy is a bit outdated.. |
18:46:19 | JdGordon | jdgordon.info/rockbox/test-themesize.z7">http://usa.jdgordon.info/rockbox/test-themesize.z7 |
18:46:25 | JdGordon | bah :p 7z |
18:47:04 | domonoky | the problem with the vmware image is, that it isnt exactly the same as the theme server, so it needs some changes and hacks. (but is good enough for my own tests). |
18:48:34 | kugel | ok, I would like to have a look |
18:50:34 | domonoky | oki, if you wait a bit, i will make a current image available. |
18:51:09 | kugel | sure |
18:51:32 | kugel | I want to have a look at keeping the download count when updating |
18:51:56 | kugel | otherwise if I update my theme I lose the (overall) 1st place in the download count stats |
18:52:32 | *** | Saving seen data "./dancer.seen" |
18:53:19 | * | JdGordon has a small epiphany |
18:53:40 | JdGordon | plugins can cause skin fonts to be unloaded and then reuse that buffer :p |
18:54:14 | JdGordon | so even if the skin uses all 7 user fonts, the plugins can use just as many. and if they both want the same then there is no special handling |
18:55:04 | gevaerts | so that would just cause some reloading on plugin entry/exit I guess? |
18:55:38 | kugel | couldn't you write a wrapper for plugins so that they don't reuse existing buffers? |
18:56:41 | domonoky | kugel: keeping download counters sounds good. ~40min till i uploaded the image. |
18:57:13 | JdGordon | kugel: yes, thats how it willl be done |
18:57:35 | * | JdGordon runs to work |
19:00 |
19:02:31 | | Join zu [0] (~zu@bucketheaded.eu) |
19:06:03 | kugel | what the heck |
19:06:08 | kugel | I'm getting "Invalid dirname: .rockbox/wps/BlacknBlue_Glass/ (should be classic_statusbar.)" all over |
19:06:40 | kugel | oh nevermind |
19:08:36 | | Quit bluebroth3r (Ping timeout: 265 seconds) |
19:09:51 | | Join bluebrother [0] (~dom@g226069170.adsl.alicedsl.de) |
19:09:51 | | Quit bluebrother (Changing host) |
19:09:51 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
19:13:11 | | Quit _zic (Quit: Ex-Chat) |
19:15:12 | bluebrother | domonoky: have you tried installing the bootloader on an ipod using windows lately? |
19:15:30 | domonoky | bluebrother: i dont have a (working) ipod :-) |
19:15:59 | bluebrother | too bad. There seems to be a problem in rbutil (including the latest release :( ) |
19:17:19 | | Join Omlet [0] (omlet05@49.145-242-81.adsl-dyn.isp.belgacom.be) |
19:18:26 | domonoky | maybe someone else here has a ipod and windows and can test it ? |
19:19:16 | domonoky | bluebrother: did you upload the fixed macosx binary, or is the one on the download server still broken ? |
19:20:00 | bluebrother | it's still broken. |
19:20:36 | bluebrother | I'm on the ipod issue right now, as I assume that there are more windows users around |
19:21:15 | | Join _zic1 [0] (~user@91-165-236-145.rev.libertysurf.net) |
19:21:46 | | Join Horscht [0] (~Horscht2@xbmc/user/horscht) |
19:22:56 | bluebrother | domonoky: I'm wondering if there's a way to prevent such issues in the future. Unfortunately testing is quite hard due to the device dependencies :( |
19:24:27 | * | Unhelpful pokes jdgordon with a stick |
19:24:54 | domonoky | automatically testing the bootloader installs is quit impossible. but the mac issue should be preventable. |
19:26:28 | domonoky | but if we some day have our testing results page and include rbutil into it, we might find most issues before we release :-) |
19:26:33 | gevaerts | domonoky: actually, with pamaury's usb-in-sim work, I think that automatically testing the bootloader installs isn't entirely impossible these days :) |
19:27:14 | * | bluebrother thinks he found the issue and has to blame himself :( |
19:28:04 | domonoky | gevaerts: the sims dont use bootloaders, so we might be able to simulate what ipodpatcher/sansapatcher does, but we still dont know if the result works. |
19:29:07 | pamaury | gevaerts, domonoky: what do you want to test with the bootloader ? |
19:29:12 | Unhelpful | sim... os == "booatloader" |
19:29:13 | gevaerts | domonoky: true, but I'd assume that whoever releases the bootloader actually tests it, so the only thing you have to validate when testing the install is that the resulting disk image is exactly right |
19:30:08 | * | bluebrother still gets a segmentation fault on exit |
19:30:50 | pamaury | gevaerts: any plan to commit close-handles-on-plugin-exit one day ? |
19:32:13 | pamaury | Has anyone seen Slasheri recently ? How often does he go on the channel ? |
19:33:16 | gevaerts | pamaury: hm, maybe we should just commit it right now |
19:33:25 | bluebrother | pamaury: you can ask logbot |
19:33:36 | pamaury | bluebrother: how ? |
19:33:47 | bluebrother | pamaury: /query logbot seen Slasheri |
19:34:18 | bluebrother | try /query logbot help. Just did that myself to figure out how it worked ;-) |
19:34:30 | * | gevaerts used to not like the splash in that patch, but he thinks he changed his mind |
19:35:09 | pamaury | gevaerts: did I used a splash ? I used a yesno gui, no ? |
19:35:30 | pamaury | bluebrother: /query logbot help doesn't do anything for me :( |
19:36:07 | | Quit toffe82 (Read error: Connection reset by peer) |
19:36:27 | pamaury | Ah, ok. |
19:37:30 | | Quit phanboy4 (Read error: Connection reset by peer) |
19:37:32 | bluebrother | pamaury: sorry, use /msg |
19:37:51 | pamaury | I found out by myself ;) |
19:39:28 | | Nick fxb__ is now known as fxb (~felixbrun@h1252615.stratoserver.net) |
19:39:47 | * | bluebrother reinstalls the bootloader for a felt 1000th time for tonight |
19:40:44 | amiconn | JdGordon: Yes, something like that. Only on multiscreen targets of course |
19:41:29 | | Join Strife89 [0] (~michael@168.16.237.214) |
19:41:44 | amiconn | Loading a font already triggers a full redraw anyway, so that request shouldn't cause problems with screen contents |
19:41:59 | | Join toffe82 [0] (~chatzilla@12.169.218.14) |
19:43:10 | amiconn | It could even allow to cancel the .fnt loading as a 3rd option (although - single screen doesn't allow that) |
19:46:03 | bluebrother | how (un)stable is voice? I'm experiencing quite a lot data aborts since I started using the voice I generated on OS X |
19:47:54 | | Quit piotrekm (Ping timeout: 256 seconds) |
19:49:21 | | Join JdGordon_ [0] (~836b0052@gateway/web/freenode/x-netztotbusamhqac) |
19:50:25 | | Quit dmb_ (Ping timeout: 246 seconds) |
19:50:39 | domonoky | voice should be stable at least for older targets, but maybe someone made it unstable ? |
19:50:52 | CIA-88 | New commit by bluebrother (r24597): Restore width of the Autodetect button. |
19:51:30 | | Quit gevaerts (Disconnected by services) |
19:51:39 | | Join gevaerts [0] (~fg@rockbox/developer/gevaerts) |
19:53:53 | JdGordon_ | Unhelpful: hey, I wanted to ask if you know how safe the AA stuff is when playback is stopped? |
19:53:58 | | Quit toffe82 (Read error: Connection reset by peer) |
19:54:21 | JdGordon_ | the code looks like it shold be fine, but apparently it breaks on target (in the fm screen), so maybe there is something missing? |
19:54:59 | | Quit zu (Ping timeout: 260 seconds) |
19:59:43 | pixelma | bluebrother: I have no problem with voice (I'm using my M5 the most currently).Just one freeze while browsing in a long time and the problem has been there for a looong time on coldfire (or at least the Iaudios) but isn't reproducible |
20:00 |
20:00:46 | pixelma | JdGordon_: ok, going to test in a bit |
20:01:09 | JdGordon_ | great :) thanks |
20:01:35 | amiconn | The occasional freeze also happens on cf irivers |
20:02:16 | gevaerts | pamaury: a yesno is a sort of splash in my view :) |
20:02:36 | amiconn | (and it's more annoying on them since resetting needs a paperclip) |
20:03:14 | pamaury | gevaerts: you're right but I really prefer the yesno in that case ! |
20:03:36 | DerPapst | hey all. quick question... i have viewport and display a menu in it but i want to clear the backdrop before. any advice how to do that? |
20:03:38 | | Join piotrekm [0] (~pm@adeo178.neoplus.adsl.tpnet.pl) |
20:03:44 | | Join zu [0] (~zu@bucketheaded.eu) |
20:03:53 | gevaerts | pamaury: anyway, do you want to commit, or shall I? |
20:04:11 | JdGordon_ | DerPapst: why? |
20:04:14 | gevaerts | I think you spend more time on it than I did in total |
20:04:18 | gevaerts | *spent |
20:04:45 | JdGordon_ | DerPapst: passing false to the last param of do_menu() will disable the theme and bckdrop |
20:04:58 | pamaury | Ok. But just one thing: the patch enables the check by default and you said it should not be enabled for release builds. Do we put a memo somewhere for that ? |
20:05:01 | DerPapst | JdGordon_: because it looks real weird in the plugin :P |
20:05:15 | DerPapst | JdGordon_: thanks, i'll try |
20:05:44 | JdGordon_ | are you using your own backdrop in the plugin? or a non full screen menu? |
20:06:04 | gevaerts | pamaury: my view on that has also changed a bit :) We have three months before the next release, so by then there should be no more filehandle bugs, so leaving it enabled should be fine :) |
20:06:29 | pamaury | ok :) |
20:07:36 | pixelma | bluebrother, pamaury: you can also query logbot and then "talk" to him. E.g. just type "help" into the query or "seen xyz" |
20:07:56 | pamaury | pixelma: that's what I did actually :) |
20:08:07 | | Join Schmogel [0] (~Miranda@p3EE21746.dip0.t-ipconnect.de) |
20:08:55 | DerPapst | JdGordon_: this is what the menu currently looks like http://derpapst.eu/rockbox/battleship.png the last parameter is false already btw |
20:10:20 | JdGordon_ | DerPapst: woops, logic is backwards :p the last param shuold be true |
20:10:34 | kugel | yea, it's "hide_theme" so it should be true :) |
20:10:37 | DerPapst | heh... lets try. thanks |
20:10:46 | JdGordon_ | are you using a viewport for the menu also? |
20:11:21 | kugel | JdGordon_: I wondered what's the matter with calling viewport_set_defaults on every theme_enable() call, is that needed? |
20:11:59 | kugel | I wouldn't expect this function to touch the viewport and all, and it also seems wasteful |
20:12:36 | DerPapst | yay, works. thank you :) |
20:12:53 | JdGordon_ | kugel: which one? |
20:12:53 | bluebrother | pixelma: I know :) |
20:13:25 | kugel | viewportmanager_theme_enable() calls viewport_set_defaults() for the passed viewport |
20:13:49 | amiconn | JdGordon: Btw, a mini-selector might be useful for other filetypes with more than one action as well |
20:13:58 | bluebrother | pixelma: hmm, I'm on PP (mini2g). I'm somewhat wondering if the problem could be related to the voice −− otoh the voice file is just a voice file and I have no idea why a voice file should (even could) cause crashes |
20:14:01 | DerPapst | JdGordon_: yes. two viewports. one for the menu and one for the logo |
20:14:14 | amiconn | (bmp, ...) |
20:14:32 | JdGordon_ | kugel: only if its passed in, the idea there is so you can setup the viewport+theme in one call instead of 2 |
20:15:07 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
20:15:07 | | Quit pixelma (Disconnected by services) |
20:15:19 | | Quit amiconn (Disconnected by services) |
20:15:20 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
20:15:23 | JdGordon_ | amiconn: well, how many extensions would this apply to? we don;t want to duplicate the open with code |
20:15:24 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
20:15:39 | CIA-88 | New commit by pamaury (r24598): Commit FS #10889: detect file handles leaks in plugins and automatically close them on exit and warn the user. |
20:15:42 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
20:16:15 | amiconn | Actually it may be possible to reuse the "open with" code. Just that the default would show less choices |
20:16:44 | JdGordon_ | not in its current form |
20:17:15 | | Quit flydutch (Quit: /* empty */) |
20:17:21 | JdGordon_ | the code for it is heavily linked to the filetypes and only loads a plugin |
20:17:59 | * | amiconn also thinks that single-buffer font caching (possibly reusing the buflib) may pave the way to get lru caching going for other stuff |
20:18:08 | amiconn | E.g. voice on lowmem targets |
20:21:41 | JdGordon_ | if we had malloc we would have far better memory usage overall! |
20:22:01 | JdGordon_ | there are probably at least 5 differen buffer allocation implementations in the core already |
20:22:10 | * | JdGordon_ hides |
20:22:26 | * | gevaerts tracks JdGordon_ down so he can partially agree |
20:22:49 | amiconn | That's not correct |
20:22:57 | JdGordon_ | which part? |
20:22:59 | | Nick Ypsy is now known as YPSY (~ypsy@geekpadawan.de) |
20:23:10 | amiconn | malloc needs a very different approach on memory usage. |
20:23:13 | gevaerts | The big problem is that while I think that malloc might be OK, free() and realloc() aren't exactly straightforward |
20:23:28 | amiconn | Basically you must never use up all available memory. |
20:23:56 | amiconn | But we do want that, i.e. use all available memory for buffering |
20:24:28 | JdGordon_ | no, but we can allocate say 1MB for malloc and then the rest is audio |
20:24:44 | amiconn | There are other problems with malloc, but this design issue is the most fundamental one |
20:24:59 | amiconn | Set aside 1MB of about 1.3MB free in total? Certainly not |
20:25:15 | amiconn | And that's archos - there are swcodec targets with even less free memory than that |
20:25:58 | JdGordon_ | no, obviuosly i wasnt talking about low mem here |
20:26:22 | amiconn | Further issues: memory fragmentation (you *cannot* avoid that on a system without mmu), increased complexity of all code needing buffers (you need to handle malloc failure) etc etc |
20:28:22 | DerPapst | the reason why you have to reboot iPL after using some modules/programs... |
20:28:31 | * | amiconn really doesn't want his dap to display "malloc failure" in response to a user action |
20:28:50 | amiconn | If I wanted that, I would have installed ipl on my ipods.... but no thanks |
20:29:11 | pamaury | memory fragmentation in inherent to malloc, with or without mmu because you usually allocate small things. |
20:29:13 | DerPapst | *pffff* iPL is teh best! |
20:31:11 | gevaerts | We could move to a handle-based model instead of the current pointer-based model for the bits that need lots of memory in a non-predictable way. That would fix the fragmentation issue. If we then also change buffering to handle changing amounts of available memory, we'd have most cases covered I think. |
20:31:16 | gevaerts | Lots of work though |
20:31:25 | | Join grndslm [0] (~grndslm@174-126-14-4.cpe.cableone.net) |
20:31:49 | amiconn | gevaerts: That's why I mentioned buflib... |
20:32:00 | | Join toffe82 [0] (~chatzilla@12.169.218.14) |
20:32:20 | gevaerts | hm, yes. I wasn't paying attention... |
20:32:55 | gevaerts | Anyway, a pure traditional malloc()/free() model won't ever work properly |
20:33:05 | * | amiconn wonders whether it would be possible to extend buflib so that it handles *all* variable buffers |
20:33:39 | amiconn | I.e. font loading, voice, theming etc would all allocate from the same pool |
20:33:47 | saratoga | we can't really do malloc since we have no VM |
20:33:58 | amiconn | saratoga: huh? |
20:33:59 | saratoga | but we could have a better dynamic memory allocation system |
20:34:15 | saratoga | without VM you can't really implement free effectively |
20:34:20 | kugel | we as well just use a real malloc then |
20:34:20 | gevaerts | You still have to be careful. You don't want to allocate so much to voice+theme that you have to choose between a font and playback.... |
20:34:23 | amiconn | You *can* do malloc without virtual memory. It's just nasty |
20:34:33 | amiconn | AmigaOS is an example |
20:34:52 | saratoga | sure you can malloc, you just can't expect free() to do what you want it to . . . |
20:34:55 | kugel | how does tlsf malloc in the codeclib do it? |
20:35:08 | gevaerts | by luck I assume |
20:35:10 | saratoga | kugel: it doesn't really do free correctly |
20:35:23 | saratoga | it just depends on the codec being reinited every track |
20:35:25 | amiconn | free() is possible, but you get fragmentation over time |
20:35:38 | gevaerts | If you have lots of small allocations it will mostly work, but if you have some large ones, expect trouble |
20:35:55 | | Nick fxb is now known as fxb__ (~felixbrun@h1252615.stratoserver.net) |
20:35:57 | amiconn | That's why codecs can get away with it - they "just" have to survive one track |
20:36:05 | saratoga | the original malloc i tried for the codecs didn't work because it didn't get reinited on track change, which meant that eventually you ran out of buffer memory and it would crash |
20:36:41 | saratoga | if you played too many tracks of the same codec in a row (it was fine if you kept changing formats) |
20:37:10 | amiconn | AmigaOS does some clever tricks to minimize fragmentation (like doing small allocations from one end and large allocations from the other), but it can't be avoided completely |
20:37:35 | saratoga | but a system where you could request some (large) fixed size chunks might work, or one where you periodically have to give back the chunk so it can be defragmented |
20:38:06 | saratoga | in a sense thats what we do now for various things (enabling tagcache or db), except we do it by forcing the user to reboot |
20:38:37 | | Part zu |
20:39:40 | JdGordon_ | buflib would still have problems with beign careful to not store pointers so they can be moved around right? |
20:39:53 | | Join guesswho [0] (~54fdcb7d@giant.haxx.se) |
20:39:58 | guesswho | evening |
20:40:08 | JdGordon_ | which means a MASSIVE amount of work for skins |
20:40:10 | saratoga | yes, you have to update all pointers when you check back a block |
20:40:13 | gevaerts | JdGordon_: yes, but if you make sure that they get moved around, that won't be a problem for very long :) |
20:40:39 | | Nick guesswho is now known as guesswhoooo (~54fdcb7d@giant.haxx.se) |
20:40:43 | saratoga | alternatively you can just request largish blocks (e.g. multiples of 64KB) and then try to minimize freeing |
20:40:49 | amiconn | saratoga: The idea is to avoid pointers and use handles |
20:40:52 | saratoga | that should make fragmentation less of a problem |
20:41:03 | saratoga | amiconn: yeah thats what i figured |
20:41:17 | saratoga | sounds like a great place to use OO |
20:41:23 | saratoga | lets switch to c++ |
20:41:30 | JdGordon_ | AAAAAAAAAAAAAHHHHHHHHHHHH |
20:41:47 | gevaerts | saratoga: uh? How does OO relate to this? |
20:42:07 | saratoga | put all this into objects and make them worry about the pointers |
20:42:23 | guesswhoooo | umm.. evening.. is there a way to edit rockbox_default wps? I'd like to swap the places for artist and song name |
20:42:26 | JdGordon_ | isnt that exactly what the handle does |
20:42:38 | JdGordon_ | guesswhoooo: no, you can just copy that to a new .wps |
20:42:46 | JdGordon_ | the code for that wps is in the wiki i tinhk |
20:42:55 | guesswhoooo | ah, I'll take a look from there |
20:42:58 | saratoga | i assume you have to handle that yourself though, by dereferencing things like *handle+offset ? |
20:43:19 | gevaerts | saratoga: **handle I'd say |
20:43:31 | JdGordon_ | void* get_handle_data(int handle) |
20:44:18 | gevaerts | The old Mac way was to just have a pointer to an entry in a pointer table, which was located at the start of the heap |
20:44:29 | | Join pamaury_ [0] (~pamaury@ALyon-551-1-70-117.w92-137.abo.wanadoo.fr) |
20:45:23 | | Quit pamaury (Read error: Operation timed out) |
20:45:27 | * | amiconn thinks that buflib could in fact work quite well for the core, but it would need to be extended with a special buffer type |
20:45:38 | | Quit krazykit` (Read error: Operation timed out) |
20:45:42 | | Join krazykit [0] (~kkit@ppp-70-236-33-106.dsl.ipltin.ameritech.net) |
20:45:58 | saratoga | buflib is buffering.c? |
20:46:05 | amiconn | A type that's marked as "can be thrown away when allocation isn't possible otherwise". This type would then be used by playback |
20:46:37 | kugel | saratoga: nope, it's in the pluginlib |
20:46:45 | kugel | pictureflow uses it |
20:46:49 | gevaerts | yes and no. You don't want to throw away the bit you're decoding right now |
20:47:08 | amiconn | This way playback wouldn't need to be stopped anymore e.g. when loading a larger theme |
20:48:00 | gevaerts | yes, and the dircache limit could probably go away |
20:48:00 | kugel | sounds like a shit lot of work (*cough* GSoC *cough*) |
20:48:14 | amiconn | gevaerts: Yeah, actually those blocks need more differentiation |
20:49:46 | gevaerts | the allocation code should ask the playback code I guess |
20:49:53 | amiconn | saratoga: No, buflib is apps/plugins/lib/buflib.c |
20:50:24 | amiconn | gevaerts: A callback? "Call me for futher information about this block" |
20:50:33 | JdGordon_ | no more callbacks :( |
20:50:50 | pixelma | JdGordon_: this build boots but I only get a blank radio screen (except statusbar) and if I'm lucky, I get some scrolling text from the menu... this is without having a fms set which I don't even know where to set it |
20:50:57 | gevaerts | possibly, yes |
20:51:05 | amiconn | Nah, just a function in playback that buflib could call would be sufficient |
20:51:22 | amiconn | Blayback would then do the deallocation itself |
20:51:26 | amiconn | *Playback |
20:51:37 | JdGordon_ | pixelma: ok, bugger |
20:51:53 | CIA-88 | New commit by bluebrother (r24599): Restore the default message handler on application shutdown. Fixes sporadic segfaults on exit. |
20:52:09 | JdGordon_ | would buflib allow us to use the entire buffer for everything? so there is no problem of choosing how much to allocate betwen it and playback? |
20:52:10 | pixelma | should there be a menu option under theme settings? |
20:52:15 | | Join jobec_ [0] (paulus@viherharakka.cs.tut.fi) |
20:52:18 | gevaerts | Actually, a function that asks playback to free some memory (with an argument to tell it how much is needed) should be enough. The allocator can then move things around to make the newly requested block fit |
20:52:20 | JdGordon_ | no, fms: setting in the config |
20:52:24 | amiconn | JdGordon_: yes |
20:52:32 | JdGordon_ | ok, sweet |
20:52:36 | *** | Saving seen data "./dancer.seen" |
20:52:39 | amiconn | Probably except for code (plugins, codecs) |
20:52:44 | pixelma | noooo :( |
20:52:54 | JdGordon_ | its just not added yet |
20:52:57 | | Join petur [0] (~petur@rockbox/developer/petur) |
20:53:00 | gevaerts | JdGordon_: yes, with (I guess) some rules so you have minima for some of them. You want to have more than 100 bytes left to load a WPS :) |
20:53:12 | amiconn | But everything that is data would be completely flexible |
20:53:42 | gevaerts | well yes, as long as code is linked to a fixed address, you can't change that anyway |
20:54:15 | JdGordon_ | well, if we had dynalically loadable plguins/codecs... :) |
20:54:25 | * | amiconn wonders how large a coff loader would be |
20:54:35 | amiconn | preglow wanted to experiment with that iirc |
20:55:19 | JdGordon_ | ok, 1 tiny possible issue. the skin allocator has 2 ways of allocing a block, one if the usualy "give me a block X size", the other is "give me the whole buffer and ill tell you how much I used when I'm done" |
20:55:24 | | Join dmb [0] (~Dmb@unaffiliated/dmb) |
20:55:28 | JdGordon_ | I assume buflib cant handle the second? |
20:55:30 | | Quit jobec (Ping timeout: 246 seconds) |
20:55:44 | amiconn | It would mean further core binsize increase, but at the same time it would do away with overlay plugins, and users not using plugins would waste zero buffer... |
20:55:50 | gevaerts | JdGordon_: maybe not right now (I haven't looked), but it should |
20:56:04 | gevaerts | There's no real reason not to handle it |
20:56:12 | guesswhoooo | hmm, can't seem to find the default wps |
20:56:22 | JdGordon_ | (the second is used to alloc space for things of unkown sizes, like images (although that can be known) and the token array which *is* unkown) |
20:56:22 | amiconn | JdGordon_: The second case is rather bad |
20:56:33 | JdGordon_ | yes, yes it is :p |
20:57:12 | amiconn | Basically it would either mean that skin loading might fail (if "give me all" is interpreted as "give me all that's free atm") or stopping playback |
20:57:15 | | Join jobec [0] (paulus@viherharakka.cs.tut.fi) |
20:57:16 | gevaerts | JdGordon_: don't expect too much performance, but realloc()-style things aren't impossible. It just may require lots of memmov() |
20:57:21 | bluebrother | domonoky: any thoughts on pushing a rbutil 1.2.5 shortly? I've found the line that crashes the bootloader installation |
20:57:46 | domonoky | then it would be good to make a new release :-) |
20:57:59 | amiconn | gevaerts: buflib compacts anyway, using memmove |
20:58:24 | JdGordon_ | well, those 2 cases can be removed if we read the bmp required size before actually loading it, and go to a linked list instead of a regular array for the tokens |
20:58:27 | amiconn | What might be tricky is to make the compactor not delay things too much. It might need to yield during compaction |
20:58:28 | gevaerts | JdGordon_: can that be changed to e.g. "give me 100K. I'll tell you to grow if I see I need more, and I'll shrink when done"? |
20:58:47 | JdGordon_ | it can grow and shrink? |
20:58:59 | bluebrother | anyone wants to update a rbutil translation? |
20:59:45 | gevaerts | JdGordon_: again, not *now* (I think...), but there's no reason why it can't be done |
20:59:54 | | Quit jobec_ (Ping timeout: 252 seconds) |
20:59:59 | JdGordon_ | ok, then it could work |
21:00 |
21:00:40 | gevaerts | You check if there's enough free space after the buffer you want to grow, if there isn't, you shuffle buffers around until there is, and then you change the size field |
21:01:08 | gevaerts | Shrinking is slightly easier, as it's guaranteed that there's room for it :) |
21:01:55 | JdGordon_ | well, if there is any seriousness in wanting to do this, then I'll work the skin engine to use it as a proof of concept if someone does the buflib side of things |
21:03:29 | gevaerts | Basically the only differences with a full malloc() system are that (a) you use double indirection, so you have to be a bit more careful with pointers, and (b) allocation and resizing can be *a lot* slower. I'd never use this model for something that allocates and frees all the time, but for things like the skin buffer, dircache, or the database, it should work well |
21:04:05 | amiconn | Plus font buffer(s), voice, ... |
21:04:11 | CIA-88 | New commit by b0hoon (r24600): Packard Bell Vibe 500: correct main keymaps, enable full keyboard editing, enable morse input. |
21:04:19 | JdGordon_ | i.e all the allocation rockbox does :p |
21:04:21 | pixelma | JdGordon_: blank screen with the fms: line and a file I had prepared (on target), can't tell for sure if the fms should work or not, I'd need to test in a sim but am not sure if it's worth it |
21:04:30 | gevaerts | "like" :). Yes, things that don't change without user intervention basically |
21:04:45 | amiconn | Hmm, also the browser buffer |
21:04:57 | JdGordon_ | pixelma: can you try loading a fms which just has some text and %wd? |
21:04:59 | pixelma | or maybe prepare a super simple .fms |
21:05:00 | amiconn | No more maunual limit setting |
21:05:24 | amiconn | The browser would start with a small buffer and resize if a folder doesn't fit |
21:05:40 | JdGordon_ | small being a reasoably size? or tiny? |
21:05:47 | amiconn | (of course not in single-entry steps...) |
21:05:55 | kugel | is this all threading safe? what if I get my data and another thread causes movement? |
21:06:16 | amiconn | kugel: You always have to re-evaluate handles after a yield() |
21:06:24 | | Join FOAD_ [0] (~dok@dinah.blub.net) |
21:06:29 | amiconn | That's where cooperative threading helps a lot |
21:06:49 | gevaerts | yes, a handle based system with preemptive threading sounds like a nightmare |
21:06:50 | | Join Tomis2 [0] (~Tomis@70.134.86.166) |
21:06:52 | amiconn | Dual core might have special implications |
21:07:12 | gevaerts | hm, there are some edge cases |
21:07:13 | JdGordon_ | can we add a simple locking mecahnism for dual core? |
21:07:36 | amiconn | As a first step, I'd just says the COP must never use handle allocated memory |
21:07:37 | gevaerts | Does file I/O yield? |
21:07:42 | amiconn | yes |
21:08:18 | gevaerts | ok, that needs some care then |
21:08:31 | JdGordon_ | would a call to check if the pointer is still valid be a bad idea? |
21:08:33 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
21:08:38 | amiconn | It yields e.g. when waiting for disk spinup |
21:08:58 | gevaerts | JdGordon_: how do you know that? |
21:09:13 | amiconn | JdGordon_: Why ask if it's still valid? That's slower than just re-requesting it |
21:09:16 | JdGordon_ | pass the handle and pointer into the function |
21:10:00 | | Quit FOAD (Ping timeout: 264 seconds) |
21:10:00 | | Nick FOAD_ is now known as FOAD (~dok@dinah.blub.net) |
21:10:07 | gevaerts | oh, you mean a pointer to the start of the buffer? Yes, that would work, but as amiconn says, why? |
21:10:45 | amiconn | I am imagining it as a function that replaces the dereferencing operator |
21:11:00 | gevaerts | I guess core functions that handle memory might need a way to lock a buffer |
21:11:01 | | Quit Tomis (Ping timeout: 252 seconds) |
21:11:02 | | Nick Tomis2 is now known as Tomis (~Tomis@70.134.86.166) |
21:11:06 | amiconn | I.e. instead of *pointer, you write geth(handle) |
21:11:42 | amiconn | In C++ you could do this by operator overloading, I think (I never wrote C++ code myself...) |
21:12:10 | gevaerts | Depending of how you implement it, it can just be **handle, possibly with some volatile magic |
21:12:12 | amiconn | Ah yes, buffer locking would be another option |
21:12:42 | amiconn | You don't really need volatile in a cooperative threading environment, as long as no interrupts are involved |
21:13:20 | CIA-88 | New commit by b0hoon (r24601): fix typo |
21:13:27 | gevaerts | buffer locking only works if no buffer is more than 1/3 of the total buffer space I think (that number might be wrong, and I'm assuming that only one lock is active at a time. More locks need smaller maximum chinks) |
21:13:43 | pixelma | JdGordon_: still a blank screen and the fms just containing: "%wd" "Station:" "%TnX" "%tf" (separate lines each) |
21:13:59 | JdGordon_ | pixelma: change that X to 0 |
21:14:06 | amiconn | Hmm, of course it's just **handle |
21:14:35 | JdGordon_ | oh yuck, deadlock issues if we add locking :( |
21:15:18 | | Join linuxstb [0] (~linuxstb@rockbox/developer/linuxstb) |
21:15:51 | gevaerts | I wouldn't add locking for general "users", only inside firmware/* functions that get passed a pointer to a buffer (are there more of those than just disk I/O?) |
21:16:29 | | Join p3tur [0] (~petur@rockbox/developer/petur) |
21:17:01 | JdGordon_ | well, if bufflib is in firmware/ we could just has a second set of i/o functions to use a handle instead? |
21:17:12 | gevaerts | yes, that's the other option |
21:17:16 | | Quit petur (Disconnected by services) |
21:17:20 | | Nick p3tur is now known as petur (~petur@rockbox/developer/petur) |
21:17:23 | pixelma | JdGordon_: still blank - and the %wd doesn't seem to take effect. I can still see the statusbar or more precisely the classic_statusbar.sbs |
21:17:30 | JdGordon_ | int *handle, so the caller doesnt even need to allocate one first |
21:17:55 | pixelma | it was still set from my work on it back then |
21:18:00 | gevaerts | Buflib-allocated pointers would be forbidden for the other firmware/* functions then |
21:18:39 | JdGordon_ | can anyone think of any other functions where thats a problem though? |
21:19:33 | amiconn | You only need to consider functions which work on buflib-allocated memory |
21:19:35 | JdGordon_ | pixelma: crud, ok, can you try the 2nd last patch instead of the latest one please? |
21:19:59 | amiconn | E.g. lcd update and rawing are no problem. The framebuffer is fixed size, no need to allocate that via buflib |
21:20:12 | pixelma | yeah, would be nicer if you could start numbering your patch versions btw. |
21:20:13 | amiconn | s/rawing/drawing/ |
21:20:16 | | Join tipi^ [0] (pihlstro@lehtori.cc.tut.fi) |
21:20:31 | gevaerts | You also only need to consider functions that may yield |
21:20:47 | amiconn | Yes, although that might be difficult to predict across targets |
21:20:57 | JdGordon_ | lcd_set_backdrop() will be problematic |
21:20:59 | amiconn | E.g. lcd_update[_rect]() yields on some targets |
21:21:31 | JdGordon_ | we can lock the buffer in the lcd_update()'s though |
21:21:38 | amiconn | (just an example for the difficulty - of course lcd_update() isn't a problem) |
21:22:05 | amiconn | JdGordon_: It would be all drawing functions, no the update |
21:22:13 | JdGordon_ | it sort of is a problem, the yield could cause playback to buffer more data and move handles |
21:22:34 | amiconn | Hence you need to re-evaluate handles after a yield... |
21:23:04 | JdGordon_ | or add a yeild_and_lock() for formare so it doesnt cause problems 15 layers up |
21:23:13 | amiconn | The drawing functions don't need locking |
21:23:31 | gevaerts | exactly. And then another thread can do file IO which also locks some bits, and then another thread does something, and as soon as you have lots of locks some thread asks for a new buffer (or a resize) that would require moving a locked buffer to be moved |
21:23:33 | amiconn | öcd_set_backdrop() needs to work differently though - it needs to store the handle |
21:23:34 | JdGordon_ | the thing that calls it might need to reget its handles |
21:23:59 | amiconn | Each drawing function would then need to get the backdrop handle before starting to draw -> problem solved |
21:24:08 | amiconn | Drawing functions never yield half-way |
21:24:10 | | Quit alexbobp (Ping timeout: 260 seconds) |
21:24:15 | * | gevaerts thought this was a good idea when we didn't have any details yet |
21:25:18 | | Join p3tur [0] (~petur@rockbox/developer/petur) |
21:25:20 | | Quit jd (Quit: Ω) |
21:25:21 | | Quit petur (Disconnected by services) |
21:25:25 | | Nick p3tur is now known as petur (~petur@rockbox/developer/petur) |
21:25:26 | JdGordon_ | I'm talking about locking the whole buffer, not individual handles |
21:25:39 | | Join jd [0] (~jd@modemcable207.134-202-24.mc.videotron.ca) |
21:25:39 | | Quit jd (Changing host) |
21:25:39 | | Join jd [0] (~jd@Wikipedia/HellDragon) |
21:25:40 | JdGordon_ | nothing shuold block waiting for a new handle |
21:25:47 | JdGordon_ | hence, no deadlocking |
21:25:49 | | Join pyro_maniac_ [0] (~quassel@77-23-96-13-dynip.superkabel.de) |
21:25:54 | | Part watto |
21:26:17 | amiconn | If you lock the whole buffer and something wants to do an allocation, what should happen? |
21:26:29 | JdGordon_ | that should never be allowed |
21:26:30 | * | amiconn thinks that locking is bad, and should be avoided altogether |
21:26:35 | JdGordon_ | activily avoided |
21:26:45 | amiconn | how? |
21:26:47 | JdGordon_ | or at least, we lock any movement |
21:27:04 | amiconn | Then you have to deal with failures... |
21:27:26 | gevaerts | no, you just have to make movement yield and wait |
21:27:46 | amiconn | -> deadlock |
21:27:50 | JdGordon_ | not really. The bulk of allocations are done at init, worst case is file i/o waiting for a block |
21:28:10 | | Quit anewuser (Quit: Another edition of chiptune gig WinterChip5! :O http://xrl.us/WinterChipV =ooo) |
21:28:25 | gevaerts | why? Only firmware/* functions should ever lock, and only apps/* should ever allocate or change allocations |
21:28:33 | JdGordon_ | ^ bingo |
21:28:51 | amiconn | Well, firmware/ may also allocate |
21:28:57 | JdGordon_ | a yield() in lcd_update() will never cause a forced alloc, buffering might ask to alloc, but it will be refused a handle |
21:29:17 | JdGordon_ | i/o is different and wont do a full lock |
21:29:40 | JdGordon_ | unless someone comes up with somewhere else not in the main thread which allocs, its not an issue |
21:29:59 | amiconn | Playback isn't in the main thread |
21:30:18 | amiconn | Dircache is yet another thread |
21:30:23 | pixelma | hmm... exchangint the firmware file should be enough on the Ondio to test the radio screen patch, or am I missing something? |
21:30:26 | kugel | JdGordon_: as a note, the multifont patch breaks checkwps so beware :p |
21:30:27 | JdGordon_ | right, so it needs to make sure not to while(1) if the buffer is locked |
21:30:31 | pixelma | exchanging too |
21:30:42 | JdGordon_ | pixelma: yes, it should |
21:30:50 | JdGordon_ | kugel: yeah, I know (or expected it anyway) |
21:31:19 | * | gevaerts somehow thinks that there won't be any deadlock, but he has no proof |
21:31:37 | JdGordon_ | anything outside the main thread just does a non blocking handle request and yields if it fails |
21:31:56 | JdGordon_ | those threads are not critical if the alloc waits longer (except maybe buffering) |
21:32:07 | JdGordon_ | which would just lead to a playback stall, not a crash/lockup |
21:32:07 | gevaerts | possibly wrapped in a loop that sleeps until unlock |
21:32:29 | JdGordon_ | no, it should be handled by the main thread loop properly |
21:32:32 | gevaerts | buffering can't use the allocation anyway until the disk is ready for it |
21:32:50 | gevaerts | And non-disk locks should be *very* short |
21:32:56 | JdGordon_ | right |
21:33:15 | JdGordon_ | as lnog as yield changes to a lock yield we shuold be safe |
21:33:35 | CIA-88 | New commit by amiconn (r24602): Remove leftover from before make system rework. |
21:33:36 | | Join fml [0] (~53ecea55@giant.haxx.se) |
21:34:08 | amiconn | linuxstb: ^^ That's not the one for the standalone build, of course |
21:34:21 | JdGordon_ | the COP is then the last major obstacle |
21:34:59 | fml | JdGordon: hey! I still think UI fonts shouldnot get special treatment in the firmware. Just allocate the buffer at RB start and load the UI fonts (if specified) like all other skin fonts. |
21:35:21 | | Quit yosafbridge (Quit: Coyote finally caught me) |
21:35:22 | pixelma | JdGordon_: I get a blank screen with the previous version of the patch too |
21:35:27 | JdGordon_ | bugger |
21:35:29 | JdGordon_ | ok thanks |
21:35:34 | JdGordon_ | fml: I replied in the thread |
21:35:41 | JdGordon_ | and I duisagree :) |
21:35:48 | fml | JdGordon: we could even treat the sysfont the same way if it can be described as a "font", i.e. as a struct font |
21:36:18 | JdGordon_ | it *is* a font struct |
21:36:19 | | Join yosafbridge [0] (~yosafbrid@li14-39.members.linode.com) |
21:36:20 | fml | JdGordon: yes, I saw your reply, but you agree with all I said actually. |
21:37:14 | JdGordon_ | I said it wasnt worth the effort to move it, and it doesnt really make it cleaner |
21:37:23 | fml | JdGordon: then we can handle the sysfont the same way. Just "load" it in the beginning so the ref gets 1. And then it will stay forever since noone will unload it. |
21:37:27 | JdGordon_ | sysfont has to be handled differently, ui font may as well also be |
21:38:09 | fml | JdGordon: why does it have? I don't think so. And it *will* make firmware cleaner. |
21:38:36 | fml | Wemy of course have different understanding of cleanness :-) |
21:39:09 | JdGordon_ | no, because the setting MUST be loaded into the FONT_UI or FONT_SYS slots, we have no way to force that, and that font will change during a session |
21:39:39 | JdGordon_ | and FONT_UIREMOTE for them |
21:40:00 | | Quit yosafbridge (Client Quit) |
21:40:04 | amiconn | pamaury_: Do we really need this handle leaking check? It costs quite some binsize for what it does |
21:40:08 | | Join yosafbridge [0] (~yosafbrid@li14-39.members.linode.com) |
21:40:21 | amiconn | Imo the plugin author has to make sure his plugin doesn't leak handles |
21:40:56 | | Join alexbobp [0] (~alex@66.112.249.238) |
21:41:53 | pamaury_ | that was proposed by gevaerts :) I'm *just* the implementer. |
21:41:53 | kugel | hehe, I got a PM already for my plans to keep the download count when updating |
21:42:12 | fml | JdGordon: we can. We can load them at the start so that they will get loaded at the right positions. |
21:42:15 | amiconn | It could be useful for developers, but imo it should be disabled by default |
21:42:32 | JdGordon_ | fml: and then how do you load a different font to that slot? |
21:43:14 | | Quit fml (Quit: CGI:IRC (EOF)) |
21:43:39 | pamaury_ | amiconn: I also think it's useful for developpers but you're right, perhaps it should be disabled by default. Ask gevaerts, he thinks it should be enabled by default. |
21:43:48 | | Join fml [0] (~53ecea55@giant.haxx.se) |
21:44:10 | | Quit guesswhoooo (Quit: CGI:IRC) |
21:44:51 | fml | JdGordon: we can have a replace function or something like that. |
21:44:57 | amiconn | gevaerts: ^^ |
21:45:40 | JdGordon_ | fml: but then you are at the point where you are adding more code for that case than to just have the 4 line special case for them that it is now |
21:46:07 | JdGordon_ | and then it even more so special cases those fonts |
21:46:13 | fml | JdGordon: and 10K wasted memory for nothing |
21:46:27 | JdGordon_ | remote font you talking about? |
21:46:33 | fml | yes |
21:46:35 | pixelma | interresting. I never noticed that the radio kept playing in the USB screen on the OndioFM at least |
21:46:40 | fml | (besides other things) |
21:47:01 | JdGordon_ | the remote buffer is the only thing wasted if its unused |
21:47:10 | fml | pixelma: I think it's always been like that, on other targets too |
21:47:34 | JdGordon_ | I was thinking about spitting the main screen remote buffer 50:10, but dont know a nice way to see if the split is needed |
21:47:50 | JdGordon_ | without reloading both font files |
21:47:54 | fml | JdGordon: and then we wouldn't have that special treatment for the NULL params which is not very nice IMO |
21:48:27 | JdGordon_ | you just move the special casing out of firmware into more volatile code |
21:48:31 | JdGordon_ | it doesnt make sense imo |
21:48:34 | pixelma | I guess on the M5 the 5:1 split wouldb't be fair |
21:49:13 | fml | JdGordon: btw: I've only looked at the patch, not the whole file. But I noticed that there are fd's (file descriptors) in the app level font descriptors (skin fonts) as well. Why that? |
21:49:58 | fml | JdGordon: I won't be able to properly look at that until after the next week |
21:50:16 | JdGordon_ | no, there is an id, not an fd |
21:51:40 | kugel | domonoky: the guy was worried about people abusing the rating and dl count feature |
21:51:52 | pixelma | although album art doesn't work (correctly) on the remote which reminds me that I should write some bug reports |
21:52:16 | domonoky | kugel: he PMd me too..:-) |
21:52:25 | fml | JdGordon: ah, right! |
21:52:44 | kugel | I'm not sure if dl count should be made unique |
21:53:12 | kugel | if I download the theme again after it was updated it should count again; but downloading every 2 seconds is surely too abusive |
21:53:54 | domonoky | its also not easy to prevent abuse of download counts and ratings. currently only the sum is store. |
21:54:41 | kugel | maybe it could work like this: if an ip downloads/rates 3 times within 15min then lock it out for 2h (numbers could be changed) |
21:56:03 | domonoky | you still need to store the ips in the db somewhere, which is probably not preferable. but maybe some cookie magic could help ? ( i dont know much about web things) |
21:56:34 | | Quit Strife89 (Quit: Going home.) |
21:56:58 | kugel | me neither :( |
21:57:08 | kugel | but cookes are easier to trick around |
21:58:28 | JdGordon_ | do we really care that much if peopkle want to be stupid? |
21:59:08 | domonoky | sure, but it might but the barrier high enough so we dont get much abuse, and it might be easier (if someone knows cookie magic) |
21:59:40 | CIA-88 | New commit by kugel (r24603): Save download count when updating a theme |
21:59:49 | | Join phanboy4 [0] (~benji@gate-22.spsu.edu) |
22:00 |
22:01:12 | domonoky | kugel: commit looks good. i will make it live now :-) |
22:01:23 | kugel | JdGordon_: we not so much, but the themers do |
22:01:29 | kugel | domonoky: thanks :) |
22:01:55 | dionoea | the issue with ip restrictions is people behind NATs (or countries behind NATs) |
22:02:35 | | Join Dhraakellian [0] (~ntryon@cpe-66-66-76-248.rochester.res.rr.com) |
22:03:09 | Dhraakellian | was USB support for the 5G iPod Video disabled in the 3.5 build? |
22:03:24 | kugel | the captcha idea sounds bloody annoying if you ask me |
22:04:20 | domonoky | captchas for rating are too much hassle i think. For reporting themes its fine. |
22:04:35 | kugel | the guy suggested it for downloading too |
22:04:44 | domonoky | and we surely dont want captchas for downloads ! |
22:05:04 | dionoea | can't you just use cookies? (we use that for theme rating on the vlc skins download page and have never had any abuse, at least not significant enough to notice) |
22:05:32 | domonoky | dionoea: that was my idea, but i dont know cookies.. maybe you could help out ? :-) |
22:06:15 | kugel | sure, it's just that cookies never convinced me for preventing abusing. and someone needs to code it of course |
22:06:16 | pixelma | Dhraakellian: I think it was disabled for all Ipods in the release because they still have issues with charging in Rockbox USB which can actually lead to discharge |
22:06:28 | JdGordon_ | oh, just tell the themers to get stuffed, their ego's arnt that important that a few fake downloads will change anything |
22:06:33 | dionoea | domonoky: in php its easy: $_COOKIE["somekey"] = somevalue; |
22:06:53 | Dhraakellian | pixelma: ah |
22:07:15 | Dhraakellian | well, given that my Woot BoC iPod only gets ~2h battery life *anyway* |
22:08:19 | JdGordon_ | AlexP: you wouldnt like to try the latest patch and let me know if you get a blank screen would you? |
22:08:49 | pixelma | hmm, I could test on my M5 |
22:09:10 | domonoky | dionoea: sounds easy :-) |
22:09:32 | kugel | setcookie("prevent_abuse" ,"asd" ,time()+60*5 ); seems to do it too |
22:10:03 | dionoea | yeah, it's equivalent except that setcookie allows setting the experiation (while the array method doesn't) |
22:10:07 | kugel | then just if(isset($_COOKIE["prevent_abuse"])) "Stop abusing"; |
22:10:38 | | Join webguest [0] (~4fb85cb2@giant.haxx.se) |
22:10:41 | bluebrother | domonoky: what do you think about completely dropping the turkish and chinese translations? They are in really bad shape and turkish itself is broken on file level |
22:10:54 | dionoea | we basically do $_COOKIE["skinrated_{$_POST["skin_id"]}"] != $_POST["skin_id"] that way people can only rate a skin once |
22:11:02 | bluebrother | or should we rather remove them from the resource file to exclude them from the binaries? |
22:11:08 | | Join froggyman [0] (~sopgenort@pool-72-69-210-48.chi01.dsl-w.verizon.net) |
22:11:24 | domonoky | bluebrother: removing them from the resource is good. |
22:11:43 | kugel | dionoea: I think we want download count to count again after some time |
22:11:43 | | Quit webguest (Client Quit) |
22:11:54 | dionoea | cookies timeout |
22:12:01 | domonoky | and maybe remove the broken turkish, but chinese should stay in svn to ease a future update. |
22:12:33 | domonoky | kugel: use two cookies, one with short timeout for download counting, and one with long/ever for rating. |
22:12:38 | dionoea | kugel: it looks like the default timeout is a browser session |
22:12:48 | dionoea | which seems decent enought |
22:12:49 | dionoea | -t |
22:12:58 | dionoea | see php.net/setcookie |
22:13:04 | | Quit phanboy4 (Ping timeout: 248 seconds) |
22:13:24 | rasher | bluebrother: Why would you drop a translation simply for being in bad shape? |
22:13:26 | JdGordon_ | the rating cookie only lasts untill the update though yeah? |
22:14:35 | * | kugel has a look |
22:14:42 | domonoky | JdGordon_: if we use the themeid as cookie value, its not a problem. updates get new ids. |
22:15:13 | JdGordon_ | just makign sure :) |
22:15:26 | | Join FOAD_ [0] (~dok@dinah.blub.net) |
22:15:30 | bluebrother | rasher: well, is a translation that has less than 40% of all strings translated actually useful? At least I prefer untranslated applications rather than reading two languages all the time |
22:15:59 | rasher | bluebrother: But you understand English perfectly. For someone who doesn't, 40% is pretty good |
22:16:13 | bluebrother | and turkish itself is broken on file level −− all non-ascii characters are encoded as the same character. Which makes it really useless. |
22:16:33 | rasher | Well yes, that's no good obviously |
22:16:52 | bluebrother | rasher: I wouldn't call my english anything about perfect. But admittedly, I'm not uncomfortable using that language :) |
22:18:33 | kugel | hrm, I don't get a cookie |
22:18:57 | | Join yosafbri` [0] (~yosafbrid@adsl-71-142-225-118.dsl.scrm01.pacbell.net) |
22:19:06 | | Join phanboy4 [0] (~benji@gate-22.spsu.edu) |
22:19:13 | | Quit FOAD (Ping timeout: 272 seconds) |
22:19:13 | | Nick FOAD_ is now known as FOAD (~dok@dinah.blub.net) |
22:19:13 | | Nick pamaury_ is now known as pamaury (~pamaury@ALyon-551-1-70-117.w92-137.abo.wanadoo.fr) |
22:19:17 | kugel | ah wait there it is |
22:19:51 | | Quit pyro_maniac_ (Remote host closed the connection) |
22:20:04 | Llorean | Maybe it would make sense to define some strings as "core strings" (being the minimum necessary to use the player; which would just be very basic things like the label for the file browser and some error messages) and as long as those are still translated leave "old" translations in? |
22:20:06 | kugel | the expiration time doesn't seem to work |
22:20:57 | kugel | or firefox doesn't show it properly... |
22:22:57 | kugel | ok, that was really easy |
22:23:44 | Mode | "#rockbox -o Llorean" by ChanServ (ChanServ@services.) |
22:24:48 | kugel | domonoky: http://pastie.org/820844 that blocks downloadcount += 1 for 30s |
22:25:04 | CIA-88 | New commit by bluebrother (r24604): Rockbox Utility russian translation update. ... |
22:26:15 | kugel | commit? |
22:26:26 | domonoky | kugel: shouldnt it check for the theme id ? so it counts if i download different themes ? |
22:26:41 | kugel | ah you mean it should be on a per-theme basis? |
22:26:49 | dionoea | that would be coherent |
22:26:50 | domonoky | jup |
22:27:05 | kugel | makes sense |
22:28:05 | domonoky | also i would name the cookie a bit different (downloadcnt or similar) so we dont get conflicts when we add a cookie for the ratings. |
22:29:16 | kugel | yes, I didn't plan to leave it at "abuse" actually :) |
22:29:25 | domonoky | :-) |
22:31:47 | kugel | hm, that means mutiuple cookies, else downloading two themes in an alternating fashion still permits abuse |
22:32:44 | domonoky | just sprintf the themeid into the cookie name. |
22:32:55 | kugel | jup did so and it works |
22:33:28 | kugel | http://pastie.org/820863 |
22:34:17 | * | kugel needs to push his themes before this goes in! |
22:34:20 | kugel | ;) |
22:34:54 | domonoky | if it works, but it in. |
22:35:39 | domonoky | one small "bug" downloadcnd :-) |
22:35:55 | kugel | yep, saw it :) is 30 ok or do we want a higher value?= |
22:35:57 | kugel | 30s |
22:36:28 | domonoky | it could easily be more and wont hurt. |
22:36:41 | * | JdGordon_ whips up a script to download my theme every 31 seconds |
22:37:18 | bluebrother | what are you currently implementing? Hammering prevention? |
22:37:29 | | Quit Omlet (Read error: Connection reset by peer) |
22:37:37 | domonoky | bluebrother: abusing download counts. |
22:38:03 | domonoky | you can still download the theme multiple times in 30 seconds, it just wont be counted. |
22:38:34 | bluebrother | ah, ok. Just was wondering if it would conflict with rbutil's ability to download multiple themes :) |
22:39:02 | | Join Range_304 [0] (~Guest_721@79-68-226-184.dynamic.dsl.as9105.com) |
22:39:16 | | Part Range_304 |
22:39:24 | kugel | do rbutil downloads count at all? |
22:39:46 | * | kugel goes for 3min |
22:39:48 | domonoky | yes, they count too.. i also give rbutil the download.php link |
22:40:35 | bluebrother | reminds me that I wanted to display the theme name instead of the download filename in the progress dialog. With that script link it became pretty ugly (and longish) |
22:40:42 | domonoky | see rbutil.tpl |
22:41:14 | pixelma | JdGordon_: on my M5 I get something on the default radio screen without an fms set - but it looks a bit messy. First line reads literally: |
22:41:17 | pixelma | Station: %d, %02d MHz (and then it's showing the current frequeny) |
22:41:42 | JdGordon_ | you didnt update english.lang then? |
22:41:57 | JdGordon_ | and yes, thats the other minor fix thats still needed |
22:42:12 | pixelma | I did a complete build and unzipped it fully |
22:42:12 | JdGordon_ | I need to deprecate LANG_FM_STATION and come up with a new one for "Station:" |
22:42:41 | JdGordon_ | ok thats odd, english or another language? |
22:42:45 | pixelma | oh, hmm. That's with the previous patch |
22:43:01 | pixelma | currently German |
22:43:16 | pixelma | and so it's showing "Sender:" actually |
22:43:27 | JdGordon_ | that could be the problem then |
22:43:27 | CIA-88 | New commit by kugel (r24605): Prevent abusing download count a bit using cookies, downloads don't increment the counter until 3min have passed after the previous counted one |
22:43:45 | pixelma | I don't like the bare look of the default radio screen though |
22:43:45 | JdGordon_ | kugel: 3min? thats a bit drastic |
22:43:56 | JdGordon_ | pixelma: isnt t the same as svn? |
22:44:04 | kugel | is it? |
22:44:05 | pixelma | mo |
22:44:10 | pixelma | err... no |
22:44:26 | | Quit petur (Remote host closed the connection) |
22:44:52 | domonoky | JdGordon_: why is that drastic ? its only the counter and only if you download the same theme again and again. |
22:45:03 | | Join petur [0] (~petur@rockbox/developer/petur) |
22:45:17 | pixelma | there is no progressbar as far as I remember but more info about mono/stereo and preset/scan mode - need to recheck but it's definitely more than 3 lines |
22:45:30 | JdGordon_ | domonoky: oh, you can keep downloading, just the coutner doesnt inc? |
22:45:40 | domonoky | JdGordon_: sure |
22:45:45 | JdGordon_ | ok then its fine |
22:46:10 | | Join flydutch [0] (~flydutch@host66-209-dynamic.15-87-r.retail.telecomitalia.it) |
22:46:23 | domonoky | now we need todo the same for rating, just with a verylong timeout. :-) |
22:46:43 | * | JdGordon_ thinks this is all wasted effort |
22:46:53 | B4gder | just note that doing the same operation with curl and wget won't get stopped by that cookie thing |
22:46:54 | JdGordon_ | can you make it show a listing of what is included in each theme? |
22:47:12 | pixelma | JdGordon_: ok, the first line problem really has to do with the language but that shouldn't happen if the patch goes in... |
22:47:32 | JdGordon_ | yes, see about about deprecating that string |
22:48:10 | domonoky | B4gder: sure, it isnt bullet proove, but probably good enough. |
22:48:27 | kugel | an updated theme is given a new id, right? |
22:48:28 | B4gder | yeah, it'll prevent the repeated pretting of the link a browser |
22:48:39 | B4gder | uh pressing |
22:48:43 | domonoky | kugel: correct, a updated theme is technically a new theme. |
22:49:15 | domonoky | and theme ids are just the line numbers of the theme table. |
22:49:54 | pixelma | SVN is four lines on swcodec: (1) Preset number. Preset name (2) Station: frequency (3) Mono/Stereo (4) Mode: scan/preset |
22:50:13 | | Quit petur (Remote host closed the connection) |
22:50:52 | | Join petur [0] (~petur@rockbox/developer/petur) |
22:51:52 | pixelma | hwcodec also has the peakmeter below |
22:52:06 | domonoky | kugel: maybe we should reset download counters and ratings after we have implemented both abuse measures ? |
22:52:23 | kugel | I have thought about it too |
22:52:30 | kugel | we could, but I don't care much |
22:52:38 | *** | Saving seen data "./dancer.seen" |
22:52:48 | * | pixelma has 3 radios running currently ;) |
22:53:26 | kugel | awesome: "DB Error: not an error (0)" |
22:53:47 | * | kugel posts that to tdwtf |
22:56:16 | kugel | ok, that works |
22:57:09 | Unhelpful | amiconn: i had put *some* thought into making the audio buffer (including buffer_alloc stuff done before playback) buflib-like. the idea i had was to add a flags or types bitmask to buflib allocations, with certain flags indicating that certain types of data would exist before the actual buffer item. one of these flags would've marked "relatively static" data that was not supposed to be moved without calling a callback to let its owner know |
22:57:09 | Unhelpful | . the idea was that "static" allocations could replace most of buffer_alloc uses, but we would still be able to make changes to these, for example to get a larger buffer on skin load, etc. |
22:57:20 | | Part Dhraakellian ("Meddle not in the affairs of ircops, for they are (not so) subtle, and quick to anger.") |
22:57:22 | JdGordon_ | pixelma: preset name/number doont really mean anything in scan mode |
22:57:45 | Unhelpful | also a bunch of stuff the audio buffer has now, like filename, would be gone except for items that actually want it. |
22:58:33 | | Quit evilnick_B (Quit: Page closed) |
22:58:47 | pixelma | JdGordon_: yes, and? |
22:59:06 | JdGordon_ | so having them on the screen is pointless? |
23:00 |
23:01:18 | CIA-88 | New commit by kugel (r24606): Prevent abusing rating a bit by using cookies, this time we try to prevent rating a theme mulitple times entirely (which basically can't work with ... |
23:02:34 | domonoky | kugel: nice commit message :-) |
23:02:49 | pixelma | current SVN shows a blank line instead. That's not a reason to combine the mode line with it - especially as there are only 4 lines at most and I think it looks a bit better as it makes better use of the scree, especially if you think about the fact that preset number and name get more room (hence less scrolling) when not combined with the mode line |
23:03:30 | | Quit bertrik (Quit: De groeten) |
23:03:33 | pixelma | and preset mode is what I use much more than scan mode |
23:05:16 | kugel | domonoky: so do we reset now? |
23:05:29 | | Join kramer3d_ [0] (~kramer@unaffiliated/kramer3d) |
23:05:46 | domonoky | kugel: just done it :-) |
23:06:44 | gevaerts | amiconn: I agree that in general plugin authors should make sure they don't leak, but right now we have lots of plugins that may or may not be buggy. Maybe we should just leave it enabled for a week or two, and then disable it again? |
23:08:30 | | Quit kramer3d (Ping timeout: 256 seconds) |
23:09:04 | | Quit dmb (Ping timeout: 256 seconds) |
23:09:56 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
23:10:02 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
23:10:05 | pixelma | hmm... my .fms doesn't seem to be loaded. The cfg file should contain a line "fms: /.rockbox/WPS/my.fms" and my.fms should be at the place specified. correct? |
23:10:59 | * | petur has the recording histogram finally working in the sim \o/ |
23:11:16 | pixelma | provided I use my.wps too (I actually copy-pasted the wps line in the config and edited the file extension and the beginning of the line |
23:11:37 | pixelma | ^ JdGordon_ |
23:12:17 | JdGordon_ | lower case wps in the setting |
23:12:33 | JdGordon_ | unless our file code is case insensitive |
23:12:39 | JdGordon_ | that is correct otherwsie |
23:12:40 | pixelma | yes, caps lock error |
23:13:09 | pixelma | when typing it here I mean |
23:13:15 | gevaerts | uhm, it's case sensitive? |
23:14:11 | pixelma | don't know and doesn't matter as it is all lower case on target |
23:14:13 | * | gevaerts thinks that if so, that's a bug |
23:14:31 | gevaerts | ah, ok |
23:14:32 | JdGordon_ | I dont think it is |
23:14:47 | * | gevaerts shuts up |
23:14:53 | | Nick kramer3d_ is now known as kramer3d (~kramer@unaffiliated/kramer3d) |
23:16:21 | CIA-88 | New commit by stripwax (r24607): Get rid of ffmpeg_scratchpad member (not needed since previous commit - accidentally left it in) |
23:17:20 | kugel | the playlistviewer is annoying |
23:17:31 | JdGordon_ | in what way? |
23:17:41 | | Join mt [0] (~mtee@rockbox/developer/mt) |
23:17:51 | kugel | in the way that pressing down opens the 2nd context menu instead of going to the main menu |
23:18:33 | pixelma | which target? |
23:18:58 | pixelma | and huh? |
23:19:13 | kugel | fuze, e200 and probably more |
23:19:36 | CIA-88 | New commit by bluebrother (r24608): Don't close the disk handle to the ipod too early. ... |
23:19:48 | pixelma | can't follow your description what it does or not |
23:21:21 | pixelma | which might be related to the fact that I don't have a scrollwheel target and I currently think that it is this way on those targets |
23:21:52 | | Quit bluebrother (Ping timeout: 265 seconds) |
23:22:02 | kugel | possible, but the playlist viewer has 2 context menus |
23:22:15 | kugel | and one of them overrides the global go-to-mainmenu button |
23:22:51 | | Quit GeekShado_ (Quit: The cake is a lie !) |
23:23:22 | | Join bluebrother [0] (~dom@g224236039.adsl.alicedsl.de) |
23:23:23 | | Quit bluebrother (Changing host) |
23:23:23 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
23:23:24 | | Join Adnyxo [0] (~aaron@adsl-065-013-002-216.sip.asm.bellsouth.net) |
23:24:41 | bluebrother | AlexP: that was the bug discussed in the forums. I'm still considering not to answer in that thread. |
23:25:01 | pixelma | true. I never noticed as I never wanted to go to the main menu from there, just view or manipulate the playlist or get back to the WPS |
23:25:22 | pamaury | Is there any tagcache expert here ? |
23:25:35 | | Part _zic1 |
23:25:36 | | Quit Horscht (Ping timeout: 264 seconds) |
23:26:39 | CIA-88 | New commit by bluebrother (r24609): Remove turkish from the languages bundled in the binary. The translation is broken on file level (all non-ASCII characters are broken). |
23:31:12 | | Quit fml (Quit: CGI:IRC) |
23:32:31 | pamaury | How does rockbox detect the type of a music file ? Is it only extension based ? |
23:32:39 | kugel | pamaury: I'm no expert but I have looked at the code time ago |
23:32:48 | pixelma | JdGordon_: in my current preset list, there are 6 of 17 stations that don't fit completely on the line with "Preset: " in front which fit before - and the line doesn't even scroll |
23:33:13 | pixelma | that's on my M5 with 12-Nimbus |
23:33:20 | domonoky | pamaury: it starts with the file extension. this maps to a metadataparser, which can change the codec type again. |
23:33:26 | kugel | pamaury: IIRC it's primary extension based, but the metadata parser can change that |
23:33:41 | pamaury | kugel: I'm not sure. Suppose I have a complete file path P and I want to retrieve some properties of the tagcache entry associated to this name. How can I do that ? Do I use the tagcache_find_index function ? |
23:34:16 | pixelma | and it seems I can't load an own fms, or I did something wrong but you didn't tell me before |
23:35:02 | kugel | pamaury: get_metadata() IIRC |
23:35:42 | pamaury | kugel: yes but then if I want to be able to modify some data associated to the file, then it's better to use tagcache (it's for MTP), no ? |
23:35:59 | * | domonoky thinks get_metadata() wont use tagcache, but directly parse that file. |
23:36:16 | pixelma | pamaury: it doesn't find the mp3s put on my c200 in MTP mode with the SansaOF which are renamed to <some arbitrary string>.dat but work fine if I change the extension back to .mp3 |
23:36:23 | kugel | tagcache doesn't allow modifying the files IIRC |
23:36:51 | JdGordon_ | pixelma: ok, so the %s needs to be added |
23:36:54 | pixelma | so I *assume* from a user perspective that it's extension based |
23:36:55 | pamaury | no, but it allows to modify some data associated to it like the rating no ? |
23:37:14 | kugel | rating and playcount, that's all |
23:37:25 | kugel | and you need gather runtime data enabled for that |
23:37:41 | kugel | the other meta data is read-only afaik |
23:38:06 | pixelma | JdGordon_: I would also prefer these things broken into two lines again |
23:38:07 | pamaury | Ok, so assume that I code a read-only version of MTP, then you think get_metadata is better than tagcache to retrieve meta data ? |
23:38:40 | JdGordon_ | pixelma: would you like to write the code and ill copy it into the .c? |
23:38:58 | | Join dmb [0] (~Dmb@unaffiliated/dmb) |
23:39:00 | kugel | pamaury: if domonoky is right then get_metadata() maybe isn't the best candidate |
23:39:22 | pamaury | Yes, because it will be awfully slow |
23:39:30 | domonoky | pamaury: get_metadata() will work even without tagcache, but it will be slooow |
23:39:50 | pixelma | you already have it almost, just shuffling it around a bit. And I'd like a way to test (other than recompile ;) ) |
23:40:13 | * | pixelma could also try on c200, maybe things are also different there... |
23:40:18 | pamaury | Then, I can suppot both. But tagcache is mandatory on all targets, on ? |
23:40:19 | CIA-88 | New commit by b0hoon (r24610): Packard Bell Vibe 500: Start committing plugin keymaps. |
23:40:36 | kugel | pamaury: if ramcache is available then I'd use tagcache_fill_tags() |
23:40:56 | pamaury | Hum, interesting |
23:41:13 | pixelma | JdGordon_: I could do that of course. Btw. the progressbar is quite nice |
23:41:16 | kugel | you need to perform a tagcache search for it though |
23:41:47 | pamaury | It a tagcache search slow ? |
23:41:49 | pamaury | *Is |
23:41:57 | JdGordon_ | it is if its not in ram |
23:42:14 | kugel | pamaury: err no, you dont |
23:42:18 | kugel | bool tagcache_fill_tags(struct mp3entry *id3, const char *filename) |
23:42:20 | domonoky | pamaury: tagcache is not avaiable be default. users need to initialise the database first. |
23:42:59 | kugel | I'm not sure whether tagcache is useful for mtp |
23:43:12 | pamaury | That's my current question :) |
23:43:56 | kugel | I rather doubt it |
23:44:26 | kugel | mtp would be pretty useless if you could really only transfer audio files with it |
23:44:30 | pixelma | tagcache still chokes on large collections, right? Also with no music files (can't remember seeing a fix at least= |
23:45:05 | kugel | pixelma: someone noticed strange behavor with 65k+ songs, and I fixed that |
23:45:25 | kugel | the upper limit is theoretically 4 billion now |
23:45:41 | domonoky | and would .databaseignore files also be effective for mtp ? |
23:45:50 | pamaury | The possible use I see of tagcache is to retrieve metadata. Indeed, MTP provides a way to retrieve metadata without retrieving the whole file. Here tagcache helps because it contains all the necessary data. |
23:46:16 | pixelma | hmm... maybe I confused it with dircache |
23:46:19 | pamaury | domonoky: what is .databaseignore ? |
23:46:36 | pamaury | I already use dircache :) |
23:46:38 | kugel | pamaury: in that case, it may be useful |
23:46:44 | domonoky | a file to let the database ignore files in certain directorys. |
23:46:50 | pixelma | it's database.ignore |
23:47:25 | domonoky | there is also database.unignore :-) |
23:47:39 | amiconn | JdGordon_, gevaerts: vfat is case preserving, but case insensitive |
23:47:40 | pamaury | ok, I didn't know it existed :) |
23:47:56 | pixelma | it's more of a :( though |
23:48:19 | | Join Casainho [0] (~chatzilla@87-196-74-13.net.novis.pt) |
23:48:20 | pamaury | Why ? |
23:49:05 | | Quit Adnyxo (Quit: Leaving) |
23:49:53 | domonoky | if other mtp features still work for everthing, it might be ok to just provide metadata access for those which are in the db. |
23:50:08 | pixelma | because it slows things down again. Before the unignore tagcache didn't look deeper into the directory structure if you had a .ignore at the top level. Now it still goes ahead looking into subfolders searching for an unifnore |
23:50:14 | pixelma | *unignore |
23:50:26 | domonoky | and mtp could ofcourse update tagcache while transfering new files :-) |
23:50:37 | | Quit flydutch (Quit: /* empty */) |
23:50:56 | pamaury | True |
23:52:40 | | Quit jgarvey (Quit: Leaving) |
23:52:43 | pamaury | Ok, I think my code will try to use tagcache_fill_tags if the db is in RAM, then tagcache_find_index if not and finally get_metadata if db is not initialized. I think performance will get worse in this order |
23:53:51 | | Quit efyx_ (Remote host closed the connection) |
23:53:57 | | Join checker [0] (~621342a9@giant.haxx.se) |
23:54:50 | checker | anyone know what "BM6" means? My wps isn't showing up, instead i get this little code |
23:55:34 | JdGordon_ | pixelma: ok, yeah, if you make the default fms I'll happily use it. I'll work on trying to figure out the last issues tonight |
23:55:34 | | Quit ender` (Quit: The problem with political jokes is they get elected. -- Henry Cate, VII) |
23:56:59 | pixelma | and I would be opposed to this going in without a fms browser, editing the cfg is a bah |
23:57:34 | JdGordon_ | yes yes, dont worry about that |
23:57:50 | JdGordon_ | its not done yet because its the boring finiky bits at the end which I just havnt done yet |