| 00:00:29 | kugel | why that? |
| 00:01:57 | | Join T44 [0] (i=Topy44@g228213029.adsl.alicedsl.de) |
| 00:02:34 | bluebrother | why not? It sometimes "interleaves" with log messages from the build client, and the details about the running upload aren't that interesting anyway. |
| 00:03:31 | kugel | Imo they're interesting |
| 00:06:15 | | Quit bluebrother ("enough headache for today") |
| 00:06:45 | CIA-61 | New commit by mt (r22337): Move bitstream.[ch] to codecs/lib/ffmpeg_bitstream.[ch] to avoid duplicate copies across codecs. |
| 00:06:49 | | Nick geo|kvirc is now known as geo|afk (n=kvirc@c-76-97-30-122.hsd1.ga.comcast.net) |
| 00:15:13 | kugel | seems the build system is struggling a bit without gevaerts server :) |
| 00:17:45 | Zagor | wow, that was a crappy round |
| 00:18:36 | | Join kirilla [0] (n=kirilla@c83-253-242-21.bredband.comhem.se) |
| 00:20:05 | | Quit Topy44 (Read error: 110 (Connection timed out)) |
| 00:20:28 | CIA-61 | New commit by mt (r22338): Fix red. |
| 00:20:37 | kugel | qnap-jdgordon doing massive parallel building agian. His client seems particularly hit by that |
| 00:22:03 | * | kugel wonders if ccache can handle the constantly changing build directory |
| 00:22:55 | Zagor | that's a valid question. and we learned yesterday that we can't trust the ccache manpage, so a test would be good |
| 00:23:20 | kugel | the cache hit / cache miss ratio is horrible, 15/66 |
| 00:23:58 | kugel | I'm getting *very few* hits duing this round, but I don't know if I get the same builds |
| 00:25:48 | kugel | Zagor: can we have a −−ccache parameter for rbclient.pl that's passed to configure? I don't feel like messing with it locally :( |
| 00:26:24 | Zagor | what would it do? |
| 00:27:04 | kugel | you can deactivate the use of ccache with configure IIRC |
| 00:27:50 | Zagor | you are right, ccache only caches compiles in the same path |
| 00:28:34 | kugel | wow, qnap-jdgordon is seriously broken this round |
| 00:28:50 | kugel | ah no, it did fine actually |
| 00:29:12 | kugel | Zagor: so ccache can't help us actually? |
| 00:29:26 | Zagor | it seems like the bug is accelerating |
| 00:29:36 | Zagor | kugel: it seems not |
| 00:29:45 | | Part kirilla |
| 00:30:24 | | Quit AsaelReiter ("CGI:IRC") |
| 00:30:46 | kugel | ah, not "fine", but it did do his duty (queued) build at least before starting the massive parallel building this time at least |
| 00:31:24 | kugel | Zagor: is the client still asking for builds after one is done? maybe that's not working correctly anymore (i.e. asking for a new one even if it didn't complete yet) |
| 00:32:36 | kugel | rasher's work is really helpful |
| 00:32:45 | Zagor | indeed |
| 00:32:50 | | Join HBK [0] (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
| 00:35:21 | Zagor | I think it is the generated .h files (autoconf.h and lang.h for example) that break ccache. |
| 00:36:08 | Zagor | though I can't think of a way to get around that |
| 00:37:20 | | Quit jernejovc ("I divided by zero. Oh shi--") |
| 00:37:37 | kugel | we could deactive ccache for all clients to see if it makes a difference |
| 00:37:37 | Zagor | kugel: yes they do. but I doubt that is what is broken. the server is much more likely. |
| 00:38:13 | Zagor | kugel: that's a very crude way of testing. client speeds vary anyway. |
| 00:38:46 | kugel | yea true :( |
| 00:39:05 | kugel | but pts/s don't change, does it? |
| 00:39:20 | Zagor | simply create two different build dirs and run "time make" in both. you'll see they take nearly the same amount of time. then run "make clean; time make" in one, and see how fast it goes... |
| 00:39:29 | kugel | actually, I had 220 pts/s last week, and since about amiconn's fix for ccache I only have 180 |
| 00:39:36 | | Quit efyx ("Quitte") |
| 00:40:15 | Zagor | we discovered yesterday that different compilers run at different speed on different machines. so the average speed is actually not terribly correct. |
| 00:41:17 | kugel | meh, if I disable ccache in configure locally it will produce M builds :( |
| 00:41:22 | Zagor | on the reference machine they all run att 100 pts/sec, but on one of my machines the arm gcc is nearly twice as fast as the m68k one. |
| 00:41:37 | Zagor | kugel: you don't have to disable ccache to test this |
| 00:42:05 | kugel | but I'd be interested to see if I'd reach the 220 again after deactivating |
| 00:42:19 | Zagor | aren't you reading what I'm writing? |
| 00:42:25 | kugel | and 180 after reactiving, and so on |
| 00:42:38 | kugel | I am |
| 00:42:50 | Zagor | ccache is not a factor in this |
| 00:42:58 | Zagor | since it's disabled anyway |
| 00:43:28 | Zagor | your average speed sunk because you were given other builds |
| 00:43:39 | kugel | it's disabled? |
| 00:43:54 | kugel | it's clearly running here, I watched ccache -s during the round |
| 00:44:01 | | Quit GeekShadow ("The cake is a lie !") |
| 00:44:19 | Zagor | kugel: do the test |
| 00:44:26 | kugel | I will |
| 00:48:03 | | Quit mt (Read error: 113 (No route to host)) |
| 00:50:35 | kugel | it's obvious that ccache doesn't handle differing build dirs |
| 00:50:57 | Zagor | yes |
| 00:51:53 | | Quit dfkt ("-= SysReset 2.53=- Ph'nglui mglw'nafh Cthulhu R'lyeh wgah'nagl fhtagn.") |
| 00:51:58 | Zagor | it's quite embarassing that we haven't noticed earlier... |
| 00:52:15 | Zagor | I did think it was strange that amiconn |
| 00:52:38 | kugel | let's see how fast it goes w/o ccache |
| 00:52:40 | Zagor | 's fix didn't seem to improve the times any significantly. but I was preoccupied with other issues. |
| 00:53:48 | kugel | faster |
| 00:53:54 | kugel | just a tad bit though |
| 00:54:02 | Zagor | yes, ccache is rather efficient |
| 00:55:13 | kugel | the build round was 1:48, the second 1:38, the last (no ccache) 1:31 |
| 00:57:26 | kugel | my machine is comparably fast, with big ram, I think the overhead isn't as high as on slower clients |
| 00:57:50 | Zagor | well, the fix isn't to remove ccache. the fix is to remove the different build dirs. |
| 00:58:23 | Zagor | all we have to do is run "make zip" before starting the next build |
| 00:58:27 | kugel | I guess that'd do the trick better indeed |
| 00:58:53 | Zagor | (and, obviously, moving away the zip so it can be uploaded while the new build is running) |
| 01:00 |
| 01:00:15 | Zagor | there's still the parallell builds server bug, too |
| 01:00:22 | kugel | but ccache still only helps if getting the same builds (which is quite likely though) |
| 01:02:31 | Zagor | only if you have a very small cache. the default size is 1GB which is room for quite a lot of builds |
| 01:02:48 | kugel | 2GB should be plenty :) |
| 01:02:57 | BdN3504 | bluebrother: what's your opinion on this? http://tinyurl.com/viewersmanual also to the others, what do you think if we split up the viewers section of the manual or include such a table? |
| 01:02:57 | | Quit roolku () |
| 01:08:49 | kugel | Zagor: I'm now wondering how proper ccache use would complicate best-fit |
| 01:11:11 | Zagor | it will make speeds vary more, which will of course make prediction more difficult. |
| 01:12:06 | Zagor | but over time (as clients populate their ccache) the average speed will still be a speed indicator |
| 01:12:31 | Zagor | it just won't match each build during the round as well |
| 01:13:12 | Zagor | it will certainly be interesting! :-) |
| 01:15:19 | *** | Saving seen data "./dancer.seen" |
| 01:15:24 | | Quit Zagor ("bed time") |
| 01:19:10 | | Quit robin0800 (Read error: 54 (Connection reset by peer)) |
| 01:23:49 | | Join mylogic [0] (n=matt@74-36-196-3.dsl1.stbo.ga.frontiernet.net) |
| 01:25:58 | T44 | i am trying to manually apply the (outdated) histogram patch to the current svn |
| 01:30:10 | T44 | but i am having trouble to see where one certain recording.c reject is sopposed to go... |
| 01:35:07 | | Join efyx [0] (n=efyx@lap34-1-82-224-140-171.fbx.proxad.net) |
| 01:35:44 | | Quit w1ll14m (Read error: 60 (Operation timed out)) |
| 01:36:07 | | Join w1ll14m [0] (n=wpeters@mail.setict.nl) |
| 01:45:48 | | Quit gevaerts (Nick collision from services.) |
| 01:46:00 | | Join gevaerts [0] (n=fg@rockbox/developer/gevaerts) |
| 01:54:04 | | Join bubsy [0] (i=Bubsy@94.139.72.137) |
| 02:00 |
| 02:01:18 | | Quit Rondom (Nick collision from services.) |
| 02:01:29 | | Join Rondom [0] (n=Rondom@dslb-084-057-161-175.pools.arcor-ip.net) |
| 02:07:32 | | Quit z35 (simmons.freenode.net irc.freenode.net) |
| 02:07:32 | NSplit | simmons.freenode.net irc.freenode.net |
| 02:38:11 | | Part DarkDefender ("Leaving") |
| 02:41:13 | | Join saratoga [0] (i=9803c6dd@gateway/web/freenode/x-dpnyzspyadfxjiog) |
| 02:49:19 | NHeal | (timeout) simmons.freenode.net irc.freenode.net |
| 02:53:49 | | Join Voluntaryist [0] (n=torment@pool-71-180-185-107.tampfl.fios.verizon.net) |
| 02:54:16 | Voluntaryist | if i install RB on a fuze there is 100% surety that DRM will not work after? |
| 02:55:07 | kugel | I guess yes. I haven't seen it myself, but that's because I have 0 DRM'd files |
| 02:55:28 | Voluntaryist | i have a e200 w/ rb on it, but i just bought a fuze to listen to audible |
| 02:55:36 | Voluntaryist | so i guess i wont do that :-/ |
| 02:58:00 | | Quit martian67 (Remote closed the connection) |
| 03:00 |
| 03:00:06 | | Quit domonoky1 (Read error: 104 (Connection reset by peer)) |
| 03:07:46 | | Join martian67 [0] (n=martian6@about/linux/regular/martian67) |
| 03:10:30 | | Quit martian67 (SendQ exceeded) |
| 03:12:27 | | Join martian67 [0] (n=martian6@about/linux/regular/martian67) |
| 03:15:22 | *** | Saving seen data "./dancer.seen" |
| 03:19:54 | | Quit SUSaiyan (Read error: 104 (Connection reset by peer)) |
| 03:20:16 | | Join SUSaiyan [0] (n=SUSaiyan@cc84863-b.zwoll1.ov.home.nl) |
| 03:25:21 | Unhelpful | is there a reason it wouldn't still work in the OF? some kind of firmware checksumming or such? |
| 03:26:07 | saratoga | Unhelpful: its been speculated that its due to changes to the RTC, which is used for determining if DRM keys are still valid |
| 03:26:27 | saratoga | probably fixable if anyone was really interested in it |
| 03:29:54 | Unhelpful | saratoga: oh! yes, i believe you're not actually allowed to *change* the RTC with the current gen of DRM. you're supposed to handle "setting" the clock by modifying a display offset for the real RTC time, or something like that, to prevent the user adjusting the time to make time-validated DRM work after it expires. |
| 03:34:08 | kugel | Unhelpful: I don't know if we still do, but I think we changed the rtc in the early days (re-used v1 code before we noticed the different offset) |
| 03:36:14 | saratoga | atrac seems to like to copy things between buffers that could probably be done in place |
| 03:38:27 | | Quit BdN3504 ("CGI:IRC (EOF)") |
| 03:39:17 | saratoga | ugh and why are "channel_unit" structs refered to by the variable "pUnits" |
| 03:51:12 | | Quit ehntoo (Read error: 110 (Connection timed out)) |
| 03:51:15 | Unhelpful | kugel: but the offset is probably variable, too, since as i understand it it's the only thing you're allowed to vary. the basic idea is that your subcription content is validated for the current RTC date plus one month, or whatever. :/ |
| 03:54:06 | kugel | IIUC we changed (or still change) the rtc date (at least when setting the time) |
| 03:54:16 | kugel | no idea really |
| 04:00 |
| 04:02:37 | | Nick daurn is now known as daurnimator (i=daurn@freenode/staff/daurnimator) |
| 04:15:05 | | Quit chandoo ("Leaving") |
| 04:15:32 | saratoga | mt: (for the logs) do you have a real media test file that uses joint stereo? none of the ffmpeg ones do and I'd like to test some changes with JS files before commiting |
| 04:21:20 | | Join SUSaiyan` [0] (n=SUSaiyan@cc84863-b.zwoll1.ov.home.nl) |
| 04:21:31 | | Quit SUSaiyan (Read error: 104 (Connection reset by peer)) |
| 04:22:38 | kugel | Unhelpful: I'm looking into anti-aliased font right now and struggle again :( |
| 04:24:03 | kugel | stupid mpegplayer :( |
| 04:24:13 | CIA-61 | New commit by saratoga (r22339): Make the IMDCT temporary working buffer a local variable instead of keeping one for each channel. Saves 4KB of RAM. |
| 04:29:29 | saratoga | hmm I bet we could compute ATRAC3 spectra on a per QMF band basis rather then doing them all upfront and having to store the output |
| 04:30:56 | | Join Hillshum [0] (n=chatzill@75-165-224-160.slkc.qwest.net) |
| 04:35:21 | | Nick intrados_ is now known as intrados (n=intrados@cpe-71-67-138-190.woh.res.rr.com) |
| 04:35:36 | | Join SUSaiyan [0] (n=SUSaiyan@cc84863-b.zwoll1.ov.home.nl) |
| 04:40:51 | | Quit kugel (Remote closed the connection) |
| 04:41:19 | | Quit efyx (Remote closed the connection) |
| 04:46:55 | | Quit SUSaiyan` (Read error: 104 (Connection reset by peer)) |
| 04:53:39 | | Quit bah_ (Read error: 104 (Connection reset by peer)) |
| 04:53:51 | saratoga | heh if the ATRAC3 decoder doesn't have a subband coded, it zeros the buffer and then adds it anyway, instead of just not adding . . . |
| 04:59:33 | | Join bah [0] (n=bah@c-76-105-203-173.hsd1.wa.comcast.net) |
| 05:00 |
| 05:06:37 | | Quit J-23 (Read error: 110 (Connection timed out)) |
| 05:09:52 | | Quit bah (Nick collision from services.) |
| 05:10:07 | | Join bah_ [0] (n=bah@c-76-105-203-173.hsd1.wa.comcast.net) |
| 05:15:23 | *** | Saving seen data "./dancer.seen" |
| 05:31:06 | | Join Mig [0] (n=chatzill@69.85.81.69) |
| 05:31:07 | | Quit SUSaiyan (Read error: 104 (Connection reset by peer)) |
| 05:31:11 | | Join SUSaiyan` [0] (n=SUSaiyan@cc84863-b.zwoll1.ov.home.nl) |
| 05:31:29 | saratoga | do plugins need to handle USB inserts specifically? i notice test_codec tends to freeze on insert |
| 05:31:33 | | Nick Mig is now known as Guest44401 (n=chatzill@69.85.81.69) |
| 05:34:53 | | Part Guest44401 ("I'm not here right now.") |
| 05:35:22 | | Quit parafin (Nick collision from services.) |
| 05:36:45 | | Join parafin|away [0] (n=operator@paraf.in) |
| 05:54:58 | | Quit Hillshum ("ChatZilla 0.9.85 [Firefox 3.5.2/20090729225027]") |
| 06:00 |
| 06:02:44 | | Join webguest76 [0] (n=4a681d95@gateway/web/cgi-irc/labb.contactor.se/x-qzrfamjtkavnmsbj) |
| 06:05:07 | | Quit webguest76 (Client Quit) |
| 06:05:42 | CIA-61 | New commit by saratoga (r22340): Put QMF coefficients into IRAM. Update test program to still compile with IBSS_ATTR defined. |
| 06:07:27 | | Quit mylogic (Client Quit) |
| 06:11:15 | | Quit geo|afk ("good night everyone!") |
| 06:13:50 | saratoga | the atrac IQMF is going to be impossible to make fast as its currently written |
| 06:20:14 | saratoga | hmm where is buschel when i need him |
| 06:22:34 | | Quit saratoga ("Page closed") |
| 06:41:16 | | Join inigomontoya [0] (i=621cc255@gateway/web/freenode/x-sokuwxrrovaqxsfw) |
| 06:42:29 | inigomontoya | Hello, I am having a problem with new builds, I can't turn off crossfade, something i'm doing wrong? should i file a bug report? |
| 06:42:52 | maraz | stop listening to mixtapes |
| 06:53:33 | | Quit inigomontoya ("Page closed") |
| 06:56:19 | | Join mylogic [0] (n=matt@74-36-196-3.dsl1.stbo.ga.frontiernet.net) |
| 06:56:52 | | Join ehntoo [0] (n=ehntoo@24-177-172-69.dhcp.mrqt.mi.charter.com) |
| 06:57:44 | | Quit mylogic (Client Quit) |
| 06:58:00 | | Join mylogic [0] (n=matt@74-36-196-3.dsl1.stbo.ga.frontiernet.net) |
| 07:00 |
| 07:15:24 | *** | Saving seen data "./dancer.seen" |
| 07:19:07 | | Quit JdGordon ("Leaving.") |
| 07:36:13 | | Quit CaptainKwel (Remote closed the connection) |
| 07:55:36 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 07:57:45 | | Join Horscht86 [0] (n=Horscht2@p4FD4C210.dip.t-dialin.net) |
| 08:00 |
| 08:00:36 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 08:05:00 | | Join Anu [0] (n=chatzill@117.200.164.54) |
| 08:08:12 | Anu | hello everyone |
| 08:08:52 | Anu | I failed to remove password using ATA PWD for dell optiplex 330 so what should i do ? |
| 08:09:29 | | Join Grahack [0] (n=chri@78.251.44.237) |
| 08:12:41 | | Join Rob2222 [0] (n=Miranda@p4FDCEBA9.dip.t-dialin.net) |
| 08:15:51 | | Join jayren [0] (n=jayren@67-132-69-188.dia.static.qwest.net) |
| 08:16:03 | | Join Marvin_ [0] (n=5abfca64@gateway/web/cgi-irc/labb.contactor.se/x-tdhvivdjnglcggyq) |
| 08:16:11 | | Quit Horschti (Read error: 110 (Connection timed out)) |
| 08:16:33 | Marvin_ | remember the nano 1g i told about ... that froze and was "dead" well it started to charge after a week of being dead |
| 08:16:38 | Marvin_ | and this night this happened http://bsd.ee/~olev/IMG_7124.jpg |
| 08:16:51 | Marvin_ | i am lucky that i woke up, saw the table literlly burning |
| 08:17:00 | Marvin_ | with open fire |
| 08:17:09 | Marvin_ | http://bsd.ee/~olev/IMG_7123.jpg |
| 08:17:37 | Marvin_ | seems that it did not know when to stop charging |
| 08:23:13 | jayren | i am assuming the english.voice file is combined speech fragments for the various voice prompts and such, is there some way to uncombine it? the link to the tour of the rockbox for the blind gave me an invalid server and i've been wanting to hear the voice of the thing since I read about it about a year ago. |
| 08:24:16 | * | jayren is back. |
| 08:25:30 | Marvin_ | the room still smells, god knows how much cancerogen it released to air |
| 08:30:30 | | Quit Rob2223 (Read error: 110 (Connection timed out)) |
| 08:32:14 | | Quit jayren ("Time wasted on IRC: 18 minutes 49 seconds") |
| 08:32:38 | | Quit Anu (Read error: 60 (Operation timed out)) |
| 08:33:12 | | Quit daurnimator (Read error: 110 (Connection timed out)) |
| 08:33:14 | | Join jayren [0] (n=jayren@67-132-69-188.dia.static.qwest.net) |
| 08:34:04 | | Part jayren |
| 08:42:25 | | Join daurnimator [0] (i=daurn@freenode/staff/daurnimator) |
| 08:46:25 | markun | Marvin_: wow |
| 08:48:09 | Marvin_ | i'll take it to apple shop on monday for laughs before breaking it apart to see the damage inside |
| 08:48:34 | Marvin_ | being 2 years old it has no warranty damn |
| 08:52:00 | T44 | i think as it WENT UP IN FIRE i believe you should AT LEAST get a new device out of it :) |
| 08:52:13 | T44 | if not compensation or whatever |
| 08:53:58 | Marvin_ | yeah, but we'll see |
| 08:55:31 | Marvin_ | i'm lucky that i was not drinking, god knows that it might have ignited |
| 08:56:50 | | Quit n1s (Read error: 110 (Connection timed out)) |
| 09:00 |
| 09:14:31 | | Quit fyrestorm ("lamers envy me like they envy bill g -- main boot xp, just the way it should be!") |
| 09:15:26 | *** | Saving seen data "./dancer.seen" |
| 09:33:14 | | Join mt [0] (n=MTee@rockbox/developer/mt) |
| 09:49:01 | | Join TheCoolGman [0] (n=446e5e7d@gateway/web/cgi-irc/labb.contactor.se/x-zuzqtmwcxeekgrat) |
| 09:49:10 | | Join flydutch [0] (n=flydutch@host214-146-dynamic.15-87-r.retail.telecomitalia.it) |
| 09:50:54 | TheCoolGman | Hi I was wondering the possiblites of MPEG Player being able to read the size of the video and then if it is bigger than 320x240 it would play it at a temp resized version |
| 09:52:42 | | Quit Tristan (Nick collision from services.) |
| 09:53:23 | | Join Tristan [0] (n=Tristan@i.dont.want.to.die.virgin.net.in) |
| 09:53:42 | | Quit Tristan (Nick collision from services.) |
| 09:54:29 | | Join Trista901 [0] (i=tristan@i.dont.want.to.die.virgin.net.in) |
| 10:00 |
| 10:14:30 | | Quit TheCoolGman ("CGI:IRC (EOF)") |
| 10:16:50 | | Nick Horscht86 is now known as Horscht (n=Horscht2@p4FD4C210.dip.t-dialin.net) |
| 10:31:51 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 10:32:41 | | Quit Grahack (Read error: 110 (Connection timed out)) |
| 10:44:41 | | Join daurn [0] (i=daurn@freenode/staff/daurnimator) |
| 10:49:35 | | Join domonoky [0] (n=Domonoky@rockbox/developer/domonoky) |
| 10:50:32 | mt | Any coldfire owners around and willing to test a patch ? |
| 11:00 |
| 11:02:11 | | Quit daurnimator (Read error: 110 (Connection timed out)) |
| 11:07:13 | | Quit n17ikh|Server ("moving day") |
| 11:09:21 | | Quit n17ikh () |
| 11:09:59 | | Join JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
| 11:12:15 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 11:15:28 | *** | Saving seen data "./dancer.seen" |
| 11:18:14 | | Join Moomoc [0] (n=streiche@port-92-206-86-219.dynamic.qsc.de) |
| 11:19:32 | T44 | mt: h340 is coldfire, right? yeah, sure |
| 11:20:04 | T44 | just tell me what you need |
| 11:20:18 | mt | T44: Yeah, Thanks. here's the patch : http://pastebin.com/d22efe101 |
| 11:20:44 | T44 | firing up vm... |
| 11:20:48 | mt | you know how to do speed tests for codecs ? |
| 11:21:20 | T44 | no, are there instructions in the wiki? |
| 11:21:28 | T44 | wait, i think i saw it yesterday |
| 11:22:32 | mt | Just add "test_codec.c" to apps/plugins/SOURCES |
| 11:23:41 | mt | it will compile the test_codec plugin with your build, then you will open the target file with test_codec. |
| 11:24:11 | T44 | sec, have to update first |
| 11:29:58 | T44 | so... applying the patch |
| 11:30:36 | amiconn | mt: I may try some things later |
| 11:31:20 | mt | amiconn: Great, thanks. |
| 11:32:00 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
| 11:35:26 | T44 | hm... for some reason its not accepting the patch, **** Only garbage was found in the patch input. |
| 11:35:56 | mt | weird .. one minute. |
| 11:36:18 | T44 | sec, meaby the copypaste added cr/lf... |
| 11:36:33 | T44 | nope, thats not it |
| 11:36:58 | pixelma | did you use "download", that'll give you the raw text |
| 11:37:28 | T44 | nope, copypasting into the vm seemed easier, let me try |
| 11:38:33 | T44 | ah, did the trick |
| 11:39:23 | pixelma | for c+p, be sure to copy from the lower window (the above has line numbers added) and cr/lf can also be a problem on Windows |
| 11:39:52 | mt | OK good. Did you add test_codec.c to apps/plugins/SOURCES ? |
| 11:39:55 | T44 | yeah, probably thats the problem, even though i told my editor to use unix style newlines |
| 11:39:59 | T44 | yeah, compiling |
| 11:41:25 | mt | Here are some samples of atrac3_rm files in case you don't have any : http://samples.mplayerhq.hu/real/AC-atrc/ - Just one file should be enough. |
| 11:43:09 | T44 | transferring to device |
| 11:43:41 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
| 11:44:11 | T44 | booting |
| 11:45:49 | pixelma | T44: you don't have to let us know every detail, especially on seperate lines... remember that the channel is logged. Thanks |
| 11:45:58 | T44 | sorry |
| 11:46:08 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 11:47:16 | | Join fyrestorm [0] (n=nnscript@cpe-68-173-235-106.nyc.res.rr.com) |
| 11:47:38 | T44 | hm, just realized make produced an error... sec |
| 11:49:50 | T44 | i am getting a region PLUGIN_IRAM is full error, when compiling atrac3_rm.codec i believe |
| 11:50:15 | T44 | can you compile it? (device 11, iriver h3x0) might be on my side |
| 11:51:05 | * | pixelma will try for M5 (same CPU) |
| 11:52:53 | mt | T44: No the problem is with my patch, putting more buffers in iram than there's actually available, as it seems. Thanks for testing. :) |
| 11:53:01 | T44 | :) |
| 11:53:18 | T44 | tell me when it compiles, i'll test it again |
| 11:56:26 | | Quit bubsy ("I'll be back somewhere in time...") |
| 12:00 |
| 12:03:34 | | Quit evilnick_home (Read error: 113 (No route to host)) |
| 12:10:24 | | Join dfkt [0] (i=dfkt@unaffiliated/dfkt) |
| 12:23:26 | mt | amiconn: Why would this be wrong : http://pastebin.com/d3ce7b79e ? ( This is supposed to do 32bit x 32bit -> return result >> 16 bit ) |
| 12:44:14 | | Join bubsy [0] (i=Bubsy@94.139.72.137) |
| 12:46:01 | | Quit lyngaas (Read error: 110 (Connection timed out)) |
| 12:48:51 | mt | pixelma, T44: (Whoever is free) another patch that needs coldfire testing - http://pastebin.com/d2ca5cc9f :) |
| 12:50:20 | n1s | is it important for us to make uses of gcc-specific features conditional on __GNUC__ etc |
| 12:53:14 | mt | It's not needed imho. |
| 12:58:28 | | Quit mt (Read error: 104 (Connection reset by peer)) |
| 12:59:06 | | Join mt [0] (n=MTee@rockbox/developer/mt) |
| 13:00 |
| 13:02:49 | | Quit Moomoc ("leaving") |
| 13:04:07 | amiconn | mt: I already told you what's wrong with that ~14 hours ago |
| 13:05:00 | mt | amiconn: Yes, I don't see why I should do a 15-bit left shift rather than a 16-bit one ? |
| 13:05:24 | amiconn | Because that's how emac works |
| 13:05:58 | amiconn | For this to work, emac needs to be in fractional mode. If you use it for iteger, accN contains bit 62..31 of the result, not bit 63..32 |
| 13:06:40 | * | amiconn hands mt a copy of CFPRM.pdf |
| 13:06:47 | mt | Oh, OK. Thanks. :) |
| 13:07:09 | amiconn | And you need to make sure (in a suitable place) that the emac actually is in that mode |
| 13:07:58 | mt | Could that be in the init_codec function for example? |
| 13:15:30 | *** | Saving seen data "./dancer.seen" |
| 13:18:53 | amiconn | Hmm, saratoga isn't here :\ |
| 13:19:40 | amiconn | Putting the imdct buffer on the stack was a bad idea |
| 13:19:56 | amiconn | It causes an immediate Stkov codec |
| 13:20:18 | | Join DarkDefender [0] (n=rob@78-69-30-229-no36.tbcn.telia.com) |
| 13:26:33 | | Join bertrik [0] (n=bertrik@d90-128-154-247.cust.tele2.nl) |
| 13:27:40 | | Join BdN3504 [0] (n=5ce53b7b@gateway/web/cgi-irc/labb.contactor.se/x-hbrrsyitxntsdnsn) |
| 13:30:40 | BdN3504 | Who wants to comment/commit this manual patch? http://www.rockbox.org/tracker/task/10531 |
| 13:32:36 | | Quit BdN3504 (Client Quit) |
| 13:32:45 | CIA-61 | New commit by amiconn (r22341): Revert r22339 as it causes immediate stack overflow on target, at least on coldfire. |
| 13:35:48 | amiconn | mt: Ahem, fixp_math.c has only inline functions. You know they will never be used inline this way? |
| 13:36:37 | amiconn | They should go into fixp_math.h and made 'static inline'. fixp_math.c can be deleted |
| 13:37:45 | mt | No, didn't know that. |
| 13:38:10 | amiconn | Functions can obviously not be inlined if they reside in another .c file |
| 13:38:36 | * | amiconn starts playing around with fixp_math |
| 13:38:50 | amiconn | Unfortunately I didn't take an arm target with me |
| 13:39:42 | mt | I could test on my e200 |
| 13:48:01 | | Quit bertrik (Read error: 110 (Connection timed out)) |
| 13:48:31 | | Nick parafin|away is now known as parafin (n=operator@paraf.in) |
| 13:48:53 | | Quit T44 (Read error: 104 (Connection reset by peer)) |
| 13:48:55 | | Quit parafin ("So long and thanks for all the fish") |
| 13:48:58 | | Join z35 [0] (n=z35@ool-45701ce5.dyn.optonline.net) |
| 13:48:58 | | Join parafin [0] (i=parafin@paraf.in) |
| 13:49:10 | | Join AndyI [0] (i=AndyI@212.14.205.32) |
| 13:51:24 | * | amiconn needs to wait for test_codec to finish |
| 13:51:27 | amiconn | I need base data |
| 13:52:42 | | Join Lear [0] (i=chatzill@rockbox/developer/lear) |
| 13:53:24 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 13:54:21 | | Join efyx [0] (n=efyx@lap34-1-82-224-140-171.fbx.proxad.net) |
| 13:56:09 | | Join robin0800_ [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 14:00 |
| 14:01:15 | | Quit AndyIL (Read error: 110 (Connection timed out)) |
| 14:02:57 | | Quit robin0800 (Read error: 60 (Operation timed out)) |
| 14:07:50 | | Join robin0800__ [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 14:07:52 | | Quit pandrew (Remote closed the connection) |
| 14:09:27 | | Quit robin0800_ (Read error: 60 (Operation timed out)) |
| 14:10:55 | | Join teru [0] (n=teru@KD059133112132.ppp.dion.ne.jp) |
| 14:11:00 | | Quit robin0800__ (Remote closed the connection) |
| 14:11:26 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 14:13:27 | amiconn | mt: http://rockbox.pastebin.ca/1531393 Could you test whether this works on arm? |
| 14:13:41 | amiconn | I get roughly 20% speedup on coldfire |
| 14:14:38 | kugel | just due to inlining? |
| 14:14:40 | kugel | not bad |
| 14:15:29 | amiconn | Will be even better with cpu specific optimisations |
| 14:15:46 | | Join bertrik [0] (n=bertrik@d90-128-154-247.cust.tele2.nl) |
| 14:17:59 | | Quit pixelma (".") |
| 14:18:46 | | Join billenium [0] (n=billeniu@c-98-237-8-252.hsd1.pa.comcast.net) |
| 14:18:51 | billenium | Does rockbox play ogg files? |
| 14:19:19 | | Quit z35 (simmons.freenode.net irc.freenode.net) |
| 14:19:19 | NSplit | simmons.freenode.net irc.freenode.net |
| 14:19:19 | bluebrother | yes. |
| 14:19:46 | | Join pixelma [0] (i=quassel@rockbox/staff/pixelma) |
| 14:19:56 | billenium | Hm, they seem not to play for me... maybe i have to update |
| 14:20:19 | bluebrother | which version are you running? Ogg is supported since ages |
| 14:20:30 | mt | amiconn: Building now - Could you try this for the CF : http://pastebin.com/d2ca5cc9f (it just puts the lookup tables into iram). |
| 14:20:37 | bluebrother | you should make sure the files don't contain ID3 tags. |
| 14:21:49 | billenium | Version r18669-080929 |
| 14:21:58 | billenium | How would i check? |
| 14:22:55 | bluebrother | depends on the system you're using, obviously. On linux try id3v2 |
| 14:23:05 | billenium | k |
| 14:24:52 | CIA-61 | New commit by bluebrother (r22342): Remove svn:executable on source file and set keywords. |
| 14:25:59 | kugel | billenium: that's really old, you should update nevertheless |
| 14:26:31 | amiconn | mt: The patch doesn't apply. "patch: **** Only garbage was found in the patch input." |
| 14:26:49 | amiconn | The leading @@ of all chunks seem to be missing |
| 14:27:07 | bluebrother | it should still play ogg files. Ogg not playing is not related to a version several months old |
| 14:27:24 | CIA-61 | New commit by teru (r22343): fix crossfade setting broken by r22192. |
| 14:27:46 | kugel | bluebrother: yes, hence I added "nevertheless" |
| 14:28:16 | bluebrother | :) |
| 14:28:55 | * | amiconn fixed it manually |
| 14:30:42 | amiconn | mt: Your patch gives about 2% speedup on cf |
| 14:31:15 | n1s | mt: it might be a good idea to look at using more compiler optimizations for the codecs you've added as iiuc they currently use the same optimization level as the core rockbox except for librm that uses O3 (which sometimes is slower than O2 btw) |
| 14:36:50 | Lear | amiconn: Replacing fixmul31 with an assembler macro looks useful (and easy, since there's MULT31 in Tremor)... |
| 14:37:24 | amiconn | Lear: I will play with cf macros soon. Just want to know whether the inline stuff is committable and then commit first |
| 14:37:54 | * | amiconn is playing with fat time+date in the meantime |
| 14:38:50 | mt | amiconn: about 10% speedup on arm. |
| 14:39:24 | n1s | amiconn: am i correct in thinking that both libcook ad libatrac are compiled with the same optimizations as core? |
| 14:41:07 | CIA-61 | New commit by amiconn (r22344): Make those functions actually inline. Around 20% speedup on coldfire, 10% speedup on arm. |
| 14:42:27 | amiconn | mt: Will you commit the iram change for the tables? |
| 14:42:51 | mt | Sure. |
| 14:50:02 | kugel | bluebrother: yes, hence I added "nevertheless" |
| 14:50:14 | kugel | bluebrother: oops sorry |
| 14:51:07 | | Quit cg_ (Remote closed the connection) |
| 14:51:20 | CIA-61 | New commit by mt (r22345): Put the lookup tables into iram ; 2% speedup on ColdFire 1% on ARM |
| 14:51:27 | NHeal | (timeout) simmons.freenode.net irc.freenode.net |
| 14:52:50 | | Quit billenium (Read error: 110 (Connection timed out)) |
| 15:00 |
| 15:00:54 | | Join cg_ [0] (n=cromos@cable-kmi-fe71de00-186.dhcp.inet.fi) |
| 15:03:20 | mt | using the am multiply functions on arm gives a 32% speedup ! |
| 15:05:16 | mt | s/am/asm/ |
| 15:15:27 | | Join qurvel [0] (n=qurvel@i125-203-133-246.s02.a013.ap.plala.or.jp) |
| 15:15:34 | *** | Saving seen data "./dancer.seen" |
| 15:18:03 | | Join petur [0] (n=peter@94-225-44-197.access.telenet.be) |
| 15:18:23 | mt | and a 2.5% speedup by putting ATRAC3Context in iram - This should be more for CF I think, amiconn ? |
| 15:20:01 | | Part qurvel |
| 15:21:16 | mt | How much IRAM is available on the CF targets ? |
| 15:26:35 | amiconn | For codecs you can use 48KB IRAM on MCF5249 (iriver H1x0, H300, iAudio M3), and 64KB IRAM on MCF5250 (iAudio M5, X5) |
| 15:30:33 | | Join BdN3504 [0] (n=5ce53b7b@gateway/web/cgi-irc/labb.contactor.se/x-mtvsfrkgquarcegm) |
| 15:31:02 | * | n1s wonders why wma data aborts on the beast with all the samples he's tried |
| 15:32:11 | BdN3504 | can you please have a look at http://www.rockbox.org/tracker/task/10531 ? i value your opinion. |
| 15:34:50 | bluebrother | BdN3504: why are you adding hard newlines in the LaTeX code? One should not do that unless absolutely necessary. |
| 15:34:52 | mt | n1s: The beast is an arm core right ? |
| 15:35:25 | amiconn | Yes, ARM1136EJ-S (ARMv6) |
| 15:35:25 | BdN3504 | because the notes automatically stick to the text if you don't |
| 15:35:27 | bluebrother | also, I find the description for the png viewer confusing. It doesn't open files "in the file browser" |
| 15:36:07 | BdN3504 | bluebrother: it doesn't? i didn't know that. sorry. |
| 15:36:14 | bluebrother | you still shouldn't to this. If you need a new paragraph put a new paragraph. |
| 15:36:36 | n1s | i'll try to work out when it broke later gtg |
| 15:36:49 | bluebrother | well, you can't open anything *in* the file browser. You open it *from* the file browser ;-) |
| 15:37:37 | mt | n1s: I guess I know why it broke. |
| 15:37:44 | bluebrother | btw, \note{} not including a \par is intended. I can't see any reason why a note should be a new paragraph in every case. |
| 15:39:09 | | Join Thundercloud [0] (n=thunderc@84-51-130-71.judith186.adsl.metronet.co.uk) |
| 15:41:00 | BdN3504 | bluebrother: i looked at the other parts of the manual and they looked different, than what i made. in section 1.3 for example is a note and that is clearly on a new paragraph, so i went that same route. |
| 15:41:29 | bluebrother | how is the new paragraph done there? You put in a newline, which is *not* a new paragraph. |
| 15:42:16 | bluebrother | \\ forces a newline. If you want to force a new paragraph use \par. Besides, starting a new paragraph is usually done by two consecutive newlines (i.e. leaving one line empty). |
| 15:42:47 | BdN3504 | so using \par would be ok? |
| 15:43:42 | bluebrother | well, it would be at least better. You really should not mess with LaTeX' layout as that is done by the document class |
| 15:47:09 | | Join IuDeX [0] (n=52a0f8f7@gateway/web/cgi-irc/labb.contactor.se/x-yfdvbhaukwujlhwo) |
| 15:47:15 | | Quit IuDeX (Client Quit) |
| 15:47:32 | | Join geo|ppc [0] (n=geo@h155.250.213.151.static.ip.windstream.net) |
| 15:47:39 | bluebrother | BdN3504: and an ellipsis is \dots, not ... (it really looks different) |
| 15:48:52 | BdN3504 | ok. |
| 15:50:01 | bluebrother | hmm, something is strange here. \note *does* add a \par unless in inner mode. |
| 15:50:09 | bluebrother | which is not the case here. |
| 15:50:28 | BdN3504 | but actually that's not an ellipsis, the setting in the context menu read open with... |
| 15:50:48 | bluebrother | it does, but in typography it's an ellipsis. |
| 15:51:04 | BdN3504 | aha ok |
| 15:51:13 | bluebrother | an ellipsis is three dots, slightly far spaced than three consecutive dots. |
| 15:51:44 | bluebrother | as I said, it looks different (but you might not notice it if you're not aware of it :) |
| 15:52:00 | n1s | mt: good, will you fix or should i try to gigure it out? |
| 15:52:30 | mt | n1s: I'm on it now .. could you just sen me links to the samples you're using ? |
| 15:52:37 | mt | s/sen/send |
| 15:53:26 | bluebrother | BdN3504: http://en.wikipedia.org/wiki/Ellipsis Better explanation that I would be able to give ;) |
| 15:54:10 | n1s | mt: i took some random samples from mplayerhq and from supposedly fixed taks in our tracker |
| 15:54:13 | | Join qurvel [0] (n=qurvel@i125-203-133-246.s02.a013.ap.plala.or.jp) |
| 15:54:31 | | Part qurvel |
| 15:57:18 | BdN3504 | all but vbrfix look nice with /par |
| 16:00 |
| 16:00:14 | BdN3504 | two newlines don't work, /par doesn't work, only // makes the note in vbrfix look consistent to the other notes in that section. |
| 16:00:45 | bluebrother | if you put in slashes instead of backslashes it won't work at all ... |
| 16:01:41 | | Join Lss [0] (i=Lss@cm26.delta97.maxonline.com.sg) |
| 16:02:14 | bluebrother | for the jpegviewer, the note does not make sense with the note directly following it. |
| 16:02:26 | BdN3504 | yea, you know what i mean stuüid typo in irc |
| 16:03:17 | bluebrother | well, I've seen people writing links using backslashes and wondering why the server has problems, so I don't know what you mean. At least not if you're aware of the difference. |
| 16:04:33 | bluebrother | ok, seems I know now :) |
| 16:04:35 | BdN3504 | i'm using a tex editor and it marks \ blue. |
| 16:06:36 | BdN3504 | ok, /par doesn't create a new line on any instance. that's not how i wanted it to be, but if you say the guidelines are not to use \\, so be it. |
| 16:06:44 | BdN3504 | i mean \par |
| 16:06:45 | BdN3504 | lol |
| 16:07:13 | | Quit Hadaka (simmons.freenode.net irc.freenode.net) |
| 16:07:13 | NSplit | simmons.freenode.net irc.freenode.net |
| 16:07:13 | | Quit AB3JU (simmons.freenode.net irc.freenode.net) |
| 16:07:28 | * | bluebrother fails to recognize the colors in BdN3504's editor |
| 16:07:59 | NHeal | simmons.freenode.net irc.freenode.net |
| 16:07:59 | NJoin | Hadaka [0] (n=naked@naked.iki.fi) |
| 16:07:59 | NJoin | AB3JU [0] (n=dz@alt.dissonance.nl) |
| 16:08:24 | bluebrother | something seems to be broken in the plugins sections. Someone needs to figure what ... |
| 16:08:27 | bluebrother | anyway, bbl. |
| 16:09:07 | | Quit kachna (Read error: 113 (No route to host)) |
| 16:09:11 | | Quit geo|ppc ("Pocket IRC 1.2.5 (http://pocketirc.com/) UNREGISTERED") |
| 16:09:42 | | Quit mt (Remote closed the connection) |
| 16:10:29 | BdN3504 | i don't understand this. now all the other notes don't look right, only the one on vbrfix... i thought latex is not like word, where so much inexpliccable stuff happens :( |
| 16:11:08 | | Join mt [0] (n=MTee@rockbox/developer/mt) |
| 16:12:43 | | Join shaggy-h [0] (n=kiwi@host-87-74-127-193.dslgb.com) |
| 16:13:28 | | Join rkjnsn [0] (n=4c69cb73@gateway/web/cgi-irc/labb.contactor.se/x-hgjjgrroiwbtnxgr) |
| 16:13:56 | BdN3504 | maybe it has something to do with my compiler. |
| 16:17:35 | rkjnsn | Do you guys have time for a quick user question? |
| 16:18:03 | pixelma | try it ;) |
| 16:18:42 | n1s | BdN3504: are you not using the regular manual build system? |
| 16:19:38 | rkjnsn | Okay. I am trying to get album art to work using the "everything in 'albumart' under '.rockbox'" approach, but I am having trouble with artists which non-ascii characters. |
| 16:21:33 | pixelma | non-ascii but valid in FAT, like cyrillic or ä or so? |
| 16:22:43 | BdN3504 | i am using miktex to build the manual |
| 16:22:50 | rkjnsn | Hmm... not sure on that. I think my OS transferred it as UTF-8 encoded (which may be the problem). |
| 16:23:03 | pixelma | rkjnsn: or do you mean things like questionmarks? |
| 16:23:24 | rkjnsn | No, I did all the conversion to _ for those. |
| 16:24:38 | BdN3504 | nls: the vmware image debian distro gives me tons of errors although i have utf 8 support installed and builds a manual which does not have working links. miktex' output is fine though |
| 16:24:46 | n1s | BdN3504: if that gives different results it sounds like a bad solution, any reason you don't do it the regular way? |
| 16:24:50 | pixelma | rkjnsn: hmm... sounds like a codepage problem then. How were your files tagged and does the filename show correctly in the browser (and the tag in the WPS or database? |
| 16:25:10 | rkjnsn | I mean things like "Claude-Michel Schönberg" and "æ¤æ¾ä¼¸å¤«" |
| 16:25:35 | | Join evilnick_home [0] (n=evilnick@pool-173-52-149-141.nycmny.east.verizon.net) |
| 16:25:38 | rkjnsn | The files are tagged in UTF-8, which shows up correctly in the WPS/Database |
| 16:25:56 | markun | rkjnsn: sounds like a problem with your mount options |
| 16:26:05 | markun | VFAT is unicode |
| 16:27:20 | rkjnsn | "rw,nosuid,nodev,noatime,uhelper=hal,shortname=lower,flush,uid=1000" |
| 16:27:38 | BdN3504 | i updated the patch |
| 16:27:52 | rkjnsn | It's the automount thingy with Gnome/HAL |
| 16:30:05 | Torne | markun: he said they were tagged in utf-8 |
| 16:30:14 | Torne | markun: that's fine (and preferable) |
| 16:30:22 | markun | Torne: but the tags are not the problem, right? |
| 16:30:25 | Torne | indeed |
| 16:30:28 | pixelma | how do the filename look in Rockbox's filebrowser (you might need to set "show files" to "all" to look inside the .rockbox folder? |
| 16:30:30 | Torne | and vfat is *always* unicode |
| 16:30:31 | markun | the filenames |
| 16:30:49 | Torne | at least in theory |
| 16:31:17 | * | pixelma needs some more ) |
| 16:31:35 | rkjnsn | pixelma: The filenames are incorrect. I looks like what happens when you view UTF-8 as Latin-1 |
| 16:32:01 | markun | rkjnsn: then it sounds like the default codepage for mounting is latin-1 |
| 16:32:27 | rkjnsn | markun: In Rockbox or on my PC? |
| 16:33:26 | | Quit efyx (Remote closed the connection) |
| 16:33:32 | Torne | you need 'utf8' in the mount options, slightly counterintuitively |
| 16:33:46 | Torne | well, maybe. |
| 16:33:55 | Torne | vfat is always unicode on disk |
| 16:34:12 | Torne | but the fs needs to know what to translate that to for the purposes of the unix API |
| 16:34:12 | markun | rkjnsn: PC |
| 16:34:26 | Torne | by default it may try latin1 or similar. |
| 16:34:31 | Torne | linux generally wants utf8 |
| 16:34:35 | Torne | (these days) |
| 16:35:43 | Torne | (on disk it's utf16 which is not suitable for POSIX APIs) |
| 16:37:08 | markun | but I don't know how to change these settings for HAL |
| 16:37:20 | Torne | indeed :) |
| 16:37:25 | Torne | Works For Me, also |
| 16:37:36 | | Quit BdN3504 ("CGI:IRC") |
| 16:37:43 | Torne | my player gets automounted by hal and utf8 translation is already on |
| 16:38:12 | Torne | linux works in utf8, vfat driver writes to disk as utf16, all is happy. |
| 16:39:35 | rkjnsn | Okay. I guess I'll have to look into getting HAL to use the utf8 option. |
| 16:39:36 | rasher | rkjnsn: Open gconf-editor and go to system > storage > default_options > vfat |
| 16:41:12 | rkjnsn | rasher: Thank you, that did the trick. |
| 16:41:34 | rkjnsn | Thank you all. Everything seems to be working now. |
| 16:47:01 | | Join mt_ [0] (n=MTee@41.233.150.77) |
| 16:47:13 | | Quit robin0800 (Remote closed the connection) |
| 16:48:49 | mt_ | n1s: The problem is with my commit which created one set of bitstream.[ch] files for libwma/cook/atrac in codeclib. Reverting this change for libwma fixes the data aborts, do you think I should just revert it for now for libwma ? |
| 16:54:22 | | Quit mt (Read error: 113 (No route to host)) |
| 16:54:37 | | Join robin0800 [0] (n=robin080@general-ld-216.t-mobile.co.uk) |
| 16:55:54 | kkurbjun | How do we go about adding a new theme section for a new player? |
| 16:57:18 | CIA-61 | New commit by teru (r22346): pluginlib_bmp: fix bug that bitmap isn't saved correctly if width is not a multiple of 4. optimize a bit. |
| 16:58:57 | n1s | mt_: i don't know, maybe a revert is best unless it gets fixed pretty quickly |
| 17:00 |
| 17:09:37 | | Join CaptainKwel [0] (n=jason@207.237.172.77) |
| 17:15:35 | *** | Saving seen data "./dancer.seen" |
| 17:16:08 | | Join Omlet [0] (n=Omlet05@99.225-200-80.adsl-dyn.isp.belgacom.be) |
| 17:16:17 | | Quit Thundercloud (Remote closed the connection) |
| 17:20:59 | | Quit Omlet (Client Quit) |
| 17:29:10 | | Quit teru ("Quit") |
| 17:30:32 | | Quit faemir ("Leaving") |
| 17:30:54 | | Join keef [0] (n=5b547ee0@gateway/web/cgi-irc/labb.contactor.se/x-cwsgrvassxbexftr) |
| 17:33:09 | keef | Hello. I have installed Rockbox (release 3.3) on my Iriver H10 6GB MTP, together with the H10 MTP Bootloader (from http://download.rockbox.org/manual/rockbox-h10_5gb/rockbox-buildch2.html#x4-60002). The installation itself seems to be successful - on startup Rockbox loads and reaches the main menu. But then no buttons or keys have any effect, including the on-off button - so I have to remove the battery to shut |
| 17:35:29 | | Join raphi [0] (n=raphi@pub082136118205.dh-hfc.datazug.ch) |
| 17:36:35 | CIA-61 | New commit by mt (r22347): revert r22337 for libwma for now since it broke wma playback on ARM due to alignment issues. |
| 17:45:13 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 17:46:32 | | Join GeekShado_ [0] (n=Antoine@86.69.8.142) |
| 17:51:48 | | Quit raphi ("leaving...") |
| 17:57:36 | | Join FOAD_ [0] (n=dok@dinah.blub.net) |
| 17:58:56 | | Quit FOAD (Read error: 60 (Operation timed out)) |
| 17:58:57 | | Nick FOAD_ is now known as FOAD (n=dok@dinah.blub.net) |
| 18:00 |
| 18:02:43 | | Quit mt_ (Read error: 110 (Connection timed out)) |
| 18:03:50 | | Quit GeekShadow (Read error: 110 (Connection timed out)) |
| 18:08:44 | | Join efyx [0] (n=efyx@lap34-1-82-224-140-171.fbx.proxad.net) |
| 18:19:23 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
| 18:19:47 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
| 18:20:10 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
| 18:20:31 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
| 18:20:55 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
| 18:21:17 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
| 18:21:41 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
| 18:22:02 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
| 18:22:28 | | Quit jfc (Read error: 104 (Connection reset by peer)) |
| 18:22:49 | | Join jfc [0] (n=john@dpc6682208002.direcpc.com) |
| 18:27:41 | | Join domonoky1 [0] (n=Domonoky@g229187005.adsl.alicedsl.de) |
| 18:27:41 | | Quit domonoky (Read error: 104 (Connection reset by peer)) |
| 18:29:43 | | Quit efyx (Remote closed the connection) |
| 18:42:50 | | Quit Lear ("ChatZilla 0.9.85 [Firefox 3.5.2/20090729225027]") |
| 18:48:27 | | Join mcuelenaere [0] (n=mcuelena@78-21-191-122.access.telenet.be) |
| 18:49:21 | mcuelenaere | kkurbjun: http://themes.rockbox.org/admin.php (if you have an account) |
| 18:50:25 | mcuelenaere | only thing is, themes having the same resolution will show up for different players (which isn't good for touchscreen- vs non-touchscreen-compatible themes) |
| 18:50:58 | kkurbjun | mcuelenaere: thanks for the link - I don't think I have an account - who would I talk to to get one? |
| 18:51:16 | mcuelenaere | anyone with SSH access to that server I guess (which I have) |
| 18:51:19 | kkurbjun | that won't be a problem for the M:Robe right now since it is the only player that runs at 640x480 |
| 18:51:40 | mcuelenaere | hmm and it isn't a problem for the Onda either (400x240), but that doesn't fix the problem :) |
| 18:51:46 | mcuelenaere | eh 240x400 |
| 18:51:48 | kkurbjun | :), true |
| 18:52:19 | kkurbjun | actually, one other thing - I modified the onda and D2 touchscreen drivers to hopefully act like the m:robe |
| 18:52:38 | kkurbjun | I think reversi and calculator should be working on those targets now |
| 18:52:45 | mcuelenaere | ah |
| 18:52:54 | mcuelenaere | I saw the commit, but hadn't look into what it actually did :) |
| 18:53:19 | kkurbjun | on calculator I just enabled the work that you had done - minesweeper did not seem to work for me |
| 18:54:02 | | Quit Tuplanolla (Read error: 110 (Connection timed out)) |
| 18:54:27 | kkurbjun | how much work is it to add a target for themes.rockbox.org? If it is just a few settings that need to be enabled it might be easier to do that than for me to get an account |
| 18:54:57 | mcuelenaere | it seems you need a picture of the player |
| 18:55:10 | mcuelenaere | but I can't seem to find my logs, better to ask rasher for an account |
| 18:55:24 | kkurbjun | ok, I'll do that |
| 19:00 |
| 19:02:52 | | Quit AB3JU (simmons.freenode.net irc.freenode.net) |
| 19:02:52 | NSplit | simmons.freenode.net irc.freenode.net |
| 19:02:52 | | Quit Hadaka (simmons.freenode.net irc.freenode.net) |
| 19:02:55 | mcuelenaere | kkurbjun: regarding that commit, I'm wondering whether it shouldn't be done in firmware/drivers/touchscreen.c instead |
| 19:03:45 | NHeal | simmons.freenode.net irc.freenode.net |
| 19:03:45 | NJoin | Hadaka [0] (n=naked@naked.iki.fi) |
| 19:03:45 | NJoin | AB3JU [0] (n=dz@alt.dissonance.nl) |
| 19:05:37 | | Join mt [0] (n=MTee@rockbox/developer/mt) |
| 19:09:57 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 19:11:38 | | Quit robin0800 (Remote closed the connection) |
| 19:15:39 | *** | Saving seen data "./dancer.seen" |
| 19:17:56 | | Quit cg_ (Read error: 110 (Connection timed out)) |
| 19:18:49 | CIA-61 | New commit by amiconn (r22348): FAT timestamp handling improvements for non-RTC targets: ... |
| 19:19:46 | | Quit AB3JU (simmons.freenode.net irc.freenode.net) |
| 19:19:46 | NSplit | simmons.freenode.net irc.freenode.net |
| 19:19:46 | | Quit Hadaka (simmons.freenode.net irc.freenode.net) |
| 19:20:48 | NHeal | simmons.freenode.net irc.freenode.net |
| 19:20:48 | NJoin | Hadaka [0] (n=naked@naked.iki.fi) |
| 19:20:48 | NJoin | AB3JU [0] (n=dz@alt.dissonance.nl) |
| 19:21:11 | | Quit AB3JU (simmons.freenode.net irc.freenode.net) |
| 19:21:11 | | Quit Hadaka (simmons.freenode.net irc.freenode.net) |
| 19:21:14 | NJoin | Hadaka [0] (n=naked@naked.iki.fi) |
| 19:21:15 | NJoin | AB3JU [0] (n=dz@alt.dissonance.nl) |
| 19:21:15 | | Quit keef ("CGI:IRC (EOF)") |
| 19:21:38 | JdGordon | anyone got a ipod video handy? does the icatcher theme work correctly in svn? |
| 19:26:21 | | Join cg_ [0] (n=cromos@cable-kmi-fe71de00-186.dhcp.inet.fi) |
| 19:27:29 | JdGordon | damn... it does |
| 19:30:49 | JdGordon | ah FUCK! it works in the sim fine with my patch, but not on target |
| 19:31:40 | JdGordon | can I get some testers for 8996 please? I really want to get it in today |
| 19:35:19 | JdGordon | sweet :) using a vairable before initialising it causes wierd shit to happen on target :) |
| 19:35:33 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 19:35:38 | CIA-61 | New commit by amiconn (r22349): Save a bit of binsize by using a more suitable datatype and making the array static. |
| 19:36:47 | | Join mitk [0] (n=mitk@chello089077247055.chello.pl) |
| 19:37:59 | | Quit kugel (Remote closed the connection) |
| 19:38:06 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 19:39:51 | stripwax | mt - is fixmul16 in atrac really the same as MULT32 in codeclib? MULT31 and MULT32 differ by one bit shift but fixmul16 and fixmul31 differ by 15bit shift, so not sure I understand how they can be used as drop-in replacements |
| 19:41:34 | mt | No it isn't this was wrong, but the asm stuff were reverted some revisions ago ? |
| 19:42:44 | stripwax | oh, so it was - sorry, I'd missed the revert. |
| 19:43:21 | mt | np - could you perhaps, ahem, profile atrac ? :) |
| 19:43:36 | kugel | ah, which reminds me |
| 19:44:04 | stripwax | mt - not easily (not for a couple weeks I'm afraid) |
| 19:44:10 | kugel | I tried to play a bit with profiling recently, but it didn't work at all |
| 19:44:20 | kugel | plugins just freezed |
| 19:44:32 | mt | stripwax: Sure, whenever you feel like it. |
| 19:44:34 | stripwax | kugel - which plugins? which functions did you want to instrument? |
| 19:45:04 | stripwax | kugel - did you at least get a profile.out when you were done? |
| 19:46:17 | kugel | no, I started a plugin and rockbox freezed |
| 19:46:49 | kugel | I used rb->profile_thread() and rb->profstop(), and added the cflag to plugins.make |
| 19:47:15 | stripwax | kugel - which plugin |
| 19:47:18 | mt | stripwax: here are the correct arm multiplies http://pastebin.com/d23846cc7 |
| 19:47:21 | stripwax | oh, all plugins? |
| 19:47:33 | kugel | all, although I just inserted the stuff into bubbles |
| 19:47:45 | amiconn | mt: ~20% speedup on coldfire due to ATRAC3Context q IBSS_ATTR; |
| 19:47:52 | amiconn | (in atrac3_rm.c) |
| 19:48:11 | mt | That's nice ! only 2.5% for arm. |
| 19:48:24 | amiconn | You mean PP502x |
| 19:48:38 | mt | Yes. |
| 19:48:43 | amiconn | On PP5002 it will also be quite a bit more (but I can't test that now) |
| 19:49:20 | stripwax | kugel - you should only put the cflags into the plugin you want to profile, ideally. but presumably the plugin that froze rockbox was the bubbles plugin, is that right? not some unrelated one |
| 19:49:39 | kugel | every plugin froze |
| 19:49:46 | kugel | including bubbles |
| 19:50:00 | kugel | there's only 1 make file for all plugins |
| 19:50:16 | kugel | (except for the subdir ones I think) |
| 19:50:46 | mt | amiconn: Are you going to commit it, or should I do it ? |
| 19:51:23 | stripwax | oh, right. and then you had to reboot rockbox, no way to quit or anything? it shouldn't do that [obviously] but I haven't seen it do that before so I don't know why it didn't work for you |
| 19:51:42 | stripwax | I've only tried profiling codecs in this way, not all plugins |
| 19:51:45 | kugel | right |
| 19:51:55 | stripwax | which device? |
| 19:52:08 | kugel | e200 |
| 19:52:27 | kugel | same happened when profiling the core btw, although I don't know if that was ever possible |
| 19:52:43 | stripwax | kugel - does profiling codec work for you? |
| 19:53:02 | kugel | I didn't try |
| 19:53:09 | stripwax | btw I don't see why codec would work and other plugins not; but I know codec profiling works [for me] |
| 19:53:23 | kugel | without modifications? |
| 19:54:42 | stripwax | you still need to put $PROFILE_OPTS in cflags and ci->profile_thread in codec_main |
| 19:55:13 | kugel | I mean other modifications, like fixing profiling :) |
| 19:55:35 | stripwax | I'm pretty sure it works without any other modifications, at least on iriver h120/140 and ipod 5g .... but I can't test right now. I don't think I needed to make any changes to profiling code to fix anything |
| 19:55:50 | kugel | I'm wondering if I did a complete rebuild, enabling profiling breaks the api |
| 19:55:58 | stripwax | that's likely |
| 19:56:23 | kugel | codecs worked though |
| 19:56:26 | stripwax | I would recommend a complete rebuild - since you need to reconfigure with Profile anyway. I used a separate build dir for that. |
| 19:56:37 | | Join Creposucre [0] (n=53c3f041@gateway/web/cgi-irc/labb.contactor.se/x-gfuvkpkncbeadtpn) |
| 19:57:01 | Creposucre | Hi |
| 19:57:09 | kugel | I mean, I didn't try profiling a codec, but the codec api breaks as well, and since they worked I think I did a complete rebuild |
| 19:57:53 | Creposucre | JdGordon: still need a tester for video? |
| 19:58:49 | JdGordon | tester on any target would be good |
| 19:59:12 | JdGordon | I tihnk its all good now though |
| 20:00 |
| 20:00:07 | | Part mitk ("Leaving") |
| 20:00:14 | kugel | JdGordon: are you sure you really need testers for 8996? |
| 20:00:26 | Creposucre | JdGordon: which task? FS #8996 is "Usb charge-only mode is not documented" |
| 20:00:28 | stripwax | kugel - I would still recommend a totally separate build directory if you want to be able to switch between regular and profile builds |
| 20:00:35 | JdGordon | 9886 maybe? |
| 20:00:48 | | Join mitk [0] (n=mitk@chello089077247055.chello.pl) |
| 20:00:52 | Creposucre | ok |
| 20:00:53 | JdGordon | :p yeah 9886 |
| 20:01:34 | bluebrother | any thoughts on the wording about the icons in the about dialog? http://www.alice-dsl.net/dominik.riebeling/rockbox/rbutil-aboutbox.jpg |
| 20:03:44 | JdGordon | seems ok? |
| 20:05:41 | stripwax | mt - on arm, you should be able to combine the orr and the shift into one instruction. fixmul31 would be the same as MULT31. but I thought it doesn't affect the condition code so "cc" should not be in the clobbers list. |
| 20:06:52 | | Join Tuplanolla [0] (n=jani@unaffiliated/tuplanolla) |
| 20:07:48 | JdGordon | disk access should cause the calling thread to yield yeah? |
| 20:09:07 | | Join robin0800 [0] (n=robin080@cpc3-brig8-0-0-cust436.brig.cable.ntl.com) |
| 20:13:36 | Creposucre | JdGordon: patch hunks with latest build |
| 20:14:10 | JdGordon | dw, that patch is old.. commiting anyway |
| 20:15:31 | Creposucre | JdGordon: anything that I can do? |
| 20:15:59 | JdGordon | get the new build on your ipod once its built and let me know if anything breaks :) |
| 20:16:21 | Creposucre | lol |
| 20:16:22 | Creposucre | ok |
| 20:16:42 | JdGordon | is apps/gui/skin_engine/skin_engine.h in svn? |
| 20:17:37 | Creposucre | the .h is not |
| 20:17:52 | JdGordon | its not? |
| 20:18:56 | Creposucre | skin_buffer.c, skin_buffer.h, skin_engine.c and skin_tokens.h only are in skin_engine |
| 20:19:40 | * | kugel slaps JdGordon again for not doing the renaming in a seperate commit |
| 20:19:59 | JdGordon | I didnt know.... *cries* I didnt know... |
| 20:23:04 | JdGordon | BOOM! |
| 20:23:04 | CIA-61 | New commit by jdgordon (r22350): more wps->skin engine work.. ... |
| 20:23:55 | | Part Voluntaryist |
| 20:26:04 | Creposucre | jdgordon: is there something special to do to test the functionalities of this patch? |
| 20:26:15 | JdGordon | bah, forgot to mention that its advisable to load themes from .cfg files and NOT .wps files |
| 20:26:33 | JdGordon | Creposucre: there is no new functionality... you shouldnt notice any badness |
| 20:26:39 | JdGordon | or anything different at all hopefully |
| 20:26:49 | kugel | "Unlimited "string" tokens now (limit was 1024 which was rediculously wasteful) " really sounds funny |
| 20:27:27 | JdGordon | well.. hmm.. yeah, considering hardly any themes would use more than 20.. the 1000 was silly |
| 20:28:11 | JdGordon | damn fine delta if you ask me :D |
| 20:28:19 | | Quit stripwax ("http://miranda-im.org") |
| 20:28:34 | JdGordon | and thats not cheating by using buffer_alloc() for the buffer :) |
| 20:28:48 | kugel | that'll give a decent delta won't it? |
| 20:29:07 | JdGordon | changing to tha will be a HUGE fake delta |
| 20:29:11 | kugel | 1bit remotes with tiny displays basically get the same buffer as a big color screen, right? |
| 20:29:31 | JdGordon | (((LCD_HEIGHT*LCD_WIDTH*LCD_DEPTH/8) + (2*LCD_HEIGHT*LCD_WIDTH/8)) * NB_SCREENS) |
| 20:30:48 | kugel | heh, people with remotes suffer a bit :) |
| 20:30:52 | * | amiconn slaps JdGordon around with 82 points |
| 20:31:03 | JdGordon | yeah yeah, I'm fixing it |
| 20:31:13 | JdGordon | but come one.. the delta more than makes up for that :) |
| 20:31:36 | JdGordon | -e |
| 20:32:13 | kugel | JdGordon: I think it would've been wiser to not just multply with NB_SCREENS, but rather something like "+((LCD_REMOTE_HEIGHT*LCD_REMOTE_WIDTH*LCD_REMOTE_DEPTH/8) + (2*LCD_REMOTE_HEIGHT*LCD_REMOTE_WIDTH/8)) |
| 20:32:27 | CIA-61 | New commit by jdgordon (r22351): fix player |
| 20:32:51 | JdGordon | are those defined to 0 for non remote targets? |
| 20:32:52 | | Join lyngaas [0] (n=staale@19.81-167-149.customer.lyse.net) |
| 20:33:03 | Creposucre | JdGordon: looks correct |
| 20:34:27 | Unhelpful | JdGordon: not really fake when you consider how little of the static buffer size most real themes need. removing artificial limits *and* saving memory for the common case is a win all around. :P |
| 20:34:34 | kugel | JdGordon: no, but you can easiy do that as "#define REMOTE_BUFFER [...]" within #ifdef HAVE_REMOTE_LCD, then buffer_size = IMG_BUFSIZE + REMOTE_BUFFER |
| 20:34:59 | | Quit rkjnsn ("CGI:IRC (EOF)") |
| 20:35:24 | JdGordon | Unhelpful: yeah :) |
| 20:36:32 | Creposucre | JdGordon: i don't see anything different after test, i guess it's a sucess :) |
| 20:36:41 | | Join DarkFlash [0] (n=DarkFlas@host232-113-dynamic.24-79-r.retail.telecomitalia.it) |
| 20:36:44 | JdGordon | :) |
| 20:37:26 | * | kugel still thinks the buffer on targets with remote is unreasonably higher than on those without remote |
| 20:38:08 | Creposucre | JdGordon: by the way, if you want to have some rest, you can test 10494, i added a function for your dock |
| 20:38:18 | Creposucre | hopefully it'll work |
| 20:38:35 | pixelma | JdGordon: unfortunately the delta is quite some plus on Archos |
| 20:38:53 | pixelma | (about the "make up for it" bit |
| 20:38:57 | pixelma | ) |
| 20:39:02 | | Join stripwax [0] (n=fircuser@genkt-048-077.t-mobile.co.uk) |
| 20:39:18 | JdGordon | blame its remote :) |
| 20:39:28 | pixelma | huh? |
| 20:39:50 | JdGordon | the talk above about the buffer size... maybe kugel wants to fix it and take credit :) |
| 20:40:21 | pixelma | Ondios and Player (?) don't have a remote and the Recorder one is screenless |
| 20:41:32 | DarkFlash | Hi everybody. I want tu buy an mp3 player where a can install rockbox. I want an player with about 40 GB of memory... Any advice? |
| 20:41:33 | JdGordon | oh right |
| 20:42:12 | JdGordon | meh |
| 20:43:04 | JdGordon | Creposucre: ok, I'll try to test it today |
| 20:43:12 | bertrik | I think skin_buffer.c should also #include skin_buffer.h |
| 20:43:59 | JdGordon | why? it doesnt actually need to |
| 20:44:47 | | Join efyx [0] (n=efyx@lap34-1-82-224-140-171.fbx.proxad.net) |
| 20:44:51 | bertrik | it's ensures that the implementation is consistent with the declaration (gcc warns if it is not) |
| 20:45:49 | rasher | JdGordon: urgh |
| 20:45:59 | JdGordon | ? |
| 20:46:34 | rasher | You can no longer pick a wps independently of the theme? |
| 20:47:13 | JdGordon | no, you can |
| 20:47:36 | JdGordon | just if you keep doing it, it will stop working untill you load a .cfg |
| 20:48:01 | pixelma | what? |
| 20:49:36 | | Quit Grahack ("Leaving.") |
| 20:51:49 | rasher | That sounds pretty terrible, to be honest |
| 20:52:28 | JdGordon | umm why? how often would you manuall load a .wps? the theme site even forces the correct .cfg structrue |
| 20:52:49 | linuxstb | JdGordon: Why is loading a .wps a problem now? |
| 20:53:00 | JdGordon | its not a problem... |
| 20:53:09 | linuxstb | You just said it was... |
| 20:53:11 | rasher | It just might randomly stop working |
| 20:53:18 | rasher | I'd say that qualifies as a problem |
| 20:53:22 | JdGordon | you will just run out of room after loading image heavy .wps files mulitple times |
| 20:54:07 | rasher | So basically if you only switch wps, but not the entire theme, you're likely screwed |
| 20:54:19 | JdGordon | 3 or 4 times depending on the theme |
| 20:54:21 | kugel | is that only true if you don't reboot inbetween? |
| 20:54:44 | JdGordon | you can just load a .cfg (ANY .cfg.. it can even be blank) and it will work again |
| 20:54:48 | * | linuxstb thinks that's clearly a problem, but has a flight to catch... |
| 20:55:08 | rasher | JdGordon: That sounds spectacularly braindead |
| 20:55:17 | kugel | it sounds weird indeed |
| 20:55:29 | pixelma | JdGordon: did you perhaps higher the WPS buffer on Archos for some reason? |
| 20:55:32 | JdGordon | then you make it work correctly |
| 20:55:41 | rasher | JdGordon: I wasn't the one who broke it |
| 20:55:43 | JdGordon | pixelma: not on purpose |
| 20:55:49 | JdGordon | ITS NOT BROKEN |
| 20:56:05 | rasher | "if you keep doing it, it will stop working" |
| 20:56:06 | pixelma | it sounds like it is though |
| 20:56:07 | rasher | that's broken. |
| 20:56:14 | JdGordon | come up with a way to safely reload every .wps if just one changes (im tihnking remote targets here) and by all means it wil be done |
| 20:56:51 | | Quit linuxstb (Read error: 60 (Operation timed out)) |
| 20:57:01 | | Quit moos ("Rockbox rules the DAP world") |
| 20:57:03 | rasher | JdGordon: Again, I wasn't the one who broke it. |
| 20:57:11 | | Quit domonoky1 (Read error: 54 (Connection reset by peer)) |
| 20:57:14 | JdGordon | its not broken |
| 20:57:19 | rasher | "if you keep doing it, it will stop working" |
| 20:57:23 | rasher | Broken. |
| 20:57:37 | rasher | You should know better than to commit half-finished work. |
| 20:58:02 | pixelma | JdGordon: will you answer the reports of "my WPS suddenly stopped working, what's wrong"? |
| 20:58:16 | JdGordon | thats not going to happen |
| 20:58:20 | | Quit stripwax (Read error: 110 (Connection timed out)) |
| 20:58:35 | JdGordon | rasher: I posted on the ml.,. if you dont care enough to comment then thats your bad luck |
| 20:58:37 | pixelma | sure "the one I loaded yesterday doesn't work when I load it now" |
| 20:58:45 | JdGordon | THATS NOT GOING TO HAPPEN |
| 20:59:12 | JdGordon | that will only ever happen if you load .wps files from the file browser MUTLIPLE TIMES and they are image heavy |
| 20:59:23 | JdGordon | doing a .cfg load avoids that entirely |
| 20:59:25 | rasher | JdGordon: That's no excuse for committing broken code. Apparently you knew of this brokenness, yet just dumped it in anyway |
| 20:59:29 | pixelma | umm... some people do |
| 21:00 |
| 21:00:21 | pixelma | and those who change their WPSs multiple times usually have image heavy ones too |
| 21:00:31 | rasher | I'm pretty sure I wouldn't be using the theme feature on my H120 if I still had it |
| 21:00:41 | rasher | Just loading wpses and fonts as I saw fit |
| 21:01:13 | | Join domonoky [0] (n=Domonoky@rockbox/developer/domonoky) |
| 21:02:11 | kugel | -84k sounds a bit too much for just making remote buffer smaller |
| 21:04:36 | | Quit flydutch ("/* empty */") |
| 21:05:23 | | Join J-23_ [0] (n=zelazko@unix.net.pl) |
| 21:05:36 | kugel | JdGordon: what's the problem with just reloading every wps? why's loading an empty cfg magically fixing things? |
| 21:05:37 | JdGordon | pixelma: if yo change your wps multiple times, chances are you are changing the whole damn theme in one hit |
| 21:05:55 | JdGordon | kugel: resetting the buffer on the 2nd one blows away the first one |
| 21:06:10 | pixelma | *I* usually don't |
| 21:06:13 | JdGordon | right now this only affects remote targets, but think about radio skin or rec skin |
| 21:06:15 | | Nick J-23_ is now known as J-23 (n=zelazko@unix.net.pl) |
| 21:06:47 | | Quit CaptainKwel (Remote closed the connection) |
| 21:06:51 | pixelma | but I don't change my WPS that often |
| 21:07:22 | pixelma | it still sounds like broken by design and I'm awaiting the first bug report |
| 21:07:30 | | Quit J-23 (Remote closed the connection) |
| 21:07:46 | rasher | JdGordon: *you* might change the whole theme at once, I guarantee not everyone does. |
| 21:07:50 | | Join J-23 [0] (n=zelazko@unix.net.pl) |
| 21:08:13 | | Quit mt (Read error: 101 (Network is unreachable)) |
| 21:08:21 | JdGordon | are you suggesting that people are so stupid they cant work the rediculously simple work around? |
| 21:09:25 | kugel | I still don't understand this |
| 21:09:37 | pixelma | JdGordon: you can't be serious |
| 21:09:46 | kugel | why can't you just use the code that works for loading an empty .cfg for loading wpses to? |
| 21:10:08 | JdGordon | kugel: loading an empty .cfg will call settings_apply() which reloads everything |
| 21:10:45 | JdGordon | there is a single large buffer for all .wps files... there is no safe way without really big complication to reset that buffer and load skins again |
| 21:10:56 | JdGordon | WHICH I WAS GOING TO PROBABLY DO LATER ANYWAY |
| 21:11:28 | rasher | JdGordon: But you figured you'd just go ahead and commit broken code? |
| 21:12:02 | JdGordon | fuck... its not broken |
| 21:12:21 | rasher | Then why are there suddenly things we can't do anyomre? |
| 21:12:45 | rasher | You latest mail doesn't really make things any better. |
| 21:15:20 | CIA-61 | New commit by kugel (r22352): Recalibrate the skin buffer size calculation so that targets with remote get a reasonable extra buffer (depending on the remote) instead of just ... |
| 21:15:43 | *** | Saving seen data "./dancer.seen" |
| 21:16:19 | kugel | my rockbox-info.txt numbers were weird |
| 21:16:59 | JdGordon | kugel: yeah thats the mess I wanted to avoid... |
| 21:17:38 | kugel | mess? |
| 21:17:49 | JdGordon | yep |
| 21:19:02 | kugel | I can't spot mess |
| 21:19:45 | | Join keef [0] (n=5b547ee0@gateway/web/cgi-irc/labb.contactor.se/x-hjgevjfkxqcldnba) |
| 21:20:06 | JdGordon | did you fuck something up? those deltas look way too big |
| 21:20:30 | JdGordon | actually what was the old buffer size? |
| 21:21:03 | kugel | no idea |
| 21:21:30 | JdGordon | it must have been a worse waste on remote targets than I thoguht |
| 21:22:06 | kugel | it's definitely not wrong that the mr500 needs additional 700k just for its 64x16x1 (???) remote |
| 21:22:13 | kugel | not right* |
| 21:23:01 | | Join BdN3504 [0] (n=4e3435ff@gateway/web/cgi-irc/labb.contactor.se/x-nfuscojxwlcbvkgt) |
| 21:23:20 | pixelma | and what's with the difference between e.g. the M5 and the X5 - it's the same remote |
| 21:23:23 | JdGordon | yeah, but the delta from my commit shuld have been different |
| 21:24:04 | kugel | pixelma: it was twice the main screen before |
| 21:24:06 | pixelma | ah, colour main screen or greyscale mainscreen |
| 21:24:42 | pixelma | that's why it was so big on the M:Robe500 before - huge main screen |
| 21:25:23 | kugel | JdGordon: well, right now I can't figure out how the remote was taken into account before your commit |
| 21:25:37 | | Quit petur (Remote closed the connection) |
| 21:25:43 | JdGordon | pretty sure its 2*IMG_BUF |
| 21:25:53 | BdN3504 | i'm with you jd. i already made a couple of WPSs that i had to recode or change because of the buffer size restricitions. and as we have a strict themes upload policy requiring intact folder structures and cfg files |
| 21:25:55 | JdGordon | bucause its an array of wps_data which is statically sized |
| 21:27:25 | BdN3504 | and i dont think it's likely anybody would change the wps file 4 times in a row. in the end themes are called themes, because the creators thought about which bg will accompany the wps best |
| 21:27:33 | kugel | ah right, wps_data[NB_SCREENS] |
| 21:27:48 | JdGordon | BdN3504: thank you.. exactly |
| 21:28:43 | kugel | BdN3504: that doesn't mean the code is intact. it's based of rather subjective assumptions |
| 21:30:07 | BdN3504 | is this only happening with the wps file? what about large font files? say i load a unifont ten times in a row, will that cause something weird? |
| 21:30:20 | JdGordon | no, *only* .wps files |
| 21:30:35 | JdGordon | well.. and rwps |
| 21:30:37 | kugel | why don't you just call settings_apply() when loading a wps? |
| 21:30:37 | n1s | if we don't want people to load wsp's directly we should drop that functionality IMO |
| 21:30:54 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 21:30:56 | | Join petur [0] (n=peter@94-225-44-197.access.telenet.be) |
| 21:31:06 | JdGordon | n1s: yeah, I dont think we need to go that far |
| 21:31:13 | funman | hi, i wanted to let you know i had no news from this AMS guy today ... |
| 21:31:18 | JdGordon | :( |
| 21:31:40 | kugel | n1s: I think that feature is questionable anyway, wps are designed around a certain font size for example, mostly loading another wps doesn't work out well for that reason |
| 21:31:49 | BdN3504 | do they work on sundays? :) |
| 21:32:13 | kugel | but I wouldn't remove it :> |
| 21:32:17 | kugel | funman: no call? |
| 21:32:22 | amiconn | kugel: Not all wps are designed for fixed font sizes. Also, it's a handy feature if you're testing your wps |
| 21:32:38 | * | amiconn s standard wps doesn't care about font size |
| 21:33:01 | | Quit Creposucre ("CGI:IRC (Ping timeout)") |
| 21:33:24 | kugel | JdGordon: is this overflow at least detectable? |
| 21:33:45 | JdGordon | its not an overflow :) its protected.. |
| 21:33:48 | kugel | as in, throwing a "please reboot to enable" in case it happens |
| 21:33:56 | JdGordon | but yes it can be checked for |
| 21:34:08 | JdGordon | I plan on adding a "skin ram usage" line to the rockbox info screen |
| 21:34:11 | kugel | I'd rather see it fixed, though |
| 21:34:19 | JdGordon | its not a reboot to fix issue |
| 21:34:49 | | Join Creposucre [0] (n=53c3f041@gateway/web/cgi-irc/labb.contactor.se/x-zsrybcspjddnevef) |
| 21:34:51 | | Join toffe82 [0] (n=chatzill@adsl-75-23-148-166.dsl.frs2ca.sbcglobal.net) |
| 21:35:06 | kugel | a reboot wouldn't get around it? |
| 21:35:21 | JdGordon | it would,m but all that is needed is to run an empty .cfg file |
| 21:35:40 | kugel | I wonder why doing settings_apply() on wps loading doesn't work |
| 21:35:59 | JdGordon | you want to reapply every setting to reload one .wps file?! |
| 21:36:07 | JdGordon | slooow and wasteful and sucky |
| 21:36:29 | | Quit Creposucre (Client Quit) |
| 21:37:33 | | Quit Lss (Read error: 104 (Connection reset by peer)) |
| 21:37:37 | | Join Riku [0] (n=Lss@cm26.delta97.maxonline.com.sg) |
| 21:37:41 | kugel | then hack a small version together |
| 21:38:17 | | Quit amiconn (Nick collision from services.) |
| 21:38:19 | | Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn) |
| 21:38:39 | | Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn) |
| 21:39:57 | | Quit pixelma (Nick collision from services.) |
| 21:39:57 | | Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma) |
| 21:40:15 | | Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma) |
| 21:40:15 | | Quit BdN3504 ("CGI:IRC (EOF)") |
| 21:41:56 | kugel | JdGordon: like this http://pastebin.ca/1531752 |
| 21:42:42 | JdGordon | lunch bbs |
| 21:43:48 | funman | kugel: no call / no SMS |
| 21:44:53 | | Quit petur (Remote closed the connection) |
| 21:44:54 | | Quit keef ("CGI:IRC (EOF)") |
| 21:45:36 | | Join petur [0] (n=peter@94-225-44-197.access.telenet.be) |
| 21:46:16 | kugel | funman: :'( |
| 21:46:44 | CIA-61 | New commit by bluebrother (r22353): Add Save icon and add a note to the about dialog about the origin of the icons. Should have been there much earlier ... |
| 21:48:54 | | Quit funman ("free(random());") |
| 21:50:45 | mcuelenaere | Why is there an Archos AV300 target in config.h when the config-av300.h file doesn't exist? |
| 21:51:10 | | Join Grahack [0] (n=chri@ip-222.net-82-216-222.rev.numericable.fr) |
| 21:51:26 | kugel | mcuelenaere: historical reasons? :) |
| 21:51:36 | mcuelenaere | probably :) so it can be removed? |
| 21:52:19 | pixelma | ask linuxstb and scorche |
| 21:53:07 | mcuelenaere | linuxstb, scorche: ? |
| 21:56:23 | kugel | JdGordon: that code actually works |
| 21:57:05 | kugel | mcuelenaere: I think we never remove ports, and I really hate this situation in case of clearly dead ones |
| 21:57:31 | kugel | especially since re-adding is a matter of calling svn diff |
| 21:57:39 | mcuelenaere | kugel: it's not that I want to remove the port, but it just doesn't have a config-av300.h file and that breaks my script :) |
| 21:57:52 | mcuelenaere | (although I could work around it, but I would prefer not to) |
| 21:58:53 | kugel | I'd prefer to remove a bunch of others too in that run :> |
| 22:00 |
| 22:02:56 | | Join aaron424 [0] (n=chatzill@adsl-065-013-002-216.sip.asm.bellsouth.net) |
| 22:04:32 | rasher | The gminis were removed |
| 22:10:08 | | Quit DarkFlash (Read error: 113 (No route to host)) |
| 22:10:52 | | Quit petur (Remote closed the connection) |
| 22:12:20 | CIA-61 | New commit by kugel (r22354): Fix loading of multiple .wps/.rwps files in a row, by reloading all wpses upon loading one so that the skin buffer can be reset properly. |
| 22:12:38 | JdGordon | arg |
| 22:12:45 | JdGordon | a bit of pateince would kill you? |
| 22:12:48 | JdGordon | THATS A BAD IDEA |
| 22:13:04 | kugel | lol, I just looked before hitting the button if you answered |
| 22:13:17 | | Join petur [0] (n=peter@rockbox/developer/petur) |
| 22:13:17 | kugel | it's better than what we have now |
| 22:14:08 | JdGordon | 1) its not.. 2) its a bad idea |
| 22:14:28 | JdGordon | its a very bad band aid for a issue which will never show up |
| 22:14:42 | rasher | JdGordon: Just because you keep saying it will never show up doesn't make it true. |
| 22:15:16 | JdGordon | of course, the bad side effects of that change wont show up for most poeple for a while |
| 22:15:17 | rasher | We don't leave bugs in just because someone thinks it'll never happen. And we certainly don't knowingly commit them. Oh wait... |
| 22:15:36 | kugel | JdGordon: lol, I just reproduced the problem very quickly |
| 22:15:51 | JdGordon | in regular use? |
| 22:16:01 | kugel | what's bad about the fix? |
| 22:17:14 | JdGordon | its not really bad right now... at most there are 2 skins being loaded... think about what happens when fm gets skins... and on remote targets... thats 4 loads every time |
| 22:17:20 | JdGordon | then add other screens |
| 22:17:22 | JdGordon | its a bad idea |
| 22:17:48 | kugel | it isn't |
| 22:17:53 | | Quit Riku (Nick collision from services.) |
| 22:17:53 | JdGordon | tis! |
| 22:18:11 | bluebrother | OMG can't you guys get your changes *finished* before committing? |
| 22:18:13 | | Join Lss__ [0] (i=Lss@cm26.delta97.maxonline.com.sg) |
| 22:18:21 | kkurbjun | JdGordon: I load wps files direct... |
| 22:18:25 | kugel | the problem you created will get even worse if you add more skins, the the user just needs to load a wps, a rwps and a wts and BANG it's broken |
| 22:18:33 | kugel | that doesn't even require reloading another skin |
| 22:19:07 | kugel | the disk is already spinning if you load one skin, it doesn't even create much delay |
| 22:19:18 | * | JdGordon gives up |
| 22:19:23 | kugel | good |
| 22:19:25 | kugel | :p |
| 22:19:49 | * | bluebrother shakes head |
| 22:20:14 | | Part Grahack |
| 22:21:31 | kugel | kkurbjun: happy about the 700k ram saving? |
| 22:22:01 | kkurbjun | rasher - on themes, I was wondering if it would be possible to add a section for the m:robe 500? mcuelenaere pointed me to the admin panel, but I don't have an account - is that something you could help me out with? |
| 22:22:14 | | Join moonscapex [0] (n=4b9fb4d0@gateway/web/cgi-irc/labb.contactor.se/x-thmcyhcdgdszdqeg) |
| 22:22:15 | kkurbjun | kugel: :), yes that is nice |
| 22:22:18 | rasher | kkurbjun: certainly, hang on |
| 22:22:44 | | Quit moonscapex (Client Quit) |
| 22:23:50 | kkurbjun | kugel: oh, I didn't see the impact that changing the remote allocation made, that's REALLY nice :-D |
| 22:24:00 | kugel | :> |
| 22:25:25 | kugel | kkurbjun: you basically had enough space to run a fully featured 640x480 wps on the remote too :p |
| 22:25:35 | kkurbjun | haha |
| 22:25:48 | kkurbjun | yeah, and for such a small screen that made no sense |
| 22:25:57 | JdGordon | dont give him all the credit.... it was my apparently devasting change which made that saving possible! |
| 22:26:18 | kugel | haha |
| 22:26:22 | | Join kadoban [0] (n=mud@cpe-24-93-17-195.rochester.res.rr.com) |
| 22:26:22 | kkurbjun | rasher, how do I add an image for the m:robe |
| 22:26:28 | kkurbjun | :) |
| 22:26:41 | rasher | kkurbjun: get it into rockbox.org - that's where they come from |
| 22:26:55 | kugel | JdGordon: "JdGordonthe talk above about the buffer size... maybe kugel wants to fix it and take credit :)" :D |
| 22:27:17 | rasher | kkurbjun: http://www.rockbox.org/playerpics/ is where they are expected to be |
| 22:27:18 | kugel | yes I take it :p |
| 22:28:49 | kkurbjun | rasher, ok, so I just need to check out the WWW section in svn and add the player image? |
| 22:29:20 | JdGordon | kugel: credit yes.. not *all* the credit :D |
| 22:29:21 | pixelma | kkurbjun: do you have an svg? |
| 22:29:26 | rasher | kkurbjun: and then poke the swedes, since rockbox.org doesn't run directly out of svn |
| 22:29:33 | kkurbjun | pixelma: I do |
| 22:29:41 | JdGordon | can some ams targets be added also? |
| 22:29:41 | CIA-61 | New commit by learman (r22355): Fix realloc bug in TLSF. |
| 22:29:47 | JdGordon | they are no less suported than the mr500 |
| 22:31:16 | kugel | I think so yes |
| 22:31:32 | kkurbjun | pixelma: is there a good place to store svgs publically? |
| 22:31:41 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 22:31:51 | kugel | although e200v2 doesn't need an extra image |
| 22:32:00 | JdGordon | there hasnt been any desicisons to not put working-in-progress targets on theme site has there? |
| 22:32:01 | | Join pamaury [0] (n=pamaury@sal63-1-82-243-96-220.fbx.proxad.net) |
| 22:32:09 | rasher | kkurbjun: the manual holds the other svgs |
| 22:32:12 | bluebrother | kkurbjun: put it with the proper naming for use in the manual svn :) |
| 22:33:10 | | Quit aaron424 (Remote closed the connection) |
| 22:33:10 | | Quit suom1 (Remote closed the connection) |
| 22:33:54 | JdGordon | kugel: back to that commit... part of the motivation for the whole work was to make some screens easier to draw.. the time/date screen is a good example which wastes lots of code drawing the time when the skin engine could do it free... reloading the skins like that is a waste when the time screen doesnt always need to be loaded |
| 22:34:02 | JdGordon | what I was going to do was add an event which is triggered when the buffer is reset so screens can decide to reload or not |
| 22:34:43 | kugel | but since you didn't add a possiblity to reload skins seperately, it's the only way to fix *right now* |
| 22:35:50 | | Quit thegeek_ (Read error: 104 (Connection reset by peer)) |
| 22:36:21 | kugel | there's nothing holding you back from doing a better fix |
| 22:37:58 | kugel | I also think the buffer could managed better. for example a pointer for each skin pointing to the buf_start for that skin in the buffer, and the size. Then the skins could be memmoved when another skin needs reloading to make it fit |
| 22:38:24 | kugel | I think Unhelpful's buflib also wouldn't do such a bad job here |
| 22:38:28 | JdGordon | that doesnt guarentee it will always git |
| 22:38:35 | JdGordon | fit |
| 22:39:03 | CIA-61 | New commit by mcuelenaere (r22356): Rockbox Utility: add preliminary support for installing the bootloader (+ dual boot) on ChinaChip targets |
| 22:39:04 | | Quit GeekShado_ (Read error: 145 (Connection timed out)) |
| 22:39:08 | kugel | well, it does, unless all of your skins exceed the skin buffer |
| 22:39:16 | mcuelenaere | crap, forgot to mention FS#.. |
| 22:39:40 | CIA-61 | New commit by kkurbjun (r22357): M:Robe 500 add SVG for player |
| 22:39:53 | JdGordon | and no... the memroy cant move, unless lots of work is done to use offsets instead of pointers |
| 22:40:03 | kugel | yea, why not? |
| 22:40:09 | JdGordon | but sure, if you want to do the work, be my guest |
| 22:40:15 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 22:40:21 | kugel | nah, I don't feel like it, that's your job |
| 22:40:22 | JdGordon | why not hwat? |
| 22:40:28 | kugel | storing offsets |
| 22:40:53 | JdGordon | offsets are slower and more likeloy to be done wrongly |
| 22:41:15 | kugel | then I prefer being slow *once* on loading, instead of being slow always at drawing |
| 22:41:16 | mcuelenaere | hmm I should probably also add the vx777 and vx747+ |
| 22:41:42 | JdGordon | oy gevalt |
| 22:41:47 | * | JdGordon is off again |
| 22:42:02 | kugel | oy gevalt? |
| 22:42:40 | JdGordon | yiddish :) |
| 22:42:44 | JdGordon | thats how annoyed I am :p |
| 22:43:41 | rasher | kkurbjun: That's certainly a very basic svg! |
| 22:43:56 | kkurbjun | rasher, I can't say that I'm an artist :-D |
| 22:44:15 | kkurbjun | Is there more that should be in it? |
| 22:44:42 | rasher | The manual expects button labels for any physical buttons. I assume there is at least one? |
| 22:45:00 | kkurbjun | yeah, there is a power button on the top |
| 22:45:21 | kkurbjun | any preference on the font used? |
| 22:45:26 | JdGordon | hmm... is any validation done on themes when they are uploaded? |
| 22:45:31 | rasher | kkurbjun: Take a look at the other svgs. |
| 22:45:37 | kugel | I hope that's getting scaled down a bit for the theme site :) |
| 22:45:45 | rasher | JdGordon: A good deal. |
| 22:45:46 | JdGordon | is checkwps or whatever its called run? |
| 22:46:01 | rasher | If it builds these days.. |
| 22:46:28 | JdGordon | I'll try getting that to output the ram usage stats for the skins, would be useful |
| 22:46:43 | kkurbjun | rasher - I was, but it doesn't look like the font type is consistent |
| 22:46:58 | rasher | JdGordon: Looks like you'll need to fix checkwps building first.. |
| 22:47:27 | rasher | It's been broken for months |
| 22:48:16 | domonoky | maybe we should try to make automated build of checkwps so it doesnt get broken ? |
| 22:48:39 | pixelma | kkurbjun: I also drew side buttons on the sides at least a bit so that you could see where they are even if you couldn't see them in reality from the top view. The font introduced in the beginning and which I used (so *most* svgs have it from the Palatino family |
| 22:48:47 | bluebrother | having a checkwps target in the build system would be nice. Plus a bootloader and simulator target :) |
| 22:49:19 | mcuelenaere | hmm is there a difference between TARGET_ID and MODEL_NUMBER? |
| 22:50:36 | mcuelenaere | domonoky: that was going to be added in the build server rework, but because checkwps was so badly broken; it didn't got added |
| 22:50:50 | | Join mcscruff [0] (n=mcscruff@host86-152-183-144.range86-152.btcentralplus.com) |
| 22:52:09 | | Join Omlet [0] (n=Omlet05@99.225-200-80.adsl-dyn.isp.belgacom.be) |
| 22:53:35 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
| 22:55:35 | kkurbjun | pixelma: thanks I'll add the button on the top along with the text |
| 22:56:49 | JdGordon | rasher: also, are remotes going to be added to the theme site? |
| 22:56:57 | CIA-61 | New commit by kkurbjun (r22358): M:Robe 500: Add power button to SVG. |
| 22:57:18 | * | JdGordon doesnt even know how to build checkwps |
| 22:57:56 | rasher | JdGordon: Ideally they would be :| |
| 22:58:35 | rasher | JdGordon: cd tools/checkwps and run buildall.sh |
| 22:58:35 | domonoky | JdGordon: if wasnt broken, it would just be a tools\checkwps\buildall.sh |
| 22:58:48 | rasher | domonoky: \!? |
| 22:58:58 | JdGordon | ok, i'm looking at fixing it now |
| 22:58:59 | domonoky | windows :-) |
| 22:59:07 | pixelma | M3 (Iaudio remote) could also be added - at least as a main screen |
| 22:59:25 | mcscruff | hi all, is there any development going on for Sony Walkmans |
| 22:59:56 | JdGordon | yikes, seems very very broken |
| 23:00 |
| 23:01:44 | | Quit mitk ("Leaving") |
| 23:02:44 | bluebrother | mcscruff: no |
| 23:03:08 | * | bluebrother wonders how domonoky plans to invoke a sh script using windows path delimiters |
| 23:03:25 | CIA-61 | New commit by mcuelenaere (r22359): * Onda VX747/VX767: fix target_id & MODEL_NUMBER ... |
| 23:04:32 | JdGordon | checkwps is very very fubar |
| 23:11:16 | CIA-61 | New commit by kkurbjun (r22360): M:Robe 500: Add small image. |
| 23:14:12 | | Join suom1 [0] (i=markus@viitamaki.net) |
| 23:15:20 | kkurbjun | ok, I got the theme section added and I added the picture - who should I follow up with to get the www updated so that the theme site shows the M:robe 500 small image? |
| 23:15:46 | *** | Saving seen data "./dancer.seen" |
| 23:16:12 | | Join moos [0] (i=mustapha@rockbox/staff/moos) |
| 23:17:11 | | Quit n1s ("Lämnar") |
| 23:18:07 | JdGordon | kkurbjun: hhe the mr500 sim is broken if you show the backdrop :) |
| 23:18:30 | JdGordon | and, can you test a quick patch? my mr500 is out of action... |
| 23:18:40 | JdGordon | fiddle with the region array in the wps |
| 23:20:37 | * | JdGordon wonders if there is a nice way to deal with all the different struct skin_token_list * members in the wps_Data struct |
| 23:21:38 | | Quit mcscruff ("Leaving") |
| 23:23:00 | CIA-61 | New commit by jdgordon (r22361): remove the limit of touch regions in touchscreen skins |
| 23:26:39 | | Quit HBK () |
| 23:26:46 | kkurbjun | JdGordon: region array? |
| 23:27:24 | JdGordon | Unhelpful: would it be theoretically doable to have more than one AA image loaded? (either the same image at different res, or different files |
| 23:27:26 | JdGordon | kkurbjun: ? |
| 23:27:34 | kkurbjun | what is the region array? |
| 23:27:39 | CIA-61 | New commit by moos (r22362): Use a more appropriated word than the string commited in r21223. |
| 23:28:23 | | Quit ehntoo (Read error: 110 (Connection timed out)) |
| 23:28:28 | | Quit Omlet () |
| 23:28:57 | kkurbjun | or I guess more importantly how would you like it tested? Just to see if the user defined buttons work? |
| 23:29:21 | JdGordon | yeah |
| 23:29:29 | JdGordon | it shouldnt have broken anything, just changed how they are stored |
| 23:30:15 | CIA-61 | New commit by jdgordon (r22363): remove an unused #define |
| 23:30:15 | pixelma | I don't believe my buildclient did 2 bootloaders in 1 second each |
| 23:30:45 | pixelma | it can't be true |
| 23:30:50 | JdGordon | ccache... its entirely cached |
| 23:30:58 | JdGordon | there was no change at all for them |
| 23:31:14 | bluebrother | and does it work if the output folder changes on each run? |
| 23:31:19 | pixelma | but that's a VM on a not too fast laptop |
| 23:31:45 | JdGordon | bluebrother: ccache? no idea.. and apparently its man page cant be trusted |
| 23:32:03 | bluebrother | JdGordon: yes, that's the reason I'm asking. |
| 23:32:19 | bluebrother | maybe it would make more sense to name the output folder build-<targetname>? |
| 23:32:29 | Bagder | the location of the output or source surely can't matter |
| 23:32:39 | JdGordon | yeah, it sounds really wierd if it did |
| 23:32:59 | bluebrother | that way ccache could also catch differently preprocessed sources. |
| 23:33:19 | moos | oops, /me wanted to refer to r21364 in his commit message :( |
| 23:35:48 | JdGordon | lets say Unhelpful says yes to above... how would people feel about allowing that? |
| 23:36:19 | JdGordon | it would be pretty impressive to show front+back covers.. no idea how it would work in the .wps though.. but would be cool |
| 23:36:48 | mcuelenaere | when one taps on the image, it could be switched? (touchscreen only) |
| 23:37:26 | JdGordon | both on the screen at the same time maybe also |
| 23:38:05 | pixelma | rotate the cover in 3D? |
| 23:38:16 | JdGordon | rasher: whats the correct way to deprecate a string? |
| 23:38:35 | JdGordon | I'm gong to remove the AA size line in the rockbox info because its unseless now with resizing aa |
| 23:38:55 | bluebrother | hmm. If ccache doesn't bother if the output directory changed, why does my ccache show that much cache misses? |
| 23:39:05 | kkurbjun | JdGordon: the buttons still work |
| 23:39:13 | JdGordon | *phew* :) |
| 23:39:21 | kkurbjun | :) |
| 23:40:04 | rasher | JdGordon: tools/langtool.pl −−deprecate −−id LANG_ALBUMART −−inplace apps/lang/*.lang |
| 23:40:10 | bluebrother | useless? The AA size line still shows the size of the AA the theme is resizing too. The use of that line is quite limited, admittedly. |
| 23:40:27 | JdGordon | so dont remove it? |
| 23:40:57 | JdGordon | it had its use, now its just a waste imo |
| 23:41:08 | bluebrother | well, I don't object to removing. I just don't think it's completely useless. |
| 23:41:08 | JdGordon | but I'm happy to leave it if others want it |
| 23:41:28 | bluebrother | it's just calling it useless :) |
| 23:41:31 | pixelma | IMO it could be removed |
| 23:41:45 | JdGordon | I cant spell unnesacery, so i try to avoid using the word :) |
| 23:41:57 | bluebrother | I found that line quite confusing in the past :D |
| 23:41:59 | rasher | I agree, it doesn't really add much |
| 23:42:39 | | Quit bertrik (Read error: 110 (Connection timed out)) |
| 23:43:06 | bluebrother | JdGordon: remove it! (just to make that clear :) |
| 23:43:12 | | Join bertrik [0] (n=bertrik@d90-128-154-247.cust.tele2.nl) |
| 23:43:15 | JdGordon | done :) |
| 23:44:30 | moos | you want to remove it for replace it with a skin buffer line? ;) |
| 23:46:39 | JdGordon | yes, the usage line is useful |
| 23:46:52 | JdGordon | or at least, mildly interesting :p |
| 23:47:07 | moos | hehe :) |
| 23:47:37 | JdGordon | and wrong :) |
| 23:47:43 | JdGordon | "skin usage: 18.0MB" ! |
| 23:47:53 | pixelma | is there a way to check if it's really the WPS/skin buffer which is responsible for the bin size growth on Archos? |
| 23:48:03 | pixelma | and maybe why? |
| 23:48:26 | pixelma | all others saw a decrease |
| 23:48:37 | JdGordon | well.. a fair bit of code was added... and because its small screen and generally small arrays anyway, not enough of the new code was offset |
| 23:48:44 | JdGordon | I wouldnt worry about 500b |
| 23:49:00 | pixelma | it could break my voice file(s) |
| 23:49:47 | JdGordon | you can shrink the buffer if you really want |
| 23:49:54 | JdGordon | assuming you dont use the full buffer for your wps |
| 23:50:05 | pixelma | maybe I could but others not |
| 23:50:45 | amiconn | It's almost exclusively bss increase. Binsize itself is +36 bytes (recorder) |
| 23:51:15 | amiconn | Hint: hover over the delta to see the details |
| 23:53:10 | pixelma | so it's not the added code but somehow more space is reserved for the WPS/skin buffer? |
| 23:54:10 | | Quit matsl ("Leaving") |
| 23:54:29 | JdGordon | the buffer size didnt change so that shouldnt be possible... |
| 23:55:04 | | Quit ej0rge (Remote closed the connection) |
| 23:55:25 | JdGordon | "buffer_pos-buffer_start" is the correct count of usage isnt it? (_pos is incremeneted with usage, _start is constant) |
| 23:55:31 | kugel | pixelma: bload-o-meter or objdiff may help (in utils/analysis), just throw the rockbox.elf before and after the commit at one of them |
| 23:58:59 | pixelma | have to pass |
icon identifies that the person is a core developer (has commit access).