**** BEGIN LOGGING AT Thu Jan 12 02:59:57 2012 Jan 12 04:58:55 * PuffTheMagic wonders where the font fix is Jan 12 04:59:52 not here! Jan 12 05:58:46 PuffTheMagic: lol i have things to do :) Jan 12 06:04:03 really? Jan 12 06:04:12 I thought you were just waiting around for something to happen. Jan 12 06:18:09 ka6sox-farfarawa: taking a break from waiting around :D Jan 12 06:18:28 aha Jan 12 06:18:36 I am in need of a break. Jan 12 06:25:45 :( Jan 12 06:53:06 dtzWill, http://paste.pocoo.org/show/533652/ Jan 12 06:53:48 this is segfaulting on hasCacheLine(i, index) = false; Jan 12 07:02:38 well presumably Jan 12 07:02:44 b/c nFonts is initialized to zero Jan 12 07:02:53 while you replaced it with a static constant 4 Jan 12 07:03:14 which no longer has any relation to the size of haveCAcheLine Jan 12 07:03:25 but since haveCacheLine now is a fixed size, you might as well allocate it statically Jan 12 07:23:09 dtzWill, well i get no segfault now, but i also get no text ;) Jan 12 07:23:15 http://paste.pocoo.org/show/533661/ Jan 12 07:23:35 lol Jan 12 07:23:36 this is terrible Jan 12 07:23:47 this method of debugging via textual patch i mean Jan 12 07:23:47 lol Jan 12 07:25:41 dtzWill, i think its stuck in nextPowerOfTwo() Jan 12 07:25:44 cpu it going crazy Jan 12 07:26:48 TTF_Font *m_font; "(TTF_Font*)&m_font" - that looks very broken Jan 12 07:28:29 stbuehler: good call Jan 12 07:28:59 same goes for the SDL_Color cols[...]; (SDL_Color*)&cols Jan 12 07:29:08 PuffTheMagic: while you're at it, maybe enable warnings in Makefile :) Jan 12 07:29:20 (SDL_Color*)&cols was dtzWill's code ;) Jan 12 07:30:30 hmm getting closer Jan 12 07:30:36 i have text and color Jan 12 07:30:39 but so so wrong Jan 12 07:30:55 so it is, damn. how did that work at all? Jan 12 07:33:04 https://github.com/PuffTheMagic/wTerm/commit/1157beaafd737a78e80647202030eedfa1a3682f Jan 12 07:34:15 http://ompldr.org/vYzdseA/wterm_2012-12-01_023042.png Jan 12 07:34:18 LOL Jan 12 07:34:25 idk why the screencaptures look so grainy Jan 12 07:34:29 they dont on my screen Jan 12 07:35:38 and the worst part Jan 12 07:35:42 the font loading bug still exists Jan 12 07:37:33 wait what Jan 12 07:37:46 PuffTheMagic: commenting on the code, fwiw Jan 12 07:38:25 in my master branch i pushed a change to the Makefile.inc Jan 12 07:38:28 export DEBUG=1 Jan 12 07:38:31 when building Jan 12 07:38:34 for -g -Wall Jan 12 07:39:43 dtzWill, its hard to tell what you are referring to in your comments Jan 12 07:39:54 they're inline? Jan 12 07:39:56 memcpy(this->fnt, fnt, sizeof(TTF_Font*)); - sweet :D Jan 12 07:39:56 cause i suck at github Jan 12 07:40:08 how about this->fnt = fnt ? Jan 12 07:40:37 stbuehler, i had that at one point Jan 12 07:40:46 dtzWill, i cant tell if a comment is for the line above or below Jan 12 07:41:08 below Jan 12 07:41:23 the only one tightly coupled with the lines it discusses is that this->Fnt = fnt and this->cols = cols Jan 12 07:41:51 are 4 lines below malloc'ing to cols and then clearing it.... Jan 12 07:41:55 and then you set it equal to cols Jan 12 07:42:26 (which is a stack-allocated array in the color function, and almost certainly is causing much of your problem) Jan 12 07:42:34 lol 'color fucntion' Jan 12 07:42:37 it's been a long day. Jan 12 07:42:40 *caller function Jan 12 07:45:28 PuffTheMagic: regarding the memcpy(this->fnt, fnt, sizeof(TTF_Font*)); Jan 12 07:45:38 if you really wanted to copy the contents, you would need memcpy(this->fnt, fnt, sizeof(TTF_Font)); Jan 12 07:45:42 (without the "*") Jan 12 07:46:15 but this assumes that you actually can copy a TTF_Font, which is unlikely Jan 12 07:47:21 as not every font will use the same amount of memory, a TTF_Font certainly contains pointers to dynamically allocated data, so you only have a "shallow" copy, which is unusable if you free the original one. Jan 12 07:47:33 dtzWill, your comments are totally showing up below the lines u are talking about Jan 12 07:47:49 i thought u meant that your comments refer to the lines below it Jan 12 07:48:09 if you click on the "comment this line" the form appears below, so that would be the default :) Jan 12 07:48:49 i meant my comments were below it, i'm sorry. Jan 12 07:49:01 it makes sense now Jan 12 07:49:08 just first time ive encountered this Jan 12 07:49:22 stbuehler: if you see anything related to a TTF_Font being referred to by value that needs to be ripped out :) Jan 12 07:51:27 so dtzWill u say to remove this->fnt = fnt; Jan 12 07:51:32 and stbuehler says to keep it Jan 12 07:53:03 i.. Jan 12 07:53:07 drop the fnt memcpy Jan 12 07:53:14 the cols is the offender, the fnt is just redundant Jan 12 07:53:38 how is fnt redundant Jan 12 07:54:05 u used to have an array called fnts Jan 12 07:54:08 which held the 4 fonts Jan 12 07:54:14 i changed that to 1 fnt Jan 12 07:54:18 and i have to set it Jan 12 07:56:32 line 87 Jan 12 07:56:33 ahh i get what u are saying about the cols Jan 12 07:56:38 i have better color now Jan 12 07:56:38 you both memcpy and set it Jan 12 07:57:07 which is redundant, nevermind the fact that memcpy'ing a single pointer might be silly (I do understand you were donig a quick refactoring) Jan 12 07:57:10 hooray better color Jan 12 07:57:17 the memcopy stuff was commented out Jan 12 07:57:22 it was only doing the setting Jan 12 08:11:35 hmmm Jan 12 08:11:44 setting these attributes is being weird Jan 12 08:13:25 dtzWill, https://github.com/PuffTheMagic/wTerm/commit/301f480581a146012c77d38a066c5b731871e3f1 Jan 12 08:16:12 color is back to looking normal for setting the font properties is not working correctly Jan 12 08:16:23 what's wrong? Jan 12 08:16:40 i can only set 2 fonts Jan 12 08:16:55 what happens if you enable a third? Jan 12 08:16:57 if i add more they go wrong Jan 12 08:17:07 like right now i have normal and underline Jan 12 08:17:12 if i add bold Jan 12 08:17:22 then underline becomes bold underline Jan 12 08:17:26 or something like that Jan 12 08:17:31 it makes no sense Jan 12 08:19:19 Ihttp://pastebin.com/zseKPa3B Jan 12 08:19:31 eek, minus that I :3 Jan 12 08:20:26 Brybry: which part are you drawing our attention to? Jan 12 08:21:04 it looks to me like it's adding there but isn't it supposed to replace? Jan 12 08:21:18 Brybry: well it's 'adding' to font->face_style Jan 12 08:21:28 presumably face_style is a constant property of the font Jan 12 08:21:34 and 'style' is the ...current..style? Jan 12 08:21:34 idk Jan 12 08:21:50 but maybe you're right Jan 12 08:21:57 oh Jan 12 08:21:57 well Jan 12 08:22:01 i misunderstood what puff was saying Jan 12 08:22:14 maybe you're right then :) Jan 12 08:27:58 nah, I didn't read it right. it's definitely replacing Jan 12 08:54:58 PuffTheMagic: hm, i begin to understand why it worked Jan 12 08:55:18 the old or new code? Jan 12 08:55:22 PuffTheMagic: you had broken casts, but it didn't matter as the resulting pointers where only used in memcpy, which doesn't care about types Jan 12 08:59:21 then you changed from array of pointers to fonts to pointer to font, and i guess you got confused by the wrong types Jan 12 09:00:11 the memcpy was used to copy an array of pointers; if you want to use it to copy a single pointer you need memcpy(&his->fnt, &fnt, sizeof(TTF_Font*)); - or just this->fnt = fnt :) Jan 12 09:01:51 may i ask what your problem with the fonst was in the first place? Jan 12 09:17:03 we had multiple TTF_OpenFont() to the same file/font which were causing (at least indirectly) font files to not be copied correctly during an ipk install while the application was still running Jan 12 09:30:35 wtf. why would it matter whether you open a file more than once? Jan 12 09:36:07 as changing font styles will flush caches i'm not sure you really want to use only one TTF_Font* Jan 12 11:29:46 PuffTheMagic: class SDLTerminal : public SDLCore, public ExtTerminal, public ExtTerminalContainer Jan 12 11:30:22 it is "bad style" to derive from more than one real class (interfaces only contain virtual methods, no data members) Jan 12 11:30:35 but i guess this isn't your code, right? :) Jan 12 16:15:27 stbuehler, not sure how long u have been with "webos" but the code base for the wterm plugin came from sdlterminal Jan 12 16:15:38 so all the classes is not our code Jan 12 16:15:47 just have been working within it Jan 12 17:52:08 dtzWill, Brybry stbuehler ping Jan 12 17:52:11 here is something interesting Jan 12 17:52:21 m_fontNormal = TTF_OpenFontRW(file, 1, nSize); Jan 12 17:52:21 m_fontBold = TTF_OpenFontRW(file, 1, nSize); Jan 12 17:52:21 m_fontUnder = TTF_OpenFontRW(file, 1, nSize); Jan 12 17:52:21 m_fontBoldUnder = TTF_OpenFontRW(file, 1, nSize); Jan 12 17:52:34 it always fails on the 3rd font Jan 12 17:52:42 regardless of which one is the 3rd Jan 12 20:01:30 PuffTheMagic: well, i guess it is time to run it with strace :) Jan 12 21:43:36 stbuehler, Brybry dtzWill ping Jan 12 21:43:44 pongish Jan 12 21:44:12 * Brybry yawns Jan 12 21:44:35 im having problems with something and how to handle it Jan 12 21:45:18 i need to change something like \033[11~ Jan 12 21:45:29 which has a literal \ character Jan 12 21:45:40 into \033[11~ Jan 12 21:45:47 where \ is an escape Jan 12 21:48:30 for nested escape codes? Jan 12 21:49:31 no Jan 12 21:49:36 this is for custom keybindings in js Jan 12 21:49:44 i suppose i could use hex instead of dec Jan 12 21:49:59 you mean for transmitting data from js to plugin? Jan 12 21:50:17 shouldn't json be able to handle that? Jan 12 21:51:35 sure but the issue is displaying the escape code in form that a user can edit/see Jan 12 21:52:28 stbuehler, also, whats your full name, so i can update the contibutors list Jan 12 21:53:55 PuffTheMagic: check the git commits :) "Stefan Bühler" Jan 12 21:54:29 ok, where would the user be able to edit the escape codes? i still don't get it i suppose Jan 12 21:54:52 i've added a keybindings pane in the prefs Jan 12 21:54:56 not pushed yet though Jan 12 21:54:57 hm k Jan 12 21:55:24 but the javascript doesn't use escape codes yet, right? Jan 12 21:55:30 Bühler? Bühler? Jan 12 21:55:33 sorry had to Jan 12 22:00:02 woot Jan 12 22:00:03 got it Jan 12 22:00:19 enyo.json.parse('"' + str + '"') ? Jan 12 22:00:50 or eval? :D Jan 12 22:03:15 no Jan 12 22:03:48 the escape codes / keybindings are stored in escaped hex on the js prefs Jan 12 22:03:50 like '\\x1b[11~' Jan 12 22:04:28 then i use this http://dpaste.com/686782/ Jan 12 22:04:36 before sending it to the plugin Jan 12 22:05:01 well, this only works for hex :) Jan 12 22:05:10 not for the octal \033 Jan 12 22:05:21 yes well Jan 12 22:05:34 its better than nothing Jan 12 22:05:40 kk Jan 12 22:05:49 idk that I can make it support both Jan 12 22:06:38 /\\([0-7]{3})/g and parseInt(..., 8) Jan 12 22:07:10 think I could chain that up in the decodeEscape function? Jan 12 22:07:24 yep Jan 12 22:07:35 wel Jan 12 22:07:48 someone could encode \ with \x(whatevercodr) Jan 12 22:07:54 and append 000 or similar Jan 12 22:08:41 so the correct solution would be /\\(x[0-9A-Fa-f]{2}|[0-7]{3})/g as regex Jan 12 22:08:57 and then checking whether the first char is 'x' and using 16 and 8 otherwise Jan 12 22:09:15 hm, and the x needs to be skipped Jan 12 22:12:34 i don't get why there have to be two "terminals" linked to each other Jan 12 22:20:51 stbuehler> i don't get why there have to be two "terminals" linked to each other Jan 12 22:20:53 huh? Jan 12 22:49:17 PuffTheMagic: in the main there is a "Terminal" and a "SDLTerminal" that get linked via the ExtTerminal/-Container interfaces Jan 12 22:50:43 oh right Jan 12 22:50:52 code is a mess Jan 12 23:08:16 basically SDLTerminal is the graphics side thread and Terminal is the tty terminal device communication thread? Jan 13 00:24:37 i hate C++ Jan 13 00:25:19 and it hates you! Jan 13 00:25:24 what's your preferred language, not including js Jan 13 00:25:59 could write language bindings for java/ruby/python/etc Jan 13 00:26:16 halfhalo, eat a bowl of dicks Jan 13 00:26:27 dwc-, python and lisp Jan 13 00:39:58 stbuehler, i'd merge your pull request but it seems to not handle orientation changes well Jan 13 00:47:51 dwc-, Brybry stbuehler first cut of custom keybindings and input schemes pushed Jan 13 00:47:59 only supports function keys atm Jan 13 00:48:03 can be extended **** ENDING LOGGING AT Fri Jan 13 02:59:57 2012