| 00:02:37 | | Join matsl [0] (n=matsl@1-1-4-2a.mal.sth.bostream.se) |
| 00:03:35 | CIA-85 | New commit by BobC (r23451): Replace hard-coded value of memory size |
| 00:15:35 | CIA-85 | New commit by mc2739 (r23452): FS #10744 - Fuze: Last line of screen's pixels doesn't change when switching themes and using plugins |
| 00:21:48 | | Quit StealthyXIIGer (Read error: 60 (Operation timed out)) |
| 00:32:16 | | Join robin0800 [0] (n=quassel@cpc3-brig8-0-0-cust436.brig.cable.ntl.com) |
| 00:35:21 | CIA-85 | New commit by BobC (r23453): Replace hard coded memory size |
| 00:51:32 | CIA-85 | New commit by mc2739 (r23454): Add Sascha Wilde to docs/CREDITS for bug fix committed in r23449 |
| 00:54:52 | | Quit flydutch ("/* empty */") |
| 00:58:42 | | Join Utchybann [0] (n=lolo@ede67-1-81-56-102-26.fbx.proxad.net) |
| 00:59:37 | | Quit robin0800 (Remote closed the connection) |
| 01:00 |
| 01:04:57 | | Quit bertrik ("Leaving") |
| 01:11:52 | pixelma | hrrmm... why does pictureflow not work without own album art even though it provides a default "no art" picture? |
| 01:12:10 | | Quit Bob_C_ ("ChatZilla 0.9.85 [Firefox 3.5.4/20091016092926]") |
| 01:20:17 | Unhelpful | pixelma: i don't quite understand? it requires the database, but should work with AA disabled in core, and should display the no art picture for albums for which art is not found... |
| 01:20:47 | | Join GeekShado_ [0] (n=Antoine@77.204.157.218) |
| 01:25:40 | | Quit Hillshum (Remote closed the connection) |
| 01:26:03 | pixelma | yes, but it fails when there is no own album art at all... I just put a few songs into my simdisk dir without an art picture, initialised database, rebooted - tried to enter pictureflow and it fails with a splash "No album art found" and "Could not create album art cache. Press any button to continue." |
| 01:26:31 | | Join Hillshum [0] (n=hillshum@75-165-237-208.slkc.qwest.net) |
| 01:26:46 | pixelma | it works if I throw one bmp into the folder |
| 01:27:58 | pixelma | I mean the simdisk where the music files are |
| 01:30:39 | | Join saratoga [0] (i=9803c6dd@gateway/web/freenode/x-iowmjjkvsfrjiiac) |
| 01:31:01 | saratoga | is there a command in rockbox to get the offset in a file of a file handle? |
| 01:32:29 | amiconn | pos = lseek(fd, 0, SEEK_CUR); |
| 01:34:16 | Unhelpful | ah! you mean that there's *no* album art. for any albums. |
| 01:35:17 | pixelma | yes, I still would expect it to not fail and just use the no art picture all the way, it is my fault if it doesn't look right |
| 01:37:40 | | Quit GeekShadow (Read error: 110 (Connection timed out)) |
| 01:38:04 | Unhelpful | pixelma: i think you're probably right... and it would even save code. ;) |
| 01:39:07 | Unhelpful | oh, i meant to ask something, although you might not be the right person... though you do have an m3. it there a keymap context that includes remote+main buttons? |
| 01:40:27 | Unhelpful | it would be really nice on devices with remotes to use something *like* CONTEXT_TREE, but which will trigger on any keys mapped in CONTEXT_TREE *or* in CONTEXT_TREE|CONTEXT_REMOTE... |
| 01:40:53 | | Quit DerPapst ("Leaving.") |
| 01:41:31 | pixelma | I don't have an M3, amiconn does |
| 01:41:45 | pixelma | I have an M5 but without the remote |
| 01:42:51 | | Quit n1s ("Lämnar") |
| 01:43:40 | pixelma | and I am just trying to make sense of the keymapping in pictureflow to rework the manual button table a bit |
| 01:44:35 | | Quit matsl (Read error: 110 (Connection timed out)) |
| 01:47:31 | Unhelpful | ah! such a context would also make PF testable on M3 sim... |
| 01:50:28 | | Quit ender` (" If Java had true garbage collection, most programs would delete themselves upon execution. -- Robert Sewell") |
| 01:52:31 | pixelma | well, it would be nice if the remote buttons were simulated too to test all the other things (mostly plugins) too. The remote hold switch already is simulated |
| 01:53:45 | | Quit Hillshum (Read error: 110 (Connection timed out)) |
| 01:55:21 | Unhelpful | yes, simulating all remote buttons, too, would be good. but it seems like there should be, for any given context on a remote target, a "main+remote" version of the context... don't almost all core screens respond to either remote or device buttons? |
| 01:55:24 | *** | Saving seen data "./dancer.seen" |
| 01:59:08 | pixelma | it should if the amounts of available buttons on the remote is not too different from the main device. Hmm, I think your idea would be hard to do on the H100/H300 where you can use one of three different remotes (with different button layout, the worst one being the non-lcd remote with just a few buttons) |
| 01:59:50 | pixelma | on the M3 the mappings are quite similar though |
| 02:00 |
| 02:06:11 | TheSeven | ok, backlight is working now :-) |
| 02:09:38 | saratoga | TheSeven: on the 4g? |
| 02:09:43 | TheSeven | yes |
| 02:12:08 | | Quit amiconn (Nick collision from services.) |
| 02:12:11 | | Join amiconn_ [0] (i=quassel@rockbox/developer/amiconn) |
| 02:12:33 | | Nick amiconn_ is now known as amiconn (i=quassel@rockbox/developer/amiconn) |
| 02:14:21 | | Join pixelma_ [0] (i=quassel@rockbox/staff/pixelma) |
| 02:14:21 | | Quit pixelma (Nick collision from services.) |
| 02:14:41 | | Nick pixelma_ is now known as pixelma (i=quassel@rockbox/staff/pixelma) |
| 02:15:39 | | Quit domonoky1 (Read error: 54 (Connection reset by peer)) |
| 02:17:32 | saratoga | TheSeven: how similar are the Nano 4Gs? |
| 02:17:47 | TheSeven | to the 2g? well, a little. |
| 02:18:18 | TheSeven | but they're very similar to the ipod touch 2g, which makes me wonder if anyone wants to start to work on that one :-) |
| 02:18:37 | saratoga | is the FTL similar? |
| 02:18:46 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 02:18:51 | | Quit jfc^3 (Read error: 104 (Connection reset by peer)) |
| 02:19:00 | saratoga | i might buy one while apple still has old 4Gs left in stock |
| 02:19:31 | TheSeven | saratoga: there are refurbs for 99$ (8GB) or 119$ (16GB) |
| 02:19:41 | saratoga | yeah thats what i'm looking at |
| 02:19:50 | TheSeven | and the FTL is also very iphone-like, it's a newer version of the nano2g one |
| 02:21:29 | | Join panni_ [0] (i=hannes@ip-95-222-21-143.unitymediagroup.de) |
| 02:24:28 | kugel | mc2739: interesting, the off-by-1 was a off-by-2 ? |
| 02:24:38 | | Nick Horschti is now known as Horscht (n=Horscht2@xbmc/user/horscht) |
| 02:27:32 | saratoga | TheSeven: but you think the 4G port will be no more difficult then the 2G? i ask because I don't have much rockbox time lately but i do eventually want to look at codec optimization on the new nanos |
| 02:28:39 | TheSeven | no, the 4g will be harder than itouch |
| 02:29:17 | TheSeven | it looks like we can't even dump the bootrom, and there's code signing everywhere, even for low level bootloader stages |
| 02:29:47 | TheSeven | we'll permanently need a bootrom exploit to boostrap our loader, and without a bootrom dump this is going to be very hard |
| 02:30:40 | TheSeven | right now I'm playing around through the same application level exploit we used to break the 2g nano, but this means booting the whole apple firmware which will run into a payload that may be a maximum of 3KB |
| 02:31:07 | mc2739 | kugel: I guess so. I reverted 23359 to test FS #10744 and there were two rows of pixel remnants. |
| 02:33:12 | mc2739 | kugel: It was most notable when switching between a light theme background and a dark background |
| 02:33:59 | saratoga | oh just realized the 4G is using a much faster arm core then earlier Nanos |
| 02:34:17 | | Join jfc^3 [0] (n=john@dpc6682208002.direcpc.com) |
| 02:34:34 | kugel | mc2739: i think the calculation in there is nevertheless not entirely correct |
| 02:35:53 | saratoga | although the linux4nano wiki says its arm940t which i think is wrong |
| 02:36:01 | kugel | I had a quick go at doing it right, but it resulted in garbage, mostly |
| 02:38:14 | | Join webguest46 [0] (n=60f0b7dd@giant.haxx.se) |
| 02:38:36 | | Quit webguest46 (Client Quit) |
| 02:43:22 | | Quit kugel (Remote closed the connection) |
| 02:45:37 | | Quit CaptainKwel (Remote closed the connection) |
| 02:49:31 | | Join mc2739_ [0] (n=mc2739@rockbox/developer/mc2739) |
| 02:51:36 | | Quit mc2739 (Read error: 60 (Operation timed out)) |
| 02:53:07 | | Nick mc2739_ is now known as mc2739 (n=mc2739@rockbox/developer/mc2739) |
| 02:59:41 | TheSeven | saratoga: it's some ARM11 |
| 03:00 |
| 03:00:44 | Unhelpful | pixelma: in some contexts, in some remotes, there might not be any sensible mappings equivalent to the main keys. but the basic idea doesn't require that every main-keys action have a remote-key equivalent - the idea is just to make whatever remote keys would manipulate a particular screen available at the same time as device keys, in a single context. probably it could be done, with chaining? |
| 03:01:25 | | Quit adiroiban (Read error: 110 (Connection timed out)) |
| 03:11:26 | | Join Sajber^ [0] (n=Sajber@h-65-31.A213.priv.bahnhof.se) |
| 03:13:58 | | Quit gevaerts (Nick collision from services.) |
| 03:14:10 | | Join gevaerts [0] (n=fg@rockbox/developer/gevaerts) |
| 03:23:46 | JdGordon | who knows where set_vol() is? |
| 03:23:52 | JdGordon | setvol() even |
| 03:26:07 | TheSeven | JdGordon: grep will, probably |
| 03:26:17 | JdGordon | got it first guess :) misc.c |
| 03:26:26 | saratoga | what a great filename |
| 03:26:46 | TheSeven | oh nice. there's a hidden LCD ID test in diagmode :-) |
| 03:30:27 | * | JdGordon has no freeking idea how the volume gets changed in the settinng menu... (which is probably a bad thing) |
| 03:31:13 | JdGordon | I cant see any callback that does it |
| 03:34:22 | | Join Strife1989 [0] (n=michael@adsl-220-119-35.mcn.bellsouth.net) |
| 03:36:54 | CIA-85 | New commit by jdgordon (r23455): Fix FS #10745 - %mv not working in sbs... ... |
| 03:37:48 | | Quit Strife89 (Nick collision from services.) |
| 03:37:54 | | Nick Strife1989 is now known as Strife89 (n=michael@adsl-220-119-35.mcn.bellsouth.net) |
| 03:43:28 | | Quit GeekShado_ ("The cake is a lie !") |
| 03:47:39 | CIA-85 | New commit by jdgordon (r23456): haloween night is when red monsters are vanquished right? or is that green monsters? |
| 03:50:51 | JdGordon | Zagor: apparently my .au build client was complainig about a modified svn tree? I did asvn diff but nothing was changed, but apparently it was locked? |
| 03:55:26 | *** | Saving seen data "./dancer.seen" |
| 03:56:03 | | Join Topy [0] (n=Topy44@f054173242.adsl.alicedsl.de) |
| 03:56:39 | CIA-85 | New commit by jdgordon (r23457): YELLOW ZOMBIES! |
| 03:59:59 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 04:00 |
| 04:07:57 | | Quit Rondom (Nick collision from services.) |
| 04:08:08 | | Join Rondom [0] (n=Rondom@dslb-084-057-181-033.pools.arcor-ip.net) |
| 04:09:31 | | Join Addi2 [0] (n=adam@157-157-105-127.dsl.dynamic.simnet.is) |
| 04:14:57 | | Quit T44 (Read error: 110 (Connection timed out)) |
| 04:47:31 | | Quit Strife89 ("My number of files is OVER 9000!") |
| 04:55:08 | | Quit TheSeven (Nick collision from services.) |
| 04:55:24 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 04:55:35 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 05:00 |
| 05:14:35 | | Quit daggett ("Ex-Chat") |
| 05:19:07 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 05:26:04 | | Join psudo [0] (i=7aa7f099@gateway/web/freenode/x-phdebyzmdxfehewy) |
| 05:33:07 | psudo | Hi |
| 05:34:51 | | Quit MethoS- (Remote closed the connection) |
| 05:40:16 | | Join froggyman [0] (n=sopgenor@pool-72-69-220-194.chi01.dsl-w.verizon.net) |
| 05:46:12 | | Join ShapeShifter499 [0] (n=chatzill@adsl-68-121-53-108.dsl.scrm01.pacbell.net) |
| 05:54:58 | | Join evilnick__ [0] (n=evilnick@ool-4571af51.dyn.optonline.net) |
| 05:55:28 | *** | Saving seen data "./dancer.seen" |
| 05:56:06 | | Join Llorean [0] (n=DarkkOne@99.4.146.40) |
| 05:56:42 | | Quit Res1 (Read error: 60 (Operation timed out)) |
| 06:00 |
| 06:00:49 | | Join JdGordon| [0] (n=Miranda@c-24-22-210-83.hsd1.wa.comcast.net) |
| 06:13:27 | | Quit evilnick_ (Read error: 113 (No route to host)) |
| 06:14:47 | | Part froggyman |
| 06:16:01 | | Join slam_ [0] (n=grndslm@174-126-14-4.cpe.cableone.net) |
| 06:20:09 | | Quit ShapeShifter499 ("ChatZilla 0.9.85 [Firefox 3.5.4/20091028153816]") |
| 06:20:31 | | Join Res1 [0] (n=Res@user-0c6s6ir.cable.mindspring.com) |
| 06:21:16 | | Quit psudo ("Page closed") |
| 06:24:59 | | Quit Utchybann (Read error: 113 (No route to host)) |
| 06:30:50 | | Join bburtin [0] (n=boris@adsl-75-35-1-43.dsl.pltn13.sbcglobal.net) |
| 06:32:00 | bburtin | I'd like to get write access to the Rockbox Wiki to clear up some inconsistencies in the Linux instructions for Sansa e200R. Can someone help me out? My Wiki username is BorisBurtin. Thanks! |
| 06:33:36 | | Quit panni_ (Read error: 104 (Connection reset by peer)) |
| 06:36:38 | | Quit bburtin ("Leaving.") |
| 06:37:35 | | Quit Addi2 ("Leaving") |
| 07:00 |
| 07:00:16 | DEBUG | Received signal 15 (SIGTERM), terminating (snapshot: fplrun.c line 385) |
| 07:00:16 | *** | Cleanup |
| 07:00:16 | *** | Saving seen data "./dancer.seen" |
| 07:00:16 | *** | Exit |
| 07:08:05 | *** | Started Dancer V4.16 |
| 07:08:05 | *** | Connected to irc.freenode.net on port 6667 |
| 07:08:05 | *** | Logfile for #rockbox started |
| 07:08:07 | *** | Server message 501: 'logbot :Unknown MODE flag' |
| 07:08:07 | Mode | "logbot :+i" by logbot |
| 07:08:07 | | Join logbot [0] (n=rockbox@giant.haxx.se) |
| 07:08:07 | | Join Res1 [0] (n=Res@user-0c6s6ir.cable.mindspring.com) |
| 07:08:07 | | Join slam_ [0] (n=grndslm@174-126-14-4.cpe.cableone.net) |
| 07:08:07 | | Join JdGordon| [0] (n=Miranda@rockbox/developer/JdGordon) |
| 07:08:07 | | Join Llorean [0] (n=DarkkOne@rockbox/user/Llorean) |
| 07:08:07 | | Join evilnick__ [0] (n=evilnick@ool-4571af51.dyn.optonline.net) |
| 07:08:07 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 07:08:07 | | Join TheSeven [0] (n=theseven@rockbox/developer/TheSeven) |
| 07:08:07 | | Join Rondom [0] (n=Rondom@dslb-084-057-181-033.pools.arcor-ip.net) |
| 07:08:07 | | Join faemir [0] (n=faemir@78.33.109.163) |
| 07:08:07 | | Join Topy [0] (n=Topy44@f054173242.adsl.alicedsl.de) |
| 07:08:07 | | Join gevaerts [0] (n=fg@rockbox/developer/gevaerts) |
| 07:08:07 | | Join Sajber^ [0] (n=Sajber@h-65-31.A213.priv.bahnhof.se) |
| 07:08:07 | | Join mc2739 [0] (n=mc2739@rockbox/developer/mc2739) |
| 07:08:07 | | Join jfc^3 [0] (n=john@dpc6682208002.direcpc.com) |
| 07:08:07 | | Join pixelma [0] (i=quassel@rockbox/staff/pixelma) |
| 07:08:07 | | Join amiconn [0] (i=quassel@rockbox/developer/amiconn) |
| 07:08:07 | | Join saratoga [0] (i=9803c6dd@gateway/web/freenode/x-iowmjjkvsfrjiiac) |
| 07:08:07 | | Join n17ikh [0] (n=n17ikh@host-69-59-126-212.nctv.com) |
| 07:08:07 | | Join AlexP [0] (n=alex@rockbox/staff/AlexP) |
| 07:08:07 | | Join FOAD [0] (n=dok@dinah.blub.net) |
| 07:08:07 | | Join toffe82_ [0] (n=chatzill@adsl-75-23-148-240.dsl.frs2ca.sbcglobal.net) |
| 07:08:07 | | Join antil33t [0] (n=Mudkips@119.224.12.185) |
| 07:08:07 | | Join phanboy4 [0] (n=benji@c-24-98-43-198.hsd1.ga.comcast.net) |
| 07:08:07 | | Join BlakeJohnson86 [0] (n=bjohnson@c-24-118-162-123.hsd1.mn.comcast.net) |
| 07:08:07 | | Join killan [0] (n=nnscript@c-0efa70d5.06-397-67626721.cust.bredbandsbolaget.se) |
| 07:08:07 | | Join mt [0] (n=mtee@rockbox/developer/mt) |
| 07:08:07 | | Join z35 [0] (n=z35@ool-45714f83.dyn.optonline.net) |
| 07:08:07 | | Join Horscht [0] (n=Horscht2@xbmc/user/horscht) |
| 07:08:07 | | Join liar [0] (n=liar@83.175.83.185) |
| 07:08:07 | | Join gtkspert [0] (n=gtkspert@124-169-228-197.dyn.iinet.net.au) |
| 07:08:07 | | Join mikroflops [0] (n=yogurt@90-231-195-226-no112.tbcn.telia.com) |
| 07:08:07 | | Join yosafbridge [0] (n=yosafbri@li14-39.members.linode.com) |
| 07:08:07 | | Join Bob_C [0] (n=chatzill@host86-144-201-0.range86-144.btcentralplus.com) |
| 07:08:07 | | Join scorche|sh [50] (n=scorche@rockbox/administrator/scorche) |
| 07:08:07 | | Join Rob2223 [0] (n=Miranda@p4FDCD5F7.dip.t-dialin.net) |
| 07:08:07 | | Join bluebrother [0] (n=dom@rockbox/developer/bluebrother) |
| 07:08:07 | | Join niekie [0] (i=quasselc@dreamworld.bergnetworks.com) |
| 07:08:07 | | Join feisar- [0] (i=jljhook@irkki.fi) |
| 07:08:07 | | Join HellDragon [0] (i=jd@Wikipedia/HellDragon) |
| 07:08:07 | | Join avacore [0] (i=nobody@1008ds1-rdo.0.fullrate.dk) |
| 07:08:07 | | Join parafin [0] (i=parafin@paraf.in) |
| 07:08:07 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
| 07:08:07 | | Join TopyMobile [0] (n=topy@xdsl-78-34-69-123.netcologne.de) |
| 07:08:07 | | Join Galois [0] (i=djao@efnet.math.uwaterloo.ca) |
| 07:08:07 | | Join scorche [50] (n=scorche@rockbox/administrator/scorche) |
| 07:08:07 | | Join maraz [0] (i=maraz@xob.kapsi.fi) |
| 07:08:07 | | Join linuxstb [0] (n=linuxstb@rockbox/developer/linuxstb) |
| 07:08:07 | | Join ps-auxw [0] (n=arneb@dyn37.ps-auxw.de) |
| 07:08:07 | | Join bzed [0] (n=bzed@devel.recluse.de) |
| 07:08:07 | | Join goffa [0] (n=goffa@70.33.8.114) |
| 07:08:07 | | Join tmzt [0] (n=tmzt@adsl-99-51-211-69.dsl.akrnoh.sbcglobal.net) |
| 07:08:07 | | Join Shaid [0] (i=adam@lurking.shaid.net) |
| 07:08:07 | | Join tha [0] (i=1038@ccc2.rbg.informatik.tu-darmstadt.de) |
| 07:08:07 | | Join FlynDice [0] (n=FlynDice@c-24-19-225-90.hsd1.wa.comcast.net) |
| 07:08:07 | | Join freqmod_gq [0] (n=quassel@dhcp208-240.ed.ntnu.no) |
| 07:08:07 | | Join linuxguy3 [0] (n=timj@adsl-75-57-190-229.dsl.emhril.sbcglobal.net) |
| 07:08:07 | | Join HBK [0] (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
| 07:08:07 | | Join CIA-85 [0] (n=CIA@208.69.182.149) |
| 07:08:07 | | Join Torne [0] (i=torne@rockbox/developer/Torne) |
| 07:08:07 | | Join jvd [0] (n=syscrash@poipu/developer/syscrash) |
| 07:08:07 | | Join Zambezi [0] (i=Zulu@80.67.9.2) |
| 07:08:07 | | Join lyngaas [0] (n=staale@19.81-167-149.customer.lyse.net) |
| 07:08:07 | | Join Kopfgeldjaeger [0] (n=nicolai@monitor-mode-enabled-on-mon0.phy0.de) |
| 07:08:07 | | Join courtc [0] (n=court@unaffiliated/courtc) |
| 07:08:07 | | Join SUSaiyan [0] (n=SUSaiyan@cc84863-b.zwoll1.ov.home.nl) |
| 07:08:07 | | Join rvvs89 [0] (n=ivo@pdpc/supporter/base/rvvs89) |
| 07:08:07 | | Join advcomp2019 [0] (n=advcomp2@unaffiliated/advcomp2019) |
| 07:08:07 | | Join tchan [0] (n=tchan@lunar-linux/developer/tchan) |
| 07:08:07 | | Join explore [0] (n=msparker@pool-173-57-92-51.dllstx.fios.verizon.net) |
| 07:08:07 | | Join crashd [0] (i=foobar@lostnode.org) |
| 07:08:07 | | Join sbhsu [0] (n=a6530466@Zion.dorm.au.edu.tw) |
| 07:08:07 | | Join preglow [0] (n=thomj@tvilling2.pvv.ntnu.no) |
| 07:08:07 | | Join crwl [0] (n=crwlll@a91-156-100-168.elisa-laajakaista.fi) |
| 07:08:07 | | Join thegeek [0] (n=nnscript@s168c.studby.ntnu.no) |
| 07:08:07 | | Join JdGordon [0] (n=jonno@rockbox/developer/JdGordon) |
| 07:08:07 | | Join kkurbjun [0] (n=kkurbjun@c-98-245-170-51.hsd1.co.comcast.net) |
| 07:08:07 | | Join GodEater [0] (n=bibble@rockbox/staff/GodEater) |
| 07:08:07 | | Join markun [50] (n=markun@rockbox/developer/markun) |
| 07:08:07 | | Join ej0rge [0] (n=alhaz@alhaz.fttp.xmission.com) |
| 07:08:07 | | Join Bagder [241] (n=daniel@rockbox/developer/bagder) |
| 07:08:07 | | Join gitster [0] (n=user@ip68-225-240-211.oc.oc.cox.net) |
| 07:08:07 | | Join togetic [0] (n=togetic@unaffiliated/ibuffy) |
| 07:08:07 | | Join Tuplanolla [0] (n=jani@unaffiliated/tuplanolla) |
| 07:08:07 | | Join Zarggg [0] (n=zarggg@65-78-69-194.c3-0.eas-ubr6.atw-eas.pa.cable.rcn.com) |
| 07:08:07 | | Join Overand [0] (i=overand@crappy.domain.name) |
| 07:08:07 | | Join AB3JU [0] (n=dz@alt.dissonance.nl) |
| 07:08:07 | | Join elcan [0] (i=user36@pr0.us) |
| 07:08:07 | | Join J-23 [0] (n=zelazko@unix.net.pl) |
| 07:08:07 | | Join rasher [50] (n=rasher@rockbox/developer/rasher) |
| 07:08:07 | | Join jhulst [0] (n=jhulst@jhulst.com) |
| 07:08:07 | | Join Xerion [0] (i=xerion@82-170-197-160.ip.telfort.nl) |
| 07:08:07 | | Join Hadaka [0] (n=naked@83.246.72.254) |
| 07:08:07 | | Join jasio [0] (n=yann@cpc2-rdng20-2-0-cust902.15-3.cable.virginmedia.com) |
| 07:08:07 | | Join jon-kha [0] (i=jon-kha@83.150.91.127) |
| 07:08:07 | | Join blithe [0] (n=blithe@blakesmith.me) |
| 07:08:07 | | Join jordan` [0] (n=jordan@78.235.252.137) |
| 07:08:07 | | Join xavieran_ [0] (n=xavieran@ppp118-209-209-209.lns20.mel6.internode.on.net) |
| 07:08:07 | | Join Unhelpful [0] (n=quassel@rockbox/developer/Unhelpful) |
| 07:08:07 | | Join Tristan [0] (i=tristan@66.252.24.153) |
| 07:08:07 | | Join ThomasAH [0] (n=thomas@aktaia.intevation.org) |
| 07:08:07 | | Join jds2001 [0] (n=jds2001@fedora/jds2001) |
| 07:08:07 | | Join beta2k [0] (n=beta@24.36.68.97) |
| 07:08:07 | | Join dionoea [0] (n=dionoea@videolan/developer/dionoea) |
| 07:08:07 | | Join pjm0616 [0] (n=user@61.250.113.98) |
| 07:08:07 | | Join topik [0] (i=awesome@wtf.grmpf.org) |
| 07:08:07 | | Join rphillips [0] (n=rphillip@66-90-184-91.dyn.grandenetworks.net) |
| 07:08:07 | | Join r00s [0] (n=ru@zentrale.profitables.biz) |
| 07:08:07 | | Join fish_ [0] (n=fish@freigeist.org) |
| 07:08:07 | | Join lostlogic [50] (n=lostlogi@rockbox/developer/lostlogic) |
| 07:08:07 | | Join chaos [0] (n=ch4os@gentoo/user/ch4os) |
| 07:08:07 | | Join zu_ [0] (n=zu@bucketheaded.eu) |
| 07:08:07 | | Join fxb__ [0] (n=felixbru@85.214.97.64) |
| 07:08:07 | | Join Kohlrabi [0] (n=Kohlrabi@frustrum.nosebud.de) |
| 07:08:07 | | Join shodanX [0] (n=shodanX@jazz.informatik.uni-erlangen.de) |
| 07:08:07 | | Join YPSY [0] (n=ypsy@87.106.45.183) |
| 07:08:07 | | Join Dhraakellian [0] (n=ntryon@cpe-72-226-197-191.rochester.res.rr.com) |
| 07:08:07 | | Join Slasheri [0] (i=miipekk@rockbox/developer/Slasheri) |
| 07:08:07 | | Join @ChanServ [0] (ChanServ@services.) |
| 07:08:40 | Ctcp | Version from freenode-connect!freenode@freenode/bot/connect |
| 07:08:40 | *** | Server message 477: 'logbot #rockbox :[freenode-info] if you're at a conference and other people are having trouble connecting, please mention it to staff: http://freenode.net/faq.shtml#gettinghelp' |
| 07:20:12 | | Part toffe82_ |
| 07:29:18 | | Quit bluebrother (Nick collision from services.) |
| 07:29:21 | | Join bluebroth3r [0] (n=dom@rockbox/developer/bluebrother) |
| 07:47:42 | | Quit Res1 (Read error: 60 (Operation timed out)) |
| 07:51:13 | | Join Horschti [0] (n=Horscht2@xbmc/user/horscht) |
| 08:00 |
| 08:00:52 | | Quit BHSPitLappy (Remote closed the connection) |
| 08:04:06 | | Join Rob2222 [0] (n=Miranda@p4FDCDAB0.dip.t-dialin.net) |
| 08:06:10 | | Join AndyI [0] (n=pasha_in@212.14.205.32) |
| 08:10:14 | amiconn | Unhelpful: In fact all standard screens should react both on main + remote buttons |
| 08:10:17 | | Quit Horscht (Read error: 110 (Connection timed out)) |
| 08:11:25 | amiconn | Imo it's not a problem if some actions aren't possible on both keypads due to lack of buttons. This is normal - on a target with a simple remote, only the most frequent actions need to be controllable from the remote |
| 08:13:19 | amiconn | examples: Archos remote with just 6 buttons and no release, repeat, or simultaneous presses, hence max. 6 actions, or also the H300 simple remote (5 buttons + pure hardware hold) |
| 08:22:43 | | Quit Rob2223 (Read error: 110 (Connection timed out)) |
| 08:23:22 | | Quit JdGordon| ("Miranda IM! Smaller, Faster, Easier. http://miranda-im.org") |
| 08:44:34 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
| 08:50:30 | | Join Grahack [0] (n=chri@ACaen-156-1-117-170.w90-17.abo.wanadoo.fr) |
| 09:00 |
| 09:00:16 | | Join petur [0] (n=peter@d54C6F9B2.access.telenet.be) |
| 09:05:44 | | Quit parafin (Nick collision from services.) |
| 09:05:53 | | Join parafin [0] (i=parafin@paraf.in) |
| 09:08:06 | *** | Saving seen data "./dancer.seen" |
| 09:10:15 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 09:23:12 | Unhelpful | amiconn: what's the right way to use get_action* to achieve that? it looks to me like you can either use a context with or without |CONTEXT_REMOTE... |
| 09:44:24 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 09:48:55 | | Quit parafin (Read error: 60 (Operation timed out)) |
| 09:49:00 | | Quit faemir (Read error: 110 (Connection timed out)) |
| 09:49:27 | | Join parafin [0] (i=parafin@paraf.in) |
| 09:52:04 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
| 09:55:18 | | Quit Grahack (Read error: 110 (Connection timed out)) |
| 10:00 |
| 10:04:27 | amiconn | Unhelpful: You don't tackon |CONTEXT_REMOTE explicitly. get_action_worker does that if it receives a remote button press |
| 10:04:35 | amiconn | See apps/action.c |
| 10:10:00 | | Quit GodEater (Remote closed the connection) |
| 10:13:44 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 10:13:53 | | Join flydutch [0] (n=flydutch@host248-201-dynamic.15-87-r.retail.telecomitalia.it) |
| 10:15:06 | | Quit parafin (Read error: 60 (Operation timed out)) |
| 10:16:42 | | Join Grahack [0] (n=chri@ACaen-156-1-122-103.w90-51.abo.wanadoo.fr) |
| 10:16:54 | | Join parafin [0] (i=parafin@paraf.in) |
| 10:21:36 | | Join lifeless_ [0] (n=lifeless@188.16.95.54) |
| 10:27:16 | | Quit phanboy4 ("Leaving") |
| 10:27:30 | Unhelpful | amiconn: ahhhh. that explains some things. so by using CONTEXT_TREE alone i *will* be getting any remote-button mappings, and i just need to add mappings for remote buttons, if appropriate, for any new actions that i add? |
| 10:48:10 | pixelma | a 3 column button table with the first column blank looks a bit... silly :\ |
| 10:59:46 | | Join Mr_XP [0] (n=Brunzel@chello084113243005.3.14.vie.surfer.at) |
| 11:00 |
| 11:00:09 | | Part Mr_XP |
| 11:02:52 | | Join robin0800 [0] (n=quassel@general-ld-216.t-mobile.co.uk) |
| 11:08:08 | *** | Saving seen data "./dancer.seen" |
| 11:14:36 | | Quit n1s ("Lämnar") |
| 11:19:29 | | Join Utchybann [0] (n=lolo@ede67-1-81-56-102-26.fbx.proxad.net) |
| 11:27:03 | bluebroth3r | AlexP: ok, I've retested beastpatcher and used a different machine than the one used for development. Something seems to be broken with the binary :( |
| 11:27:18 | bluebroth3r | at least I can reproduce the issue as reported in the forums |
| 11:27:21 | | Nick bluebroth3r is now known as bluebrother (n=dom@rockbox/developer/bluebrother) |
| 11:27:47 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 11:29:31 | | Join evilnick [0] (n=evilnick@ool-4571af51.dyn.optonline.net) |
| 11:34:05 | | Quit evilnick__ (Read error: 60 (Operation timed out)) |
| 11:48:00 | CIA-85 | New commit by bluebrother (r23458): Adjust mknkboot sources path to recently changed location. |
| 12:00 |
| 12:15:26 | | Quit Grahack (Client Quit) |
| 12:18:29 | | Join domonoky [0] (n=domo@g229117248.adsl.alicedsl.de) |
| 12:23:00 | | Join GeekShadow [0] (n=Antoine@reactos/tester/GeekShadow) |
| 12:26:34 | bluebrother | anyone with beast and windows around? I'd like to verify something regarding beastpatcher. |
| 12:27:28 | | Quit Horschti (Read error: 104 (Connection reset by peer)) |
| 12:33:59 | | Join Horscht [0] (n=Horscht2@xbmc/user/horscht) |
| 12:34:22 | | Quit ender` (" Do not believe any statistic you didn't falsify yourself.") |
| 12:37:23 | | Quit domonoky () |
| 12:43:14 | | Quit mc2739 (Read error: 60 (Operation timed out)) |
| 12:43:47 | | Join mc2739 [0] (n=mc2739@rockbox/developer/mc2739) |
| 12:51:00 | | Join domonoky [0] (n=Domonoky@g229117248.adsl.alicedsl.de) |
| 12:53:46 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 12:54:17 | | Quit stripwax (Client Quit) |
| 12:57:19 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 12:57:19 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 12:57:41 | | Nick fxb__ is now known as fxb (n=felixbru@85.214.97.64) |
| 12:59:08 | kugel | JdGordon: I've had a fix too..... |
| 13:00 |
| 13:02:15 | | Join MoMonster [0] (n=4db3cb07@giant.haxx.se) |
| 13:03:34 | | Quit TopyMobile (Read error: 110 (Connection timed out)) |
| 13:04:01 | MoMonster | Is there a way to adjust the volume using +/- buttons on a touchscreen wps? All I found was a volumebar which stes the volume depending on where you press it |
| 13:08:10 | *** | Saving seen data "./dancer.seen" |
| 13:12:39 | | Quit antil33t (Read error: 104 (Connection reset by peer)) |
| 13:12:47 | | Join antil33t [0] (n=Mudkips@119.224.12.185) |
| 13:14:29 | | Join casainho [0] (n=chatzill@bl15-106-33.dsl.telepac.pt) |
| 13:16:03 | casainho | /msg NickServ identify chave |
| 13:16:41 | bluebrother | casainho: you might want to change your nickserv password ... |
| 13:16:44 | stripwax | casainho hmm.. |
| 13:17:33 | casainho | bluebrother: hello :-) −− why should I change the password? |
| 13:17:47 | linuxstb | See the bottom of http://www.rockbox.org/irc/current.txt |
| 13:17:51 | bluebrother | casainho: because you just put it to the public? |
| 13:18:47 | casainho | bluebrother: ah. Oh, it was just a space before and system didn't accepted it |
| 13:19:15 | casainho | bluebrother: anyway, I will leave like that, no one will want to use my nick, there is no point |
| 13:19:28 | casainho | bluebrother: thanks. |
| 13:19:33 | linuxstb | casainho: You shouldn't type nickserv commands in the #rockbox window - type them in the server window. (just in case of mistakes like that). |
| 13:20:10 | bluebrother | btw, most irc clients allow you to automatically authenticate. |
| 13:21:47 | casainho | bluebrother: ok, thanks. |
| 13:22:24 | pixelma | casainho: maybe you wondered why you got disconnected right after the last time this happened to you... that's what someone can do if he has your password (and this was just a very simple demonstration, there are other people who would use it for something else) |
| 13:22:54 | casainho | pixelma: ok, I will chanhe now |
| 13:35:28 | | Quit casainho (Nick collision from services.) |
| 13:35:37 | petur | he lied :) |
| 13:35:51 | | Join casainho [0] (n=chatzill@bl15-106-33.dsl.telepac.pt) |
| 13:36:07 | petur | so, when are you going to change it? |
| 13:37:16 | | Join bubsy_ [0] (n=bubsy@94.139.72.137) |
| 13:38:42 | | Join Omlet [0] (i=omlet05@91.182.43.181) |
| 13:38:42 | | Join Omlet05 [0] (i=omlet05@91.182.43.181) |
| 13:39:07 | | Quit Omlet (Client Quit) |
| 13:40:36 | | Join Bob_C_ [0] (n=chatzill@host86-144-201-0.range86-144.btcentralplus.com) |
| 13:41:18 | | Join MethoS- [0] (n=clemens@134.102.106.250) |
| 13:41:35 | MoMonster | Is there a way to adjust the volume using +/- buttons on a touchscreen wps? All I found was a volumebar which stes the volume depending on where you press it |
| 13:44:00 | | Quit bubsy (Read error: 60 (Operation timed out)) |
| 13:47:59 | | Join teru [0] (n=teru@KD059133112132.ppp.dion.ne.jp) |
| 13:48:08 | | Quit casainho ("ChatZilla 0.9.85 [Firefox 3.5.4/20091028153816]") |
| 13:49:20 | | Nick bubsy_ is now known as bubsy (n=bubsy@94.139.72.137) |
| 13:59:09 | | Join casainho [0] (n=chatzill@bl15-106-33.dsl.telepac.pt) |
| 14:00 |
| 14:01:11 | | Join adiroiban [0] (n=adiroiba@h194-54-129-79.teleson.ro) |
| 14:04:24 | | Quit casainho ("ChatZilla 0.9.85 [Firefox 3.5.4/20091028153816]") |
| 14:05:38 | | Quit stripwax (Read error: 54 (Connection reset by peer)) |
| 14:08:44 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 14:08:59 | CIA-85 | New commit by kugel (r23459): Cleanup includes a bit. |
| 14:13:38 | | Join casainho [0] (n=chatzill@bl15-106-33.dsl.telepac.pt) |
| 14:15:42 | bluebrother | tomers: *please* talk about things first before filing bugs. Some things *are* intentional the wey they are. Doesn't mean that this can't be questioned, but it's really not helpful to add tasks for everything that seems wrong at first. |
| 14:16:23 | | Quit AlexP (Read error: 113 (No route to host)) |
| 14:17:02 | | Join tomers [0] (n=86bfe845@giant.haxx.se) |
| 14:17:19 | tomers | bluebrother: Do you mean the langs not getting compiled? |
| 14:17:58 | bluebrother | tomers: yes. That was intentional −− what's the point in delaying a debug build with translations? They are also not added to the resources for debug builds |
| 14:18:22 | | Join KBH [0] (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
| 14:19:23 | tomers | Is that so bad to post an invalid bug? |
| 14:20:09 | tomers | You can just close it as 'not a bug' with that explanation... but I'll ask first on IRC next time :-) |
| 14:20:54 | bluebrother | didn't I close it as "not a bug"? |
| 14:21:39 | tomers | yep... just noticed :-) |
| 14:22:19 | | Quit petur ("real life") |
| 14:22:32 | bluebrother | well, having such things around is in general more work than simply explaining it. Plus (and that's the reason I really don't like having too much tasks) it helps loosing the overview of open and important tasks. |
| 14:22:37 | tomers | can I do distclean, then compile normally, then qmake -config dbg and compile again, so that I can use the app with language files? |
| 14:22:53 | tomers | ok. understood |
| 14:23:52 | bluebrother | I don't think there's a distclean target, but you don't need that at all. make clean && qmake && make is sufficient. Or, if you want to be sure, just delete the output folder. In case of in-tree building it's simÃply the build/ folder. You need to run qmake after you deleted that folder though. |
| 14:24:02 | kugel | arg, stupid macro :/ |
| 14:24:19 | | Quit HBK (Read error: 60 (Operation timed out)) |
| 14:24:26 | | Nick KBH is now known as HBK (n=hbk@rrcs-97-77-51-170.sw.biz.rr.com) |
| 14:25:18 | tomers | bluebrother: What do you asy regarding 'FS #10747 - Disable actions if rbutil is not configured properly' |
| 14:25:24 | tomers | ? |
| 14:25:37 | tomers | s/asy/say/ |
| 14:28:04 | bluebrother | let me check the diff. Haven't gotten around that yet, beastpatcher is acting up :( |
| 14:28:57 | kugel | hrm, why does the red hit so little targets? |
| 14:29:10 | CIA-85 | New commit by BobC (r23460): Fix debug/multidrive build error |
| 14:29:23 | kugel | the macro should expand for almost every target |
| 14:31:13 | pixelma | many of those are new ports, maybe something that was forgotten there? |
| 14:36:33 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 14:36:54 | CIA-85 | New commit by kugel (r23461): Remove a wps function call from skin engine, hopefully the last one. |
| 14:36:59 | CIA-85 | New commit by kugel (r23462): Fix red - statusbar_position() is a macro. |
| 14:41:29 | * | bertrik realises that meizu M6 firmwares are the same for models M6SP and M6SL |
| 14:43:17 | * | teru found skin_display.c includes abrepeats.h and abrepeats.h includes settings.h if ab repeat is enabled. |
| 14:43:50 | | Join webguest64 [0] (n=57b9da10@giant.haxx.se) |
| 14:44:05 | kugel | teru: ah, that explains things, thanks |
| 14:44:27 | | Quit webguest64 (Client Quit) |
| 14:48:55 | | Quit stripwax (Read error: 104 (Connection reset by peer)) |
| 14:48:59 | bluebrother | tomers: looks ok to me so far. There's only one (really minor) thing: the first hunk swaps two lines with is simply unnecessary. |
| 14:49:54 | bluebrother | though I'm not sure if disabling the actions menu is a good idea for screen readers. Haven't checked though. |
| 14:50:02 | MoMonster | Is there a way to adjust the volume using +/- buttons on a touchscreen wps? All I found was a volumebar which stes the volume depending on where you press it |
| 14:51:14 | bertrik | I was afraid that a M6 rockbox port would have to support stuff like dual-codec and two display drivers, but M6SP and M6SL can probably each just use a single codec and a single LCD driver in rockbox firmware |
| 14:52:12 | bertrik | AMS sansa USB is something nice to work on, I think I am able to make that work (given sufficient time) |
| 14:52:24 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 14:52:41 | bertrik | bughunter2 was already working on that, but AFAIK he has no prior USB experience |
| 14:55:04 | tomers | bluebrother: OK. I'll work on it later. I'll fix this minor issue, and test with a screen reader today (NVDA). |
| 14:55:37 | funman | bertrik: I suppose having a Sansa AMS connected to computer with JTAG would help a lot into reverse engineering the USB initialization |
| 14:56:13 | funman | there were at least 2 successful JTAG connections made but all the players eventually broke |
| 15:00 |
| 15:00:17 | bluebrother | tomers: I guess that reading the menu entry itself won't be a problem. The question I have is if it will confuse blind users as the menu "won't work". I guess answering this would kinda require input from someone who actually uses a screenreader on a daily basis. |
| 15:00:56 | bluebrother | btw, nvda had less issues with reading the tabs than Jaws had. ItÄ's been a while since I tried that, though. |
| 15:01:01 | kugel | amiconn: ping |
| 15:01:23 | tomers | bluebrother: As far as I remember, the screen reader says it is disabled |
| 15:02:37 | bluebrother | tomers: in that case I guess it shouldn't be an issue. |
| 15:02:54 | bertrik | funman, I think we just need a good debugging screen for USB, and a lot of patience :P |
| 15:03:39 | funman | debugging screen = dump registers content in debug menu ? |
| 15:07:25 | | Join jerl92 [0] (n=irchon@ip-66-254-35-19.mqdsl.megaquebec.net) |
| 15:07:48 | | Quit jerl92 (Remote closed the connection) |
| 15:08:12 | *** | Saving seen data "./dancer.seen" |
| 15:10:55 | | Join krazykit [0] (n=kkit@cpe-24-210-70-248.columbus.res.rr.com) |
| 15:11:05 | | Join jerl92 [0] (n=irchon@ip-66-254-35-19.mqdsl.megaquebec.net) |
| 15:11:09 | * | domonoky detects that his failing plugin load on mini2440 is the first call to sd_read with incount > 1, at least in the main-binary... lets check the multiple sector handling.. |
| 15:12:15 | | Quit jerl92 (Client Quit) |
| 15:12:15 | kugel | every other read only reads 1 sector? |
| 15:12:56 | | Quit tomers ("CGI:IRC") |
| 15:13:21 | domonoky | yes.. i made sure with a call to panicf in sd_read if incount > 1 |
| 15:15:25 | CIA-85 | New commit by BobC (r23463): Replace hardcoded constants (fix 32/64MB bug) |
| 15:16:33 | | Join jerl92 [0] (n=irchon@ip-66-254-35-19.mqdsl.megaquebec.net) |
| 15:17:59 | * | funman just recorded his own voice on a Sansa Fuze |
| 15:18:28 | domonoky | hm, incount is 11, so also bigger then UNALIGNED_NUM_SECTORS. So maybe the problem also happens when it uses two transfers... |
| 15:18:32 | kugel | funman: \o/ |
| 15:18:36 | domonoky | funman: nice ! |
| 15:18:37 | | Quit jerl92 (Remote closed the connection) |
| 15:19:54 | funman | http://pastie.org/678826 < base64 encoded mp3 of the recording : "hello, how are you ?" |
| 15:22:04 | Bob_C_ | domonoky: which issue are you looking at? |
| 15:22:53 | kugel | domonoky: I guess you didn't see it coming that test_disk exposes bugs in that way :) |
| 15:23:12 | domonoky | Bob_C: i enabled plugins, but the plugin buffer only contains FFFF...FFF after the read.. |
| 15:23:30 | domonoky | kugel: i still can load test_disk itself :-) |
| 15:23:34 | domonoky | can not |
| 15:23:53 | Bob_C_ | The FF..s are caused by the 32/64MB mapping bug. The latest r23463 should fix this issue |
| 15:24:23 | | Quit stripwax ("Miranda IM! Smaller, Faster, Easier. http://miranda-im.org") |
| 15:24:28 | domonoky | will try that :-) |
| 15:24:38 | kugel | domonoky: I mean failing to load test_disk is the new way of test_disk reporting plugins |
| 15:24:48 | kugel | s/plugins/errors/ |
| 15:24:49 | | Join DerPapst [0] (n=DerPapst@p4FE8FBBD.dip.t-dialin.net) |
| 15:25:41 | CIA-85 | New commit by teru (r23464): Show insert_last_shuffled item and queue_last_shuffled item when playlist file is selected. ... |
| 15:26:13 | kugel | Bob_C_: I still think crt0.S should just call the C version |
| 15:26:39 | Bob_C_ | I tried that, but there is no stack setup so crt0.s can only call leaf functions |
| 15:26:44 | | Join GodEater [0] (n=bibble@bb-87-80-121-64.ukonline.co.uk) |
| 15:27:14 | kugel | ah I see |
| 15:28:13 | * | kugel wonders how it works on other targets then |
| 15:28:44 | funman | usually stack setup is done in crt0.S |
| 15:28:48 | kugel | well, memory_init() is just a wrapper around other asm functions |
| 15:29:01 | Bob_C_ | The s3c2440 crt0.s version is frankly pretty weird, it copies itself to phys RAM and remaps, then jumps to virt addr |
| 15:29:27 | Bob_C_ | Basically a can of worms... |
| 15:29:52 | kugel | funman: other targets do memory_init() before stack setup |
| 15:30:11 | Bob_C_ | How does push lr work then?? |
| 15:30:19 | kugel | Bob_C_: maybe due to F/X being run from OF-environment |
| 15:30:48 | funman | you could use say 512 bytes of memory for a stack only used in early init |
| 15:31:27 | Bob_C_ | sure, it's probably due to OF, but I wonder if it has to be so unconventional |
| 15:32:47 | domonoky | Bob_C: you where right.. i can load test_disk.c now :-) |
| 15:33:21 | Bob_C_ | I had the same problem loading codecs |
| 15:34:24 | kugel | Bob_C_: I think it works because memory_init() is not called from the bootloader, and the bootloader sets up some stack |
| 15:36:03 | kugel | ie. even though running in the main binary, it uses the bootloader stack |
| 15:39:05 | Bob_C_ | so what stack does the bootloader use... |
| 15:41:29 | kugel | crt0.S is used for both, bootloader and main binary |
| 15:41:55 | kugel | so it does the same, except that the addresses are setup app.lds and boot.lds respectively |
| 15:44:28 | Bob_C_ | but boot.lds sets the C stack, which cannot be initialised until the MMU has been setup... |
| 15:44:51 | kugel | no, the MMU doesn't have to be set up |
| 15:44:59 | Bob_C_ | Yes it does |
| 15:45:10 | kugel | you can operate without it |
| 15:45:41 | Bob_C_ | Not when NOR flash is mapped where you are expecting RAM to be |
| 15:46:29 | kugel | surely you can only use physical addresses without mmu |
| 15:47:02 | | Quit AndyI () |
| 15:47:17 | Bob_C_ | Err.... exactly. Boot.lds is the virtual address map |
| 15:48:38 | kugel | nor flash is at 0x0? |
| 15:48:54 | Bob_C_ | physical address yes |
| 15:49:16 | kugel | can we remap if we wanted to access it from rockbox? |
| 15:50:59 | Bob_C_ | Sure, if a VM section is mapped to address 0 |
| 15:53:39 | | Join AndyI [0] (n=pasha_in@212.14.205.32) |
| 15:55:33 | * | domonoky made a successfull test_disk.c run with SD_DEBUG undefined. 300MB successfully written and compared, so at least for me (and my sd-card) the driver works flawlessly without debug output. (But it still pretty slow). |
| 15:56:14 | Bob_C_ | What happens if you remove the calls to mci_delay? |
| 15:56:34 | domonoky | will try.. |
| 15:57:12 | Bob_C_ | There is a call to sd_wait_for_state which has a #if 0... I think that should be reinstated |
| 16:00 |
| 16:00:51 | Bob_C_ | The error I encountered is that a data transfer command fails with a "Data timeout" status bit set |
| 16:01:04 | | Join AlexP [0] (n=alex@rockbox/staff/AlexP) |
| 16:01:11 | mt | saratoga: ping |
| 16:03:02 | AlexP | bluebrother: That's a bugger |
| 16:03:56 | AlexP | I can find a windows meachine in ~2 hours if needs be |
| 16:05:15 | funman | do you use DMA for SD on mini2440 ? |
| 16:05:33 | Bob_C_ | Yes we do |
| 16:08:21 | domonoky | Bob_C if i remove the mci_delay at the end of send_cmd it doesnt work anymore. but it looks like the delay at the beginning is not needed (running a test_disk test again now) |
| 16:09:23 | funman | the Clip OF records mono at 24kHz on the microphone |
| 16:10:42 | Bob_C_ | I wonder if the problem is that the previous DMA transfer is not complete before the next command is started |
| 16:11:28 | funman | 24kHz for the radio as well |
| 16:11:46 | funman | Bob_C_: don't you block until the transfer finished ? |
| 16:11:57 | Bob_C_ | does the test_disk plugin do a write/read/verify pattern? |
| 16:12:08 | funman | yes |
| 16:12:22 | funman | there are 2 entries: write/read/verify and performance |
| 16:12:26 | domonoky | Bob_C: yes, with random sizes alignements etc, so should be a good test. |
| 16:13:06 | Bob_C_ | The driver waits for an interrupt from the SD controller to indicate transfer has finished. |
| 16:14:43 | funman | if the sd mutex is locked then there shouldn't be other requests in the meantime |
| 16:14:53 | Bob_C_ | This driver was adapted from the SD as3525 driver, so I used that as a model |
| 16:14:59 | domonoky | test passed successfully, with only the mci_delay at the end of send_cmd() |
| 16:16:42 | funman | Bob_C_: be careful, i wrote a lot of this driver and let a lot of bugs in it ;) they might have all been fixed by FlynDice but you never know .. |
| 16:17:52 | domonoky | the as3525 also does some delay in send_cmd() if the command is the enable command. maybe thats what missing the s3c2440 sd driver ? (me starts some experiments) |
| 16:17:59 | Bob_C_ | I did make an assumption there... |
| 16:19:30 | domonoky | also my sd-card is a small non-sdhc card, so results for other cards may differ :-) |
| 16:19:34 | | Quit teru ("Quit") |
| 16:22:06 | Bob_C | I have tried a 2GB sd and a 4GB SDHC and I got similar results I think |
| 16:25:33 | | Quit TheSeven (Nick collision from services.) |
| 16:25:49 | domonoky | interestingly it doesnt matter if if remove the delay from the beginning or from the end of send_cmd(). As long as one is there, it works. So that clearly hints that we have to somehow wait for something to finish. |
| 16:25:52 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 16:26:04 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 16:28:15 | Bob_C_ | I compare all the SDI controller regs before and after the delay, and they appear identical, so I conclude the problem is somewhere else, e.g. DMA, the card itself, or "something else" |
| 16:34:16 | | Quit krazykit (Remote closed the connection) |
| 16:34:53 | | Join gingermig [0] (n=gingermi@136.84-49-195.nextgentel.com) |
| 16:37:08 | FlynDice | domonoky: re sd card slow, Do you enable 4bit widebus in the controller also?. I see the card gets the command but I can't tell if the controller does. |
| 16:37:54 | domonoky | FlynDice: no i first wanted to make sure it works fine, before trying to make it speedy :-) |
| 16:38:31 | casainho | FlynDice: do you have the Mini2440 board also? |
| 16:39:39 | Bob_C_ | I set the S3C2410_SDIDCON_WIDEBUS which means 4 bit I hope... |
| 16:39:41 | FlynDice | casainho: No, I just saw that you guys used some of the as3525 code for the sd driver and that's what I've been working with for awhile... |
| 16:40:58 | casainho | FlynDice: do you have the Mini2440 board also?f |
| 16:41:12 | casainho | FlynDice: ok, nice to know ;-) −− Thanks |
| 16:41:38 | | Join toffe82 [0] (n=chatzill@ppp-71-130-76-7.dsl.frs2ca.pacbell.net) |
| 16:42:19 | FlynDice | re widebus: there's also an ACMD42 SET_CLEAR_CARD_DETECT command which disables a pullup resistor on DAT3 for widebus. The linux guys don't seem to think it's needed but i't in the sd spec and a lot of the SD card product manuals. |
| 16:48:34 | CIA-85 | New commit by kugel (r23465): Move find_viewport() and find_image() to skin_parser.c where the reset of the linked-list handling sits. That enables removing duplicated code from ... |
| 16:49:24 | | Join mcuelenaere [0] (n=mcuelena@78-21-191-122.access.telenet.be) |
| 16:54:55 | | Quit robin0800 (Read error: 54 (Connection reset by peer)) |
| 16:55:03 | kugel | "#ifndef __PCTOOL__" of course |
| 16:57:11 | bluebrother | AlexP: it's quite interesting, beastpatcher seems to work fine if WMP11 installed but fails with WMP10. That's quite strange. |
| 16:58:34 | | Join panni_ [0] (i=hannes@ip-95-222-21-143.unitymediagroup.de) |
| 17:00 |
| 17:00:24 | | Part gingermig |
| 17:01:15 | kugel | what is the magic to generate sysfont.h? |
| 17:02:17 | funman | make $PWD/sysfont.h ? |
| 17:04:45 | | Quit casainho (Remote closed the connection) |
| 17:07:33 | domonoky | at least with only one mci_delay at the end of send_cmd(), the transfer rate has gone a good bit up (now 144KB/s -> 1500KB/s depending on size. It was as low as 70KB/s earlier) |
| 17:08:14 | *** | Saving seen data "./dancer.seen" |
| 17:11:22 | Bob_C_ | I imagine it would, mci_delay is quite a long delay |
| 17:11:56 | | Join Paleskin [0] (n=Paleskin@fm-ip-118.137.61.71.fast.net.id) |
| 17:12:08 | | Quit bertrik ("Leaving") |
| 17:12:18 | | Join FOAD_ [0] (n=dok@dinah.blub.net) |
| 17:12:28 | Paleskin | automatic install in rbutil doesn't work on sansa fuze |
| 17:13:33 | | Join bertrik [0] (n=bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
| 17:13:39 | | Quit liar (Read error: 104 (Connection reset by peer)) |
| 17:13:57 | | Quit advcomp2019 (Read error: 104 (Connection reset by peer)) |
| 17:14:03 | | Join Res1 [0] (n=Res@user-0c6s6ib.cable.mindspring.com) |
| 17:14:11 | * | bertrik wonders if it's possible to use a target-specific crt0.s file (not just SoC specific) |
| 17:15:13 | | Join liar [0] (n=liar@83.175.83.185) |
| 17:15:20 | funman | bertrik: everything is possible |
| 17:15:41 | funman | Paleskin: what's the problem ? |
| 17:16:12 | kugel | bertrik: s3c2440 does soc specific, and there's a generic arm one too. it pretty much just depends on SOURCES |
| 17:16:13 | Bob_C_ | bertrik: if the build scripts are changed, yes |
| 17:16:29 | Bob_C_ | the linker scripts are not in SOURCES |
| 17:16:37 | Paleskin | everytime I ran bootloader install, it saiy unrecognized firmware |
| 17:16:53 | Paleskin | but it is the original firmware |
| 17:17:32 | bertrik | I suspect the meizu M6 SP and meizu M6 SL use different kinds of DRAM memory |
| 17:17:37 | | Quit liar (Client Quit) |
| 17:18:00 | TheSeven | great news! |
| 17:18:09 | domonoky | Paleskin: did you try with another version of the OF ? (rbutil might not know all versions, and only allows known, proven to work versions) |
| 17:18:16 | | Quit kugel (Remote closed the connection) |
| 17:18:20 | funman | Paleskin: use an older firmware, last version isn't supported yet by rbutil (in next release it will be) |
| 17:18:56 | | Join robin0800 [0] (n=quassel@general-ld-216.t-mobile.co.uk) |
| 17:18:59 | bertrik | DRAM needs to be initialised in crt0 before we call main, having a separate crt0 for each would be convenient |
| 17:19:05 | Paleskin | I prefer the last of firmware, I forgot how to manually make the bootloader |
| 17:19:25 | funman | Paleskin: then wait next rbutil release |
| 17:19:49 | domonoky | Bob_C: if i put the dump_regs thing around the last sleep in send_cmd() it outputs some differences, but i am still trying to understand what it wants to tell me :-) |
| 17:20:19 | bertrik | I was afraid of it not being defined in SOURCES :) |
| 17:20:30 | Paleskin | well, u mean even manually patch the latest of is not possible ? |
| 17:20:54 | funman | you have to build mkamsboot from svn for that |
| 17:21:17 | Paleskin | yeah, I forgot the command line |
| 17:21:48 | Bob_C_ | domonoky: can you do a pastie? |
| 17:23:21 | domonoky | Bob_C: http://pastebin.com/m5943f41c its always those two registers.. |
| 17:23:27 | | Quit FOAD (Read error: 110 (Connection timed out)) |
| 17:23:27 | | Nick FOAD_ is now known as FOAD (n=dok@dinah.blub.net) |
| 17:24:25 | domonoky | ah, its the data remaining counter and the fifo status.. thats probably dma transfering data after a read/write command ? |
| 17:25:25 | TheSeven | http://img217.imageshack.us/img217/4122/img0969.jpg |
| 17:25:35 | Paleskin | oic |
| 17:25:54 | Paleskin | mkamsboot is specific firmware dependant |
| 17:26:51 | domonoky | Paleskin: sure, it checks md5sums of the firmwares, because its dangerious if something in this goes wrong. |
| 17:27:01 | Bob_C_ | My thought was that it needs to wait for DMA transfer to complete as well as SDI complete, because SDI may signal transaction has been completed before DMA has finished |
| 17:27:42 | Paleskin | then I shall wait for the next util released yet |
| 17:27:43 | Paleskin | sigh |
| 17:29:13 | funman | TheSeven: nice :) |
| 17:29:30 | Bob_C_ | bertrik: just be careful there is a dependency between the lds file and crt0.s |
| 17:29:36 | * | funman would have expected portrait/landscape switch with accelerometers |
| 17:30:06 | TheSeven | no, ibugger doesn't do that. but rockbox should. |
| 17:31:53 | | Part Paleskin |
| 17:32:35 | | Quit dmb ("Leaving") |
| 17:35:53 | funman | i've defined HW_SAMPR_CAPS and REC_SAMPR_CAPS to SAMPR_CAP_ALL, but recording->frequencies only lists 22.05kHz |
| 17:37:48 | bertrik | Bob_C, yes I realise that, thanks |
| 17:38:40 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 17:39:09 | bertrik | I think we can clean up this crt0.s file: there's a #ifdef IPOD_NANO2G inside another #ifdef IPOD_NANO2G for example, and we could probably use #defines for hardware register instead of using the addresses directly |
| 17:43:45 | funman | I have updated the recording Sansa AMS on FS #10371, I'd appreciate if you can test it on c200v2/e200v2/m200v4 and just tell if it records something or if it locks up the player (recording quality is still not here though) |
| 17:45:09 | TheSeven | bertrik: and we could try to finally merge that protection unit code |
| 17:50:19 | | Quit flydutch ("/* empty */") |
| 17:51:41 | | Quit panni_ (Read error: 60 (Operation timed out)) |
| 17:53:48 | | Nick YPSY is now known as Ypsy (n=ypsy@87.106.45.183) |
| 17:53:52 | pixelma | funman: I'm not sure about the AMS chip in the new Sansas but the AMS codec (it is then, I believe) in the v1 models also only allows for recordings with 22.05 kHz |
| 17:54:13 | | Join Grahack [0] (n=chri@ACaen-156-1-122-103.w90-51.abo.wanadoo.fr) |
| 17:54:35 | pixelma | back then jhMikeS said it's a hardware limitation |
| 17:54:45 | funman | pixelma: the recording frequency is completely tunable (OF use 24kHz for example), what I wonder is why I only see 22.05kHz even if I allow all frequencies in the config file |
| 17:55:02 | | Join panni_ [0] (i=hannes@ip-95-222-21-143.unitymediagroup.de) |
| 17:58:40 | | Join liar [0] (n=liar@83.175.83.185) |
| 17:59:58 | AlexP | bluebrother: That is strange - do you think we can get round that, or will we have to require WMP11 for win users (which would be a pain)? |
| 18:00 |
| 18:00:54 | | Quit MoMonster ("CGI:IRC") |
| 18:01:20 | bluebrother | AlexP: well, I think we should get around that though I have no idea what's the problem. I just figured it when trying it on a different machine with WMP10 −− and the two installations with WMP11 I've tried work. |
| 18:01:37 | AlexP | Yes, we should get round it if possible :) |
| 18:01:52 | bluebrother | it could of course also be related to my installation. I hope some of the people with issues in the forums post details about their system setup |
| 18:02:12 | AlexP | bluebrother: In other news, are the edits we discussed last night correct? i.e. need to do the battery switch, plus no error messages? |
| 18:04:41 | bluebrother | AlexP: not really I need to admit :o |
| 18:04:48 | AlexP | hehe :) |
| 18:04:53 | AlexP | So what should it be? |
| 18:05:37 | bluebrother | on linux you get that error message and an immediate reboot. On Windows you don't get that error message but also no reboot. The beast however reboots when disconnecting USB, i.e. you just need to disconnect USB and reconnect immediately. |
| 18:06:14 | | Join stoffel [0] (n=quassel@p57B4C066.dip.t-dialin.net) |
| 18:06:27 | bluebrother | hmm, and there is a WMP11 success post in the forums. But that post is dated in now + 2 mins? |
| 18:06:29 | AlexP | OK, I'll change that in a mo |
| 18:06:38 | * | bluebrother checks local clock |
| 18:07:28 | | Join toens [0] (n=toens@196-209-186-53-tbnb-esr-2.dynamic.isadsl.co.za) |
| 18:08:02 | | Nick Ypsy is now known as YPSY (n=ypsy@87.106.45.183) |
| 18:08:17 | | Nick YPSY is now known as Ypsy (n=ypsy@87.106.45.183) |
| 18:09:48 | | Quit MethoS- (Remote closed the connection) |
| 18:12:10 | | Join tomers [0] (n=chatzill@bzq-84-109-85-100.red.bezeqint.net) |
| 18:16:04 | CIA-85 | New commit by tomers (r23466): FS #10747 - Disable actions if rbutil is not configured properly |
| 18:17:25 | | Quit stripwax (Read error: 104 (Connection reset by peer)) |
| 18:19:36 | | Join saratoga_ [0] (i=463f90ed@gateway/web/freenode/x-todnbbwphtjevkvw) |
| 18:20:00 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 18:20:24 | | Join akur [0] (n=akur@bl7-28-75.dsl.telepac.pt) |
| 18:21:52 | tomers | Is the following change is acceptable as a commit? It is just a cosmetic change... http://pastie.org/679013 |
| 18:24:01 | saratoga_ | mt: ping |
| 18:24:03 | tomers | I need your opinion regarding "FS #10740 - rbutil: Test Cowon D2 OF file for CRC consistency" before patching. Anyone interested in giving me feedback about it? |
| 18:24:27 | mt | saratoga_:pong |
| 18:24:44 | saratoga_ | mt: i saw you asked in the logs |
| 18:25:07 | | Join kugel [0] (n=kugel@rockbox/developer/kugel) |
| 18:25:20 | domonoky | tomers: testing for CRC consistency is always good, but i didnt look at the code till now :-) |
| 18:25:21 | mt | Yes .. I might work more tonight on the fft stuff. Anything new that I should be aware of ? |
| 18:25:30 | mt | saratoga_:^ |
| 18:25:51 | saratoga_ | mt: I only played with it for a little while, but could not get correct output, i'm not sure what i was doing wrong |
| 18:26:13 | saratoga_ | been meaning to look at it more |
| 18:26:28 | mt | Are you free for a minute or so, to send me the inputs you were using so that I can test ? |
| 18:27:29 | | Join froggyman [0] (n=sopgenor@pool-72-69-220-194.chi01.dsl-w.verizon.net) |
| 18:27:47 | saratoga_ | i think i have them on this machine, let me check |
| 18:29:06 | | Join MoMonster [0] (n=4db3cb07@giant.haxx.se) |
| 18:29:07 | | Join ukaszs2 [0] (i=ukaszs2@77.255.110.82) |
| 18:29:14 | CIA-85 | New commit by alex (r23467): Correct beast manual install instructions in Windows. |
| 18:29:28 | domonoky | tomers: for me this crc check code looks fine :-) But why do we have to wrap telechips_test_crc() ? |
| 18:30:57 | MoMonster | Hi! Is there a way to adjust the volume using +/- buttons in touchscreen-WPS? The only thing I found was the volume bar, which sounds like a dangerous idea to me |
| 18:31:12 | domonoky | bluebrother: about this WMP11/10 problem. Afaik WMP comes with the MTP Drivers, so it may be a problem with specific MTP Driver versions ? |
| 18:31:25 | bluebrother | tomers: the first part in telechips.c simply renames a variable without any obvious reason. Why? |
| 18:31:44 | bluebrother | domonoky: could be. Haven't investigated the issue further as of now. |
| 18:32:14 | bluebrother | at least it works on Vista. Just tried it a bit earlier, but Vista comes with WMP11. |
| 18:32:38 | domonoky | and surely another version of the MTP driver :-) |
| 18:32:40 | mcuelenaere | MoMonster: you mean physical +/- buttons or touchscreen areas/viewports? |
| 18:33:03 | bluebrother | domonoky: of course :) |
| 18:33:36 | MoMonster | mcuelenaere: I mean touchscreen areas |
| 18:33:38 | saratoga_ | mt: http://pastebin.ca/1651732 |
| 18:33:55 | saratoga_ | its basically the code you sent me with trivial changes |
| 18:34:11 | bluebrother | tomers: also, test_firmware_tcc() seems somewhat superfluous to me. |
| 18:34:53 | bluebrother | or are you planning to add more checks in that function later? |
| 18:35:23 | mcuelenaere | MoMonster: http://www.rockbox.org/wiki/CustomWPS lists all current WPS tags, in-/decreasing the volume doesn't seem to be listed among those; so someone will need to implement that |
| 18:35:57 | domonoky | MoMonster: looks like it isnt possible at moment, that would need a volume+ and volume- action for touchscreen areas.. But feel free to improve the code and provide a patch :-) |
| 18:36:03 | mt | saratoga_: I can't spot changes in the input, are there any ? |
| 18:36:22 | saratoga_ | mt: no i left that alone |
| 18:36:29 | saratoga_ | it works for you? |
| 18:37:24 | mt | Haven't tested yet, was just looking at the code. |
| 18:37:35 | saratoga_ | your original code worked though? |
| 18:39:41 | mcuelenaere | MoMonster: http://pastebin.com/d299d0cbd should work (untested) |
| 18:41:13 | mt | saratoga_: Yes, my original code worked. the 3 outputs were almost exact. |
| 18:41:30 | saratoga_ | maybe its something wrong with my computer then |
| 18:41:49 | mt | I sent you a plot of the error iirc ? |
| 18:41:58 | saratoga_ | thats right you did |
| 18:42:19 | saratoga_ | could you rerun your code and verify that you get good output, then send it to me? |
| 18:42:35 | | Quit Omlet05 (Read error: 113 (No route to host)) |
| 18:43:17 | | Quit freqmod_gq (Read error: 60 (Operation timed out)) |
| 18:44:18 | | Part froggyman |
| 18:46:25 | | Join Omlet [0] (i=omlet05@119.145-241-81.adsl-dyn.isp.belgacom.be) |
| 18:48:15 | MoMonster | Thank you for your help. Too bad I don't know how to code and build patches and all that. I'm just trying to make a theme :/ |
| 18:49:22 | CIA-85 | New commit by tomers (r23468): rbutil: Fix Cowon D2 bootloaderfile had no slash prefix. ... |
| 18:49:56 | mt | saratoga_: Figured where the problem is I guess :) .. There's a loop in main() that keeps performing fft repeatedly. I used that to measure the speed differences between ffmpeg's and a52's. |
| 18:51:28 | tomers | domonoky: When including string.h, there was some namespace collision with the variable index. It had to be renamed. |
| 18:51:30 | saratoga_ | mt: I disabled that loop so it ran faster and it didn't seem to help |
| 18:51:45 | mt | It worked for me |
| 18:52:00 | mt | I'll just check the plot against the floating point one |
| 18:52:26 | saratoga_ | can you pastebin the workign code? maybe i broke something |
| 18:52:29 | | Join Link4u [0] (n=51aa94e9@giant.haxx.se) |
| 18:52:37 | Link4u | hi all! |
| 18:52:44 | Link4u | i am a n00b at this |
| 18:53:05 | Link4u | how to install bootloader on cowon d2? |
| 18:53:09 | tomers | domonoky, bluebrother: I did not want to include telechips.h and mess Makefile in bootloaderinstalltcc.cpp. But using test_firmware_tcc() in mktccboot.c is indeed a bit redundant. I chose to do so to make the code somehow more readable. |
| 18:54:03 | tomers | Link4u: Do it manually, or compile the rockbox utility yourself, and it should do it automatically for you (you'll have to download OF file first) |
| 18:54:03 | Link4u | does anyone have a patched commercial firmware with bootloader for the d2? |
| 18:55:13 | domonoky | tomers: ah, now i understand. So for me this would be fine to commit. |
| 18:56:27 | mt | saratoga_: There's the code : http://www.pastie.org/679034 - and the plot : http://imagebin.ca/view/hfKFRDhh.html |
| 18:56:57 | | Join Grahack1 [0] (n=chri@ACaen-156-1-65-141.w90-51.abo.wanadoo.fr) |
| 18:57:30 | mt | saratoga_: The name "difference.jpeg" is misleading though, I forgot to change that :/ . It's just a normal plot of both. |
| 18:57:32 | MoMonster | mcuelenaere: What am I supposed to do with that code? Can you get it submitted somehow? I have no clue of patching and all that, sorry :/ |
| 18:57:47 | saratoga_ | mt: works great! |
| 18:57:55 | mt | Yep ! :) |
| 18:58:06 | tomers | bluebrother: Are you OK with committing FS #10740? |
| 19:00 |
| 19:00:33 | mcuelenaere | MoMonster: it's a patch; you should apply it against SVN and compile it. I won't be committing this as I haven't tested it and I'm too busy atm |
| 19:01:25 | mt | There are 2 approaches I'm thinking about right now. One : Add more sizes to a52's and test the speed differences between both with larger datasets. Or, Two : Assume that since a52 is faster on a 64-point fft then it should be faster on larger sizes, then I'd just convert the functions to macros and see how much speedup that would give. What do you think ? |
| 19:01:31 | mt | saratoga_: ^ |
| 19:02:19 | saratoga_ | mt: converting a52 to use larger sizes is pretty easy, its only a couple lines of code |
| 19:02:31 | | Quit Grahack (Connection timed out) |
| 19:02:44 | bluebrother | tomers: is r23468 correct? d2n.bin becoming / ? |
| 19:03:22 | | Quit ukaszs2 (Read error: 104 (Connection reset by peer)) |
| 19:03:24 | tomers | Yes. I tried it on a real player. I could have rename it to /d2n.bin instead, but the filename itself is not being used |
| 19:03:40 | saratoga_ | but yes i think a52 will be faster then ffmpeg's and its more efficient with memory too |
| 19:03:43 | tomers | I wanted to emphasize it is not being used |
| 19:03:57 | | Join froggyman [0] (n=sopgenor@pool-72-69-220-194.chi01.dsl-w.verizon.net) |
| 19:04:16 | saratoga_ | i'm actually tempted to just ignore ffmpeg's but i do like that its LGPL rather then GPL which would be nice for some codecs so they can stay LGPL |
| 19:04:50 | bluebrother | as for FS #10740, please remove the renaming of the index variable. I don't see any reason for that. Apart from that the only thing I'm wondering if the inclusion of string.h is needed in telechips.c. And I have no idea if the test routine is actually ok as I know nothing about the D2, but that is up to you ;-) |
| 19:05:28 | mt | saratoga_: We can steal the code from a52, can't we ? :) |
| 19:05:38 | saratoga_ | yeah we can use it |
| 19:05:48 | saratoga_ | though if we do we just have to use GPL, which is fine for rockbox anyway |
| 19:06:42 | | Quit stripwax (Read error: 54 (Connection reset by peer)) |
| 19:06:43 | mt | No, I meant, we could modify ffmpeg to be like a52 without actually using their code, and leaving it LGPL'd. Can we do that ? |
| 19:07:31 | | Quit bmbl (SendQ exceeded) |
| 19:07:55 | | Join ukaszs2 [0] (n=ukasz@77.255.109.50) |
| 19:08:18 | *** | Saving seen data "./dancer.seen" |
| 19:08:20 | tomers | bluebrother: I've tested the test routine. It allowed valid file, and rejected invalid file. |
| 19:08:22 | saratoga_ | mt: that would be fine too |
| 19:08:41 | tomers | bluebrother: If I won't rename index, there will be some error there |
| 19:08:41 | | Quit Link4u ("CGI:IRC (EOF)") |
| 19:08:48 | tomers | or warning... |
| 19:09:03 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
| 19:09:05 | saratoga_ | before worrying about that it would be nice to see that the new fft is actually faster though :) |
| 19:09:10 | tomers | including string.h is for memcpy |
| 19:09:12 | | Join stripwax [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 19:09:31 | | Quit beta2k (SendQ exceeded) |
| 19:09:34 | mt | saratoga_: ahem .. good point. :) |
| 19:09:41 | | Join beta2k [0] (n=beta@d24-36-68-97.home1.cgocable.net) |
| 19:10:52 | bluebrother | tomers: ah, right, overlooked memcpy being in string.h :o |
| 19:11:21 | tomers | bluebrother: So are you ok with renaming the index variable? |
| 19:11:41 | bluebrother | but I don't get why you would get a warning if not renaming that variable. If there is a problem that warning should show up right now. |
| 19:11:56 | | Join shai [0] (n=Shai@l192-117-110-233.cable.actcom.net.il) |
| 19:12:09 | tomers | bluebrother: Without my patch, just include string.h and see this warning |
| 19:12:11 | | Join link4u [0] (n=51aa94e9@giant.haxx.se) |
| 19:12:12 | | Part froggyman |
| 19:12:27 | tomers | Somewhere in string.h there is a global variable called index |
| 19:13:22 | * | bluebrother checks |
| 19:13:52 | Bagder | index() is a common function |
| 19:13:57 | bluebrother | urgh. You're right, but it's not a variable. It's function |
| 19:14:13 | bluebrother | ok, so renaming it is the correct solution. |
| 19:14:23 | tomers | ok, then I'm committing :-) |
| 19:14:40 | * | bluebrother likes it when finding reasons also creates more knowledge :) |
| 19:14:49 | CIA-85 | New commit by mc2739 (r23469): OndaVX777 sim now builds so add it back to the list of sims. |
| 19:15:25 | bluebrother | yeah, no further comments from my side. Might be a good idea to note in the commit message about the index rename, just in case someone looks at the diff and is wondering the same I did ;-) |
| 19:15:30 | mt | saratoga_: Found the error plot too : http://imagebin.ca/view/W2P3b0.html |
| 19:15:57 | mt | green : a52, red : ffmpeg-fixed |
| 19:16:31 | saratoga_ | looks good |
| 19:16:38 | saratoga_ | i still don't see why your first code didn't work though |
| 19:17:10 | | Quit robin0800 (Read error: 54 (Connection reset by peer)) |
| 19:17:13 | | Join barrywardell [0] (n=barrywar@rockbox/developer/barrywardell) |
| 19:18:25 | mt | The one you sent me ? |
| 19:18:54 | funman | the frequency setting manual entry was blacklisted for Sansa AMS, perhaps there is another arbitrary blacklisting in the code |
| 19:19:49 | | Quit Sajber^ (SendQ exceeded) |
| 19:19:56 | | Quit niekie (SendQ exceeded) |
| 19:20:33 | mt | saratoga_: the code you sent me had that loop I mentioned, that's the only change I did |
| 19:20:59 | mt | Although there was a missing backslash that was breaking compilation |
| 19:21:08 | saratoga_ | i think it was the backslash |
| 19:21:15 | saratoga_ | it doesn't even generate a warning for me |
| 19:21:26 | ukaszs2 | sorry for bothering but can someone help me with my e200v2? i have problem after i unbrick it, when i try turn it on or conect to pc it freeze on sandisk logo about 120sec and it is always like that |
| 19:21:52 | mt | It gave me lots of errors - and never compiled .. it was missing from a macro. |
| 19:22:10 | | Join Sajber^ [0] (n=Sajber@h-65-31.A213.priv.bahnhof.se) |
| 19:23:43 | | Quit TheSeven (Nick collision from services.) |
| 19:23:59 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 19:24:03 | mt | saratoga_: A 1024-point fft speed test be significant enough for us, right ? |
| 19:24:11 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 19:24:14 | mt | *would be |
| 19:24:25 | funman | Generating dependencies |
| 19:24:25 | funman | arm-elf-gcc: /media/bordel/rockbox/apps/plugins/boomshine.lua: linker input file unused because linking not done |
| 19:24:34 | mcuelenaere | funman: I know |
| 19:24:37 | saratoga_ | mt: I think 512 is the biggest comonly used one |
| 19:24:39 | mcuelenaere | don't know a fix though :/ |
| 19:24:54 | saratoga_ | since a the fft size is mdct size /4 |
| 19:25:21 | mt | I doubt there would be a different result at that size but I'll test anyway. |
| 19:25:33 | mcuelenaere | funman: it doesn't do any harm though, just annoying |
| 19:25:45 | funman | mcuelenaere: i think we can blame Zagor :p |
| 19:25:56 | mcuelenaere | hehe |
| 19:26:02 | CIA-85 | New commit by tomers (r23470): FS #10740 - rbutil: Test Cowon D2 OF file for CRC consistency before patching |
| 19:29:31 | | Quit TheSeven (Nick collision from services.) |
| 19:29:46 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 19:29:58 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 19:30:10 | | Quit Grahack1 (Client Quit) |
| 19:30:38 | | Quit link4u ("CGI:IRC") |
| 19:30:46 | | Quit MoMonster ("CGI:IRC") |
| 19:35:47 | | Join phanboy4 [0] (n=benji@c-24-98-43-198.hsd1.ga.comcast.net) |
| 19:36:14 | | Quit ukaszs2 () |
| 19:37:35 | JdGordon | kugel: oh well... what was your fix? |
| 19:37:47 | funman | wget http://pastie.org/679081.txt; base64 -d 679081.txt > first_correct_mic_recording_on_sansa_fuze.mp3 |
| 19:39:18 | mt | saratoga_: Same figures approx. a52's is ~30.5% faster at a 512-point fft (was ~28% at 64-point). |
| 19:39:20 | kugel | JdGordon: similar, but not in global status, but wps_state instead |
| 19:39:33 | | Join Grahack [0] (n=chri@ACaen-156-1-65-141.w90-51.abo.wanadoo.fr) |
| 19:39:43 | | Quit TheSeven (Nick collision from services.) |
| 19:39:47 | JdGordon | wps_state should be removed imo |
| 19:40:03 | JdGordon | there is no reason we cant use global_status more for that |
| 19:40:04 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 19:40:14 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 19:40:19 | kugel | did you get the setting the volume in the sound settings to work? I assume that was the option_screen hunk? |
| 19:40:22 | JdGordon | ditto wps_sync_data also |
| 19:40:29 | JdGordon | yes |
| 19:40:48 | JdGordon | that should make the quickscreen work also |
| 19:41:05 | kugel | ok, I struggled there |
| 19:41:13 | | Quit adiroiban ("Leaving.") |
| 19:41:17 | saratoga_ | mt: WMA used to have a very simple mdct.c that called the ffmpeg fft (old version) |
| 19:41:22 | | Join adiroiban [0] (n=adiroiba@h194-54-129-79.teleson.ro) |
| 19:41:29 | saratoga_ | its mdct.c might be a good place to start testing in rockbox |
| 19:41:58 | kugel | wps_data and global status are eventually the same/equivalent, but isn't global status saved to nvram.bin or do I confuse it? |
| 19:42:00 | saratoga_ | i think it should be fairly straightforward to use with WMA (or one of your codecs if you prefer) |
| 19:42:23 | saratoga_ | i need to run some errands, then I'll head into work and take a look at it |
| 19:42:24 | JdGordon | kugel: no.. only its items which are in settings_list.c |
| 19:42:34 | saratoga_ | are you working on it now? |
| 19:42:42 | kugel | wps_sync_data is different. it's not shared between skins, just between the displays for a single skin (i.e. main screen and remote) |
| 19:43:04 | mt | saratoga_: wma still uses that till now ? |
| 19:43:27 | * | JdGordon hopes that it can be got rid of totally anyway |
| 19:43:31 | mt | saratoga_: I'll work on it now. |
| 19:43:32 | JdGordon | or.. comments :) |
| 19:43:37 | saratoga_ | mt: no I removed it a year ago July, but you could check out a build from then |
| 19:43:40 | kugel | it has comments |
| 19:43:44 | JdGordon | why did you get rid of is_wps_fading()? |
| 19:44:06 | JdGordon | "/* Holds data for all screens in a skin. */" isnt very helpful |
| 19:44:08 | saratoga_ | its what i tried to do the first time i looked at the new fft last spring |
| 19:44:23 | kugel | the members have enough comments? |
| 19:44:41 | saratoga_ | since mdct.c handles all the rotation code that turns an fft into an mdct, so its mostly just a matter of trying different fft functions and seeing which is fastest |
| 19:44:48 | | Quit gevaerts (Nick collision from services.) |
| 19:44:57 | | Join gevaerts [0] (n=fg@rockbox/developer/gevaerts) |
| 19:45:07 | saratoga_ | although wma is a bit annoying since its the only codec that uses more then 2 MDCT sizes in the same file |
| 19:45:14 | kugel | anyway, is_wps_fading() is still there for IAP. it was a ugly function that was called by the skin engine |
| 19:46:04 | mt | saratoga_: but if - supposedly - wma ran faster using the new fft, it would be indicative, is that what you're thinking of ? |
| 19:46:20 | saratoga_ | mt: yeah |
| 19:46:28 | kugel | the functionality is still the same, but it's part of the skin engine now since skin_tokens.c needs it, not the wps. |
| 19:46:43 | JdGordon | ok |
| 19:46:53 | saratoga_ | plus I tesetd the current mdct in wma to see how many MHz it used with and without ASM code |
| 19:47:21 | saratoga_ | so if the new fft uses fewer MHz then the c version of the current mdct, it'll probbaly be faster with ASM too |
| 19:48:04 | mt | saratoga_: ok, I'll take a look at it now. Have to check out an older revision first though. Or would the one you sent me before work ? |
| 19:48:52 | | Join Strife89 [0] (n=Strife89@adsl-220-119-35.mcn.bellsouth.net) |
| 19:50:46 | saratoga_ | mt: i think i sent you the right revision before |
| 19:51:13 | mt | yes |
| 19:51:13 | | Join freqmod_qu [0] (n=quassel@dhcp208-240.ed.ntnu.no) |
| 19:52:26 | funman | mc2739: can you test http://www.rockbox.org/tracker/task/10371?getfile=20848 on e200v2 ? |
| 19:55:27 | saratoga_ | funman: no line in on the AMS targets right? |
| 19:56:50 | funman | No idea |
| 19:56:58 | funman | at least it's not used by the OF |
| 19:57:12 | funman | we could try with the other input (line2 is FM) |
| 19:59:11 | saratoga_ | still pretty hard to tell if the input is connected to anything or just floating though, at least without trying all dock pins |
| 20:00 |
| 20:02:47 | funman | not sure why the Clip FM doesn't tune |
| 20:03:56 | funman | I added a safety exit to the loop in si4700_set_frequency() to avoid infinite loop |
| 20:04:59 | | Quit stoffel (Remote closed the connection) |
| 20:05:35 | pixelma | mc2739: I posted a new patch for the pictureflow button table, maybe you can take a look and help with the remaining "issues" in the rest of the chapter |
| 20:07:51 | | Join domonoky1 [0] (n=Domonoky@e179163243.adsl.alicedsl.de) |
| 20:11:33 | funman | entering recording still crashes on Clip with a patch of FS #10605 applied |
| 20:17:56 | | Quit Strife89 ("Switching machines.") |
| 20:21:36 | JdGordon | does anyone have any ideas how to handle sharing skin files between targets with the same size screen but on has rtc/fm/rec/etc and one doesnt? |
| 20:21:56 | JdGordon | rtc sort of works, unless you want to use the "time format" setting (which I do) |
| 20:22:36 | JdGordon | I was thinking make the %St parser less strict and not fail if the setting doesnt exist, but then give it another param for the default value so it will still work |
| 20:23:01 | JdGordon | like %St|time format|0|<a|b> |
| 20:23:35 | pixelma | why not put the %St inside the %?cc there? |
| 20:23:59 | JdGordon | the parser fails because time format isnt a setting on non-rtc builds |
| 20:24:08 | | Quit domonoky (Read error: 110 (Connection timed out)) |
| 20:24:15 | * | JdGordon is trying to avoid making 50 different sbs files |
| 20:25:28 | funman | domonoky1: bertrik: we need your m200v4 and c200v2 for FS #10371 ;) |
| 20:25:31 | pixelma | so the parser would look at the true branch of the %?cc too even if it wouldn't be needed for this target? |
| 20:25:58 | JdGordon | yep |
| 20:26:04 | JdGordon | the parser isnt that smart |
| 20:26:56 | JdGordon | the rec bar is my bigger problem though.. its entirely settings based, which will fail miserably :( |
| 20:27:01 | | Join seani [0] (n=4e216d46@giant.haxx.se) |
| 20:28:43 | | Quit funman ("free(random());") |
| 20:30:53 | seani | Foplks, I have a buildFolks, I have a build that includes a "pause-on-timeout" when the sleep-timeout period matures. Is the standard process that I submit the patch and wait for it to be flayed into something that's considered acceptable, or are there other steps before I should do that? |
| 20:31:18 | seani | Oops, sorry, Chrome hides part of the text in the webclient |
| 20:31:35 | JdGordon | if you want it commited... make it acceptable first |
| 20:31:38 | kugel | the parser should probably fixed |
| 20:31:52 | JdGordon | thats not a parser problem |
| 20:31:54 | | Join petur [50] (n=petur@rockbox/developer/petur) |
| 20:32:00 | domonoky1 | seani: if you want this in svn, putting a patch into flyspray and nagging us here for review, is the recommended method. |
| 20:32:05 | kugel | it could handle those feature tags better |
| 20:32:07 | JdGordon | there is a very *VERY* small set of tags which are always constant and known at build time |
| 20:32:22 | AlexP | seani: But you should have it in a good condition first |
| 20:32:24 | JdGordon | but yes, we could look into it |
| 20:32:31 | AlexP | seani: i.e. see docs/CONTRIBUTING |
| 20:32:58 | kugel | we could also just add time format to the non-rtc targets, it's ignored anyway |
| 20:33:04 | | Join stoffel [0] (n=quassel@p57B4C066.dip.t-dialin.net) |
| 20:33:20 | JdGordon | at 300bytes each setting or whatever it is!? |
| 20:33:47 | seani | Thanls all. It looks good to me and tests Ok, but I don't know if my assumptions are good. I'll test and read a bit further and check the CONTRIBUTING docs. Cheers. |
| 20:33:51 | JdGordon | we could add a list of known settings which might not be in the build, but I dont like that idea |
| 20:34:03 | JdGordon | its not just that though... all the settings needed for the rec bar |
| 20:34:03 | pixelma | JdGordon: there is a tag for "which screen I'm in", if the parser just wouldn't look at branches of that tag that can't be true for this target (that's how I understood kugel's suggestion) |
| 20:34:06 | kugel | the global_settings entry is hardly 300 bytes, the menus take most space but those wouldn't be compiled still |
| 20:34:35 | JdGordon | the setting_list entry is a considerable amount |
| 20:34:39 | | Join niekie [0] (i=quasselc@dreamworld.bergnetworks.com) |
| 20:34:48 | JdGordon | pixelma: yeah, I'm not usre thats actually possible |
| 20:35:59 | JdGordon | sure* |
| 20:36:44 | kugel | parse_cc { #ifdef HAVE_RTC <look at yes branch> #else <look at false branch> } sounds possible to me |
| 20:36:51 | | Quit barrywardell () |
| 20:37:06 | JdGordon | its possible... but messy |
| 20:37:29 | JdGordon | it needs to backtrace and remove the conditional, then manually parse untill the end of the branch it wants |
| 20:37:39 | JdGordon | then skip over to the end of the conditional |
| 20:38:04 | JdGordon | it might be doable as a post parse step, once everything is in the tokens already |
| 20:38:11 | JdGordon | but thats too late for this to work anyway |
| 20:38:20 | kugel | it's not easy. I think the 100-150 bytes for the settings entry look more promising IMO |
| 20:38:35 | kugel | the code to handle the feature tags better is probably worth more than that |
| 20:38:56 | pixelma | that won't help with the rec and fm screen though which seems to be the bigger problem |
| 20:39:26 | CIA-85 | New commit by blue_dude (r23471): Code cleanup in codec_thread, playback and pcmbuf; more elegant solution to leftover samples |
| 20:40:01 | | Quit TheSeven (Nick collision from services.) |
| 20:40:03 | | Quit saratoga_ ("Page closed") |
| 20:40:18 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 20:40:20 | | Join Blue_Dude [0] (n=chatzill@rockbox/developer/Blue-Dude) |
| 20:40:30 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 20:40:35 | JdGordon | skipping the branch at parse time does indeed sound like the best option.. even if it is going to be a PITA to code |
| 20:40:58 | Blue_Dude | 14 minute build times? Ouch. |
| 20:41:14 | AlexP | JdGordon: We have confidence in your ninja skills :) |
| 20:41:16 | pixelma | by the way, there is this fm "mute" status on some targets which couldn't be displayed in a custom sbs currently, or am I missing something? And I don't know if USB "power" can be displayed (as the classic statusbar can) |
| 20:41:41 | JdGordon | AlexP: yeah, but coding skills? :p |
| 20:41:55 | AlexP | JdGordon: Hmmm, more difficult :) |
| 20:42:07 | JdGordon | pixelma: hmm.. fm mute might be tricky.. maybe not... usb i tihnk can be shown? |
| 20:42:30 | n1s | JdGordon: you did get that settings macro-voodoo to work didn't you? :) |
| 20:42:55 | * | JdGordon becomes embarrased with humbleness.... |
| 20:43:36 | | Quit ChanServ (Nick collision from syn.) |
| 20:43:36 | | Quit Grahack (Nick collision from syn.) |
| 20:43:36 | | Quit GodEater (Nick collision from syn.) |
| 20:43:36 | | Quit adiroiban (Nick collision from syn.) |
| 20:43:37 | | Quit bubsy (Nick collision from syn.) |
| 20:43:38 | | Quit GeekShadow (Nick collision from syn.) |
| 20:43:39 | | Quit BlakeJohnson86 (Nick collision from syn.) |
| 20:43:39 | | Quit Bagder (Nick collision from syn.) |
| 20:43:40 | | Quit fish_ (Nick collision from syn.) |
| 20:43:42 | | Quit bzed (Nick collision from syn.) |
| 20:43:44 | | Quit CIA-85 (Nick collision from syn.) |
| 20:43:44 | | Quit ps-auxw (Nick collision from syn.) |
| 20:43:45 | | Quit fxb (Nick collision from syn.) |
| 20:43:45 | | Quit blithe (Nick collision from syn.) |
| 20:43:45 | | Quit AB3JU (Nick collision from syn.) |
| 20:43:45 | | Join GodEater [0] (n=bibble@bb-87-80-121-64.ukonline.co.uk) |
| 20:43:47 | | Quit dionoea (Nick collision from syn.) |
| 20:43:47 | | Quit gevaerts (Nick collision from syn.) |
| 20:43:47 | | Quit AlexP (Nick collision from syn.) |
| 20:43:49 | | Quit tha (Nick collision from syn.) |
| 20:43:49 | | Quit bertrik (Nick collision from syn.) |
| 20:43:50 | | Quit Galois (Nick collision from syn.) |
| 20:43:51 | | Quit toffe82 (Nick collision from syn.) |
| 20:43:52 | | Quit sbhsu (Nick collision from syn.) |
| 20:43:54 | | Join dionoea [0] (n=dionoea@yop.chewa.net) |
| 20:43:57 | pixelma | last I checked it didn't work on my Ondio - maybe it's different to other targets though |
| 20:43:57 | | Quit FOAD (Nick collision from syn.) |
| 20:43:58 | | Quit DerPapst (Nick collision from syn.) |
| 20:43:58 | | Quit courtc (Nick collision from syn.) |
| 20:43:59 | | Quit chaos (Nick collision from syn.) |
| 20:43:59 | | Quit domonoky1 (Nick collision from syn.) |
| 20:43:59 | | Quit beta2k (Nick collision from syn.) |
| 20:43:59 | | Quit evilnick (Nick collision from syn.) |
| 20:43:59 | | Quit bluebrother (Nick collision from syn.) |
| 20:43:59 | | Quit Llorean (Nick collision from syn.) |
| 20:43:59 | | Quit Shaid (Nick collision from syn.) |
| 20:43:59 | | Quit FlynDice (Nick collision from syn.) |
| 20:43:59 | | Quit ej0rge (Nick collision from syn.) |
| 20:43:59 | | Quit crwl (Nick collision from syn.) |
| 20:43:59 | | Quit Blue_Dude (Nick collision from syn.) |
| 20:43:59 | | Quit seani (Nick collision from syn.) |
| 20:43:59 | | Quit phanboy4 (Nick collision from syn.) |
| 20:43:59 | | Quit akur (Nick collision from syn.) |
| 20:43:59 | | Quit tomers (Nick collision from syn.) |
| 20:43:59 | | Quit Bob_C_ (Nick collision from syn.) |
| 20:43:59 | | Quit saratoga (Nick collision from syn.) |
| 20:43:59 | | Quit Bob_C (Nick collision from syn.) |
| 20:43:59 | | Quit topik (Nick collision from syn.) |
| 20:43:59 | | Quit crashd (Nick collision from syn.) |
| 20:44:01 | | Join GeekShadow [0] (n=Antoine@77.204.157.218) |
| 20:44:02 | | Join AB3JU [0] (n=dz@alt.dissonance.nl) |
| 20:44:08 | | Join Grahack [0] (n=chri@90.51.24.141) |
| 20:44:11 | | Join Bagder [0] (n=daniel@82.182.26.5) |
| 20:44:12 | | Join ps-auxw [0] (n=arneb@dyn37.ps-auxw.de) |
| 20:44:14 | | Join Shaid [0] (i=adam@lurking.shaid.net) |
| 20:44:15 | | Join ChanServ [0] (ChanServ@services.) |
| 20:44:15 | Mode | "#rockbox +o ChanServ " by irc.freenode.net |
| 20:44:15 | | Join BlakeJohnson86 [0] (n=bjohnson@24.118.162.123) |
| 20:44:15 | | Quit GodEater (Nick collision from syn.) |
| 20:44:15 | | Join sbhsu [0] (n=a6530466@192.192.120.197) |
| 20:44:15 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
| 20:44:16 | | Join fish_ [0] (n=fish@85.214.72.37) |
| 20:44:17 | | Quit dionoea (Nick collision from syn.) |
| 20:44:18 | | Join adiroiban [0] (n=adiroiba@h194-54-129-79.teleson.ro) |
| 20:44:19 | | Quit bubsy (Nick collision from syn.) |
| 20:44:19 | | Quit GeekShadow (Nick collision from syn.) |
| 20:44:20 | | Quit AB3JU (Nick collision from syn.) |
| 20:44:20 | | Quit Grahack (Nick collision from syn.) |
| 20:44:21 | | Quit fish_ (Nick collision from syn.) |
| 20:44:21 | | Quit Bagder (Nick collision from syn.) |
| 20:44:22 | | Quit ps-auxw (Nick collision from syn.) |
| 20:44:23 | | Quit adiroiban (Nick collision from syn.) |
| 20:44:23 | | Join ch4os [0] (n=ch4os@153.19.7.19) |
| 20:44:23 | | Quit ChanServ (Nick collision from syn.) |
| 20:44:24 | | Join topik_ [0] (i=awesome@213.203.214.114) |
| 20:44:24 | | Quit ch4os (Nick collision from syn.) |
| 20:44:29 | | Join bertrik [0] (n=bertrik@87.211.49.117) |
| 20:44:32 | | Join fxb [0] (n=felixbru@85.214.97.64) |
| 20:44:33 | | Join ej0rge [0] (n=alhaz@alhaz.fttp.xmission.com) |
| 20:44:34 | | Join domonoky [0] (n=Domonoky@85.179.163.243) |
| 20:44:37 | | Join phanboy4 [0] (n=benji@24.98.43.198) |
| 20:44:43 | | Join toffe82 [0] (n=chatzill@ppp-71-130-76-7.dsl.frs2ca.pacbell.net) |
| 20:44:45 | | Join akur [0] (n=akur@bl7-28-75.dsl.telepac.pt) |
| 20:44:47 | | Join bubsy [0] (n=bubsy@94.139.72.137) |
| 20:44:48 | | Join gevaerts [0] (n=fg@195-144-092-140.dyn.adsl.xs4all.be) |
| 20:44:50 | | Join Bagder [0] (n=daniel@1-1-5-26a.hud.sth.bostream.se) |
| 20:44:50 | | Join Bob_C [0] (n=chatzill@host86-144-201-0.range86-144.btcentralplus.com) |
| 20:44:51 | | Join evilnick [0] (n=evilnick@69.113.175.81) |
| 20:44:51 | | Join Grahack [0] (n=chri@ACaen-156-1-65-141.w90-51.abo.wanadoo.fr) |
| 20:44:51 | | Join DerPapst [0] (n=DerPapst@79.232.251.189) |
| 20:44:55 | | Join FlynDice [0] (n=FlynDice@24.19.225.90) |
| 20:44:57 | | Join bluebrother [0] (n=dom@78.53.154.240) |
| 20:45:01 | | Nick fxb is now known as fxb__ (n=felixbru@85.214.97.64) |
| 20:45:02 | | Join Blue_Dude [0] (n=chatzill@74.235.222.153) |
| 20:45:02 | | Join AB3JU [0] (n=dz@alt.dissonance.nl) |
| 20:45:02 | | Join AlexP [0] (n=alex@86.66.198.59) |
| 20:45:04 | | Nick fxb__ is now known as fxb (n=felixbru@85.214.97.64) |
| 20:45:05 | | Join crashd [0] (i=foobar@lostnode.org) |
| 20:45:11 | | Join Bob_C_ [0] (n=chatzill@host86-144-201-0.range86-144.btcentralplus.com) |
| 20:45:14 | | Join tomers [0] (n=chatzill@84.109.85.100) |
| 20:45:16 | | Join FOAD [0] (n=dok@dinah.blub.net) |
| 20:45:19 | | Join Llorean [0] (n=DarkkOne@adsl-99-4-146-40.dsl.hstntx.sbcglobal.net) |
| 20:45:22 | | Join GeekShadow [0] (n=Antoine@77.204.157.218) |
| 20:45:22 | | Join ps-auxw [0] (n=arneb@dyn37.ps-auxw.de) |
| 20:45:29 | JdGordon | pixelma: ah yes.. there is charger connected, and charger charging, but no usb connected... easy enough to add |
| 20:45:29 | | Join CIA-8 [0] (n=CIA@208.69.182.149) |
| 20:45:35 | | Join adiroiban [0] (n=adiroiba@h194-54-129-79.teleson.ro) |
| 20:45:38 | | Join courtc [0] (n=court@98.230.149.46) |
| 20:45:50 | | Join beta2k [0] (n=beta@24.36.68.97) |
| 20:45:58 | | Join The_Seven [0] (n=theseven@stgt-5d84b7c4.pool.mediaWays.net) |
| 20:46:03 | | Join crwl [0] (n=crwlll@a91-156-100-168.elisa-laajakaista.fi) |
| 20:46:26 | pixelma | sounds good, maybe that would have been something like the hold for software hold as well that I could figure out myself too |
| 20:46:29 | | Join ChanServ [0] (ChanServ@services.) |
| 20:46:29 | Mode | "#rockbox +o ChanServ " by irc.freenode.net |
| 20:46:33 | | Quit ChanServ (Killed by sagan.freenode.net (Nick collision)) |
| 20:46:37 | | Join ChanServ [0] (ChanServ@services.) |
| 20:46:37 | Mode | "#rockbox +o ChanServ " by irc.freenode.net |
| 20:46:41 | | Quit ChanServ (lindbohm.freenode.net irc.freenode.net) |
| 20:46:41 | NSplit | lindbohm.freenode.net irc.freenode.net |
| 20:46:55 | | Join bzed [0] (n=bzed@devel.recluse.de) |
| 20:47:05 | JdGordon | you mean adding it? yeah, should be dead simple |
| 20:47:40 | kugel | hrm |
| 20:47:51 | JdGordon | ping gevaerts to find out how to see if its connected or not :) |
| 20:47:56 | kugel | a sort of preprocessor might be easier. |
| 20:48:25 | JdGordon | oh no... not more naughty macros! |
| 20:48:43 | | Join fish_ [0] (n=fish@freigeist.org) |
| 20:48:44 | kugel | %if HAVE_FM [...] %end, for example. it would save the whole investigation of the branch which still needs to be done to find when it ends |
| 20:48:46 | | Join GodEater [0] (n=bibble@bb-87-80-121-64.ukonline.co.uk) |
| 20:48:49 | | Join dionoea [0] (n=dionoea@yop.chewa.net) |
| 20:49:15 | JdGordon | no offence... but YUCK! |
| 20:49:21 | | Join ch4os [0] (n=ch4os@hybris.inf.ug.edu.pl) |
| 20:49:33 | | Quit stoffel (Read error: 54 (Connection reset by peer)) |
| 20:50:03 | JdGordon | assuming the skin is coded correctlty... finding the right branch shouldnt be all that hard actually... |
| 20:51:10 | JdGordon | go back 2 tags, remove the start conditional and the token for %cc, then increase a count every time you see < and decrease as you see > untill you hit a | when the count is 0... that gets the second branch... easier for the first branch |
| 20:51:34 | JdGordon | actually, increase when you see a %? |
| 20:52:05 | | Join blithe [0] (n=blithe@blakesmith.me) |
| 20:52:07 | JdGordon | it will make debugging skins hard though :) |
| 20:52:18 | | Join tha [0] (i=1038@ccc2.rbg.informatik.tu-darmstadt.de) |
| 20:52:45 | NHeal | lindbohm.freenode.net irc.freenode.net |
| 20:52:45 | NJoin | ChanServ [0] (ChanServ@services.) |
| 20:52:45 | Mode | "#rockbox +o ChanServ " by irc.freenode.net |
| 20:52:58 | JdGordon | debugging is a bad after thought anyway! |
| 20:53:00 | | Nick AlexP is now known as Guest66663 (n=alex@86.66.198.59) |
| 20:53:10 | | Quit Blue_Dude ("ChatZilla 0.9.85 [Firefox 3.5.4/20091016092926]") |
| 20:53:26 | | Join Blue_Dude [0] (n=chatzill@rockbox/developer/Blue-Dude) |
| 20:54:11 | Blue_Dude | No yellows/reds this time. That's progress. |
| 20:55:32 | | Quit Guest66663 (Client Quit) |
| 20:55:49 | | Join AlexP_ [0] (n=alex@rockbox/staff/AlexP) |
| 20:55:50 | | Quit TheSeven (Nick collision from services.) |
| 20:55:50 | | Quit The_Seven (Nick collision from services.) |
| 20:56:05 | | Join _TheSeven_ [0] (n=theseven@rockbox/developer/TheSeven) |
| 20:56:46 | | Nick AlexP_ is now known as BigBambi (n=alex@rockbox/staff/AlexP) |
| 20:56:52 | | Nick BigBambi is now known as AlexP (n=alex@rockbox/staff/AlexP) |
| 20:57:37 | | Nick _TheSeven_ is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 20:58:11 | | Quit Blue_Dude (Client Quit) |
| 21:00 |
| 21:01:42 | JdGordon | umm... hmm... yeah this might not be easy after all :p |
| 21:02:16 | JdGordon | not without a fair bit of reworking the parser |
| 21:05:31 | JdGordon | yep, I tihnk the far simpler option (even if it is only a temp solution) is fix %St to allow a small set of settings |
| 21:05:39 | JdGordon | which I tihnk is a terrible solution ?:p |
| 21:05:42 | JdGordon | -? |
| 21:07:35 | | Quit n1s (Read error: 110 (Connection timed out)) |
| 21:08:20 | *** | Saving seen data "./dancer.seen" |
| 21:10:21 | JdGordon | no! yuck yuck yuck |
| 21:14:15 | | Join Galois [0] (i=djao@efnet.math.uwaterloo.ca) |
| 21:18:31 | | Quit Res1 (Read error: 60 (Operation timed out)) |
| 21:23:52 | | Join saratoga [0] (i=9803c6dd@gateway/web/freenode/x-8c1350fd36917134) |
| 21:23:54 | | Join kadoban [0] (n=mud@cpe-24-93-17-195.rochester.res.rr.com) |
| 21:28:16 | | Join krazykit [0] (n=kkit@adsl-76-251-245-92.dsl.ipltin.sbcglobal.net) |
| 21:31:33 | | Join Res1 [0] (n=Res@user-0c6s6ib.cable.mindspring.com) |
| 21:32:03 | | Join grant___ [0] (i=453f3ee5@gateway/web/freenode/x-8ceade5261d8ddc2) |
| 21:34:33 | grant___ | iv got a problem with the nano 2g version of rockbox and TheSeven couldnt answer it so he said i should ask here |
| 21:35:06 | grant___ | whenever i try to select anything from the file menu, a bent arrow appears in the corner and i cant select the item |
| 21:35:43 | grant___ | does anybody has an idea why the virtual storage led could be causing this trouble |
| 21:35:50 | bluebrother | bent arrow? |
| 21:36:14 | grant___ | yah, its kinda weird |
| 21:36:32 | domonoky | its probably the "disk busy" icon.. |
| 21:36:40 | bluebrother | in which corner? Upper right? With the status bar enabled? |
| 21:36:51 | grant___ | yah, upper right |
| 21:36:59 | grant___ | in the status bar |
| 21:37:15 | Unhelpful | JdGordon: you want to have that sort of flexibility for third-party theme files, as well as shipped ones? |
| 21:38:11 | | Join ilikenwf [0] (n=ilikenwf@h-96-15-157-136.ip.alltel.net) |
| 21:38:31 | ilikenwf | any of you have a bash script for copying music in linux and replacing unicode so that it all actually gets copied over? |
| 21:39:04 | bluebrother | does it look like in this screenshot? http://download.rockbox.org/daily/manual/rockbox-ondiosp/rockbox_interface/images/ss-file-browser-112x64x1-ondiosp.png |
| 21:39:20 | grant___ | yah exactly |
| 21:39:41 | Unhelpful | ilikenwf: there are tons. really it makes sense to sanitize filenames in the *first* place, though, because then you can just use rsync. ;) |
| 21:39:44 | bluebrother | ah, then it's the icon for access of the flash memory. |
| 21:40:02 | bluebrother | but that's just an indicator. |
| 21:40:21 | TheSeven | additional info: he says that this doesn't happen in a revision that was before implementing that indicator in the nn2g nand driver |
| 21:40:34 | TheSeven | (but no other changes to nand/ftl) |
| 21:40:41 | bluebrother | ah. strange. |
| 21:41:52 | pixelma | grant___: do you want to use voice and installed a voice file? |
| 21:42:08 | grant___ | no, theres no voice |
| 21:42:43 | saratoga | whats with the really long build times lately |
| 21:43:17 | grant___ | if anybody has any more questions, email me at leechman777@gmail.com ; ive got to go |
| 21:43:22 | | Part grant___ |
| 21:43:39 | pixelma | ooh |
| 21:44:09 | bluebrother | looks like there are extremly much abandoned builds. I guess those are related to disconnects, probably some network issues? |
| 21:44:32 | | Join ilikenwf_ [0] (n=ilikenwf@h-96-15-157-136.ip.alltel.net) |
| 21:44:52 | pixelma | weren't there reports about failed uploads (also of the build logs)? |
| 21:45:20 | bluebrother | iirc yes. |
| 21:46:29 | liar | TheSeven: i have the same bug and if i disable nand_thread in cat coding/rockbox/firmware/target/arm/s5l8700/ata-nand-s5l8700.c it works again... |
| 21:46:34 | saratoga | it says my 4xOpteron system took 2 minutes to build the ipod mini sim which can't be right |
| 21:47:00 | | Join n1s [0] (n=n1s@rockbox/developer/n1s) |
| 21:47:29 | bluebrother | saratoga: have you checked the detailed build round table at rasher's page? |
| 21:47:41 | saratoga | no do you have the link? |
| 21:47:49 | bluebrother | saratoga: rasher.dk/rockbox/buildgraphs/">http://rasher.dk/rockbox/buildgraphs/ |
| 21:48:01 | bluebrother | the "debug" view is quite interesting :) |
| 21:49:07 | saratoga | what is a queued build? |
| 21:49:08 | JdGordon | Unhelpful: yes |
| 21:49:33 | rasher | saratoga: it's the build scheduled for you by the build master |
| 21:50:17 | Unhelpful | JdGordon: that's rough. obviously for in-tree themes we can use the CPP. we could even simulate that for uploaded themes on the theme site, though it would be a bit nasty. but parsing that out on target seems just icky... |
| 21:50:59 | kugel | cpp as a plugin, why not? :P |
| 21:51:04 | | Join Strife89 [0] (n=Strife89@adsl-068-213-037-174.sip.mcn.bellsouth.net) |
| 21:51:14 | JdGordon | yeah.. I tihnk I'm just going to have to fiddle with the makethemes script and do seperate sbs' for rec and non rec targets |
| 21:51:19 | kugel | features.txt might be handy for that |
| 21:51:22 | | Quit Grahack (Read error: 60 (Operation timed out)) |
| 21:51:58 | Unhelpful | as a plugin, maybe, but the idea of doing it in the theme parser is a bit less pleasant... for the most part i'd rather have target-specific themes than more code in core. |
| 21:51:58 | kugel | or, rockbox-info.txt |
| 21:52:00 | JdGordon | need one each for rec+rtc, rec+no-rtc, no-rec+rtc, and neither |
| 21:52:43 | Unhelpful | would "ignore unknown WPS tags" not suffice? |
| 21:53:05 | JdGordon | thats already there |
| 21:53:21 | | Join Grahack [0] (n=chri@ACaen-156-1-65-141.w90-51.abo.wanadoo.fr) |
| 21:53:22 | JdGordon | the problem (once again :) ) is the %St setting value tag which fails if the setting doesnt exist |
| 21:53:32 | kugel | %St is a different problem. it's a known tag, but not necessarily the parameter |
| 21:53:40 | JdGordon | thats the problem with rtc.. and ditto the rec bar |
| 21:53:58 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
| 21:54:07 | | Quit TheSeven (Nick collision from services.) |
| 21:54:08 | JdGordon | if the recording ones are doing in functions and new tags then it wont be a problem |
| 21:54:27 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 21:54:59 | JdGordon | or alternativly.. in the rec screen we continue using the inbuilt hard coded non skinned bar |
| 21:55:38 | | Join stripwax__ [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 21:55:44 | | Quit Utchybann (Read error: 113 (No route to host)) |
| 21:55:47 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 21:56:22 | Unhelpful | perhaps have %St as a conditional that's true if the setting exists? or is that no good, since you want to use the setting value as a conditional, too? |
| 21:59:03 | JdGordon | if that token is put inside another one (i.e "time format" is inside %cc (rtc present)) then its no problem if the setting doesnt exist |
| 21:59:13 | kugel | Unhelpful: that won't help. the parser would not does not care about what branches are possible. it parses all brances |
| 21:59:59 | kugel | s/would// |
| 22:00 |
| 22:00:27 | Unhelpful | kugel: if we have a %cc that acts as a feature switch and blocks parsing the branch, surely a similar thing could be done for settings? |
| 22:00:30 | JdGordon | I dont especially like the idea of letting %St pass the parser if the setting isnt there though (which is how it used to be) |
| 22:00:48 | JdGordon | Unhelpful: it doesnt do that.. it simply always returns 0 or 1 |
| 22:00:56 | JdGordon | the full parse still happens |
| 22:01:14 | | Quit TheSeven (Nick collision from services.) |
| 22:01:29 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 22:01:30 | Unhelpful | JdGordon: ah. what exactly happens when the parser hits a setting tag for a setting that doesn't exist? |
| 22:01:37 | JdGordon | it craps out |
| 22:01:52 | JdGordon | which is by design |
| 22:02:17 | | Quit stripwax__ (Read error: 104 (Connection reset by peer)) |
| 22:02:19 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 22:02:25 | Unhelpful | oh. i was going to say it could just be made to display the unrecognized setting tag an an empty string... |
| 22:02:31 | | Quit ilikenwf (Read error: 110 (Connection timed out)) |
| 22:02:49 | Unhelpful | we want it to die why, so that checkwps will show an error? |
| 22:03:04 | JdGordon | yeah |
| 22:03:13 | JdGordon | it used to ignore it, but we decided that was bad |
| 22:03:46 | pixelma | the %St tag can also be used conditionally (but only for the values I think, not for the setting itself) |
| 22:04:07 | JdGordon | it would have to be 2 seperate tags |
| 22:04:08 | Unhelpful | perhaps it could be special-cased to warn in checkwps, but would just be ignored as before on targets? |
| 22:04:09 | | Quit Res1 (Read error: 60 (Operation timed out)) |
| 22:04:12 | | Quit ilikenwf_ (Read error: 110 (Connection timed out)) |
| 22:04:18 | pixelma | even the former doesn't work correctly |
| 22:05:30 | | Join stripwax_ [0] (n=Miranda@87-194-34-169.bethere.co.uk) |
| 22:05:55 | | Join ukaszs2 [0] (i=ukaszs2@77-254-215-181.adsl.inetia.pl) |
| 22:08:05 | | Quit tomers ("ChatZilla 0.9.85 [Firefox 3.5.4/20091028153816]") |
| 22:08:43 | ukaszs2 | hello, i have porblem with recovery mode in my e200v2, i bridge this two points but nothing happend when connect to pc |
| 22:08:56 | | Quit n1s ("Lämnar") |
| 22:09:52 | | Quit TheSeven (Nick collision from services.) |
| 22:10:08 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 22:10:17 | * | JdGordon grumbles |
| 22:10:22 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 22:11:55 | | Quit stripwax (Read error: 110 (Connection timed out)) |
| 22:15:15 | JdGordon | i tihnk the least annoying way to fix this is: 1) create a new %c<something> tag to use the user setting for 12/14 hour, 2) for the rec settings.. split them off into functions instead of direclty useing the settings, both wont affect non-blaa builds so shouldnt have any bad effect |
| 22:15:29 | JdGordon | I dont think the radio actually has any problems like this (just yet anyway) |
| 22:15:45 | gevaerts | People use 14 hour clocks? |
| 22:17:10 | kugel | why not do it correctly now ? |
| 22:18:48 | | Quit slam_ (Read error: 60 (Operation timed out)) |
| 22:25:18 | | Join webguest25 [0] (n=4c4e9189@giant.haxx.se) |
| 22:25:22 | | Quit Omlet ("( www.nnscript.com :: NoNameScript 4.22 :: www.esnation.com )") |
| 22:26:33 | JdGordon | kugel: do what correctly now? |
| 22:30:43 | | Quit webguest25 ("CGI:IRC (Ping timeout)") |
| 22:33:31 | | Quit petur ("Zzzzzz") |
| 22:35:37 | CIA-8 | New commit by jdgordon (r23472): new skin token: %cx - 24 hour time format enabled in the setting.. e.g %?cx<24 hour|12 hour> |
| 22:36:10 | JdGordon | not pretty, but cheap :p |
| 22:36:41 | JdGordon | ah crud.. I meant to give the reson in that message also... oh well |
| 22:37:17 | kugel | JdGordon: I think I have something working that skips the true case |
| 22:37:32 | JdGordon | oh? |
| 22:37:50 | kugel | http://pastie.org/679284 |
| 22:37:59 | JdGordon | skipping true isnt too difficult i tihnk... skip false though! |
| 22:38:17 | kugel | I don't think skipping the false branch is needed |
| 22:38:26 | kugel | is it? |
| 22:38:39 | JdGordon | sure it is |
| 22:38:40 | kugel | the false branch should parse on any target |
| 22:39:34 | JdGordon | hmm... yeah probably not actually :p |
| 22:40:21 | JdGordon | find_false_branch() is wrong |
| 22:40:31 | JdGordon | %< and %> will break it |
| 22:40:46 | kugel | heh, surely it's not perfect yet |
| 22:40:47 | JdGordon | level++ should happen on %? |
| 22:41:02 | kugel | no why? |
| 22:41:18 | JdGordon | its in theory safer than < |
| 22:41:28 | JdGordon | it doesnt really matter |
| 22:41:30 | kugel | it's about nested conditionals |
| 22:41:33 | | Quit stripwax_ ("http://miranda-im.org") |
| 22:41:37 | JdGordon | %? means a new conditional is starting |
| 22:41:37 | kugel | % doesn't indicate a conditional |
| 22:41:43 | JdGordon | %? |
| 22:41:45 | JdGordon | ??? |
| 22:41:46 | JdGordon | ? |
| 22:41:56 | kugel | lol I thought it was a question |
| 22:41:57 | JdGordon | %?tag |
| 22:42:06 | JdGordon | umm |
| 22:42:10 | JdGordon | one sec |
| 22:42:34 | | Join Thundercloud [0] (i=thunderc@persistence.flat.devzero.co.uk) |
| 22:42:47 | JdGordon | yeah ? doesnt need to be escaped does it? |
| 22:43:02 | JdGordon | I'll revert that checkin then if you are almost working |
| 22:43:54 | kugel | well, the main function hacks. it increments level if it finds '<', but it then checks if 2 tokens ago was a '?', some of that sort |
| 22:44:22 | JdGordon | yeah, I noticced that |
| 22:44:30 | JdGordon | not a gid deal |
| 22:44:32 | JdGordon | big |
| 22:44:57 | kugel | anyway, I think this is the way to go |
| 22:45:03 | CIA-8 | New commit by jdgordon (r23473): revert that last checkin... something better is coming |
| 22:45:19 | JdGordon | except.. crap... you are left with a trailing > |
| 22:45:34 | kugel | no |
| 22:45:39 | kugel | that's actually needed |
| 22:45:47 | JdGordon | you keep the conditional so it knows there should be a > ? |
| 22:45:50 | kugel | to close the '<' that the main function detected |
| 22:45:58 | | Quit bmbl (Client Quit) |
| 22:46:20 | | Quit dmb (Read error: 60 (Operation timed out)) |
| 22:46:50 | JdGordon | I'm not sure that last hink is in the correct spot |
| 22:47:10 | kugel | the idea is to increment the buf pointer not at all if the feature is available, or to directly after the appropriate | if it isn't |
| 22:47:10 | JdGordon | but heck.. if it works :) |
| 22:47:51 | JdGordon | wont that end up making it parse so it comes to %?tag<flase block> |
| 22:48:09 | | Join slam_ [0] (n=grndslm@174-126-14-4.cpe.cableone.net) |
| 22:48:13 | kugel | before that is done the main function finds a '<' that needs to be closed after the checking |
| 22:48:33 | kugel | JdGordon: yea, that's the idea |
| 22:48:56 | JdGordon | and the displayer uses the first option as the false option? |
| 22:49:13 | kugel | huh? |
| 22:49:26 | JdGordon | %?tag<false|true> ? |
| 22:49:30 | kugel | it's %?<true|false> |
| 22:49:49 | JdGordon | so how will %?<false block> work? |
| 22:49:54 | kugel | always, the special case is %?<true> which I didn't handle yet |
| 22:50:04 | kugel | that doesn't exist, does it? |
| 22:50:22 | JdGordon | that is what is going to exist after your change goes through the parser |
| 22:50:32 | kugel | http://www.rockbox.org/wiki/CustomWPS#Conditional_Tags |
| 22:50:55 | JdGordon | %cc returns false on non rtc |
| 22:51:05 | JdGordon | your change gets rid of the true block which is correct |
| 22:51:18 | JdGordon | but then the displayer sees false on %cc and nothing to display which is wrong |
| 22:51:23 | | Quit ukaszs2 () |
| 22:51:44 | kugel | good point, gonna check that |
| 22:51:55 | kugel | but it will parse at least :D |
| 22:52:01 | JdGordon | well yes :) |
| 22:52:17 | JdGordon | the easy answer it to invert the return in skin_token.c |
| 22:52:31 | JdGordon | so %cc actually returns true always |
| 22:52:38 | JdGordon | actually.. |
| 22:53:11 | JdGordon | screw that... add a new token which always returns true, then change the conditional to WPS_TOKEN_TRUE |
| 22:53:20 | kugel | that breaks plain %cc of course, but I think that's accaptable for plain feature tags |
| 22:53:29 | JdGordon | no it doesnt |
| 22:53:31 | JdGordon | oh |
| 22:53:32 | | Join Res1 [0] (n=Res@user-0c6s6j8.cable.mindspring.com) |
| 22:53:42 | JdGordon | yeah, rtc build with detectable rtc |
| 22:54:09 | JdGordon | igore that suggestion then... but make it always return true or rtc_Detected() |
| 22:54:42 | Unhelpful | perhaps it would just be wise to extend all of this to a generic static conditional tag that parses only one of its branches? |
| 22:54:55 | * | JdGordon was about to suggest something like that |
| 22:55:14 | JdGordon | only the true branch is possible (without major rework) to parse |
| 22:55:18 | JdGordon | to skip i mean |
| 22:55:28 | | Join petur [0] (n=peter@rockbox/developer/petur) |
| 22:55:33 | Unhelpful | ah. that's troublesome :/ |
| 22:56:16 | * | JdGordon likes where this is going :) |
| 22:56:24 | Unhelpful | how big a rework? |
| 22:57:06 | JdGordon | not sure... but more than "insignificant" |
| 22:57:07 | | Join dmb [0] (n=Dmb@unaffiliated/dmb) |
| 22:57:22 | JdGordon | without doing it in a hack way anyway |
| 22:57:40 | kugel | returning always true or detect_hw() works |
| 22:58:41 | kugel | maybe return ' return radio_hardware_present() ? "yes":NULL ' or ' "no' " |
| 22:59:57 | JdGordon | thats good |
| 23:00 |
| 23:00:36 | JdGordon | now as long as noone cares about technical consistancy we are ok |
| 23:00:36 | kugel | I don't think anyone is going to use %cc unconditionally though |
| 23:01:00 | JdGordon | I would hope not |
| 23:02:56 | JdGordon | can you add a note in that feature check function (or on it) saying the get_token() value has to return "true".. and that it only works on true/false options |
| 23:03:03 | kugel | heh, interesting problem. it shows "no" on targets without RTC at all, but nothing on my e200 since we really get NULL then |
| 23:03:17 | | Join darkham [0] (n=darkham@host106-176-dynamic.47-79-r.retail.telecomitalia.it) |
| 23:03:17 | | Join ender` [0] (i=krneki@foo.eternallybored.org) |
| 23:03:45 | JdGordon | thats correct though |
| 23:03:46 | kugel | I would not mind disallowing the unconditional use of feature tags altogether |
| 23:04:00 | JdGordon | na, not worth it |
| 23:04:03 | JdGordon | no need to |
| 23:08:22 | *** | Saving seen data "./dancer.seen" |
| 23:09:00 | pixelma | hmm... what about targets where fm is detected at runtime? (didn't follow all of the discussion though) |
| 23:09:33 | | Quit petur ("Zzzzzz") |
| 23:10:07 | | Quit domonoky (Read error: 54 (Connection reset by peer)) |
| 23:10:39 | | Quit TheSeven (Nick collision from services.) |
| 23:10:43 | JdGordon | that was the "interesting problem" :) |
| 23:10:55 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 23:11:09 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 23:12:46 | | Join MethoS- [0] (n=clemens@134.102.106.250) |
| 23:13:05 | | Join rhcp1253 [0] (n=rhcp1253@ool-182dc0d1.dyn.optonline.net) |
| 23:14:35 | rhcp1253 | is the bug fixed on the nano 2g where you couldn't enable database with panicing on startup |
| 23:15:36 | krazykit | why don't you try and find out? |
| 23:15:51 | | Quit TheSeven (Nick collision from services.) |
| 23:15:55 | JdGordon | kugel: I'm off... just remember to make sure you only change the level on %< %> and check for %|.... otherwise looking forward to it :) |
| 23:16:07 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 23:16:08 | | Quit lifeless_ (Remote closed the connection) |
| 23:16:21 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 23:16:21 | kugel | JdGordon: ok |
| 23:16:25 | | Join lifeless_ [0] (n=lifeless@90.150.121.194) |
| 23:16:49 | kugel | although wps parse checks for the %? already, or do you mean for nested branches? |
| 23:17:14 | JdGordon | yes |
| 23:17:39 | JdGordon | no. i mean when literal < and > and | should be displayed |
| 23:17:39 | kugel | that's not easy to do |
| 23:17:54 | kugel | ah I see what you mean |
| 23:18:22 | JdGordon | thats a *requirement* :) |
| 23:22:42 | | Join funman [0] (n=fun@rockbox/developer/funman) |
| 23:22:44 | kugel | JdGordon: here's the hopefully final patch: http://pastie.org/679333 I wait or put it on flyspray if you want to look at it later |
| 23:23:00 | JdGordon | ill look quickly |
| 23:23:33 | JdGordon | *(buf-1) just looks wrong to me... |
| 23:23:41 | JdGordon | even if its safe, it seems bad |
| 23:24:03 | JdGordon | how about (if *buf == '%') { if its any of those tags buf+=2} |
| 23:25:05 | JdGordon | and the NOTE comment is wrong... on builds where the feature is conditional it will be true/false... on builds without the feautre at all, or always it will be true |
| 23:25:37 | | Quit TheSeven (Nick collision from services.) |
| 23:25:54 | kugel | I just corrected the comment |
| 23:25:56 | | Join The_Seven [0] (n=theseven@rockbox/developer/TheSeven) |
| 23:26:01 | JdGordon | otherwise looks good |
| 23:26:06 | * | JdGordon heading out |
| 23:26:10 | | Nick The_Seven is now known as TheSeven (n=theseven@rockbox/developer/TheSeven) |
| 23:27:27 | | Nick Ypsy is now known as YPSY (n=ypsy@87.106.45.183) |
| 23:27:34 | | Quit Grahack ("Leaving.") |
| 23:30:04 | | Join BHSPitLappy [0] (n=BHSPitLa@unaffiliated/bhspitmonkey) |
| 23:34:34 | | Join AaronM [0] (n=Aaron@adsl-4-232-113.mem.bellsouth.net) |
| 23:41:46 | | Quit phanboy4 ("Leaving") |
| 23:48:05 | funman | should the pitch detector show something even if I'm not making a real musical note but speaking/making noise? |
| 23:48:47 | CIA-8 | New commit by funman (r23474): new skin token: %cx - 24 hour time format enabled in the setting.. e.g %?cx<24 hour|12 hour> |
| 23:48:52 | funman | hum .. |
| 23:49:06 | funman | this commit wasn't mine ... |
| 23:50:13 | | Join bmbl [0] (n=Miranda@unaffiliated/bmbl) |
| 23:51:28 | CIA-8 | New commit by funman (r23475): Revert r23474 "new skin token: %cx - 24 hour time format enabled in the setting.. e.g %?cx<24 hour|12 hour>" ... |
| 23:51:33 | CIA-8 | New commit by funman (r23476): Sansa AMS recording support (Microphone and FM) ... |
| 23:55:20 | CIA-8 | New commit by funman (r23477): Add Fred Bauer to THANKS |
| 23:55:35 | kugel | funman: \o/ |
| 23:55:47 | funman | kugel: still disabled though :/ |
| 23:56:02 | kugel | that's why I'm wondering why the manual sections are enabled |
| 23:56:12 | funman | are they ? |
| 23:57:19 | kugel | at least the recording related bits |
| 23:57:58 | funman | What I did is add recording keymap for Fuze, and change a section of the recording options |
| 23:58:11 | CIA-8 | New commit by BobC (r23478): Improvements to mini2440/UDA1341 audio |
| 23:58:41 | | Quit bmbl ("Bye!") |
icon identifies that the person is a core developer (has commit access).