00:00:28 | stripwax | kugel - do you have a 'current build' (or equivalent :-) for RaaA Android host? |
00:00:55 | kugel | stripwax: nope :( |
00:01:08 | kugel | you can build one yourself though! |
00:01:15 | gevaerts | kugel: briefly. I hope I'll get back to following things properly tomorrow. |
00:01:19 | gevaerts | Nice work though |
00:01:19 | stripwax | can now! :) |
00:01:31 | kugel | thanks :) |
00:01:38 | | Quit piggz (Ping timeout: 265 seconds) |
00:06:10 | | Join krazykit [0] (~kkit@206.183.185.8) |
00:11:56 | kugel | stripwax: if you have any questions feel free to ask |
00:13:18 | | Quit petur (Quit: Leaving) |
00:13:24 | stripwax | kugel - none yet (although I don't have eclipse on this machine so won't be able to try out the latest version for a little while), but looking great so far! I especially liked the nugget that rockbox seems less cpu intensive than the default media player. |
00:15:23 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
00:23:20 | freddyb | Torne: http://pastie.org/1071768 |
00:23:41 | freddyb | 0.22 seconds? Am I reading that right? |
00:23:42 | CIA-6 | New commit by bieber (r27671): Theme Editor: Fixed rendering of %ca tag |
00:24:11 | Torne | freddyb: yes |
00:25:31 | CIA-6 | r27671 build result: All green |
00:25:48 | bieber | AlexP: Fixed your %ca bug :) |
00:26:17 | AlexP | bieber: ta :) |
00:26:57 | CIA-6 | New commit by bieber (r27672): Theme Editor: Added a default label to switch block in devicestate.cpp to be safe |
00:27:08 | | Quit bmbl (Quit: Bye!) |
00:27:09 | | Quit ender` (Quit: Whenever you cut your fingernails, you will find a need for them an hour later.) |
00:28:13 | jthundley | S_a_i_n_t: thank you so much for the tip earlier, I just finished translating my old custom theme to the new format :) |
00:28:49 | CIA-6 | r27672 build result: All green |
00:32:39 | CIA-6 | New commit by bieber (r27673): Theme Editor: SkinDocument class now checks file extension when loading a document and sets the cs device setting accordingly |
00:32:59 | | Quit stripwax (Quit: http://miranda-im.org) |
00:34:30 | | Quit domonoky (Ping timeout: 265 seconds) |
00:34:47 | CIA-6 | r27673 build result: All green |
00:35:46 | | Join JdGordon [0] (~jonno@rockbox/developer/JdGordon) |
00:37:51 | | Quit Jerom (Read error: Connection reset by peer) |
00:38:43 | | Join Jerom [0] (~heidi@79.132.46.225) |
00:49:03 | *** | Saving seen data "./dancer.seen" |
00:57:13 | | Part jthundley ("Konversation terminated!") |
01:00 |
01:02:08 | JdGordon | kugel: \o/ nice commit |
01:02:20 | | Quit n1s (Quit: Lämnar) |
01:03:21 | | Quit evilnick_B (Quit: Page closed) |
01:09:37 | | Quit Jerom (Read error: Connection reset by peer) |
01:11:47 | kugel | JdGordon: thanks :) |
01:14:08 | | Quit JdGordon (Ping timeout: 265 seconds) |
01:14:35 | kugel | phew, that was tricky |
01:14:46 | kugel | I need to change over to a service for proper background music |
01:18:08 | bieber | kugel: How does the interface work in RaaA? Does it just behave like a normal RB touchscreen target? |
01:18:27 | bieber | I'm totally wishing I had a functioning Android phone to try it out on |
01:18:57 | kugel | bieber: yes |
01:24:00 | | Join Strife89 [0] (~Strife89@adsl-80-169-18.mcn.bellsouth.net) |
01:29:32 | | Quit bug2000 (Ping timeout: 240 seconds) |
01:29:43 | | Quit MethoS- (Read error: Connection reset by peer) |
01:30:30 | TheSeven | fyi, the ipod nano 3g nor flash was just dumped and decrypted. we're facing yet another efi hell. |
01:31:05 | TheSeven | all classic generations will be very similar to this and can very likely be dumped using the same code |
01:31:10 | | Join saratoga [0] (9803c57f@gateway/web/freenode/ip.152.3.197.127) |
01:34:31 | S_a_i_n_t | TheSeven: Linky, please? |
01:35:11 | TheSeven | i don't have the dump yet |
01:35:23 | TheSeven | serpilliere just reported success some minutes ago |
01:35:29 | saratoga | is the linux4nano channel still logged? |
01:35:44 | TheSeven | theoretically yes |
01:35:58 | TheSeven | however our logbots are quite shaky sometimes |
01:36:08 | saratoga | could you PM me the link |
01:36:18 | saratoga | gmail seems to have lost the email |
01:37:01 | TheSeven | however, there wasn't much being said regarding this on irc |
01:38:12 | S_a_i_n_t | bieber: Is there a way I can load an existing theme in its entirity into the themeeditor? |
01:38:24 | bieber | Do you mean from a ZIP file? |
01:38:26 | S_a_i_n_t | I assume it's possible, but I can't figure it out. |
01:38:40 | S_a_i_n_t | bieber: Not necessarily. |
01:38:48 | S_a_i_n_t | just, "load the whole theme" |
01:39:11 | bieber | There's no unzipping feature (I should probably add one), but if you just extract the theme into a folder, "Open project," and open the .cfg file it should all be there for you |
01:40:19 | kugel | I think I get hit by http://code.google.com/p/android/issues/detail?id=9663 |
01:40:45 | S_a_i_n_t | bieber: Hmmmm...that's what I thought. |
01:40:56 | S_a_i_n_t | For some reason none of the images are displaying. |
01:41:07 | | Join bug2000 [0] (~bug@unaffiliated/bug2000) |
01:41:28 | S_a_i_n_t | I'll grab a newer version of the editor. |
01:41:43 | bieber | Do you mean in the theme previews? |
01:42:02 | bieber | They don't show up in the "Project Files" list, if that's what you mean |
01:43:26 | S_a_i_n_t | bieber: Theme Prievew, yes. |
01:43:47 | kugel | shit |
01:43:51 | S_a_i_n_t | Another thought I had was the option to switch fonts...is that possible? |
01:43:59 | S_a_i_n_t | (for the editor itself) |
01:44:05 | bieber | Hmm, could you show me the theme? I haven't had any problems getting images to display before |
01:44:18 | bieber | File -> Preferences lets you change fonts/colors in the editor :) |
01:44:31 | S_a_i_n_t | Seeing my code in a non-monospaced font looks *weird* ;) |
01:44:35 | S_a_i_n_t | oh, cool. |
01:45:08 | bieber | It _should_ try to select a monospace font by default, but Qt doesn't seem to be the best at finding an ideal default font |
01:45:24 | S_a_i_n_t | bieber: I may be using a more than slightly dated version...I'll grab the newest build I can and try it there. |
01:45:54 | bieber | Okay, let me know how it goes |
01:48:32 | S_a_i_n_t | do all the files need to be in the same directory, or can it have the subdir structure as intended for use on device? |
01:48:46 | bieber | It has to be in the same dir structure as the device |
01:49:02 | bieber | wps images in /wps/themename/ and so on |
01:49:33 | bieber | The only real difference is that there's no obligation to make your root directory .rockbox, it'll automatically change that for you when you export |
01:50:16 | S_a_i_n_t | Sweet, one last thing *very minor) is I find it a little overwhelming that the editor "fullscreens" on first run. |
01:50:33 | S_a_i_n_t | If you have a rather large monitor its a little overwhelming ;) |
01:50:39 | bieber | Hmm |
01:50:56 | bieber | I could have sworn I set it to 1/4 the screen size on first run, lemme check on that |
01:51:04 | bieber | You don't have multiple screens perchance, do you? |
01:51:06 | S_a_i_n_t | As I said though, in the scheme of things that's *very* minor. |
01:51:33 | S_a_i_n_t | Re: multiple monitors, no. |
01:51:40 | * | kugel tries the emulator |
01:53:33 | bieber | Hmm, it starts out small for me, it's got to be something wrong with getting the screen geometry on Windows |
01:54:03 | kugel | fuck fuck fuck fuck |
01:54:27 | kugel | RaaA won't be usable on a wide range of devices |
01:54:46 | S_a_i_n_t | bieber: What do the colours mean? |
01:54:51 | S_a_i_n_t | (text colour) |
01:55:11 | S_a_i_n_t | In general, I see red (perhaps it is marone in this case) as "bad" |
01:55:20 | | Join JdGordon| [0] (~jonno@rockbox/developer/JdGordon) |
01:55:21 | bieber | Do you mean in the preferences dialog? |
01:55:38 | S_a_i_n_t | I see a lot of red, yet I also see "The document parsed successfully" |
01:55:57 | bieber | Oh, the default highlighting for tags is red |
01:56:07 | S_a_i_n_t | Hmmm, no, I mean in the large central code viewing panel. |
01:56:07 | bieber | Good point there |
01:56:42 | bieber | If you want to change the highlighting colors, it's under the "Highlighting" tab in the preferences |
01:56:47 | S_a_i_n_t | It made me thing I was doing something wrong initially, until I noticed the "blah parsed correctly" |
01:56:54 | bieber | I probably should come up with a better default color |
01:56:56 | S_a_i_n_t | *think |
01:56:59 | bieber | I'm just not sure which one to use |
01:57:13 | JdGordon| | keep red for errors :) |
01:57:19 | S_a_i_n_t | various shades of green? |
01:57:28 | bieber | But green says comment |
01:57:32 | kugel | rasher: ping |
01:57:45 | bieber | Hmm, I just stumbled on a dark shade of gold I rather like |
01:57:52 | S_a_i_n_t | grren (roughly) universally means "good", and red (again, roughly) universally means "bad" |
01:58:00 | S_a_i_n_t | *green |
01:59:07 | bieber | Hehe, I think my perception is warped by my current IDE's default highlighting :P |
01:59:29 | S_a_i_n_t | heh ;) |
02:00 |
02:00:18 | S_a_i_n_t | bieber: One last, last thing...is there a way to cycle through animations/'simulated' track playback progress? |
02:00:47 | bieber | Yeah, bring up the timer panel from the "View" menu |
02:00:52 | | Quit Rob2222 (Quit: Rob2222) |
02:00:59 | S_a_i_n_t | Excellent, thanks again. |
02:01:12 | bieber | That lets you step through simulation time. The animations aren't perfect because of the intervals it updates at, but it gives you a good rough idea |
02:01:35 | bieber | The play/etc. buttons at the bottom control the player's play mode, not the timer, btw, in case that's ambiguous |
02:02:03 | kugel | saratoga: saratoga: http://www.alice-dsl.net/simonemartitz/rockbox/Rockbox.apk |
02:02:25 | bieber | S_a_i_n_t: If you come up with a set of highlighting colors you like, feel free to send them along to me and I'll switch the defaults. In general, I'll defer to just about anyone on design decisions ;) |
02:02:46 | bieber | Graphic design, that is |
02:02:56 | kugel | install it, run it, then press home/start the terminal emulator. run top at look if org.rockbox has "fg" under PCY |
02:03:13 | S_a_i_n_t | bieber: Last, last, last thing: Is it possible to specify if the "track" being "played" has AA or not? |
02:03:42 | bieber | Yep, it's under Playlist/Song info in the device options panel |
02:03:57 | | Join Rob2222 [0] (~Miranda@p4FDCA047.dip.t-dialin.net) |
02:04:16 | bieber | Towards the very bottom of the tab |
02:04:33 | saratoga | kugel: installing |
02:04:35 | S_a_i_n_t | thanks, quite easy to get lost in the tabs ;) |
02:04:42 | JdGordon| | kugel: haha we really need a new sysfont for big screen targets :) |
02:05:08 | kugel | that apk comes with a bigger font at least |
02:05:15 | bieber | Yep, those are a little bit of a mess. If you have any suggestions for rearranging them, by the way, let me know |
02:05:16 | saratoga | kugel: just gives me a blank screen |
02:05:37 | kugel | strange |
02:05:38 | saratoga | oh wait |
02:05:40 | saratoga | now its going |
02:05:44 | saratoga | took 30+ seconds |
02:05:51 | JdGordon| | kugel: no PCY column in top? |
02:06:01 | kugel | saratoga: it installs the themes etc on the first run |
02:06:34 | kugel | I have " PID CPU% S #THR VSS RSS PCY UID Name" in both my legend's and the emulators top |
02:07:21 | saratoga | how do the controls work? |
02:07:32 | kugel | grid mode :) |
02:07:38 | JdGordon| | what is PCY? |
02:07:51 | kugel | I'm assuming policy |
02:08:12 | kugel | it denotes whether that process is a fore- or background process |
02:09:10 | saratoga | ok figured out the controls, it plays music |
02:09:29 | saratoga | it seems to skip once and a while |
02:09:31 | JdGordon| | woo \0/ |
02:10:21 | kugel | saratoga: it's possible that the dream is too slow to handle the UI at the same time |
02:10:21 | saratoga | get test codec working so i can benchmark |
02:10:33 | kugel | anyway, can you run top for me? :) |
02:10:50 | saratoga | its very similar to the gigabeast CPU |
02:10:57 | saratoga | 450MHz ARM11 I think |
02:11:06 | saratoga | hmm just crashed without me touching it :( |
02:11:26 | saratoga | ha and it doesn't like it when the screen rotates :) |
02:11:36 | JdGordon| | kugel: I assume you've seen the force closes trying to open it from the notifications area? |
02:11:38 | saratoga | but i guess thats our wonderful graphics drivers |
02:11:40 | kugel | ah yes, I forgot to mention that |
02:11:45 | JdGordon| | and sound works fine on the N1 :) |
02:12:02 | | Quit efyx (Remote host closed the connection) |
02:12:05 | kugel | JdGordon|: yes, I've seen that |
02:12:38 | kugel | guys, can you finally tell me what top says under PCY on your devices? :) |
02:13:01 | JdGordon| | tell me another way to get to it... no PCY column |
02:13:42 | JdGordon| | 28% cpu playing music :/ |
02:13:42 | kugel | I don't know another way |
02:13:56 | JdGordon| | and 40% ram |
02:14:02 | kugel | that's the ui. go to the main menu at it will be around 6-7% |
02:14:13 | JdGordon| | just not good enough for a first attempt! :D |
02:14:22 | saratoga | kugel: about 20-23% CPU with the app minimized |
02:14:53 | saratoga | seems pretty stable as long as I don't have teh GUI up |
02:15:10 | kugel | quite the opposite here :) |
02:15:13 | JdGordon| | yeah, music in the background, the display shouldnt be updating at all.... this is what we were talkking about last night. I can make the themes not update at all if the display isnt visible |
02:16:22 | S_a_i_n_t | bieber: One thing I have just noticed in the editor: "%s%al%?Sr<%pe %Sx(of) %pp|%pp %Sx(of) %pe>" displays as " 10 20 " |
02:16:55 | kugel | saratoga: also no PCY under top? |
02:17:08 | S_a_i_n_t | I can hand you the theme if you like, still (aaaargh, been a LONG time) a WIP |
02:17:36 | S_a_i_n_t | by the way, images display fine in the new(er) version I am using. |
02:17:41 | S_a_i_n_t | bieber: ^ |
02:18:43 | saratoga | how do i make top only show the most busy processes, it doesn't fit all on my phones screen |
02:19:17 | kugel | can you scroll? |
02:19:50 | bieber | S_a_i_n_t: Looking at that line now |
02:19:57 | JdGordon| | kugel: can rockbox be closed? |
02:20:08 | saratoga | yeah but i get a tenth of a second before it scrolls me back :) |
02:20:09 | kugel | not yet |
02:20:47 | bieber | S_a_i_n_t: What _is_ %Sx? |
02:20:48 | S_a_i_n_t | bieber: Do you want the theme? |
02:20:54 | bieber | I can't find it in CustomWPS |
02:21:08 | S_a_i_n_t | bieber: That's because it isn't there ;) |
02:21:16 | bieber | Okay, found it |
02:21:20 | bieber | I had no idea that tag existed :P |
02:21:26 | S_a_i_n_t | It's "Translate <string> to user selected lang" |
02:21:33 | bieber | Which is why it's not rendering for you ;) |
02:21:43 | S_a_i_n_t | Aha ...makes sense. |
02:21:46 | bieber | I'll go ahead and implement that tonight, I've gotta run for now |
02:21:59 | S_a_i_n_t | I guess you also need all the lang files in the editor then. |
02:22:18 | S_a_i_n_t | (for %Sx to work) |
02:22:27 | saratoga | leaving the GUI up causes more skipping and about 60% cpu load with the app minimized |
02:22:36 | bieber | For now I'm just going to display the text |
02:22:46 | saratoga | PCY=fg |
02:23:04 | saratoga | THR#=9 |
02:23:15 | kugel | saratoga: try ctrl+c, you could scroll back then |
02:23:16 | S_a_i_n_t | bieber: Aha...good compromise. |
02:23:25 | bieber | I don't know if translation needs to be testable in the editor, but I at least want to get _something_ displaying there |
02:23:33 | saratoga | yeah thats what i ended up doing, just took me a while to figure out how to do a control c on a keypad :) |
02:23:54 | S_a_i_n_t | however, it'd be nice to have the lang files in though, as the line *wil;l* fail if the string isn't present in the lang file. |
02:23:59 | S_a_i_n_t | bieber: ^ |
02:24:08 | kugel | "ball+c" in the terminal I just installed |
02:24:14 | bieber | Interesting |
02:24:23 | S_a_i_n_t | So, I'd say translation needs to be implemented in the editor. |
02:24:30 | JdGordon| | na |
02:24:35 | bieber | I honestly know just about nothing about translation at the moment, I'll have to read up on it |
02:24:54 | S_a_i_n_t | JdGordon|: ...really? |
02:25:02 | kugel | saratoga: did you go to the main menu in rockbox before leaving the app? |
02:25:20 | kugel | I have not implemented stopping lcd updates so it might help |
02:25:29 | S_a_i_n_t | The idea is for the editor to make a file that *guarantees* working-ness on device no? |
02:25:49 | JdGordon| | the most I would expect would be to have the english strings know, no need to actully translate them |
02:26:00 | S_a_i_n_t | A wrong string being translated (or attempted to) is a pretty good way to make a theme not load. |
02:26:13 | S_a_i_n_t | Ah, good point on that. |
02:26:30 | JdGordon| | how is %St handled? |
02:26:31 | S_a_i_n_t | yeah, it definitely needs a current english.lang |
02:26:48 | S_a_i_n_t | re: %St ...pass. |
02:27:05 | S_a_i_n_t | That's black magic as far as I'm concerned. Never had to use it. |
02:27:36 | bieber | S_a_i_n_t: It's really more of a suggestion than a guarantee ;) |
02:27:56 | S_a_i_n_t | I know only *roughly* how it works on device, no idea about it in the editor. |
02:28:17 | bieber | Are translation files customarily distributed with themes? If so, it should be easy enough to implement, otherwise it's really beyond the scope of what I want the editor to cover |
02:28:27 | JdGordon| | they are not |
02:28:54 | S_a_i_n_t | "translation files" are the .lang files themselves. |
02:29:17 | JdGordon| | for completeness it would be nice if it could download the latest english.lang from svn and parse it, but not a high priority imo |
02:29:23 | S_a_i_n_t | %Sx9<string>) just points to a string present in 'english.lang' |
02:29:44 | S_a_i_n_t | bah, %Sx(<string>) |
02:30:16 | saratoga | kugel: if I go to the main menu its about 20%, otherwise more like 65% or so, and music skips |
02:30:33 | | Quit fdinel (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
02:30:45 | S_a_i_n_t | ie. You can't just translate %Sx(foo) without "foo" being a valid string in englisg.lang |
02:30:52 | S_a_i_n_t | bieber: ^ |
02:31:00 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
02:32:33 | bieber | I see |
02:32:55 | bieber | I'll definitely put it on my time-permitting to-do list, it just won't be an urgent priority at the moment |
02:33:11 | S_a_i_n_t | Oh, sure. |
02:33:40 | kugel | saratoga: it's about 25% vs 6-8% on my legend, your dream is quite slow it seems |
02:33:59 | kugel | saratoga: but you at least get "fb" under PCY, that's broken on my legend |
02:34:45 | kugel | on various other htc phones as well |
02:35:10 | saratoga | maybe because I have 2.2? |
02:35:45 | kugel | either that, or because you don't have an official htc image |
02:36:26 | kugel | I need rasher to test it on his froyo'd desire |
02:36:43 | kugel | if the issue is there too I see no hope for RaaA on HTC phones |
02:36:53 | JdGordon| | why? |
02:37:22 | kugel | it makes running raaa in the background practically impossible |
02:37:54 | kugel | "bg" means a hell lot of stuttering audio when you only swipe the fingers over the display |
02:38:02 | saratoga | would this include your phone? |
02:38:12 | kugel | yea |
02:38:28 | kugel | I hope htc fixed it in 2.2 |
02:38:43 | JdGordon| | you cant bump the thread priority or something? |
02:39:02 | kugel | that doesn't help |
02:39:03 | S_a_i_n_t | bieber: Another observation is that if backspace is held, the code on screen is not updated until backspace is released. So, if you hold backspace you get the impression it does nothing, but when releasing it you realise it deleted a large chunk ;) |
02:39:05 | JdGordon| | is this still not using the asm code? |
02:39:22 | S_a_i_n_t | bieber: Same thing goes for Delete |
02:40:14 | kugel | every bg process is put into some low prio corner with all others bg process. they all together get very little cpu time from the kernel |
02:40:24 | saratoga | most of the CPU time here probably isn't the codecs themselves, which even in plain c will be pretty fast on such CPUs |
02:40:57 | saratoga | hmm actually it might be |
02:41:02 | saratoga | i need to play with this more |
02:41:12 | | Quit DerPapst (Quit: Leaving.) |
02:41:36 | JdGordon| | disableing the peak meter should help a bit also.. forces 25FPS refresh rate on the wps |
02:42:44 | saratoga | ouch my phone just rebooted :( |
02:43:00 | JdGordon| | nice |
02:43:03 | saratoga | apparently i need to get out of rockbox before plugging in USB, not just pause it |
02:45:17 | FlynDice | kugel: Cool themes now! froyo on my EVO if I can test anything for you? |
02:47:51 | kugel | FlynDice: I only need to know if rockbox runs fg process if you leave it to the home screen |
02:49:04 | *** | Saving seen data "./dancer.seen" |
02:51:12 | | Join Strife1989 [0] (~Strife89@adsl-80-159-207.mcn.bellsouth.net) |
02:51:14 | FlynDice | It continues to play and there is an icon up in the notification area if that's what you're getting at... |
02:51:48 | saratoga | flac is about 9%, so maybe the codecs are just really slow without ASM |
02:52:08 | saratoga | i would expect flac to be maybe 3x faster then MP3 |
02:52:52 | | Quit bieber (Ping timeout: 260 seconds) |
02:53:07 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
02:53:18 | kugel | FlynDice: you can see the scheduling group if you run top in a terminal or adb shell |
02:53:23 | | Quit Strife89 (Ping timeout: 240 seconds) |
02:54:28 | FlynDice | I'm not rooted, can I do that with USB debugging? |
02:55:16 | kugel | yea |
02:59:53 | kugel | saratoga: for the asm optimizations to work you somehow need apps/ know about the arm cpu, but not firmware/ |
03:00 |
03:00:34 | | Quit freddyb (Quit: ChatZilla 0.9.86 [Firefox 3.5.9/20100401213457]) |
03:01:06 | JdGordon| | just ignore it in firmware/ ifdef ANDROID? |
03:03:51 | | Quit bieber (Ping timeout: 265 seconds) |
03:04:06 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
03:06:11 | kugel | JdGordon|: did it really use 40% ram? |
03:06:21 | JdGordon| | thats what is says |
03:06:48 | kugel | does it mean 40% of the entire ram, or of the 10MB heap reserved for each dalvik vm(i.e. app) |
03:06:50 | saratoga | kugel: do you have a config file for RAAA? |
03:07:16 | kugel | saratoga: firmware/export/config/application.h if you mean that |
03:08:32 | JdGordon| | kugel: I'm giong to guess the later because the numbers are all around the 40% mark |
03:08:38 | kugel | JdGordon|: I have plenty free ram with rb running |
03:08:44 | saratoga | kugel: i guess there should probably be an application-android.h somewhere that defines CPU_ARM |
03:10:25 | kugel | JdGordon|: it should use about 10MB (a bit more probably) in total, 8 of which is the audio buffer |
03:11:03 | JdGordon| | yeah, thats what I thought... if its 40% of the heap then its fine |
03:11:40 | saratoga | eventually we'll probably want to do hardware specific optimizations on RAAA targets |
03:12:13 | kugel | do we? |
03:12:43 | | Join Strife89|Laptop [0] (~Strife89@adsl-80-159-207.mcn.bellsouth.net) |
03:13:07 | kugel | I think speed is not important enough to make it dependant on certain phones |
03:13:20 | kugel | I would rather have the same binary to run on as many phones as possible |
03:13:55 | saratoga | i suppose android phones are close enough that it doesn't matter |
03:14:11 | saratoga | but at very least the system should be aware that its on ARM |
03:14:34 | saratoga | using DSP cores might be a nice option too if the NDK exposes them |
03:16:03 | | Quit Strife1989 (Ping timeout: 240 seconds) |
03:23:05 | JdGordon| | kugel: how is the java<->c bridge done? i mean can the c code send messages or something to java? |
03:23:28 | kugel | JdGordon|: look at the source code :) |
03:23:49 | JdGordon| | simple example, can rockbox ask for the android keyboard? |
03:23:49 | kugel | you can call c functions from java and java methods from c |
03:24:27 | JdGordon| | so.. yes? |
03:25:10 | kugel | I would think so |
03:25:24 | | Quit komputes (Quit: I haven't slept for ten days, because that would be too long.) |
03:31:48 | * | JdGordon| wonders if we can use the notification area like a remote lcd :) |
03:51:36 | | Quit kugel (Remote host closed the connection) |
03:52:47 | | Join stooo [0] (~sto@e179059119.adsl.alicedsl.de) |
03:58:21 | FlynDice | kugel: (logs) EVO with froyo, cpu 21%, S, #THR 10, VSS 169804K, RSS 26404K, PCY fg |
04:00 |
04:02:52 | bieber | S_a_i_n_t: That's for performance purposes. On a good sized skin, the editor was hanging pretty badly when the delete key was held because it was rendering too frequently, so now there's a brief delay between editing and rendering |
04:14:26 | | Quit anewuser (Quit: FAMICOMPO 7 - August 15 Through August 31 JST - http://midr2.under.jp/compo/vol7/index.html) |
04:19:18 | | Quit Barahir (Ping timeout: 260 seconds) |
04:20:46 | | Join Barahir [0] (~jonathan@frnk-590ff750.pool.mediaWays.net) |
04:22:20 | | Quit bieber (Ping timeout: 252 seconds) |
04:22:32 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
04:27:54 | | Quit amiconn (Disconnected by services) |
04:27:57 | | Join amiconn_ [0] (quassel@rockbox/developer/amiconn) |
04:28:03 | | Nick amiconn_ is now known as amiconn (quassel@rockbox/developer/amiconn) |
04:28:45 | | Quit pixelma (Disconnected by services) |
04:28:47 | | Join pixelma_ [0] (quassel@rockbox/staff/pixelma) |
04:29:02 | | Nick pixelma_ is now known as pixelma (quassel@rockbox/staff/pixelma) |
04:31:51 | | Quit RemDeprived () |
04:39:12 | | Quit TheSeven (Ping timeout: 276 seconds) |
04:40:36 | | Quit elcan (Remote host closed the connection) |
04:40:39 | | Join elcan [0] (user36@pr0.us) |
04:43:24 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
04:49:06 | *** | Saving seen data "./dancer.seen" |
04:58:18 | | Part toffe82 |
05:00 |
05:02:29 | | Join larrythechamp [0] (~larrythec@c-98-222-44-4.hsd1.il.comcast.net) |
05:05:15 | larrythechamp | Hello |
05:06:16 | | Join kramer3d [0] (~kramer@unaffiliated/kramer3d) |
05:20:01 | | Join Tarvis [0] (~Tarvis@48.193.100.97.cfl.res.rr.com) |
05:21:07 | | Quit larrythechamp (Quit: Leaving) |
05:23:55 | | Join BHSPitLappy [0] (~BHSPitLap@unaffiliated/bhspitmonkey) |
05:34:38 | | Quit Horscht (Quit: Verlassend) |
05:40:22 | | Quit Tarvis (Quit: /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit /quit) |
05:51:04 | | Quit rob (Ping timeout: 260 seconds) |
06:00 |
06:02:46 | | Nick Strife89|Laptop is now known as Strife89 (~Strife89@adsl-80-159-207.mcn.bellsouth.net) |
06:04:56 | | Quit fdinel (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
06:07:05 | | Quit panni_ (Quit: ( www.nnscript.de :: NoNameScript 3.81 :: www.XLhost.de )) |
06:16:35 | | Quit bug2000 (Ping timeout: 246 seconds) |
06:17:04 | | Join rob [0] (~nnscript@adsl-69-208-82-85.dsl.klmzmi.ameritech.net) |
06:25:47 | | Join r0b- [0] (~nnscript@adsl-76-235-186-78.dsl.klmzmi.sbcglobal.net) |
06:28:27 | | Join bunnyboi [0] (~androgyne@cpe-72-224-31-169.nycap.res.rr.com) |
06:29:37 | | Quit rob (Ping timeout: 276 seconds) |
06:31:36 | | Quit BHSPitLappy (Read error: Operation timed out) |
06:34:09 | | Join captainkewllllll [0] (~captainke@207-38-215-126.c3-0.nyr-ubr1.nyr.ny.cable.rcn.com) |
06:34:27 | | Quit captainkewllllll (Client Quit) |
06:49:07 | *** | Saving seen data "./dancer.seen" |
06:49:34 | | Quit kramer3d (Quit: Leaving) |
06:54:10 | | Join rob [0] (~nnscript@adsl-76-235-206-149.dsl.klmzmi.sbcglobal.net) |
06:54:19 | | Quit r0b- (Read error: Connection reset by peer) |
06:55:53 | | Join r0b- [0] (~nnscript@adsl-76-235-206-149.dsl.klmzmi.sbcglobal.net) |
06:58:52 | | Quit rob (Ping timeout: 276 seconds) |
07:00 |
07:01:39 | | Quit lestatar (Ping timeout: 260 seconds) |
07:19:21 | | Join bieber_ [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
07:29:50 | | Join LinusN [0] (linus@rockbox/developer/LinusN) |
07:44:51 | | Join Buschel [0] (~chatzilla@p54A3F36E.dip.t-dialin.net) |
07:48:02 | CIA-6 | New commit by bieber (r27674): Theme Editor: Implemented Sx tag in renderer |
07:49:58 | CIA-6 | r27674 build result: All green |
07:52:18 | | Join fyre^OS [0] (~nnscript@cpe-68-173-233-99.nyc.res.rr.com) |
07:55:09 | | Quit fyrestorm (Ping timeout: 240 seconds) |
07:56:46 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
08:00 |
08:05:38 | | Join lestatar [0] (~chatzilla@cpe-72-229-41-214.nyc.res.rr.com) |
08:09:06 | FlynDice | kugel: (logs) http://pastie.org/1072291 <−− cut & paste from top command on HTC EVO with 2.2 FROYO update, with rockbox playing music very nicely, after going back to home screen. |
08:13:44 | | Quit S_a_i_n_t () |
08:18:06 | | Quit saratoga (Ping timeout: 252 seconds) |
08:21:47 | | Join Zagor [0] (bjst@rockbox/developer/Zagor) |
08:22:22 | | Quit Buschel (Quit: ChatZilla 0.9.86 [Firefox 3.6.8/20100722155716]) |
08:22:39 | | Quit lestatar (Ping timeout: 240 seconds) |
08:25:30 | | Join lestatar [0] (~chatzilla@cpe-72-229-41-214.nyc.res.rr.com) |
08:26:00 | | Quit bieber_ (Quit: No Ping reply in 180 seconds.) |
08:26:16 | | Join bieber_ [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
08:26:21 | | Quit fyre^OS (Read error: Connection reset by peer) |
08:33:34 | | Quit bieber_ (Remote host closed the connection) |
08:35:30 | | Join BHSPitLappy [0] (~BHSPitLap@unaffiliated/bhspitmonkey) |
08:47:22 | | Join perfectdrug [0] (~AndChat@ip-90-186-138-224.web.vodafone.de) |
08:49:09 | *** | Saving seen data "./dancer.seen" |
08:50:39 | | Join MethoS- [0] (~clemens@134.102.106.250) |
08:59:56 | | Join funman [0] (~fun@rockbox/developer/funman) |
09:00 |
09:00:18 | | Join hebz0rl [0] (~hebz0rl@dslb-088-065-061-118.pools.arcor-ip.net) |
09:01:02 | | Join S_a_i_n_t [0] (S_a_i_n_t@203.184.2.211) |
09:07:14 | S_a_i_n_t | JdGordon|: Instead of "%xd(playmode, %mp)", howabout "%xd(playmode, %mp(5,3,2,4,1,1))" I see that as still being pretty readable, and it has the benifit of not needing an offset value if you want the first subimage to be an arbitrary number instead of "1" and also allows for the same image to be called more than once. |
09:07:20 | S_a_i_n_t | If I need to explain more, shoot. |
09:07:59 | JdGordon| | not a chance :) |
09:08:07 | S_a_i_n_t | I'd probably still use the "%xd(A,1)" + subline method myself, but it more closely represents the intended change for %xd. |
09:08:08 | JdGordon| | that would mean adding more to the language |
09:08:36 | JdGordon| | the parser wouldnt know that %mp(5,4,3,2,1) isnt a parameter list |
09:08:45 | solrize | anyone know the size of the lua plugin and how much ram it uses? tx |
09:09:12 | S_a_i_n_t | wasn't the deal with the new parser (or one of them) its apparent ability to be expanded with ease? |
09:09:25 | funman | solrize: check the lua.map file |
09:09:29 | JdGordon| | I guess if you were crazy you could do %?mp<5|4|3|2|1> but that defeats the purpose of the feature |
09:09:33 | solrize | funman, thanks |
09:09:59 | JdGordon| | S_a_i_n_t: sure, but %mp() looks like a param list to the parser |
09:10:26 | S_a_i_n_t | JdGordon|: Could you do "%?mp<5|4|3|2|1|1|2|3>" etc.? |
09:10:41 | S_a_i_n_t | And, I guess the brackets were a poor choice |
09:10:53 | JdGordon| | probably, but that seems just as silly as the long form |
09:10:57 | solrize | funman, where do i find that file? it's not in http://svn.rockbox.org/viewvc.cgi/trunk/apps/plugins/lua/ |
09:11:28 | JdGordon| | I guess I could support that with a tiny bit of work |
09:11:39 | S_a_i_n_t | It seems more sensible to me personally, and the ability to call a subimage more than once, or to call subimages out of "order" would be a bg plus IMO |
09:11:56 | S_a_i_n_t | *big |
09:12:38 | * | S_a_i_n_t is trying to compromise ;) |
09:12:55 | JdGordon| | you mean make it so complicated that noone uses it :D |
09:13:20 | funman | solrize: it's in the build folder (apps/plugins/lua) |
09:13:28 | S_a_i_n_t | I think its too restricted presently, people don't have much reason to use it. |
09:13:35 | S_a_i_n_t | It only covers very specific cases. |
09:13:37 | solrize | thanks |
09:15:17 | JdGordon| | the first step would be chaning it to the %xd(name, number) format.. alloiwing number to be skin code is a bigger change which would mean this whole thing would probably happen after 3.7 |
09:15:26 | JdGordon| | but we want 3.7 to be finisehd breaking things |
09:15:43 | S_a_i_n_t | I think the current way of doing it is fine personally, readability would be increased simply by the "%xd(A,1)" change. |
09:16:21 | | Quit r0b- (Read error: Connection reset by peer) |
09:16:22 | S_a_i_n_t | But I'm trying to find a way to (sensibly) expand the proposed addition, if it simply must go in. |
09:17:53 | S_a_i_n_t | "%?mp<%xd(A,1)|%xd(A,2)|%xd(A,3)|%xd(A,4)|%xd(A,5)>" seems pretty damn readable to me, personally. |
09:18:01 | JdGordon| | the code thing is more of a reason to want to change the format |
09:18:08 | | Join petur [0] (d408b802@rockbox/developer/petur) |
09:18:23 | | Part perfectdrug ("Leaving") |
09:18:26 | JdGordon| | like I said (probably not early enough), the code thing would be a later addition |
09:19:13 | | Join Szpila [0] (~sszpila@212.244.249.187) |
09:19:56 | S_a_i_n_t | Let me get one thing straight though...it's not *adding* any functionality is it? Just readability (which seems terribly subjective)...or am I wrong? |
09:20:09 | S_a_i_n_t | Is there something not currently possible this would add? |
09:20:31 | JdGordon| | no |
09:20:51 | | Join perfectdrug [0] (~AndChat@ip-90-186-138-224.web.vodafone.de) |
09:21:37 | S_a_i_n_t | Then I guess the poll should really be not "shpuld X or Y be added or dropped", but "is X really so terribly hard to read it warrants changing?" |
09:22:03 | | Join ender` [0] (krneki@foo.eternallybored.org) |
09:22:04 | JdGordon| | yes and no |
09:22:13 | pixelma | %xd(A,number) will probably look worse compared to %xd(Ae) once you need two digits which could be covered by one letter in the old way, so everything between 10 and 52 |
09:22:36 | JdGordon| | more "does this code feature look useful enough to break themes now with the possibility of the addition at some later time" |
09:22:58 | S_a_i_n_t | pixelma: I had thougt of that as well |
09:23:09 | JdGordon| | pixelma: without thinking... r is which image number? |
09:23:19 | JdGordon| | W is which? |
09:23:41 | JdGordon| | sure it might be more readable, but no way could it be considered more understandable |
09:24:34 | S_a_i_n_t | Regardles if its digits or numbers...you'll still have to count out the subimages one by one if you want them assigned in a "non ordered" way. |
09:25:06 | S_a_i_n_t | counting out A B C D is just the same as 1 2 3 4 to me. |
09:25:13 | pixelma | for me it doesn't matter much, in which "place" which subimage is as you usually only count the one time you write it |
09:25:19 | JdGordon| | IMO if your strips are not in the same arer as the tag you are doing it wrong |
09:25:40 | S_a_i_n_t | Why? |
09:25:47 | S_a_i_n_t | it doesn't *need* to be ordered. |
09:26:01 | S_a_i_n_t | It sensible, sure...but hardly necessary. |
09:26:33 | JdGordon| | what about the %Li (list icon) tag... that has 31(?) options... |
09:27:17 | * | S_a_i_n_t doesn't see the point there...sorry. |
09:27:22 | pixelma | I have a WPS which has one big strip combining all the same sized icons. The first are in order of the corresponding condfitional, then adds the next ones to it - still in order but not starting at 1 or a |
09:27:51 | S_a_i_n_t | pixelma: I do the same in some themes, I expect others do also. |
09:28:20 | S_a_i_n_t | If at all possible I see loading one image a lot better than loading 5 smaller ones. |
09:28:48 | JdGordon| | I think you are both crazy :) |
09:29:04 | S_a_i_n_t | No, we're themers ;) |
09:29:15 | JdGordon| | you say tomato |
09:29:19 | S_a_i_n_t | You write the code, we use/abuse it :P |
09:30:02 | pixelma | S_a_i_n_t: same here... and as the idea of the WPS already made it necessary to put the icons in own viewports this came "for free" |
09:30:20 | S_a_i_n_t | pixelma: Indeed. |
09:30:28 | pixelma | the solution to positioning was already there |
09:31:25 | | Quit stooo (Ping timeout: 265 seconds) |
09:33:39 | JdGordon| | no, I stand by what I said. There is no logical reason to *want* to put all images in a single strip. sure you dont need as many %xl lines, but if you want to have the strip ordered there is N times as many tags to fix, and it doesnt let you/others swap out the strip completly |
09:33:51 | JdGordon| | and counting with letters isnt bloody natural! |
09:34:55 | * | S_a_i_n_t considers 1 load, as opposed to say...5 a WIN. |
09:34:57 | pixelma | I really don't see how %xd(image,%mp) is more readable because you need to know how many conditional states %mp has (can be worse with other conditionak) and you lose "control" if you want to use less |
09:35:36 | pixelma | as in "not fill all conditional branches" |
09:35:58 | S_a_i_n_t | "%xd(image,%mp)" also looks completely foreign to all the other skin code. |
09:36:43 | S_a_i_n_t | Not being able to load individual images conditionally is a bit crap too IMO. |
09:36:59 | S_a_i_n_t | It does seem that this is such a *terribly* limited use case to me. |
09:37:20 | JdGordon| | pixelma: lets pretend I didnt suggest that form, do you not agree that %xd(Fh) is less understandable than %Xd(playmode, 5) ? |
09:37:44 | JdGordon| | s/X/x |
09:38:20 | pixelma | I don't agree as soon as I start imagining a long conditional line with the suggested syntax |
09:39:12 | * | S_a_i_n_t does not, if I want to see what image 'F" is, I look at the %Xl line, naming the images sensibly is just as good IMO |
09:39:34 | pixelma | %xl would be more suitable |
09:39:45 | JdGordon| | %?mp<%xd(a,0)|%xd(a,1)|%xd(a,2)|...> is no worse than %?mp<%xd(aa)|%xd(ab)|%xd(ac)|...> |
09:40:18 | S_a_i_n_t | except when it comes to could digits |
09:40:20 | | Quit BHSPitLappy (Ping timeout: 260 seconds) |
09:40:24 | S_a_i_n_t | (numerically) |
09:40:30 | pixelma | %?mp<%xd(playmode,0)|%xd(playmode,1)|%xd(playmode,2)|...> is |
09:40:45 | S_a_i_n_t | Az would be A26...which seems odd to me. |
09:40:47 | JdGordon| | pixelma: nothing is stopping you from naming them with one letter |
09:41:12 | pixelma | ok, then I load "a.bmp"? |
09:41:28 | JdGordon| | what? no... the %xl line doesnt change |
09:41:57 | S_a_i_n_t | An additional way to do something that can be done (in a better, more flexible way) isn't really necessary...that's the way I look at it. |
09:42:07 | JdGordon| | if you change the way you think about strips then the benefit is huge when the number param can be a tag |
09:42:27 | pixelma | ? |
09:42:42 | S_a_i_n_t | I only see it as limiting, for reasons expressed above. |
09:43:23 | JdGordon| | limiting suggests that it removes features which it doesnt |
09:43:28 | S_a_i_n_t | if you want to use the codec tag, but only want to display a few tags...you just...can't. |
09:43:32 | JdGordon| | you would be free to use it or not |
09:43:43 | S_a_i_n_t | limiting doesn't suggest it removes anything. |
09:43:48 | JdGordon| | my only aim is to make things more flexible and possibly nicer for new people |
09:44:07 | S_a_i_n_t | it's just why would I use X when Y can do it better? |
09:44:25 | JdGordon| | because Y is only better because your method of doing Z is wrong |
09:44:38 | JdGordon| | fix Z and X is much better than Y |
09:44:39 | S_a_i_n_t | If more flexibility is the aim, then I honestly don;t agree with using a tag as a subline param |
09:45:07 | S_a_i_n_t | for the fact that all subimages *must* be used. |
09:45:13 | JdGordon| | ok, so think about simplicity, readability |
09:45:28 | S_a_i_n_t | there's no way to say "right, I only want to use subimages 3,4,5 and 9 here" |
09:45:30 | pixelma | who decides how doing Z is right or wrong? |
09:45:54 | JdGordon| | S_a_i_n_t: untill anything is actually coded it is silly saying "X cant be done" |
09:46:53 | | Quit mc2739 (Ping timeout: 265 seconds) |
09:47:04 | S_a_i_n_t | if simplicity/readability is the case...I don;t agree either. A user would probably still have to/want to use the "subline" way of doing this...and two ways of doing a similar thing in the same set of code is not something I consider to be more readable |
09:47:22 | | Join Rob2223 [0] (~Miranda@p4FDCB786.dip.t-dialin.net) |
09:47:39 | | Quit perfectdrug (Quit: Bye) |
09:47:46 | pixelma | the "number lets you more easily understand which position the subimage is" argument is not very interesting to me. It's true, but if I want to know which part of the subimage is really shown, I'd still start counting with the whole image in front of me to see which is "number 14", then there is no difference if it is "n" or "14" |
09:48:27 | | Join einhirn [0] (~Miranda@bsod.rz.tu-clausthal.de) |
09:48:34 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
09:49:25 | S_a_i_n_t | well, there's a difference in the fact that larger numbers can be represented in a smaller fashion, but that's about it. |
09:49:54 | S_a_i_n_t | personally, I'd rather use "j" than "11" |
09:50:06 | pixelma | me too |
09:50:22 | JdGordon| | I will concede that without being able to use the tag for a number there is little point, but with that ability then all your arguments are flawed. absolutly everything can be done in less text |
09:50:58 | | Quit Rob2222 (Ping timeout: 240 seconds) |
09:50:59 | pixelma | I don't think so |
09:51:00 | S_a_i_n_t | individual conditional elements? |
09:51:04 | JdGordon| | sure |
09:51:19 | S_a_i_n_t | how would that look, as an example. |
09:53:09 | JdGordon| | give me a concrete example how it would actually be needed |
09:53:10 | S_a_i_n_t | for me, it is *really* easy t see what is happening in "%?<mp<%xd(Aa)|%?C<%xd(Ab|%xd(Ac>|%xd(Ad)|etc. |
09:53:42 | pixelma | you are missing some )s there ;) |
09:53:43 | JdGordon| | that line would NOT be a good candiadte for the change anyway |
09:54:04 | S_a_i_n_t | so...original format wins then? |
09:54:14 | pixelma | you said "everything" can be done in less text−−- |
09:54:24 | pixelma | or ... |
09:54:43 | JdGordon| | ok, %xd(blaa, %xx) |
09:54:54 | JdGordon| | where %aa evaluates to "%?<mp<%xd(Aa)|%?C<%xd(Ab|%xd(Ac>|%xd(Ad)|etc. |
09:55:37 | S_a_i_n_t | where are the conditional elements defined in "%xd(blaa, %xx)"? |
09:55:46 | JdGordon| | skin_tokens.c |
09:56:07 | S_a_i_n_t | I don't see *how* %xd(blaa, %xx) can == %?<mp<%xd(Aa)|%?C<%xd(Ab|%xd(Ac>|%xd(Ad)|etc. |
09:56:11 | JdGordon| | you gave me an arbitrary example and I was just showing you that untill anything is coded then i can do anything |
09:56:30 | S_a_i_n_t | if it can, I definitely don;t see it as more readable, as it is impossible to see what is happening. |
09:56:48 | JdGordon| | because that is not a real world example |
09:56:58 | JdGordon| | look at the cabbie image lines.. they are the ideal candidate for this |
09:57:12 | JdGordon| | brb |
09:59:51 | JdGordon| | show me a real line in a skin where adding 50% more chars to the line would make it >50 chars and I'll prove to you this is better |
09:59:59 | JdGordon| | imaginary examples are pointless |
10:00 |
10:00:31 | * | S_a_i_n_t suggests it should ultimately be considered by "what is the point, really, in having two ways to do the same thing" |
10:00:32 | JdGordon| | and 50chars isnt even that long |
10:00:44 | S_a_i_n_t | I really don;t think that "less is better" in all exaples. |
10:01:45 | JdGordon| | I dont see it as comeplete duplication |
10:01:58 | S_a_i_n_t | As far as readability goes, your proposed way is...smaller. The current way it is incredibly easy to see how the line flows, and exactly what it is doing and where...even if it is twice as long. |
10:02:13 | S_a_i_n_t | It's not like the length of the skin file is an issue. |
10:02:20 | JdGordon| | I disagree |
10:02:29 | | Join mitk [0] (~mitk@195.117.162.130) |
10:02:57 | JdGordon| | the old/current way requires you to know the tag value order is 2 places. with my proposal you need to know the order in the bmp ONLY and the number of images |
10:03:06 | | Quit Judas_PhD (Remote host closed the connection) |
10:03:27 | JdGordon| | %xd(playmode, %mp) means "I dont care what the order is, just display the right image" |
10:04:02 | S_a_i_n_t | It also prevents you from selectivly including/discarding an image from that subimage set. |
10:04:03 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
10:04:13 | JdGordon| | %?mp<%xd(Aa)|.....|> requires ou to a) know the order for the image, b) know the order for the code and NOT get it wrong, c) not get lost or miscount the |'s |
10:04:42 | S_a_i_n_t | All things people manage to do fine now. |
10:04:56 | JdGordon| | only because there isnt a cleaner option |
10:05:26 | JdGordon| | selective in/exclusion could be fixed with another tag also |
10:05:31 | S_a_i_n_t | If your proposed option indeed was cleaner, I may agree. |
10:05:45 | S_a_i_n_t | exactly...adding *another* tag. |
10:05:55 | S_a_i_n_t | what's the point? it can all already be done. |
10:06:25 | JdGordon| | %xd(playmode, %zz(%mp, +, 0,2,4,6)) is STILL easier to understand after the initial wtf |
10:06:49 | S_a_i_n_t | if you say so... |
10:06:56 | S_a_i_n_t | two themers disagree though. |
10:07:18 | JdGordon| | I'm still waiting for a counter example |
10:07:31 | S_a_i_n_t | the code already in place? |
10:08:16 | S_a_i_n_t | you're confusing "concise" with readable. |
10:08:27 | JdGordon| | %?mp<%xd(Fa)|%xd(Fb)|%xd(Fc)|%xd(Fd)|%xd(Fe)> is straight out of cabbie... => %xd(playmode, %mp) |
10:08:35 | S_a_i_n_t | being able to walk the line through in ones head is a real plus IMO |
10:08:49 | JdGordon| | you're head is screwed on backwards then! |
10:09:02 | S_a_i_n_t | "%?mp<%xd(Fa)|%xd(Fb)|%xd(Fc)|%xd(Fd)|%xd(Fe)>" is perfectly readable to me. |
10:09:14 | S_a_i_n_t | I'd have to check to see how many tags %mp has, etc. |
10:09:22 | S_a_i_n_t | (in your example) |
10:09:26 | JdGordon| | you have to do that anyway |
10:09:32 | JdGordon| | but only when writing |
10:09:41 | JdGordon| | what about when debugging or modifiying someone elses skin? |
10:09:50 | S_a_i_n_t | sure, but simply by looking at "e", "sweet, there's 5" |
10:09:59 | JdGordon| | scorche|sh: is there anyway to see who voted in the forum polls? |
10:10:27 | S_a_i_n_t | I'm the "no" if it helps. |
10:10:47 | JdGordon| | S_a_i_n_t: and *why* would you ever do that? if you want to change the image then you dont care about the skin code |
10:11:11 | JdGordon| | if you want to add something to that one option then yes this doesnt help you |
10:11:59 | pixelma | what would "%xd(playmode, %zz(%mp, +, 0,2,4,6))" mean? And it doesn't have your advertised advantage of "you only need to know in one place how many branches a conditional has" anymore |
10:12:09 | * | S_a_i_n_t fails to see how "if you want to change the image then you dont care about the skin code" works at all. |
10:12:19 | | Quit Strife89 (Quit: Bed.) |
10:13:08 | JdGordon| | pixelma: that was a stupid example to prove a point about an arbitrary example... that would mean "from the %mp tag only use the value if it is 0,2,4,6" |
10:13:19 | | Join n1s [0] (~n1s@rockbox/developer/n1s) |
10:13:39 | JdGordon| | S_a_i_n_t: if you want to change the play icon... why would you need to see where it is displayed? |
10:14:37 | S_a_i_n_t | JdGordon|: As the subimages may not be ordered exactly in the way that the tag uses them. |
10:14:47 | | Join bmbl [0] (~Miranda@unaffiliated/bmbl) |
10:14:57 | | Quit bunnyboi (Quit: Ex-Chat) |
10:14:59 | JdGordon| | BECAUSE YOU'RE IMAGES STRIPS ARE WRONG! |
10:15:06 | JdGordon| | -' |
10:15:16 | pixelma | he deiced that to be the "wrong way" anyway |
10:15:34 | S_a_i_n_t | s/he/who/? |
10:15:41 | pixelma | I can't see how S_a_i_n_t is a bitmap strip but whatever |
10:15:58 | pixelma | S_a_i_n_t: JdGordon| decided that |
10:16:04 | JdGordon| | if your strips are not sane then sure you need to look in the code, but if they in fact are sane then you only need to know which bmp it is in |
10:17:00 | S_a_i_n_t | im my "battery" bitmapstrip for example, my first 4 images are "charger plugged" "charged" and two for an animation to display during "charging"...how is that not "sane"? |
10:17:14 | S_a_i_n_t | in your way, I'd need to split all those...which is crap. |
10:17:19 | JdGordon| | no you wouldnt |
10:17:36 | JdGordon| | stop adding extra requirements and saying it cant be done |
10:17:41 | pixelma | my bitmap strip is completely sane, thanks |
10:17:45 | JdGordon| | nothing is in code so anything can be done |
10:18:05 | JdGordon| | pixelma: oh for crying out loud, dont take it personally |
10:18:15 | S_a_i_n_t | well, ok, but its certainly an example of "un-ordered" subimages != "wrong" |
10:18:57 | JdGordon| | you are yet to show me an example of a REAL WORLD usage where my answer couldnt be "this is obviously not suited to this" or "we can fix that" |
10:19:15 | S_a_i_n_t | duplication. |
10:19:20 | S_a_i_n_t | I thought I said that. |
10:19:33 | S_a_i_n_t | It can already be done, in a completely sane way IMO. |
10:19:44 | pixelma | JdGordon|: I don't. But I think the discussion becomes a bit silly. Because you claim things themers do currently and which are perfectly possibly being the "wrong way" - while stating before that you don't write WPSs |
10:20:01 | JdGordon| | dupleication also says that about half a dozen tags should be removed, namely "replay gain?" |
10:20:54 | JdGordon| | pixelma: think of it this way. as someone who doesnt code skins because they are painful to write/understand, I'm looking for ways to make it easier for new people |
10:20:57 | S_a_i_n_t | So, just because we can we should implement 2 or 3 ways to do everything? |
10:21:11 | JdGordon| | convincing you two is obviously impossible because you are set in the old way |
10:21:45 | S_a_i_n_t | No, that's totally untrue...I just fail to see the "new" way as being any "better" on "right" |
10:21:57 | S_a_i_n_t | *s/on/or/ |
10:22:19 | S_a_i_n_t | I'm totally willing to try new things, if they make sense. |
10:22:46 | | Join Kitar|st [0] (Kitar_st@BSN-142-70-168.dial-up.dsl.siol.net) |
10:22:59 | S_a_i_n_t | I already have a way which is already coded to do everything you have discussed that I (and other I assume) find perfectly readable. |
10:23:14 | S_a_i_n_t | The sheer volume of themes submitted agrees with that I think. |
10:23:18 | | Join et_ [0] (4d7eee3c@gateway/web/freenode/ip.77.126.238.60) |
10:23:35 | JdGordon| | I disagree |
10:23:46 | JdGordon| | the volume of themes just means that people managed to work out the current system |
10:23:53 | n1s | JdGordon|: pong |
10:23:58 | JdGordon| | it doesnt say anything about how easy/hard it is |
10:24:11 | JdGordon| | n1s: dont worry, was hoping to get a h300 test but commited now :) |
10:24:18 | n1s | ah, ok |
10:24:28 | S_a_i_n_t | And who's to say people will really find your way any easier? |
10:24:31 | S_a_i_n_t | honestly? |
10:24:48 | JdGordon| | which is why I was always happy to keep the old system around |
10:25:12 | pixelma | as I said earlier, I don't think your way is easier to understand - *especially* when only reading. You say that now we would only need to know how many branches a conditional has when creating the image then not anymore when writing. With the change, you would need to look up how many subimages your playmode.bmp has - so what do you gain? |
10:25:29 | S_a_i_n_t | And, that's fine...but I only see the point of adding the "new" way if i actually adds something. |
10:26:47 | pixelma | and the example line with "only use state 2,4,x is completely incomprehensible to me |
10:27:22 | JdGordon| | pixelma: I dont understand that comment, and that example was made up anyway and would probably not be needed |
10:28:50 | JdGordon| | S_a_i_n_t hasnt shown me an example where that request would even be needed |
10:29:21 | S_a_i_n_t | which? |
10:29:35 | pixelma | every other theme lets the stop image blank |
10:29:48 | S_a_i_n_t | heh, was just typing that. |
10:30:05 | JdGordon| | ? |
10:30:10 | S_a_i_n_t | a stop image is only needed in the .sbs, and will probably be a different resolution. |
10:30:26 | S_a_i_n_t | having "stop" in the wps is pointless. |
10:30:36 | S_a_i_n_t | there's a fine example. |
10:30:46 | JdGordon| | a fine example of what? |
10:31:15 | pixelma | not using all conditional branches |
10:31:18 | S_a_i_n_t | "not wanting to use the full set of states the tag offers" |
10:31:26 | S_a_i_n_t | I thought that was the current dispute. |
10:31:48 | JdGordon| | the wps will never go into the STOP state for more than a few milliseconds so no, that isnt an example |
10:33:01 | S_a_i_n_t | It's really hard to argue this when your proposed exaple is theoretically infinite. |
10:33:53 | JdGordon| | no it isnt... the worst tag (except the ones which scale) has 31 options... I am yet to see a theme with it probably because it is so hard to use |
10:33:58 | S_a_i_n_t | you're saying "but it *could* do this, and that, and that", eventually it *would* be a complete duplication of the current way of doing this. |
10:34:21 | S_a_i_n_t | JdGordon| You haven't seen Symmetry then? |
10:34:37 | S_a_i_n_t | that uses %Li |
10:34:41 | JdGordon| | that sets them all to one image doesnt it? |
10:35:00 | S_a_i_n_t | yep, but, that's besides the point. |
10:35:14 | JdGordon| | not at all |
10:35:38 | JdGordon| | obviously in that case you would use %?Li<%xd(image)> |
10:36:12 | S_a_i_n_t | assuming you wanted all the states filled though. |
10:36:16 | S_a_i_n_t | what if you didn't? |
10:36:45 | JdGordon| | it depends how many you want to actually use |
10:36:58 | S_a_i_n_t | then you either need to use the current way of doing it, or tag on some other functionality to the "new" method. |
10:37:27 | S_a_i_n_t | with the current way, it is *really* easy to see where a state doesn't have anything defined. |
10:37:30 | JdGordon| | xhjbkjblk;mnsdf;LSDC';. |
10:38:03 | JdGordon| | no, you are wrong |
10:38:35 | S_a_i_n_t | Oh...right, that's because you keep saying "well, it isn't coded yet...so it can do anything" |
10:38:43 | pixelma | worse would be my "home stereo" theme where I want to show an image when e.g. bass/treble is *not* 0 so I know I need to load the correct cfg. On the M5 bass/treble can be set in a range of -24 to +24 IIRC - even while the complete conditional is loooong and I can miscount, I don't see how a looong bitmap strip that has the same picture in 48 places and nothing in the other would help me here |
10:38:47 | S_a_i_n_t | So, how can we even argue for or against it? |
10:39:02 | JdGordon| | if you dont want to show all 30, (maybe you want ot reuse 1 5 times) the logical way is to just copy that image in the bmp strip so later you can change it much more simply |
10:39:37 | S_a_i_n_t | JdGordon|: doesn't that needlessly waste skin buffer. |
10:39:46 | S_a_i_n_t | s/doesn't that/that does/ |
10:40:01 | JdGordon| | you would use FAR more allocating the extra conditionals and options |
10:41:10 | JdGordon| | each skin element is 16bytes (I think), each token adds another 24 + strings from the skin are added ontop of that |
10:41:25 | S_a_i_n_t | When you actually have some code, I'll be happy to revisit this conversation. For now, it is nothing but unproductive. Anything we say you can counter with "well, it isn't written yet, so it can do anything" |
10:41:34 | JdGordon| | whereas a 8x8x16 bmp is barely noticable |
10:42:06 | JdGordon| | how will starting coding change that? untill i see actual real world examples I dont know what actually needs adding |
10:42:33 | CIA-6 | New commit by bieber (r27675): Theme Editor: Removed markup comments from CodeEditor files (it was originally a Nokia example, so it was marked up with comments for their ... |
10:42:40 | S_a_i_n_t | Well, based on the current code and its possibilities *nothing* needs adding...that's my point. |
10:42:56 | | Quit mc2739 (Ping timeout: 240 seconds) |
10:43:42 | S_a_i_n_t | And having actual code to show would stop this "well, it isn't written yet, so it can do anything" business. |
10:43:49 | | Join DerPapst [0] (~Alexander@dslb-088-069-136-249.pools.arcor-ip.net) |
10:44:00 | S_a_i_n_t | We would also have a specific example of what you believe "cleaner" to be. |
10:44:28 | | Quit ender` (Read error: Connection reset by peer) |
10:44:31 | CIA-6 | r27675 build result: All green |
10:44:32 | JdGordon| | %?mp<%xd(Fa)|%xd(Fb)|%xd(Fc)|%xd(Fd)|%xd(Fe)> is straight out of cabbie... => %xd(playmode, %mp) |
10:44:41 | JdGordon| | I'm waiting for a counter example |
10:45:00 | | Join mc2739 [0] (~mc2739@rockbox/developer/mc2739) |
10:45:01 | | Join ender` [0] (krneki@foo.eternallybored.org) |
10:45:04 | JdGordon| | im heading home... take a 45min break to find your worst image line example |
10:45:07 | S_a_i_n_t | I gave one "%?mp<%xd(Fa)|%xd(Fb)|%xd(Fc)|%xd(Fd)|%xd(Fe)>" is *much* easier to walk through. |
10:45:21 | | Quit JdGordon| (Quit: leaving) |
10:47:53 | pixelma | this example just proves that things can be written differently and that JdGordon thinks it's more readable while S_a_i_n_t and me don't think so... so I don't know what another example should "counter" |
10:48:31 | S_a_i_n_t | Me either. |
10:49:13 | *** | Saving seen data "./dancer.seen" |
10:51:30 | S_a_i_n_t | My ffw/rew animation is a pretty good example of "how the *fuck* would this work...and would it be any more readable than now": |
10:51:35 | S_a_i_n_t | %t(0.1)%?mp<|||%xd(Aa)|%xd(Ae)>;%t(0.1)%?mp<|||%xd(Ab)|%xd(Ad)>;%t(0.1)%?mp<|||%xd(Ac)|%xd(Ac)>;%t(0.1)%?mp<|||%xd(Ad)|%xd(Ab)>;%t(0.1)%?mp<|||%xd(Ae)|%xd(Aa)>;%t(0.1)%?mp<|||%xd(Af)|%xd(Ah)>;%t(0.1)%?mp<|||%xd(Ag)|%xd(Ag)>;%t(0.1)%?mp<|||%xd(Ah)|%xd(Af)> |
10:52:15 | S_a_i_n_t | basically, ffw loads the images in the bitmapstrip in forward order, rew loads them in reverse. |
10:52:57 | S_a_i_n_t | Once you get your hard around its sheer length, it's realy easy to see what is happening IMO |
10:56:16 | pixelma | example from cabbiev2: %?mm<|%xd(Ea)|%xd(Eb)|%xd(Ec)|%xd(Ed)> -> leaving out the conditional branch for "Repeat off" |
10:56:55 | S_a_i_n_t | All my themes do that. |
10:57:13 | pixelma | I already know his answer "put a blank image into the strip" |
10:57:50 | pixelma | and I seem to have completely baffled everyone with the bass/treble example :P |
10:57:51 | S_a_i_n_t | which needlessly wastes buffer IMO, "Oh, but we have more buffer now...so it's OK" |
10:59:10 | S_a_i_n_t | pixelma: It didn't baffle me thankyou! ;) |
10:59:47 | S_a_i_n_t | I'd like to see my seek animation in the "new" format. |
11:00 |
11:00:30 | | Quit bmbl (Ping timeout: 258 seconds) |
11:01:03 | S_a_i_n_t | s/like/possiblyvery much dislike/ ...though, I'm still open. |
11:02:18 | S_a_i_n_t | The point I think is being missed is that I'm not (nor do I think you are pixelma)saying "No, don't add this as additional functionality" but rather "is it really needed in the first place, if so...why?" |
11:02:48 | pixelma | the bass/treble example is real btw., I just didn't finish the theme yet but I want to one day (and the line is already there as a proof of concept though) |
11:03:29 | S_a_i_n_t | my seek animation is real also...I have a battery animation that I'd need to pastebin. |
11:03:34 | S_a_i_n_t | it is *looooooong* |
11:05:41 | S_a_i_n_t | pixelma: A real line, in a current theme of mine: http://pastebin.org/444590 |
11:05:58 | S_a_i_n_t | it just *seems* daunting, but couldn't be simpler IMO |
11:07:14 | pixelma | wow... iCatcher does similar things (also seek, pause and battery charging animations etc.), not to the same extreme but close |
11:09:00 | | Quit et_ (Quit: Page closed) |
11:09:48 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
11:10:09 | S_a_i_n_t | The new parser will let me cut that down *dramatically* though... |
11:10:14 | | Quit MethoS- (Remote host closed the connection) |
11:10:33 | S_a_i_n_t | It's just an example of a really intensely long line, that is still dead easy to read (IMO) |
11:15:35 | | Quit bmbl (Quit: Verlassend) |
11:15:44 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
11:16:32 | S_a_i_n_t | pixelma: The new parser lets me convert that line to: |
11:16:34 | S_a_i_n_t | http://pastebin.org/444623 |
11:16:46 | S_a_i_n_t | So, I think that is a bit of a win ;) |
11:17:17 | S_a_i_n_t | I'm all for the new parser, but "readability" is so terribly subjetive I think. |
11:17:50 | pixelma | my standard "not home" theme doesn't use different image definitions in conditionals much as the choice is made by using conditional viewports mostly. Idea was to show things in different colour depending on state (e.g. battery low, volume muted or above 0dB). I achieve this by using different viewports (with set %Vb/f) and monochrome bitmaps which take foreground and background colour |
11:18:24 | pixelma | on c200 that is - and in a bit cut down version with grey shades on M5 |
11:19:02 | S_a_i_n_t | Hmmm, have you gone through your themes looking for code the new parser lets you do better? |
11:19:30 | pixelma | no, I haven't even updated everything yet |
11:19:40 | S_a_i_n_t | ;) took me a while also. |
11:21:09 | pixelma | but I don't have a lot of animations so I doubt it ;) . I'm trying to think if %Vb/f seperately would help me here but can't think of how currently |
11:22:21 | pixelma | hmm... can you pit %Vb/f in a conditional? |
11:22:30 | pixelma | or put |
11:22:41 | S_a_i_n_t | sublines nested in conditionals is the only thing I have really found like "OMG, that is *so* much better" so far. |
11:23:15 | S_a_i_n_t | pixelma: I'm not sure...I doubt it. |
11:23:21 | S_a_i_n_t | possibly though. |
11:25:44 | pixelma | maybe then I'd only need one viewport and loading the three foreground colours conditionally, instead of 3 differnt viewports loaded conditionally. Would help readability |
11:26:16 | S_a_i_n_t | Well, not really...just make smaller code (IMO) |
11:28:00 | pixelma | well one viewport instead of three... taking care of when which is loaded. It looks like this in the old syntax (snippet) http://pastebin.com/f45x62z8 . One thing is nice though, I don't have to use conditionals in the viewport though |
11:28:51 | pixelma | -though |
11:29:18 | n1s | oooh, very nasty, our flac deoder doesn't fail gracefully for files with too large block sizes |
11:30:29 | | Quit Judas_PhD (Quit: Aaarrrggghhh!!!) |
11:31:05 | n1s | it does give some very nice rythmic bursts of whitenoise though |
11:31:44 | | Join bimbel [0] (~Miranda@unaffiliated/bmbl) |
11:31:44 | | Quit bimbel (Client Quit) |
11:33:26 | pixelma | S_a_i_n_t: but that doesn't have to do with the proposed image and conditional change - in fact, I currently only show one battery icon as "explanation" of the value that follows |
11:34:39 | | Join JdGordon [0] (~jonno@rockbox/developer/JdGordon) |
11:36:59 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
11:40:56 | | Part Szpila |
11:45:55 | | Quit S_a_i_n_t (Ping timeout: 245 seconds) |
11:48:59 | CIA-6 | New commit by nls (r27676): flac: error out if max blocksize is larger than we can handle. |
11:50:52 | CIA-6 | r27676 build result: All green |
11:51:32 | pixelma | Zagor: it's about the FM and V2 Recorder manuals. You changed the platform file name and some* options to archosfmrecorder - it was recorderv2fm or similar before because the manual "treats" both the same, you can't see this from the platform file and option names anymore though and may wonder what the V2 one does (you'll only find a few lines in the code and manual stating this) |
11:52:38 | | Quit sasquatch (Read error: Operation timed out) |
11:54:32 | pixelma | *) not all options that would have needed renaming have been. Leading to things wrongly included or excluded but I go through this now slowly step by step, because some things can also be written more correctly while at it |
11:56:57 | Zagor | pixelma: is the question "was this intentional or a mistake"? |
11:57:33 | | Join efyx [0] (~efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
11:59:39 | pixelma | the question is more or less how to deal with it - keep pretending in the naming that there is no V2 manual? It's confusing me a bit - the combined name was clear, now it's not that obvious that this is for both manuals, separating them completely would mean unnecessary code duplication |
12:00 |
12:00:08 | pixelma | the combined name wasn't "clear" but gave a hint at least |
12:07:10 | | Quit Dreamxtreme__ (Ping timeout: 245 seconds) |
12:10:49 | | Join sasquatch [0] (~username@p4FC65B08.dip.t-dialin.net) |
12:11:45 | | Join S_a_i_n_t [0] (S_a_i_n_t@203.184.2.90) |
12:12:06 | | Quit linuxguy3 (Read error: Operation timed out) |
12:13:08 | JdGordon | S_a_i_n_t: (nice timing :) ) lets try a different approach... the line you pasted really just says "I want the *correct* image for the %mp tag drawn" right? |
12:14:15 | | Join linuxguy3 [0] (~timj@adsl-76-202-219-170.dsl.emhril.sbcglobal.net) |
12:15:22 | JdGordon | actually, forget it... I'm going to do the whole patch and see how it goes, if it goes in after 3.7 and people complain that the themes are broken again, well, I wont accept the blame |
12:16:31 | funman | no hurry for 3.7 |
12:16:40 | funman | we can take the time to finish themes work |
12:17:02 | JdGordon | but in reposone to http://www.rockbox.org/irc/log-20100803#10:51:35 I'd say %?if(%mp, eq, 4)<%t(0.1)%xd(Fa);%t(0.1)%xd(Fb)> |
12:17:34 | JdGordon | funman: well, I dont tihnk there is any support for postponing releases except for bug fixes which this couldnt be considered as |
12:18:11 | JdGordon | that line would be read as "if %mp equals 4 then do the block in the true branch" |
12:18:45 | JdGordon | pixelma's supposedly scary example would be done the same way |
12:18:54 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
12:19:50 | pixelma | where, what? |
12:20:20 | pixelma | also, what's %?if |
12:20:43 | funman | JdGordon: on the other side, is there support for pushing releases? |
12:21:22 | JdGordon | funman: imo releases are stupid and a waste of time :D I'm in a very small minority though |
12:22:01 | JdGordon | pixelma: http://www.rockbox.org/irc/log-20100803#10:38:43 which I assume is what http://www.rockbox.org/irc/log-20100803#10:57:50 is referring to |
12:22:08 | funman | JdGordon: we'll be two then |
12:22:26 | JdGordon | %if is a new tag which lets you do basic logic |
12:23:00 | funman | no %switch or %for ? :/ |
12:23:24 | JdGordon | %? is bassically a swicth anyway |
12:24:19 | JdGordon | "for" could be done easily as a counter |
12:24:47 | funman | how can i exit rockbox on android? |
12:25:00 | JdGordon | force close it in the settings :) |
12:25:40 | | Quit anewuser (Ping timeout: 240 seconds) |
12:26:22 | pixelma | so, how is that %?if line more understandable, despite being shorter |
12:26:55 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
12:27:54 | JdGordon | you honestly dont tinhk that is easier to read than "%t(0.1)%?mp<|||%xd(Aa)|%xd(Ae)>;%t(0.1)%?mp<|||%xd(Ab)|%xd(Ad)>;%t(0.1)%?mp<|||%xd(Ac)|%xd(Ac)>;%t(0.1)%?mp<|||%xd(Ad)|%xd(Ab)>;%t(0.1)%?mp<|||%xd(Ae)|%xd(Aa)>;%t(0.1)%?mp<|||%xd(Af)|%xd(Ah)>;%t(0.1)%?mp<|||%xd(Ag)|%xd(Ag)>;%t(0.1)%?mp<|||%xd(Ah)|%xd(Af)>" |
12:28:06 | JdGordon | actually I just noticed my line is wrong |
12:29:49 | JdGordon | %if( %mp, eq, 3)<%xd(animation, %loop(0.1, 1, 5, 1)>%if( %mp, eq, 4)<%xd(animation, %loop(0.1, 1, 5, -1)> |
12:30:20 | JdGordon | no need to try to figure out which subline and which branch and which conditional you are in |
12:32:48 | | Quit AndyI (Ping timeout: 248 seconds) |
12:33:10 | JdGordon | we could go RPN and do %?eq(%mp, 3)<|> even :) |
12:39:35 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
12:42:12 | * | S_a_i_n_t personallt thinks %if( %mp, eq, 3)<%xd(animation, %loop(0.1, 1, 5, 1)>%if( %mp, eq, 4)<%xd(animation, %loop(0.1, 1, 5, -1)> looks *terrible* |
12:42:21 | S_a_i_n_t | And prefers his massive subline. |
12:42:53 | S_a_i_n_t | its implementing a buttload of things for stuff that can already be done. |
12:43:21 | JdGordon | that doesnt mean the current way is good |
12:43:34 | S_a_i_n_t | Nor does it mean yours is any better. |
12:44:06 | JdGordon | better is obviously subjective, so at least we can agree it is different |
12:44:32 | | Quit AndyI (Ping timeout: 248 seconds) |
12:44:57 | JdGordon | and no, %if(%mp, eq, 3) is FAR more readable and FAR less likely to cause problems than %?mp<||||...> |
12:44:58 | S_a_i_n_t | after you implement all this though, it *will* ne an almost complete duplicate of current code...so what motivation is there to keep the current code in place? |
12:45:05 | * | S_a_i_n_t thinks that answer is easy, none. |
12:45:10 | S_a_i_n_t | And, it sucks balls. |
12:45:32 | S_a_i_n_t | and no, %if(%mp, eq, 3) is FAR more readable and FAR less likely to cause problems than %?mp<||||...> <−−says who? |
12:46:02 | S_a_i_n_t | I thought the idea was to simplify it? |
12:46:03 | JdGordon | says me... and I'm sure I can get others to agree... counting bars sucks ass |
12:46:10 | S_a_i_n_t | A;;; I see there is overcomplication. |
12:46:12 | JdGordon | and your line wouldnt be possible to be removed... |
12:47:15 | S_a_i_n_t | "I bet I can get others to agree" just sounds like a majority vote threat....not looking at the "if it 'aint broke, don;t "fix" it" principle. |
12:47:33 | S_a_i_n_t | If I thought what you were doing was actually "fixing" anything, I wouldn;t care. |
12:48:11 | S_a_i_n_t | Quite simply, it ain't broke. All I have seen you say is really "I (you, personally) don;t understand it" |
12:48:26 | S_a_i_n_t | ...learn to understand it, why implement a brand new way? |
12:49:03 | JdGordon | because IMO the current way is shit, and the benefit of being able to code means I can do something about it |
12:49:15 | *** | Saving seen data "./dancer.seen" |
12:49:24 | JdGordon | I tinhk its pretty stupid to think *I* dont understand it |
12:49:51 | S_a_i_n_t | You understand the code, but not how a WPS should be written optimally IMO |
12:50:15 | S_a_i_n_t | the fact two themers outright dissagree has to say something...no? |
12:50:51 | JdGordon | I disagree |
12:50:59 | JdGordon | it just says they are close minded :) |
12:51:02 | JdGordon | stuck in the old way of doing things |
12:51:05 | S_a_i_n_t | But I know enought by now to know that whatever the conclusion, you'll do it anyway...so, fuck it. |
12:51:16 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
12:51:42 | JdGordon | I probably wouldnt have got the extra ideas if you didnt ocme up with nonesense examples :) |
12:51:59 | S_a_i_n_t | that's bullshit and you know it. |
12:53:53 | JdGordon | bugger... does the theme editor not ocmpile in svn? |
12:54:29 | JdGordon | bieber: ping? |
12:55:19 | S_a_i_n_t | I wonder, seriously why you even bothered to ask anyones opinion about this. |
12:55:48 | S_a_i_n_t | You were given it, but, you didn't like it so it doesn't matter to you...that's crap. |
12:56:06 | JdGordon | obviously to get the discussion for the ideas to be finished before starting codeing... obviously I didnt care what you were going to say.... :/ |
12:56:25 | JdGordon | *OBVISOULY* |
12:58:24 | S_a_i_n_t | Well, yeah. I don't see how that can have any air of sarcasm. |
12:59:59 | | Join stooo [0] (~sto@e179059119.adsl.alicedsl.de) |
13:00 |
13:00:53 | | Quit milz (Read error: Connection reset by peer) |
13:03:27 | pixelma | well, it boils down to "a different way" and after reading the examples 5 times I can see that it's easier to read in *some* cases but *only* if you implement all the other stuff. IMO that's much too much at once and I'd like to also be convinced that this is simpler and at least not too bin- or RAM-size heavy too. If you want so much at once, you should rather think about creating a whole new theme language, well thought through from the beginning |
13:03:27 | pixelma | :\ |
13:04:14 | | Quit stooo (Client Quit) |
13:04:38 | S_a_i_n_t | Or pick something to "imporve" that is flawed to begin with. |
13:04:45 | pixelma | that "eq" thing was only understandable btw. after realising that it stands for "equal" |
13:04:51 | S_a_i_n_t | *improve even |
13:05:49 | JdGordon | pixelma: heck, I could make it "equals" instead of "eq" to make it more readable |
13:06:42 | S_a_i_n_t | I can't see what happens (or would happen) if you want tag states to be independently conditional. |
13:07:13 | JdGordon | hu? |
13:08:30 | | Join mt_ [0] (~mtee@41.233.149.87) |
13:08:43 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
13:08:56 | S_a_i_n_t | %?mp<%xd(Aa)|%?C<%xd(Ab)|%?xd(Ac)>|%xd(Ad)|etc. |
13:09:07 | S_a_i_n_t | ...how does that work in your scenario? |
13:09:16 | JdGordon | it doesnt |
13:09:30 | JdGordon | I'm not replacing anything... im giving you another option |
13:09:35 | * | S_a_i_n_t has no use for this new implementation. |
13:09:36 | | Quit mt (Ping timeout: 240 seconds) |
13:09:40 | | Nick mt_ is now known as mt (~mtee@41.233.149.87) |
13:09:44 | JdGordon | ok, fine, good for you then |
13:10:31 | pixelma | I'd find another option a waste then too |
13:11:43 | S_a_i_n_t | IMO if you're going to duplicate features, the newer implementation should at least be able to do what the old one does first, before adding to it. |
13:11:59 | pixelma | you can already get the results with the current implementation. If the new one doesn't add much wrt possible results - having to things doing the same is a waste |
13:12:20 | pixelma | s/to/two |
13:12:46 | S_a_i_n_t | seeing different code doing the same thing in a theme defeats the "readability" argument also. |
13:12:51 | JdGordon | gevaerts: how do I get libskin_parser to compile with -g for checkwps? |
13:14:05 | funman | JdGordon: add -g in CFLAGS |
13:14:10 | JdGordon | GCCOPTS in the checkwps Makefile does have -g |
13:14:29 | funman | lib/skin_parser/Makefile ? |
13:15:34 | funman | btw perhaps we should always force -g |
13:15:59 | funman | and strip the binaries in specific cases (e.g. when creating android package) |
13:16:20 | JdGordon | ah got it |
13:16:26 | JdGordon | I think I just had to make clean && make :p |
13:16:43 | pixelma | maybe JdGordon's new syntax can be a translation layer in the theme editor > he can read it all he wants and it won't cost anything on target. No functionality is lost or duplicated on the target and one can read it the old way too </actually semi-serious> |
13:16:47 | | Join Topy [0] (~Topy44@f049112224.adsl.alicedsl.de) |
13:17:09 | * | JdGordon mashes head on keyboard |
13:17:33 | JdGordon | you understand that the current code is less optimised for target than my suggestion right? |
13:17:38 | JdGordon | in absolutly every way |
13:17:50 | S_a_i_n_t | yet, it does more. |
13:18:12 | JdGordon | THIS ISNT MEANT TO BE A BLOODY REPLACEMENT.... GOD FUCKING DAMMIT |
13:18:58 | funman | at least, it is a good flaming material |
13:19:04 | pixelma | yes, that's why it's not optimised for target. Having two implementations just can't be |
13:19:35 | pixelma | this time I really mean "size" of the parser too |
13:20:17 | S_a_i_n_t | Where did the specific need for this pop up? Did anyone (other than yourself) actually say "Gee, the current way %xd works regarding sublines is really confusing and impossible to read"? |
13:20:30 | | Quit funman (Quit: leaving) |
13:20:31 | | Quit T44 (Ping timeout: 265 seconds) |
13:24:52 | AlexP | JdGordon: Don't bother arguing - he's a "themer" and therefore all knowing about themes |
13:25:31 | S_a_i_n_t | Yeah, that's construstive...thanks for that. |
13:25:42 | AlexP | As was much of your argument |
13:25:58 | AlexP | For what it is worth, I quite like the idea, but I'm not bothered enough to insist |
13:27:19 | S_a_i_n_t | Think of it this way, would another tetris plugin, that actually does less than the current one, have any shot of being committed? |
13:27:33 | gevaerts | yes |
13:27:39 | AlexP | I just love lines like "[12:49:50] <S_a_i_n_t> You understand the code, but not how a WPS should be written optimally IMO" |
13:27:40 | gevaerts | See rockblox1d |
13:27:43 | n1s | we did include such a plugin |
13:27:44 | AlexP | So condescending |
13:27:53 | AlexP | I am the themer, only I can have an opinion! |
13:29:34 | AlexP | You can of course take any opinion you like on something, but to discount someone's opinion as they aren't a "themer" and you are the king of all themers (apparently) is just ridiculous |
13:30:05 | S_a_i_n_t | That's your perception of it, and you can have that if you want...but you're incorrect. |
13:30:17 | AlexP | It is how you come across |
13:30:24 | JdGordon | *ahem* time to shut the fuck up :) this isnt a social channel |
13:30:52 | S_a_i_n_t | it's a pretty fucking antisocial channel for the past few hours... |
13:30:52 | | Join t0rc [0] (~t0rc@unaffiliated/t0rc/x-5233201) |
13:31:17 | AlexP | "[12:50:14] <S_a_i_n_t> the fact two themers outright dissagree has to say something...no?" implies very heavily that people on the other side of the argument aren't "themers" and therefore their opinion doesn't count |
13:31:40 | S_a_i_n_t | No, it doesn't. |
13:31:47 | S_a_i_n_t | That's just the way you want to read it. |
13:33:18 | S_a_i_n_t | If I wanted to say, or even thought anyones opinion didn;t count...I'd damn well say so. |
13:34:06 | JdGordon | NOOOOOO!!!!!!!!!! CRAP! I think my parser optimisations last week may have screwed me here |
13:34:29 | | Join TehSpawn [0] (www-data@giant.haxx.se) |
13:35:18 | | Quit TehSpawn (Client Quit) |
13:35:43 | * | gevaerts doesn't really like the idea of multiple ways to achieve the same things (even if they do differ in some features), but he hasn't followed the discussion enough to state a firm opinion |
13:36:02 | | Join SpawnHyuuga [0] (www-data@giant.haxx.se) |
13:36:16 | | Join kugel [0] (~AndChat@e178058189.adsl.alicedsl.de) |
13:37:00 | * | kugel also dislikes redundancy |
13:37:21 | | Quit SpawnHyuuga (Client Quit) |
13:39:50 | | Join piggz_ [0] (~piggz@78.144.111.12) |
13:51:29 | * | JdGordon needs to thank bieber again for making it so easy to add a new tag :D |
13:55:06 | | Quit kugel (Ping timeout: 240 seconds) |
13:57:16 | | Join kugel [0] (~AndChat@e178058189.adsl.alicedsl.de) |
13:58:50 | | Quit t0rc (Quit: Give someone code, help them with one project. Teach someone to code, help them rule the world.) |
13:59:19 | | Quit piggz_ (Ping timeout: 246 seconds) |
14:00 |
14:02:39 | | Nick fxb__ is now known as fxb (~felixbrun@h1252615.stratoserver.net) |
14:04:52 | | Quit anewuser (Quit: FAMICOMPO 7 - August 15 Through August 31 JST - http://midr2.under.jp/compo/vol7/index.html) |
14:06:24 | JdGordon | ETOOMUCHINDIRECTION element->params[0].data.code->data |
14:14:52 | CIA-6 | New commit by jdgordon (r27677): make -vvv display the parse tree in checkwps. Fix a potential bug in the parser where recursive tags (the playlist viewier) would share params with ... |
14:16:51 | CIA-6 | r27677 build result: All green |
14:22:30 | | Quit feisar_ (Ping timeout: 260 seconds) |
14:24:16 | | Quit bluebrother (Ping timeout: 240 seconds) |
14:24:35 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
14:25:07 | | Quit TheSeven (Read error: Connection reset by peer) |
14:26:11 | | Join bluebrother [0] (~dom@g224237050.adsl.alicedsl.de) |
14:26:11 | | Quit bluebrother (Changing host) |
14:26:11 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
14:28:32 | kugel | FlynDice: ping |
14:40:30 | JdGordon | bieber: can you let me know what I need to change to make FS #11516 work in the editor please? |
14:41:55 | JdGordon | if anyone wants to play I would welcome comments (other than "this adds nothing") |
14:49:17 | *** | Saving seen data "./dancer.seen" |
14:55:35 | | Join AndChat| [0] (~AndChat@e178058189.adsl.alicedsl.de) |
14:55:55 | AndChat| | "this adds nothing" is not a valid reason anymore? |
14:55:55 | | Quit kugel (Ping timeout: 245 seconds) |
14:56:22 | | Nick AndChat| is now known as kugel (~AndChat@e178058189.adsl.alicedsl.de) |
14:57:02 | | Quit mitk (Quit: Leaving) |
14:57:29 | | Quit fdinel (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
14:57:43 | | Join robin0800 [0] (~robin0800@91.107.118.124) |
14:58:02 | | Quit efyx (Remote host closed the connection) |
14:59:40 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
15:00 |
15:00:40 | | Quit fdinel (Client Quit) |
15:02:15 | | Join fdinel [0] (~Miranda@modemcable235.127-131-66.mc.videotron.ca) |
15:02:58 | JdGordon | kugel: it is, I'm just making it known I would ignore that particular one |
15:03:26 | | Quit antil33t (Read error: Connection reset by peer) |
15:03:32 | | Join antil33t [0] (~Mudkips@124-197-51-80.callplus.net.nz) |
15:04:34 | | Join efyx [0] (~efyx@lap34-1-82-225-185-146.fbx.proxad.net) |
15:06:50 | | Join detrick [0] (~detmerz@pool-173-73-141-243.washdc.fios.verizon.net) |
15:07:42 | | Join feisar [0] (jljhook@irkki.fi) |
15:08:10 | | Nick feisar is now known as Guest69382 (jljhook@irkki.fi) |
15:09:05 | | Join piggz_ [0] (~piggz@78.144.111.12) |
15:14:37 | | Join r0b- [0] (~nnscript@adsl-99-156-30-200.dsl.klmzmi.sbcglobal.net) |
15:17:37 | | Join evilnick_B [0] (0c140464@rockbox/staff/evilnick) |
15:26:51 | | Quit r0b- (Ping timeout: 276 seconds) |
15:30:23 | | Join r0b- [0] (~nnscript@adsl-76-235-211-18.dsl.klmzmi.sbcglobal.net) |
15:30:50 | FlynDice | kugel: pong |
15:33:01 | kugel | FlynDice: does it play music well even if if it's in the background, I.e. if you use other apps? |
15:34:52 | FlynDice | kugel: I think so, lemmee test it some. Played fine while I downloaded a terminal app and went through all the settings pages. I'll test it harder right now. |
15:35:57 | | Part LinusN |
15:41:07 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
15:41:42 | | Join jgarvey [0] (~jgarvey@cpe-065-190-066-089.nc.res.rr.com) |
15:41:59 | preglow | kugel: you have an android build hanging around for people to test? |
15:42:53 | kugel | preglow: I linked one for 320x480 tonight |
15:43:50 | preglow | wee, that's my reso |
15:44:01 | kugel | it seems to run also on other resolutions but it'll be a bit awkward |
15:46:00 | preglow | bout when? |
15:46:16 | FlynDice | kugel: It plays fine even when using other apps. I played several games with rockbox playing and there were no hiccups at all. |
15:48:01 | kugel | preglow: 2:02, you need 2.x for it |
15:48:10 | preglow | got 2.x |
15:49:08 | kugel | FlynDice: cool, at least on your phone it works as it's supposed to |
15:49:36 | preglow | i can't find the url for the life of me |
15:49:45 | kugel | various HTC phones ship a broken Android |
15:51:59 | kugel | saratoga: saratoga: http://www.alice-dsl.net/simonemartitz/rockbox/Rockbox.apk |
15:52:38 | kugel | preglow: ^ |
15:54:48 | | Quit slck (Ping timeout: 276 seconds) |
15:55:19 | tmzt | kugel: borken how? |
15:55:26 | | Quit kugel (Quit: Bye) |
15:55:53 | | Join kugel [0] (~AndChat@e178058189.adsl.alicedsl.de) |
15:57:38 | kugel | tmzt: startforeground() doesn't work as advertised |
15:58:18 | tmzt | kugel: you might ask romainguy about that in #android during the afternoon pacific time |
15:58:44 | tmzt | are you using the media scanner or doing that internally to rockbox? |
15:58:59 | tmzt | and will rockbox have access to the sd card directly (similar to simulator I guess) |
16:00 |
16:00:05 | kugel | yes |
16:00:32 | tmzt | to which? sorry for so many questions so quickly |
16:01:01 | kugel | ? |
16:01:14 | tmzt | are you using the media scanner api? |
16:01:21 | kugel | it has direct access to the sdcard |
16:01:31 | tmzt | okay |
16:02:02 | tmzt | media scanner isn't very good anyway, thats seems to be the consensus and a source of many questions in the channel |
16:02:46 | kugel | we use whatever api rockbox has for this |
16:03:27 | | Join slck [0] (Venci@No.Real-Love-For.Me) |
16:04:42 | | Quit r0b- (Ping timeout: 240 seconds) |
16:10:15 | | Quit mt (Ping timeout: 258 seconds) |
16:12:14 | | Join mt [0] (~mtee@rockbox/developer/mt) |
16:12:29 | | Join r0b- [0] (~nnscript@adsl-76-236-190-87.dsl.klmzmi.sbcglobal.net) |
16:17:14 | kugel | tmzt: does romainguy have a clue |
16:17:43 | tmzt | kugel: he developed the main android apis, I would think so |
16:17:57 | tmzt | (he's a google employee) |
16:18:04 | tmzt | oh, that should be #android-dev not #android |
16:18:59 | kugel | I was in that channel a few times now. most of the people there are no help at all |
16:19:03 | JdGordon | bieber: please have a look at FS #11517 which adds optional tag param groupings to the parser (so it guesses which type of param has been given). let me know if you have comments... I have a few ideas which would need this but maybe there is a better way to add this? |
16:19:40 | | Quit JdGordon (Quit: Leaving.) |
16:19:41 | | Quit piggz_ (Ping timeout: 245 seconds) |
16:20:18 | tmzt | kugel: if I see him I'll mention it, they are usually good if you know what to ask |
16:30:44 | | Join panni_ [0] (hannes@ip-95-222-52-93.unitymediagroup.de) |
16:37:56 | | Quit kugel (Remote host closed the connection) |
16:45:39 | | Join saratoga [0] (9803c20d@gateway/web/freenode/ip.152.3.194.13) |
16:45:50 | preglow | how to change font in this android business? :P |
16:48:03 | * | evilnick_B points to the topic and stares at preglow :) |
16:48:58 | | Quit mt (Ping timeout: 258 seconds) |
16:49:21 | *** | Saving seen data "./dancer.seen" |
16:49:49 | pixelma | evilnick_B: he's testing RaaA... |
16:49:57 | preglow | what, i should read the android docs which dont exist? |
16:51:24 | evilnick_B | preglow, pixelma: Apologies for that |
16:54:14 | | Join mt [0] (~mtee@rockbox/developer/mt) |
16:56:04 | | Join newnicj [0] (www-data@giant.haxx.se) |
16:56:33 | newnicj | hi |
16:57:07 | | Quit newnicj (Client Quit) |
16:57:27 | | Part Zagor |
16:58:55 | | Join captainkewll [0] (2669ecc2@gateway/web/freenode/ip.38.105.236.194) |
17:00 |
17:00:13 | preglow | heh, shit crashes if i tilt the phone |
17:01:00 | | Join Judas_PhD [0] (~kevin@misterfluffy.dsl.xmission.com) |
17:03:30 | saratoga | i noticed that |
17:03:38 | saratoga | i think its because we hardcode the resolution in rockbox |
17:03:50 | saratoga | probably doesn't like the LCD resolution changing on the fly :) |
17:03:53 | gevaerts | RaaA is only stable if the phone is kept stable :) |
17:03:58 | preglow | ah, so that's how it's handled |
17:04:09 | tmzt | you basically have to recreate the activity when the orientation changes |
17:04:09 | preglow | it's pretty sluggish |
17:04:14 | preglow | but madly impressive |
17:04:43 | preglow | this has got to be one of the most complex android apps ever |
17:04:46 | preglow | if not the most complex |
17:05:12 | saratoga | is the long term goal to reimplement the lists as (hardware accelerated) Android widgets, or to leave everything using the NDK rockbox code on the CPU? |
17:05:57 | preglow | android lists would definitely be the way to go, if you ask me |
17:06:17 | tmzt | does this work on pre 2.2 versions? |
17:06:28 | tmzt | are you using the newest NDK and opengl directly? |
17:06:38 | preglow | works on 2.1 |
17:06:39 | tmzt | or should that be 2.1 |
17:07:25 | | Nick YPSY is now known as Ypsy (~ypsy@geekpadawan.de) |
17:08:42 | | Quit DarkJuju (Remote host closed the connection) |
17:10:00 | | Quit milz (Remote host closed the connection) |
17:13:12 | saratoga | i wonder how hard it would be to redo the list code for Android |
17:14:37 | tmzt | then what is the goal here? |
17:15:50 | | Quit antil33t () |
17:16:06 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
17:26:49 | | Join funman [0] (~fun@rockbox/developer/funman) |
17:34:17 | | Join stoffel [0] (~quassel@p57B4C2B2.dip.t-dialin.net) |
17:37:53 | | Join DarkJuju [0] (juju2143@gateway/shell/57o9.org/x-vthnzrchsbhntyep) |
17:40:26 | | Quit efyx (Remote host closed the connection) |
17:54:34 | | Quit petur (Quit: *plop*) |
17:55:37 | funman | kugel: ping |
17:57:10 | | Join komputes [0] (~komputes@ubuntu/member/komputes) |
17:57:28 | | Join clone4crw [0] (www-data@giant.haxx.se) |
17:59:00 | | Quit clone4crw (Client Quit) |
17:59:05 | | Quit detrick (Quit: Leaving) |
17:59:13 | | Quit Utchybann (Ping timeout: 260 seconds) |
18:00 |
18:03:58 | | Quit robin0800 (Quit: Leaving) |
18:07:36 | | Join Utchybann [0] (~Utchy@rps6752.ovh.net) |
18:13:25 | | Quit Stummi (Excess Flood) |
18:13:33 | | Join Stummi [0] (stummi@doppeldenk.org) |
18:21:00 | | Join piggz_ [0] (~piggz@78.144.111.12) |
18:26:13 | | Quit DarkJuju (Remote host closed the connection) |
18:26:59 | | Quit piggz_ (Ping timeout: 265 seconds) |
18:27:51 | | Join domonoky [0] (~Domonoky@rockbox/developer/domonoky) |
18:28:05 | | Join piggz_ [0] (~piggz@78.144.111.12) |
18:32:18 | | Quit bieber (Ping timeout: 265 seconds) |
18:32:33 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
18:39:33 | | Quit piggz_ (Ping timeout: 265 seconds) |
18:44:21 | | Join piggz_ [0] (~piggz@78.144.111.12) |
18:49:24 | *** | Saving seen data "./dancer.seen" |
18:52:36 | | Quit bieber (Ping timeout: 276 seconds) |
18:52:52 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
18:53:57 | | Quit jfc (Read error: No route to host) |
18:54:39 | | Join jfc [0] (~john@dpc6682208002.direcpc.com) |
18:55:13 | * | funman can't build rockbox for android |
18:56:49 | | Quit AndyI (Ping timeout: 248 seconds) |
19:00 |
19:00:29 | | Join clone4crw [0] (www-data@giant.haxx.se) |
19:01:36 | | Quit clone4crw (Client Quit) |
19:02:21 | | Join root [0] (~root@96-42-90-88.dhcp.roch.mn.charter.com) |
19:02:35 | | Nick root is now known as Guest3476 (~root@96-42-90-88.dhcp.roch.mn.charter.com) |
19:02:36 | | Quit bmbl (Quit: Verlassend) |
19:02:46 | Guest3476 | Hi, everyone |
19:02:53 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
19:03:08 | Guest3476 | At the moment I'm trying to unbrick a Sansa Fuze V2 that I got off ebay |
19:03:33 | funman | how do you know it's bricked? |
19:04:01 | | Join bmbl [0] (~bmbl@dsl-217-174-198.pool.bitel.net) |
19:04:03 | Guest3476 | Doesn't power up, nothing on screen |
19:04:06 | | Quit bmbl (Changing host) |
19:04:07 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
19:04:20 | tmzt | fully charged? |
19:04:21 | Guest3476 | But it seems to be recognised by Linux |
19:04:24 | funman | check SansaAMS for specific problems, this problem is known, explained, and there is a solution. |
19:04:28 | Guest3476 | I wouldn't know, I assume so |
19:04:39 | Guest3476 | Well, when I run lsusb, it shows up |
19:04:41 | funman | http://www.rockbox.org/wiki/SansaAMS#Specific_problems |
19:04:52 | | Join DarkJuju [0] (juju2143@gateway/shell/57o9.org/x-rvtgqyzjtrrbhfts) |
19:04:58 | | Join bimbel [0] (~bmbl@unaffiliated/bmbl) |
19:05:00 | Guest3476 | and when I run fdisk -l, it also is recognised, but there is no partition table |
19:05:04 | | Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org) |
19:05:05 | | Quit bmbl (Client Quit) |
19:05:13 | | Nick bimbel is now known as bmbl (~bmbl@unaffiliated/bmbl) |
19:05:35 | Guest3476 | so I tried to dd its OF, but it seems to only write about 4 MB out of the 15MB |
19:06:08 | funman | Guest3476: the unbrick page on the wiki says the device must be ~ 1GB |
19:06:29 | funman | you shouldn't have opened it anyway |
19:06:49 | Guest3476 | It's a 4gb model, and reporting about that much |
19:07:28 | funman | how much sectors? (look in dmesg) |
19:08:58 | simonrvn | 'how many' |
19:09:46 | Guest3476 | unless I just don't see it, dmesg isn't saying |
19:10:09 | Guest3476 | unless you mean blocks |
19:10:25 | Guest3476 | 8265 512-byte logical blocks |
19:10:47 | funman | 8265 * 512 is far away from "about 4gb" |
19:13:24 | | Join robin0800 [0] (~robin0800@91.107.247.148) |
19:14:16 | Guest3476 | yes, my bad on that. |
19:14:55 | | Join einhirn [0] (~Miranda@bsod.rz.tu-clausthal.de) |
19:15:06 | funman | stop messing with unbricking, pray for not having broken your fuzev2, and follow the advice on http://www.rockbox.org/wiki/SansaAMS#Specific_problems |
19:15:13 | gevaerts | Does that bit need the OF (as downloaded) written to it, or something slightly different? |
19:15:26 | funman | gevaerts: SansaAMSUnbrick should be clear about it |
19:15:37 | | Quit bieber (Ping timeout: 252 seconds) |
19:15:54 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
19:16:11 | | Quit einhirn (Client Quit) |
19:18:47 | Guest3476 | what precisely am I looking for in the Specific Problems page? |
19:19:28 | funman | last line before 'Port status' |
19:20:13 | | Quit bmbl (Quit: Verlassend) |
19:21:00 | Guest3476 | ah. no hardware power off. |
19:22:04 | Guest3476 | I don't suppose desoldering the battery would be a quicker alternative than waiting a couple days |
19:22:21 | funman | why? it's much quicker if you can solder it back |
19:22:25 | | Join bmbl [0] (~bmbl@unaffiliated/bmbl) |
19:22:43 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
19:23:00 | funman | FWIW, there is a hardware power-off, but somehow it doesn't work in this case. No idea why :/ |
19:25:30 | | Join kugel [0] (~kugel@rockbox/developer/kugel) |
19:26:02 | | Quit stoffel (Ping timeout: 260 seconds) |
19:26:50 | kugel | funman: pong |
19:27:30 | Guest3476 | i really wish i knew what state the battery was in |
19:27:41 | funman | kugel: i could build an apk after some fixes to wma and 1 hour messing with eclipse |
19:27:58 | kugel | wma doesn't built right |
19:28:07 | kugel | but that shouldn't be a problem :) |
19:28:10 | funman | however i can't install it - not sure why: my phone won't tell me why |
19:28:30 | | Join Horscht [0] (~Horscht@xbmc/user/horscht) |
19:28:36 | kugel | funman: is it signed? |
19:28:58 | funman | yes |
19:29:12 | kugel | I noticed it (silently) rejects unsigned apks, but eclipse should sign it for you if you run it |
19:29:35 | funman | ah, it works after desinstalling the previously installed apk (upgrade didn't work) |
19:29:54 | kugel | strange |
19:30:35 | funman | i just removed 'debuggable' field |
19:31:01 | | Quit Guest3476 (Quit: leaving) |
19:33:59 | | Quit milz (Remote host closed the connection) |
19:34:09 | funman | kugel: can i send you patches? |
19:34:32 | funman | i'm tracking gsoc-android-2 branch |
19:34:46 | kugel | yea |
19:35:24 | kugel | you could also commit to svn |
19:35:56 | funman | ah nice |
19:36:21 | | Quit bertrik (Ping timeout: 245 seconds) |
19:36:43 | kugel | rasher: can you try the apk I linked to tonight (at 2:02)? |
19:37:03 | kugel | I'm wondering if the problem is fixed in htc's froyo |
19:38:40 | | Join liar [0] (~liar@clnet-p09-185.ikbnet.co.at) |
19:41:06 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
19:41:31 | CIA-6 | New commit by funman (r27678): configure: fix LCD width/height dialog for android |
19:41:38 | CIA-6 | New commit by funman (r27679): wma: move inline functions into .h file ... |
19:41:55 | kugel | fix? |
19:42:13 | funman | yes |
19:42:23 | funman | help message didn't correspond |
19:42:38 | Bagder | "Please enter" would sound a lot better in my ears |
19:42:39 | funman | so it should have been 'fix help message' |
19:43:04 | funman | oops |
19:43:23 | CIA-6 | r27678 build result: All green |
19:43:25 | funman | or just 'Enter the LCD xxx' ? |
19:43:32 | Bagder | yeah |
19:43:57 | funman | kugel: + echo "Enter the LCD width (default: 320)" |
19:44:00 | funman | works for you? |
19:44:16 | kugel | sure |
19:44:29 | kugel | well, it was probably confusing becuase it says 320x480 in the parenthesis, but it said "separated with enter" :) |
19:44:43 | CIA-6 | New commit by funman (r27680): Typo |
19:45:10 | funman | ah, i skipped this, "320x480" was a long enough description ;) |
19:45:31 | CIA-6 | r27679 build result: All green |
19:45:44 | kugel | funman: the problem with wma is possibly that I added -std=gnu99 and inline of c99/gnu99 behaves differently as the inline in gnu89 (the gcc default) |
19:46:18 | pixelma | does this commit message mean you introduced a typo? ;) |
19:46:18 | kugel | the headers that ship with the android toolchain complain a lot, including about _STDC_ not defined |
19:46:37 | funman | i googled and found this (on gcc 4.3 release notes) but didn't check the exact difference between the two. I don't think we specifically need extern inline, do we ? |
19:47:16 | CIA-6 | r27680 build result: All green |
19:47:40 | kugel | it doesn't make sense to me anyway |
19:48:00 | | Quit DerPapst (Quit: Leaving.) |
19:48:26 | funman | we need a new header to put memset16() |
19:49:01 | kugel | IIRC we have a memory.h, but it would also fit into string-extra.h I guess |
19:49:28 | funman | when the host memory.h is used we don't have it |
19:50:13 | kugel | oh, I didn't know that exists on hosts |
19:50:20 | kugel | it just includes string.h on my system |
19:51:27 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
19:52:42 | funman | /media/bordel/rockbox/apps/main.c:189: warning: function declared 'noreturn' has a 'return' statement |
19:52:45 | funman | :? |
19:53:03 | | Join root [0] (~root@96-42-90-88.dhcp.roch.mn.charter.com) |
19:53:09 | | Nick root is now known as Guest8399 (~root@96-42-90-88.dhcp.roch.mn.charter.com) |
19:53:12 | | Quit bieber (Ping timeout: 240 seconds) |
19:53:26 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
19:53:40 | CIA-6 | New commit by funman (r27681): android: declare audiohw_set_volume() |
19:53:45 | CIA-6 | New commit by funman (r27682): system-android: cast void* to uintptr_t* |
19:54:39 | kugel | funman: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=486347 |
19:54:56 | funman | thx |
19:55:20 | funman | btw the code sourcery people said they 'added my bug report into the database' and it should be fixed at next upgrade ("fall") |
19:55:26 | CIA-6 | r27681 build result: All green |
19:55:31 | kugel | &stack wasn't void*, it was volatime uintptr_t* |
19:55:36 | | Quit pamaury (Remote host closed the connection) |
19:55:49 | Guest8399 | fuman: earlier I was the one dealing with a supposedly bricked fuzev2. I had to reboot my machine. I've disconnected the black (-) wire for the battery, and it the sansa is being reliably recognised. at this point, I am not sure what to do |
19:56:33 | kugel | funman: I would prefer CONFIG_PLATFORM & PLATFORM_ANDROID over defined(ANDROID) |
19:56:41 | | Quit milz (Remote host closed the connection) |
19:56:45 | funman | i copied the def from sound.c ^^ |
19:57:04 | funman | kugel: but stackbegin/end are uintptr_t* (above) |
19:57:07 | CIA-6 | r27682 build result: All green |
19:57:14 | funman | Guest8399: does it work? |
19:57:45 | kugel | funman: no |
19:57:57 | kugel | "uintptr_t *stackend;" |
19:58:17 | kugel | err, that's what you said :) |
19:58:54 | rasher | kugel: it seems to work. What am I looking for? |
19:59:09 | kugel | rasher: does it play music well when you are in other apps? |
19:59:19 | funman | kugel: why uintptr_t* btw? |
19:59:32 | funman | (why the '*' also) |
19:59:36 | rasher | kugel: ah.. let me see.. where do I put music? |
19:59:53 | Guest8399 | it is recognised, but nothing else seems to work. |
19:59:55 | rasher | ah |
19:59:58 | | Join DerPapst [0] (~Alexander@dslb-088-069-136-249.pools.arcor-ip.net) |
20:00 |
20:00:00 | CIA-6 | New commit by funman (r27683): faad: move inline function in header |
20:00:04 | funman | Guest8399: what's "recognized" ? |
20:00:25 | kugel | rasher: it might be news for you, but historically you can put music whereever you want for rockbox :P |
20:00:51 | * | funman misses something |
20:00:52 | kugel | funman: stackend/begin are declared uintptr_t[] on native builds |
20:01:05 | funman | where? |
20:01:09 | kugel | thread.c |
20:01:16 | | Join stoffel [0] (~quassel@p57B4C2B2.dip.t-dialin.net) |
20:01:18 | kugel | it comes from the linker script |
20:01:25 | rasher | kugel: I get an "error accessing playlist control file", I assume that's known? |
20:01:26 | Guest8399 | shows up in lsusb as SanDisk corp, and as a 4mb device without a partition table in fdisk |
20:01:35 | funman | kugel: why volatile? |
20:01:47 | CIA-6 | r27683 build result: All green |
20:01:48 | kugel | funman: to make gcc put it on stack |
20:01:56 | rasher | kugel: seems to work beautifully |
20:02:04 | funman | shouldn't it be static instead? |
20:02:16 | kugel | rasher: that's not expected, no. didn't it create a /sdcard/rockbox directory for you? |
20:02:18 | funman | hmm i see |
20:02:55 | rasher | kugel: hm, now I got a bit of stutter during a screen transition. Managed to open google earth and move around a bit without hiccup though |
20:03:12 | kugel | that's only to get a semi-valid stack address. but effectively it's not used anywhere anyway (only for stack usage statistics which is disabled for the main thread) |
20:03:13 | rasher | also, trying to re-enter Rockbox crashes it |
20:03:43 | kugel | rasher: alright. you'd do me a favor if you could check top when it's running |
20:03:56 | rasher | kugel: I don't see a /sdcard/rockbox, no |
20:03:57 | kugel | what it says under PCY for the org.rockbox process |
20:04:08 | | Join BRi7X [0] (~BRi7X@c-68-80-181-29.hsd1.pa.comcast.net) |
20:04:48 | kugel | it's a bug if it didn't, but you can create it yourself (it's where rockbox stores config.cfg, etc) |
20:05:34 | BRi7X | howdy, I just bought a used Gigabeat S30, the hard drive appears to be corrupted as I can't quite copy any files to it. I was wondering if it'd be possible to replace the hard drive inside of it with a 40GB hdd from a Gigabeat F? |
20:06:01 | evilnick_B | BRi7X: It's the wrong connector inside, unfortunately |
20:06:28 | evilnick_B | The S uses a ZIF connector whereas the F uses IDE |
20:07:07 | gevaerts | BRi7X: what exactly do you mean by "the hard drive appears to be corrupted"? |
20:07:08 | BRi7X | that's the same connector as an iPod 5G, the ZIF, isn't it? |
20:07:09 | rasher | kugel: 9633 35% S 10 172696K 26288K fg app_70 org.rockbox |
20:07:16 | kugel | awesome |
20:07:33 | kugel | so I can expect my phone to work properly once I get a froyo update |
20:07:50 | BRi7X | well, when i first opened the box, I've tried to sync music with WMP using the original firmware, it seemed to have gone on fine, but the interface was incredibly laggy on the device and nothing would work, that was my first clue |
20:08:01 | funman | and again: Problems: "Unable to resolve target 'android-4'" |
20:08:04 | kugel | rasher: it says "bg" on my phone, due to a bug in the OS. that bg makes rockbox barely usable if it's in the background |
20:08:12 | rasher | kugel: surely there must be a way? |
20:08:19 | rasher | I mean, there are other media players |
20:08:20 | funman | can you help me decipher this? |
20:08:31 | rasher | Also, oh my, 35% cpu usage |
20:08:34 | funman | oh done.. |
20:08:53 | kugel | rasher: http://code.google.com/p/android/issues/detail?id=9663 |
20:08:58 | BRi7X | gevaerts: secondly, after beast patching it and revealing the two partitions, copying the rockbox software onto it stayed stuck at 1% and 0kb/s. |
20:09:48 | kugel | I guess the builtin media player / mediaplayer java class doesn't have this problem for some reason |
20:10:11 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
20:10:17 | gevaerts | BRi7X: right. Yes, it could be the disk of course |
20:11:36 | BRi7X | gevaerts, might you have any suggestions as to what i should do? or is there any chance whatsoever that i'd be able to adapt the connector from my gigabeat F hard drive over onto the S? |
20:12:32 | gevaerts | BRi7X: I think the first thing to do would be to make sure this is indeed the problem, possibly by having the OF reformat the disk and see if the problems remanin |
20:12:39 | | Join watto [0] (~watto@193.203.81.165) |
20:13:45 | gevaerts | As for adapting the F drive to the S, I'm not sure. Adapters do exist, but I don't know if the case has enough room. I think the S firmware should handle the disk itself file |
20:13:56 | * | gevaerts would *never* move a disk from the F to the S. He much prefers his F (although he would get an X if he found one) |
20:14:15 | BRi7X | the F was nice, it just kept falling apart, i have a few pictures... |
20:14:25 | | Quit milz (Remote host closed the connection) |
20:14:30 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
20:14:33 | gevaerts | Yes, it tends to do that :) |
20:14:54 | BRi7X | gevaerts: i remember a while ago my friend wanted to put his 80GB ipod HDD into his gigabeat, we ran into the ZIF connector problem, and researched and researched then just sort of gave up |
20:15:06 | BRi7X | but i mean, it seemed possible, we just didn't put forth the effort |
20:15:33 | gevaerts | You'll have to decide if it's worth trying, or if getting a new ZIF disk is a better idea... |
20:16:50 | BRi7X | it seems to be working now, i'm not sure what i did haha |
20:16:53 | bertrik | slashdot has an article about apple chargers, with a link to an article revealing the magic resistor values in the charger |
20:17:23 | BRi7X | gonna test it with the new blind guardian flac really quick. |
20:18:27 | kugel | rasher: it still plays fine? |
20:18:43 | rasher | kugel: oh I stopped it |
20:18:48 | kugel | the cpu usage can be reduced |
20:19:17 | kugel | driving our UI is quite expensive, but that can of course be stopped when it's in the background or when the backlight is off |
20:19:30 | kugel | also, the peakmeter on the wps makes it even worse |
20:19:47 | rasher | I sure need a larger theme |
20:19:57 | kugel | I expect the desire can do rockbox sans UI with under 5% cpu usage |
20:20:51 | funman | kugel: do you use eclipse or CLI tools? |
20:20:55 | rasher | It doesn't help that I've never used a touchscreen target (and the screen size is invisible to me) |
20:21:09 | rasher | okay, playing music now, in the background |
20:21:10 | kugel | funman: eclipse. but I made the apk once via CLI |
20:21:31 | funman | eclipse throws me a 'Conversion to Dalvik format failed with error 1' |
20:21:39 | rasher | or not.. messed up |
20:21:43 | kugel | rasher: on my phone it already drops out when I visit the app menu and scroll a bit |
20:22:06 | rasher | how do I leave the wps? |
20:22:23 | funman | i also needed this diff: http://pastie.org/1073411 |
20:22:42 | rasher | I guess I could read the manual.. |
20:24:41 | rasher | Top-left, it turns out. Trying to make it skip now.. any ideas? |
20:25:09 | kugel | "make it skip"? |
20:26:25 | | Quit hebz0rl (Quit: Ex-Chat) |
20:26:54 | | Quit bmbl (Quit: Verlassend) |
20:27:10 | | Quit mt (Ping timeout: 258 seconds) |
20:27:45 | rasher | Rockbox |
20:27:49 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
20:28:12 | rasher | Seems to hover around 10-15% cpu usage now that I left the WPS |
20:29:08 | kugel | IIRC it still does two lcd updates per second even in the main menu |
20:30:33 | | Join kyle__ [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
20:31:30 | kugel | rasher: 10-15% already beats the built in media player, so it's not that bad. but it can go down more, I had my phone at 6-8% |
20:31:53 | | Quit stoffel (Remote host closed the connection) |
20:31:55 | funman | kugel: what's missing for using asm code? |
20:32:07 | kugel | preprocessor/makefile magic |
20:32:27 | n1s | i think performance is not the thing to focus on at this early stage |
20:32:43 | rasher | Yeah, I wasn't seriously suggesting that |
20:33:00 | rasher | Also, if we're not even using asm code yet, it should go way way way down |
20:33:33 | n1s | rasher: maybe, our asm code isn't really written for the kind of advanced cores in these phones though |
20:33:37 | funman | kugel: can i work on that without stepping on your toes? |
20:33:40 | | Quit milz (Ping timeout: 276 seconds) |
20:33:46 | rasher | n1s: well, I imagine it'd still go down? |
20:33:53 | rasher | Just not as much as it could |
20:34:09 | funman | n1s: you can only target armv5te or armv7-a, nothing in between afaiu |
20:34:45 | n1s | probably, most of it is just written for arm7tdmi so stalls etc are not taken into account |
20:34:55 | kugel | IIRC virtually all android phones have armv6 or armv7 |
20:35:00 | | Join Dreamxtreme [0] (~Dreamxtre@92.30.59.0) |
20:35:21 | n1s | on armv7 neon could be very interesting i think |
20:35:27 | kugel | n1s: most of our asm doesn't give the speed ups it gives for PP on other arm targets |
20:35:56 | kugel | I guess we could safely built armv6 binaries, even the g1/dream is armv6 |
20:36:12 | n1s | kugel: true, and that is mostly because those other targets are pipelined so need careful instruction sceduling to get good performance |
20:36:22 | | Quit Dreamxtreme (Client Quit) |
20:37:40 | | Quit DerPapst (Quit: Leaving.) |
20:38:21 | saratoga | turning on ASM will probably make most of the codecs faster just because the fixmul type functions will be written correctly |
20:38:41 | saratoga | telling gcc to do a 32x32=64 bit mul in c code tends to result in some very ugly ASM on 32 bit cores |
20:41:35 | kugel | doesn't armv6 have some nice instructions for that? |
20:42:31 | n1s | it has 32*32=32hi |
20:42:50 | funman | kugel: are you sure we can build for armv6? NDK mentions only armv5 and armv7 |
20:43:18 | kugel | funman: -march=armv6 ? |
20:43:47 | | Join Dreamxtreme [0] (~Dreamxtre@92.30.117.103) |
20:44:05 | kugel | what the NDK mentions probably refers to the build script it provices |
20:44:07 | n1s | armv5 or armv6 will make very little difference i think, both in regard to our asm (very little armv6 asm written) and gcc |
20:44:10 | kugel | provides* |
20:45:09 | funman | http://pastie.org/1073447 <- define CPU_ARM |
20:45:16 | funman | (v5) |
20:46:02 | kugel | that's all? |
20:46:26 | bertrik | funman, I could reproduce the hang on alarm powerdown on my clip+ :| |
20:46:34 | funman | kugel: it builds, but codecs don't start |
20:46:50 | kugel | did you rebuild them? |
20:46:54 | funman | s/don't start/crash rockbox/ |
20:47:18 | kugel | the __div0 breaks codec api |
20:47:50 | funman | hm i forgot something |
20:48:05 | * | gevaerts tries to remember... |
20:48:38 | funman | kugel: i forgot to remove __div0 from the .c |
20:49:28 | *** | Saving seen data "./dancer.seen" |
20:50:06 | kugel | IMO we should wait until test_codec works before switching on the asm |
20:50:18 | kugel | it might turn out that many of it is not faster or even slower |
20:51:13 | funman | what's missing for plugins? |
20:51:29 | kugel | dunno, I haven't even tried to build them |
20:51:59 | | Nick Ypsy is now known as YPSY (~ypsy@geekpadawan.de) |
20:52:03 | kugel | I guess most will work, but I have not implemented the get_user_file_path() thing for plugins (some write to disk) |
20:52:33 | funman | mp3 works fine |
20:52:48 | funman | perhaps top can give a clue? |
20:53:27 | * | funman requests a 'shutdown' menu |
20:54:17 | | Quit Dreamxtreme (Quit: Don't follow me) |
20:54:47 | funman | kugel: can I commit the diff without config.h ? __div0 / system-arm.c only apply to native builds anyway |
20:55:18 | kugel | so it only needs a single config.h change for enabling? |
20:55:23 | kugel | sounds good |
20:55:43 | | Nick fxb is now known as fxb__ (~felixbrun@h1252615.stratoserver.net) |
20:57:14 | * | funman wonders where __div0 went in sim builds |
20:57:32 | kugel | sim builds don't define CPU_ARM |
20:57:42 | kugel | because sim.h #undefs CONFIG_CPU |
20:58:03 | funman | right |
21:00 |
21:00:17 | | Join Dreamxtreme [0] (~Dreamxtre@92.30.117.103) |
21:00:31 | CIA-6 | New commit by funman (r27684): system-arm.c/__div0 are for ARM native builds only ... |
21:02:11 | CIA-6 | r27684 build result: All green |
21:03:23 | | Quit kyle__ (Remote host closed the connection) |
21:07:29 | | Join einhirn [0] (~Miranda@p54851453.dip0.t-ipconnect.de) |
21:07:35 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
21:08:19 | funman | Guest8399: any progress? |
21:10:21 | | Join DerPapst [0] (~Alexander@p5797CBD6.dip.t-dialin.net) |
21:11:28 | funman | jdgordon: http://anythingbutipod.com/forum/showpost.php?p=488098&postcount=4 |
21:11:44 | funman | "there's no way to reset the fms" |
21:12:44 | funman | saratoga: what makes you think there is a high chance of bricking the clipv2? |
21:12:57 | Guest8399 | Nope :( |
21:14:26 | funman | do you have the battery unsoldered? (1 wire is enough) |
21:14:37 | Guest8399 | Yeah, the black wire |
21:15:13 | funman | when i tried recovery mode, i had either no device at all, either the 1GB one. I don't remember if i saw a 4MB device |
21:15:52 | Guest8399 | i was looking around on the SanDisk forums earlier. This seems to be a kind-of common problem with the v2s |
21:16:54 | bertrik | ooh, that FMS looks nice |
21:17:55 | funman | Guest8399: i'm not aware of bricked fuzev2 except mine |
21:18:25 | Guest8399 | I did try shorting out those two recovery pins with a small screwdriver, and it didn't seem to make a differnce |
21:18:47 | funman | try again and again and again, it's hard to get it at first try |
21:19:03 | Guest8399 | I do not know how this got bricked, i got it off ebay for about $20. It would be real nice to know |
21:19:26 | Guest8399 | I have to get going, I will be back in an hour or so. |
21:20:34 | | Quit stripwax (Quit: http://miranda-im.org) |
21:26:03 | | Join darkenvy [0] (60fe6f59@gateway/web/freenode/ip.96.254.111.89) |
21:26:21 | darkenvy | hey can someone help me out? |
21:26:57 | darkenvy | I didn't want to post on the forums because I can see this as something someone will say 'do a search fisrt' (which I did btw) |
21:28:22 | | Part watto |
21:28:23 | funman | Guest8399: did it happen when you plugged USB cable? |
21:28:49 | funman | darkenvy: if you have a question just ask |
21:29:17 | darkenvy | remap keys on an ipod 5th generation? the play button doesnt work nor does the hard reset. The only way to reset is by removing battery cable |
21:29:49 | evilnick_B | darkenvy: You'd need to download the source, change what the buttons do then recompile |
21:30:11 | darkenvy | and always boot into rockbox. if it goes into original ipod firmware then it must die which takes forever or I must open it up and remove the battery cable |
21:31:19 | darkenvy | ah. thats the only way? no ini file or nuttin? :P |
21:36:04 | n1s | darkenvy: you can install in a special way with ipodpatcher so that the original firmware is removed and dualbooting into it is impossible |
21:36:17 | pixelma | also changing the keymap in Rockbox won't change the hard reset button combo |
21:36:35 | | Part piggz_ ("Konversation terminated!") |
21:37:24 | n1s | do this http://www.rockbox.org/wiki/IpodPatcher#3_OSOS_contains_only_the_Rockbox and just ignore the backup step if you want to be unable to dualboot |
21:38:04 | | Join Jerom [0] (~heidi@95.171.131.249) |
21:38:21 | darkenvy | oh sweet |
21:38:57 | darkenvy | okay so two problems down |
21:39:13 | darkenvy | has anyone ever heard of an unresetable ipod? |
21:39:34 | darkenvy | menu and center does absolutely nothing. both buttons work and both click too |
21:42:51 | evilnick_B | It not working would indicate a hardware issue, but if they work separately then... /me is stumped |
21:43:20 | pixelma | did you try to switch hold to on and off before and did you hold both buttons for a long time? |
21:44:10 | | Quit simonrvn (Quit: brb, silly client) |
21:45:23 | darkenvy | yes I did that too |
21:45:58 | darkenvy | once I basicly lock my ipod into rockbox ill have options to shutdown and reboot so its kinda fine..... untill it freezes :\ |
21:46:36 | | Join simonrvn [0] (simon@210.223-ppp.3menatwork.com) |
21:46:45 | darkenvy | you know what?! whats a great rockbox supported player with a large HDD? (I wanted to upgrade my Ipod hdd to 240gb soooo...... I mean LARGE) |
21:47:14 | AlexP | If you want more than 80 GB you need to upgrade the disk |
21:51:23 | amiconn | n1s, kugel: You will definitely notice the effect of bulding for armv6 whem trying ape :) |
21:51:29 | darkenvy | whats a good rockbox player that I can upgrade? (same kind of HDD right?) |
21:51:43 | BRi7X | darkenvy: gigabeat F, i'd say |
21:52:07 | AlexP | trouble with the gigabeat F is the 50 pin hd connector |
21:52:07 | BRi7X | though gigabeats can be hokey |
21:52:17 | BRi7X | i just got a gigabeat S and the hard drive is HOKEY |
21:52:26 | AlexP | So you are limited at 80 gb (and that drive is very hard to find), or you need an adaptor |
21:53:02 | AlexP | The gigabeat S OF is braindead and doesn't support drives bigger that 137 GB (so 120 GB max) |
21:53:13 | AlexP | to put a big drive in your only real option is an ipod video |
21:53:27 | AlexP | s/OF/bootloader/ |
21:53:49 | darkenvy | damnit |
21:54:02 | darkenvy | hey anyone selling an ipod here? lol |
21:54:15 | BRi7X | well i bought a used one (because they don't exist anymore, so i had to), and the 30GB hard drive in it is fairly corrupted, or so it seems. file transfers fail most of the time. I wanted to put one of the SEVERAL 40GB hard drives from my Gigabeat F laying around, but apparently it's a different connector |
21:54:17 | darkenvy | I have a 6th gen 120gb |
21:54:27 | AlexP | BRi7X: Yes, it is |
21:54:45 | BRi7X | are they adaptable though? |
21:54:46 | darkenvy | ill take it off your hands Bri7x |
21:54:49 | AlexP | I have a 120 GB drive in my S and it works great, but sadly I can't go bigger (stupid toshiba bootloader) |
21:55:09 | AlexP | BRi7X: If you can find an adaptor that'll fit in the case, then yeah |
21:55:22 | AlexP | People have put massive drives in e.g. iriver H100 with an adaptor |
21:55:48 | AlexP | The 50 pin to ZIP adaptors do exist, so many other players become upgradeable if you can fit an adaptor in |
21:55:48 | darkenvy | oh the gigabeat! I always thought it was an ugly player :\ no offense |
21:55:55 | AlexP | depends which gigabeat |
21:56:04 | AlexP | The F is fairly, yes |
21:56:11 | AlexP | The X and S are much better IMO |
21:56:34 | darkenvy | selling ipod anyone? lol |
21:57:12 | BRi7X | the black F is cool |
21:57:47 | BRi7X | will this fit in the S?: http://cgi.ebay.com/1-8-Hard-Disk-Drive-HDD-ZIF-2-5-44-pin-IDE-Adapter-/140424628175?cmd=ViewItem&pt=LH_DefaultDomain_0&hash=item20b1f5c7cf#ht_1808wt_1137 or is that even the right one? |
21:58:11 | AlexP | No |
21:58:20 | AlexP | You don't want a 2.5" HD |
21:58:22 | AlexP | D |
21:59:00 | AlexP | If you want to put a drive from the F in the S, then you want a 50 pin IDE 1.8" HDD to ZIF adaptor, if such a thing even exists |
21:59:27 | AlexP | The other way round certainly does |
21:59:35 | gevaerts | Both ways exist |
21:59:49 | AlexP | OK :) |
21:59:52 | gevaerts | That doesn't mean the case has enough room of course |
22:00 |
22:00:06 | AlexP | That'd be my primary concern |
22:00:34 | BRi7X | yeah when i got it i was surprised about how tiny it was |
22:02:39 | BRi7X | i haven't been inside the gigabeat S yet, so i don't know what fits and what won't, i need a tiny screwdriver. |
22:02:49 | saratoga | funman: not a high chance, but AFAIK the only sansa anyone bricked was a V2 clip |
22:03:45 | | Quit milz (Remote host closed the connection) |
22:03:56 | AlexP | BRi7X: The beast is a bit of a nightmare to get apart (opening the case is OK, and changing battery/disk, but to remove everything is hard) |
22:04:34 | BRi7X | well at the moment i'm concerned with the hard drive... the F was so easy to screw around with, i was fairly surgeonic (word?) with it |
22:04:45 | AlexP | surgical |
22:04:54 | funman | saratoga: was it you who proposed the AMSv2 to be stable for next release? |
22:07:23 | | Join mt [0] (~mtee@rockbox/developer/mt) |
22:07:39 | | Quit darkenvy (Quit: Page closed) |
22:11:12 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
22:12:01 | funman | bertrik: if make fullzip doesn't work, try blaming kugel :) |
22:12:15 | saratoga | funman: no, i have no opinion on that |
22:12:47 | funman | IMO these crashes are pretty annoying even if rare, it could cause even more users opening their players and breaking them |
22:12:47 | saratoga | but i would like to disable rebooting into the OF on USB insert, since that seems to crash some players badly enough that they can't reboot until the battery dies |
22:13:00 | funman | yeah i think the problem is reboot |
22:13:14 | saratoga | i think otherwise crashes are rare |
22:13:25 | funman | i'm ok with disabling reboot until we harass pamaury enough to fix USB :) |
22:13:27 | saratoga | i don't think i've seen one on my players in months |
22:14:03 | funman | same but i use more clipv1 than the amsv2 |
22:14:47 | BRi7X | :'( |
22:14:49 | saratoga | i use a mix of all 3 clips, they seem pretty similar aside from battery life |
22:18:00 | funman | clipv2 noise annoys me a lot :/ |
22:21:22 | preglow | kugel: what's broken about some phone's android install? |
22:22:07 | | Join stripwax [0] (~Miranda@87-194-34-169.bethere.co.uk) |
22:22:30 | | Quit stripwax (Client Quit) |
22:23:21 | BRi7X | hrmm, putting my gigabeat S in the freezer... bad idea or awful idea? |
22:24:41 | Guest8399 | funman: I'm back, no progress here :/ |
22:24:53 | bertrik | BRi7X, probably bad for the battery |
22:25:06 | xvello | saratoga (or others) : just got a clip+ and got familiar with the rockbox code. Are there some "junior jobs" to start contributing ? |
22:25:16 | kugel | preglow: http://code.google.com/p/android/issues/detail?id=9663 |
22:27:05 | | Join darkenvy [0] (60fe6f59@gateway/web/freenode/ip.96.254.111.89) |
22:27:23 | darkenvy | I recieved a 'can't load linux.bin' on this ipod |
22:27:39 | darkenvy | on the stable build, and current |
22:27:54 | pixelma | did you start it with the Play button (I believe it was)? |
22:28:33 | darkenvy | no |
22:28:44 | darkenvy | the ipod wont wake up at all witht that button |
22:29:10 | darkenvy | its completely broke. oddly enough though no matter which state (boot loader or official ipod firm) the menu+center wont hard reset either |
22:29:26 | darkenvy | hmmm I coulda sworn that was a firmware issue |
22:29:46 | funman | xvello: you could start by looking on the tracker |
22:31:49 | darkenvy | how to fix linux.bin error? |
22:31:50 | bertrik | xvello, all of the basic stuff has already been implemented and all of the easy bugs have probably been fixed ... :) |
22:32:11 | | Join Strife89 [0] (~Strife89@adsl-80-159-207.mcn.bellsouth.net) |
22:32:12 | BRi7X | chkdsk /R taking an ETERNITY for a 30GB hard drive, and "queefing" sounds coming from the gigabeat, tell tale sign it's partially physically corrupted? |
22:32:24 | bertrik | there's still this odd bug when powering down with an alarm set though |
22:32:54 | funman | perhaps it would be easy for someone who looks at it with a fresh and innocent mind |
22:34:16 | AlexP | xvello: Just find something you want to work on, and get going :) |
22:37:54 | mt | xvello: There's also this : http://www.rockbox.org/wiki/MrSomeonesTodoList - They're not really "junior jobs", but you could maybe find something there that might inspire you. |
22:38:26 | funman | start reading, it's full of hidden bugs ;) |
22:39:00 | xvello | funman: reading what ? |
22:39:04 | xvello | mt: thanks for the pointer |
22:39:12 | funman | the source |
22:39:42 | mt | xvello: you're welcome. |
22:39:47 | xvello | okay, will see what I can catch |
22:47:55 | | Quit bieber (Ping timeout: 276 seconds) |
22:48:11 | | Join bieber [0] (~quassel@162-78.97-97.tampabay.res.rr.com) |
22:49:29 | *** | Saving seen data "./dancer.seen" |
22:49:57 | darkenvy | any fix to cant find linux.bin? |
22:50:06 | | Quit funman (Quit: free(random());) |
22:53:24 | | Quit Guest8399 (Quit: leaving) |
22:54:53 | | Join clone4crw [0] (~root@96-42-90-88.dhcp.roch.mn.charter.com) |
22:54:59 | darkenvy | I cant find anything on this error surprisingly |
22:57:25 | gevaerts | darkenvy: the bootloader tries to load ipodlinux if "play" is held during boot. Maybe your play button is stuck? |
22:57:52 | darkenvy | oh wow that could be a possibility |
22:58:19 | darkenvy | how to I rebuild rockbox to delete this function? |
22:58:32 | saratoga | grep the source for that message, comment out the block its in |
22:58:35 | darkenvy | or the bootloader |
22:58:39 | saratoga | probably somewhere in the bootloaders folder |
22:58:46 | saratoga | then compile a new bootloader |
22:59:03 | darkenvy | okay ill be back to tell ya how it went |
23:00 |
23:01:22 | darkenvy | woah where the eff is the source outlet? all of these are just the folders you throw on the ipod |
23:02:39 | AlexP | svn://svn.rockbox.org/rockbox/trunk |
23:03:05 | | Quit milz (Remote host closed the connection) |
23:04:44 | | Quit slck (Remote host closed the connection) |
23:04:47 | | Join slck [0] (Venci@No.Real-Love-For.Me) |
23:09:52 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
23:18:05 | | Quit slck (Quit: Requested.) |
23:18:19 | | Join slck [0] (Venci@Venci-Notebook-LAN.ipv6.6bez10.info) |
23:24:38 | darkenvy | should I use the ipodpatcher to use my compiled bootloader? |
23:29:47 | | Quit clone4crw (Quit: leaving) |
23:29:48 | | Join Buschel [0] (~chatzilla@p54A3B914.dip.t-dialin.net) |
23:30:48 | Buschel | mt: did you see my comments and screenshots? |
23:31:01 | | Quit ender` (Quit: The propeller is just a big fan in front of the plane used to keep the pilot cool. When it stops, you can actually watch the pilot start sweating.) |
23:36:01 | | Join anewuser [0] (anewuser@unaffiliated/anewuser) |
23:36:20 | mt | Buschel: Yeah seen them. I'm trying to compile r27402 to be able to compare svn and your patches against the ref decoder. |
23:36:40 | mt | but I'm having troubles with my tmp dir now that I'm trying to resolve. |
23:38:07 | mt | Buschel: Also, somewhere along the optimisations, our decoder started outputting more samples than ffmpeg's, how is that ?! (I guess those could easily be the spikes we're seeing) |
23:38:32 | Buschel | mt: r27406 can be switched back to float for verification? |
23:40:06 | Buschel | mt: no, if the spikes would be caused by additional samples the diff would show massive difference from the first spike on (as the time domain would then be shifted between both output wav's) |
23:40:34 | | Quit AndyI (Ping timeout: 248 seconds) |
23:40:57 | mt | Buschel: I think yes. (also, do you mean 402 ? :) ) |
23:41:23 | mt | Buschel: The difference on matlab between 2 wav files (rockbox - ffmpeg) was massive |
23:41:39 | Buschel | mt: regarding outputting more samples -> I don't know. I assume this was introduced with the very first rockbox port's version. |
23:42:21 | Buschel | mt: our implementation needs to cut the first 1024 samples (for the 173kbps-file). it starts with 1024 zeroes. |
23:43:16 | Buschel | mt: I did this in an experimental code change. afterwards I could directly diff the wav-output from our flac-decoder with the wmapro-output |
23:43:43 | | Quit bertrik (Ping timeout: 258 seconds) |
23:44:43 | Buschel | mt: btw, the diff between svn/flac is (minor) larger than patched/flac. which could be a sign that svn might not be optimal... a direct comparison between float and fixedpoint decoder would be better, of course |
23:44:43 | | Join AndyI [0] (~pasha_int@212.14.205.32) |
23:45:30 | mt | Buschel: before the optimisations I could subtract the 2 wav files on matlab, (both matrices were of the same dimensions) ... now I have to truncate our matrix to be the same size (number of samples) as ffmpeg's. So those extra samples haven't been introduced early with the port. |
23:45:49 | mt | Buschel: ^ (re: more samples) |
23:46:42 | | Quit evilnick_B (Quit: Page closed) |
23:46:44 | mt | I'm trying to compile a non-optimised fixed point version of the code to compare it against the original ffmpeg decoder. |
23:47:25 | Buschel | mt: well, a float-version on pcsim could also be helpful. this was the starting point, right? |
23:47:54 | | Quit jgarvey (Quit: Leaving) |
23:47:56 | xvello | yay, found a bug ;) |
23:48:27 | mt | Buschel: Yes. I started with floating point decoder from ffmpeg's libavcodec. |
23:49:00 | mt | xvello: Great, get fixing then. ;) |
23:49:28 | | Quit domonoky (Read error: Connection reset by peer) |
23:49:39 | xvello | mt: yeah, tomorrow, it's almost midnight here |
23:49:43 | xvello | 'night people |
23:49:51 | mt | good night |
23:49:56 | Buschel | good night |
23:50:43 | gevaerts | xvello: what sort of bug? |
23:52:27 | | Quit milz (Remote host closed the connection) |
23:54:03 | xvello | gevaerts: in the FM tuner, when switching stations with left-right on preset mode, the station name isn't updated, while it is if one selects a station directly via the list |
23:54:16 | xvello | shouldn't be too hard to fix |
23:54:24 | | Join Xerion_ [0] (~xerion@84.25.7.202) |
23:54:29 | gevaerts | famous last words :) |
23:54:52 | AlexP | I think update issues in the fms is quite long standing |
23:55:00 | AlexP | Speak to JdGordon about it :) |
23:55:05 | | Quit Xerion (Read error: Connection reset by peer) |
23:55:05 | | Nick Xerion_ is now known as Xerion (~xerion@84.25.7.202) |
23:55:10 | xvello | gevaerts: ^^ |
23:56:54 | | Join milz [0] (~kyle@S0106002500fe5b38.cg.shawcable.net) |
23:56:58 | gevaerts | xvello: I don't actually know much about the UI code. It's just that how hard a bug is tends to be unpredictable |