Timestamps are in GMT/BST.
| 00:13:20 | | Watusimoto_ Quit (Ping timeout: 260 seconds) |
| 00:41:21 | | raptor Quit () |
| 01:22:11 | Fordcars | If bitfighter was online it would require a full rewrite |
| 01:22:20 | Fordcars | and it would be very slow |
| 01:49:40 | amgine1234567890 | not nessisaily |
| 01:49:54 | amgine1234567890 | slow wise workwise yes |
| 01:50:02 | amgine1234567890 | slow wise no works wise yes |
| 02:04:32 | | raptor has joined |
| 02:04:32 | | ChanServ sets mode +o raptor |
| 02:10:33 | | amgine1234567890 Quit (Quit: Page closed) |
| 02:29:19 | Fordcars | hi |
| 02:29:26 | raptor | hello |
| 02:29:45 | Fordcars | hello |
| 02:57:55 | | amgine1234567898 has joined |
| 02:57:59 | amgine1234567898 | how would i dedicate a server on bitfighter 18 |
| 02:58:07 | raptor | hi amgine1234567898 |
| 02:58:37 | raptor | you can either: 1. compile in dedicated mode (on Linux: make dedicated) |
| 02:58:47 | amgine1234567898 | im using windows |
| 02:58:49 | raptor | or 2. run from the command line: bitfighter -dedicated |
| 02:59:13 | raptor | on windows |
| 02:59:23 | raptor | use a commandline and do bitfighter -dedicated |
| 03:00:07 | amgine1234567898 | so would that look like start bitfighter.exe -dedicated |
| 03:00:12 | raptor | yes |
| 03:00:45 | amgine1234567898 | hmm having difficulty |
| 03:01:06 | amgine1234567898 | can using the target still work |
| 03:01:18 | amgine1234567898 | chaging the shortcut target |
| 03:01:34 | raptor | oh |
| 03:01:43 | raptor | just 'bitfighter.exe -dedicated' |
| 03:01:46 | raptor | no 'start' |
| 03:02:25 | amgine1234567898 | lol bitfighter.exe not recinigez |
| 03:02:53 | raptor | you have to be in the directory where it is located |
| 03:03:12 | raptor | you can probably copy and paste the shortcut, then edit the new one to have ' -dedicated' in it |
| 03:03:54 | amgine1234567898 | " is failing" |
| 03:05:05 | | koda Quit (Quit: k thx bai) |
| 03:05:07 | amgine1234567898 | says target box is not valid hmm |
| 03:05:58 | amgine1234567898 | its not working hmm |
| 03:06:25 | amgine1234567898 | got the dreaded crash im doing osmthing rong |
| 03:07:11 | amgine1234567898 | either a im doing somthing wrong or b im doing it right but im getting a crash |
| 03:11:06 | amgine1234567898 | need some feedback later ill keep trying |
| 03:14:53 | raptor | hi sam686 |
| 03:14:59 | amgine1234567898 | hi sam |
| 03:15:39 | amgine1234567898 | still having problems wonder if i got a bug on my hands |
| 03:16:57 | amgine1234567898 | a idea what iuf you automaticly added a server dedicate fill in all future versions that one would click and it would do it is that possible |
| 03:17:05 | amgine1234567898 | file |
| 03:17:20 | amgine1234567898 | would sure be more convienent for the users |
| 03:18:38 | raptor | instructions on running dedicated are on the wiki |
| 03:40:49 | amgine1234567898 | im trying what it said but failing sure would be more covenient still |
| 05:03:42 | Fordcars | cd mycomputer/program files.... |
| 05:03:47 | Fordcars | ? |
| 05:22:48 | | Fordcars Quit (Ping timeout: 245 seconds) |
| 05:52:05 | | BFLogBot Commit: acf03d062248 | Author: buckyballreaction | Message: Fix team spawns in team Nexus |
| 06:47:02 | | LoneWolfy has joined |
| 06:47:28 | LoneWolfy | you should add 9 presets for loadouts |
| 06:47:44 | LoneWolfy | and ability to rename them |
| 06:48:49 | raptor | you mean in the ini? |
| 07:09:20 | | Darrel has joined |
| 07:16:47 | | amgine1234567898 Quit (Quit: Page closed) |
| 07:44:03 | | LoneWolfy Quit (Ping timeout: 245 seconds) |
| 07:55:56 | raptor | all right - going to bed - couldn't solve the crash yet, but i think it is due to cleaning teams more than once... |
| 07:56:01 | | raptor Quit () |
| 09:21:49 | | LordDVG has joined |
| 09:26:13 | | Martinski has joined |
| 09:34:56 | | LordDVG Quit (Quit: Leaving) |
| 09:35:17 | | LordDVG has joined |
| 09:54:33 | Martinski | morning guys |
| 09:54:34 | Martinski | :) |
| 10:02:26 | Martinski | watusimoto are you here ? |
| 10:03:57 | | Watusimoto_ has joined |
| 10:04:08 | Watusimoto_ | hello |
| 10:08:07 | Martinski | hey |
| 10:08:16 | Martinski | I want to ask about the icons I have to make |
| 10:08:45 | Martinski | each of the 4 icons has to be in 3 different sizes, right ? |
| 10:11:45 | Watusimoto_ | that's what I wrote |
| 10:12:05 | Watusimoto_ | but for the 3 platform icons, that's less important |
| 10:13:06 | Watusimoto_ | for them, the thing that's most important is that we can integrate them into the downloads page somehow; that's why I wrote that we can be flexible on the shape/size |
| 10:13:39 | Watusimoto_ | For the main bitfighter icon, that does need to be in those sizes because that's what the os usually requires |
| 10:16:27 | Martinski | ok, and for the platforms, I have an idea about mixing the platform logo with the bitfighter logo, is that OK ? |
| 10:17:58 | Watusimoto_ | If it is a good design, anything is ok :-) |
| 10:18:12 | Martinski | good |
| 10:18:40 | Martinski | after about an hour, I'll have to go, because I have lessons and in about 7 hours, I'll be here |
| 10:18:52 | Martinski | will you be online ? |
| 10:19:19 | Watusimoto_ | I did try to design the platform icons myself, so I have one idea if you are realy stuck. My idea was good, but the implementation was not |
| 10:20:06 | Watusimoto_ | I'll be on and off today... let's see that would mean you were back aroud 19:00 my time... I might be on, but definitely will by 20:00 or 21:00ish |
| 10:20:24 | Martinski | ok |
| 10:20:36 | Martinski | my task time ends about 22:00 my time |
| 10:21:36 | Watusimoto_ | we'll get it to work out. I can always extend, right? |
| 10:21:47 | Martinski | ofcourse |
| 10:22:03 | Martinski | ok, I'm going to think for some ideas |
| 10:22:08 | Martinski | see you later |
| 10:22:10 | Watusimoto_ | we'll get it worked out |
| 10:22:11 | Watusimoto_ | bye! |
| 10:22:18 | | Martinski Quit () |
| 10:56:34 | | LordDVG Quit (Ping timeout: 252 seconds) |
| 13:33:18 | | koda has joined |
| 15:00:04 | | twodarray has joined |
| 15:00:10 | | twodarray Quit (Client Quit) |
| 15:01:16 | | twodarray has joined |
| 15:01:40 | twodarray | hello |
| 15:08:13 | | koda Quit (Quit: koda) |
| 15:14:33 | twodarray | is anybody there? |
| 15:15:23 | twodarray | I accepted the task for "implementing /announce", and I would like to start! |
| 15:33:50 | | raptor has joined |
| 15:33:50 | | ChanServ sets mode +o raptor |
| 15:35:36 | raptor | hello |
| 15:35:58 | raptor | twodarray: i will assign |
| 15:37:15 | twodarray | finally |
| 15:37:22 | twodarray | hi |
| 15:37:25 | raptor | hi |
| 15:37:30 | raptor | we're a small group :) |
| 15:37:38 | twodarray | :) |
| 15:37:42 | raptor | in different timezones... |
| 15:37:47 | twodarray | yeah |
| 15:37:47 | raptor | ok, assigned |
| 15:37:51 | twodarray | thanks |
| 15:38:04 | raptor | you're welcome |
| 15:38:31 | twodarray | so.. |
| 15:38:43 | twodarray | do I have info to start with? |
| 15:38:50 | twodarray | like some methods? |
| 15:39:07 | raptor | let's see... |
| 15:39:15 | raptor | do you have teh code checked out? |
| 15:39:33 | twodarray | no |
| 15:39:54 | raptor | i'd start there :) |
| 15:39:55 | twodarray | is there the source? |
| 15:40:00 | raptor | yes |
| 15:40:07 | twodarray | OH |
| 15:40:09 | twodarray | okay |
| 15:40:13 | twodarray | thanks |
| 15:40:16 | raptor | https://code.google.com/p/bitfighter/downloads/list |
| 15:40:42 | raptor | there is a source archive there; OR, if you want you can check out the code directly using mercurial |
| 15:41:12 | Watusimoto_ | hi |
| 15:41:27 | raptor | hi Watusimoto_ |
| 15:41:34 | Watusimoto_ | sorry, I was here and wasn't paying attention |
| 15:42:00 | Watusimoto_ | I creaeted a script to generate a portable isntaller from our windows installer |
| 15:42:15 | raptor | so I a separate nsis script? |
| 15:42:41 | Watusimoto_ | no, it's a bat file that rips apart and reassembles the installer |
| 15:42:51 | Watusimoto_ | would have been a good gci task, in retrospect |
| 15:42:55 | raptor | ha! |
| 15:43:03 | Watusimoto_ | because I HATE working with bat files |
| 15:43:14 | raptor | yeah, me too |
| 15:43:18 | Watusimoto_ | you can do almost anything, but it makes obfuscated perl easy to read |
| 15:43:24 | raptor | haha |
| 15:43:58 | raptor | REM this is a comment |
| 15:43:59 | Watusimoto_ | so I think we need to disable the updater for the portable install |
| 15:44:08 | raptor | ok |
| 15:44:45 | Watusimoto_ | it would be nice to notify users of new versions, though |
| 15:44:59 | raptor | we already do that in-game |
| 15:45:04 | raptor | big red box |
| 15:45:10 | Watusimoto_ | btw, hello twodarray! |
| 15:45:10 | raptor | when it connects to master |
| 15:45:15 | Watusimoto_ | ah, ok |
| 15:45:25 | Watusimoto_ | maybe we just diable the updater for the moment |
| 15:45:47 | raptor | ok\ |
| 15:46:19 | Watusimoto_ | lucky twodarray -- he gets to dissect my smooth message display that nearly killed me :-) |
| 15:48:02 | raptor | i'm still traccking down that crash |
| 15:48:06 | Watusimoto_ | so the teamspawn fix... |
| 15:48:16 | Watusimoto_ | are you sure that doesn't break team spawns? |
| 15:48:19 | raptor | i narrowed it - unless you've been working on it, too? |
| 15:48:25 | Watusimoto_ | no |
| 15:48:31 | raptor | oh, team spawn on nexus |
| 15:48:34 | Watusimoto_ | I can;t reproduce |
| 15:48:40 | Watusimoto_ | remember the teamspawn bug in soccer? |
| 15:48:48 | raptor | yes, i did the same for nexus |
| 15:49:14 | Watusimoto_ | oh wait, ok, I thought you set it to false |
| 15:49:16 | raptor | and no, it doesn't break anything else - the *only* thing that method is used for is for determining where ships spawn at |
| 15:49:31 | Watusimoto_ | ok, I thought you disabled it to avoid a warning |
| 15:49:37 | Watusimoto_ | I see that you actually enabled it |
| 15:49:44 | Watusimoto_ | my mistake |
| 15:49:49 | raptor | for the crash - |
| 15:49:59 | Watusimoto_ | Maybe I'm red-green colorblind |
| 15:49:59 | raptor | i narrowed it... |
| 15:50:06 | Watusimoto_ | yes |
| 15:50:07 | Watusimoto_ | to what? |
| 15:50:11 | raptor | in ~TeamManager() |
| 15:50:14 | raptor | we call clearTeams |
| 15:50:42 | raptor | but we also call clearTeams in loads of other places, like all the Game::cleanUp() methods |
| 15:50:42 | Watusimoto_ | ok... this causes problems? |
| 15:51:05 | raptor | in some of which TeamManager is already deleted |
| 15:51:18 | Watusimoto_ | This one? |
| 15:51:19 | Watusimoto_ | void TeamManager::clearTeams() |
| 15:51:19 | Watusimoto_ | { |
| 15:51:19 | Watusimoto_ | mTeams.deleteAndClear(); |
| 15:51:19 | Watusimoto_ | mTeamHasFlagList.clear(); |
| 15:51:19 | Watusimoto_ | } |
| 15:51:23 | raptor | yes |
| 15:51:29 | Watusimoto_ | ok, this looks dangerous! |
| 15:51:55 | Watusimoto_ | so... I see two obvoius solutions |
| 15:52:06 | Watusimoto_ | 1) figure out why things are deleted when they shouldn't be |
| 15:52:15 | Watusimoto_ | 2) make it a safeptr and hope for the best |
| 15:52:34 | raptor | i can't get safePtr to compile.. |
| 15:52:41 | Watusimoto_ | really? |
| 15:52:55 | raptor | really |
| 15:53:03 | raptor | so i went down the analysis route... |
| 15:53:09 | Watusimoto_ | you mean in a particular context, or it won't compile at all? |
| 15:53:29 | raptor | not at all, everything i tried to make it compile failed.. |
| 15:53:49 | raptor | i don't even understand the error that well and not even stackoverflow was helpful... |
| 15:53:55 | Watusimoto_ | weird |
| 15:54:15 | Watusimoto_ | ok, so analysis results in an unhappy raptor, I predict |
| 15:54:32 | raptor | yeah... |
| 15:54:51 | raptor | every time i comment out one clearTeams(), the game crashes elsewhere.. |
| 15:55:39 | raptor | wait! i have another idea... |
| 15:57:16 | Watusimoto_ | could use shared_ptr? |
| 15:59:30 | Watusimoto_ | should I disable update checking in standalone mode for all platforms, or just windows? |
| 15:59:53 | Watusimoto_ | I don;t know enough about how the linux/mac stuff works to make a decision |
| 16:00:01 | raptor | all platforms is fine - but just turn off the online update |
| 16:00:09 | raptor | leave the INI updating, etc... |
| 16:00:20 | raptor | i tried to make it platform agnostic.. |
| 16:01:32 | Watusimoto_ | I don;t understand. Let me ask a more specific question. Should i disable OSX updates in standalone mode? |
| 16:01:53 | Watusimoto_ | the INI updating, of course, shoudl run regardless |
| 16:02:18 | raptor | OSX has no standalone, so sure! |
| 16:02:29 | raptor | :) |
| 16:02:44 | raptor | I say disable *all* online update checks for standalone |
| 16:03:05 | raptor | it should be a one line fix |
| 16:06:28 | raptor | here is an example of the error I van't seem to compile: http://pastie.org/5498876 |
| 16:06:57 | raptor | i made TeamManager *mTeamManager into SafePtr<TeamManager> mTeamManager |
| 16:07:23 | raptor | that's it, and i get this goofy invalid static_cast error that i'm too stupid to solve |
| 16:11:03 | Watusimoto_ | can you show me the code itself? |
| 16:11:12 | Watusimoto_ | that generates this nice error? |
| 16:11:33 | Watusimoto_ | btw, how do I rename myself? |
| 16:11:57 | raptor | i have an editable box that shows my nickname |
| 16:12:35 | | Watusimoto_ is now known as bunnyfan |
| 16:12:54 | | bunnyfan is now known as howdoigetbacktom |
| 16:13:13 | | howdoigetbacktom is now known as whattheheck |
| 16:13:20 | raptor | hahaha |
| 16:13:22 | whattheheck | I can't get watusimoto back! |
| 16:13:31 | | whattheheck is now known as isreallywatusimo |
| 16:13:37 | isreallywatusimo | that's a little better |
| 16:13:47 | raptor | ok isreallywatusimo: here is the simple changes i made: http://pastie.org/5498898 |
| 16:14:05 | | watusimoto was kicked from #bitfighter by raptor |
| 16:14:14 | raptor | ok i kicked your work counterpart |
| 16:14:31 | raptor | he's the one taking your name |
| 16:15:12 | | isreallywatusimo is now known as stillcantbewatus |
| 16:15:58 | | stillcantbewatus is now known as watusimo_to |
| 16:16:34 | raptor | sam686: if you're around would you know why i get that really weird static cast error? |
| 16:16:47 | watusimo_to | btw, your code looks good |
| 16:16:54 | watusimo_to | I could try compiling here |
| 16:16:56 | raptor | yes - |
| 16:17:00 | raptor | it's so simple |
| 16:17:13 | raptor | but something is preventing me compiling... |
| 16:18:55 | | BFLogBot Commit: da915ae079be | Author: watusimoto | Message: Fix potential problem with Windows install, though the installer seems to work in practice without this fix |
| 16:18:56 | | BFLogBot Commit: 23b95953719a | Author: watusimoto | Message: Create a bat file to build a portable installer from the Windows installer |
| 16:18:58 | | BFLogBot Commit: fcc516258b66 | Author: watusimoto | Message: Merge |
| 16:18:59 | | BFLogBot Commit: 1ba66713983d | Author: watusimoto | Message: Disable updating in portable mode |
| 16:19:01 | | BFLogBot Commit: 5a460a630430 | Author: watusimoto | Message: Slaughter spurious spaces |
| 16:19:28 | Darrel | watusimo_to - ghost command :) |
| 16:19:29 | | watusimo_to is now known as watusimoto_ |
| 16:19:40 | Darrel | if your nick is registered |
| 16:19:51 | Darrel | if not... ah, well, that's bad |
| 16:19:52 | | watusimoto_ is now known as irc_spaz |
| 16:20:25 | irc_spaz | my nick might be registered by me |
| 16:20:27 | irc_spaz | usually I can use it |
| 16:20:34 | irc_spaz | without all this claptrap |
| 16:21:54 | raptor | irc_spaz: use /nickserve ghost watusimoto <password> |
| 16:22:35 | | irc_spaz is now known as watusimoto_ |
| 16:22:41 | watusimoto_ | I embrace my new identity |
| 16:23:30 | raptor | ha\ |
| 16:24:03 | Darrel | МА |
| 16:24:04 | Darrel | :D |
| 16:24:20 | twodarray | hmm |
| 16:24:31 | twodarray | I'm back! |
| 16:24:43 | watusimoto_ | hi |
| 16:24:56 | twodarray | and m2cSendChat and everything |
| 16:24:56 | twodarray | wow |
| 16:25:00 | twodarray | you guys are hardcore |
| 16:25:07 | twodarray | :) |
| 16:25:17 | watusimoto_ | the hardest! |
| 16:25:26 | twodarray | awwwww yiss |
| 16:26:24 | watusimoto_ | Error 1 error C2440: 'static_cast' : cannot convert from 'TNL::SafePtrData *' to 'Zap::TeamManager *' c:\users\chris\documents\bf-trunk\tnl\tnlnetbase.h 648 |
| 16:26:32 | raptor | yes! |
| 16:26:34 | raptor | see! |
| 16:26:36 | raptor | bonkers! |
| 16:26:53 | raptor | how on earth does SafePtr work everywhere else?? |
| 16:28:04 | watusimoto_ | do you know if TeamManager is instantiable? |
| 16:28:11 | raptor | hmmm... |
| 16:28:18 | | Darrel_ has joined |
| 16:28:20 | watusimoto_ | it looks like it is |
| 16:28:23 | raptor | missing a constructor? |
| 16:28:39 | watusimoto_ | Darrel_: are you making fun of me? |
| 16:28:40 | raptor | ha, missing an explicit constructor... |
| 16:28:40 | | Darrel Quit (Read error: Connection reset by peer) |
| 16:29:02 | Darrel_ | watusimoto_, no! how did you even think about it! :P |
| 16:29:20 | Darrel_ | just switched to the wrong keyboard layout :)) |
| 16:30:06 | watusimoto_ | :-) |
| 16:30:25 | watusimoto_ | raptor: have you tried adding the constructor? |
| 16:30:28 | raptor | yes |
| 16:30:31 | raptor | fails |
| 16:30:34 | raptor | (just did that) |
| 16:30:48 | watusimoto_ | I don't even know if being abstract is a problem |
| 16:31:05 | raptor | how is it abstract?? |
| 16:32:39 | watusimoto_ | we instantiate it |
| 16:32:43 | watusimoto_ | so it must be instantiable |
| 16:32:45 | watusimoto_ | red herring |
| 16:33:01 | | BFLogBot Commit: 3f42f1ee767a | Author: buckyballreaction | Message: Simpler solution to disabling online updates in standalone mode. Sorry watusimoto :) |
| 16:33:47 | watusimoto_ | you basically reverted to my unchecked in solution :-) |
| 16:34:00 | raptor | yes, and added a simpler one |
| 16:34:21 | watusimoto_ | I did it that way because at the time it seemed that we would be making the decsion platform by platform |
| 16:34:27 | raptor | ah ok |
| 16:34:31 | watusimoto_ | doesn't matter |
| 16:34:36 | raptor | well, then forgive me |
| 16:34:39 | raptor | please |
| 16:34:40 | watusimoto_ | done |
| 16:35:12 | raptor | so now this safeptr goofiness |
| 16:35:17 | raptor | i don't even understand... |
| 16:35:26 | watusimoto_ | wait |
| 16:35:33 | watusimoto_ | I think I know |
| 16:35:47 | raptor | c++ fail on my part.. somehow |
| 16:35:49 | watusimoto_ | the error I get is error C2440: 'static_cast' : cannot convert from 'TNL::SafePtrData *' to 'Zap::TeamManager * |
| 16:36:01 | watusimoto_ | so are we doing a static cast somewhere? |
| 16:36:13 | watusimoto_ | the answer must be yes |
| 16:37:14 | watusimoto_ | but seems to be no |
| 16:37:40 | raptor | it is done in the operator overload of SafePtr |
| 16:37:43 | watusimoto_ | must be an implicit cast |
| 16:38:18 | watusimoto_ | I think I found it |
| 16:38:20 | watusimoto_ | hold on |
| 16:39:04 | watusimoto_ | compiling |
| 16:40:30 | watusimoto_ | nope |
| 16:41:42 | watusimoto_ | but still working on this idea |
| 16:41:45 | watusimoto_ | compiling again |
| 16:42:00 | raptor | sam686 rewrote a lot of SafePtr a while ago |
| 16:42:06 | raptor | and it's worked well since |
| 16:42:09 | raptor | until now.. |
| 16:43:58 | watusimoto_ | I don't think this is a SafePtr problem, actually |
| 16:44:07 | watusimoto_ | I just found another implicit cast |
| 16:44:22 | raptor | enlighten me, oh wise one |
| 16:44:27 | watusimoto_ | I think we're casting somewhere from TeamManager to SafePtr<TeamManager> |
| 16:45:54 | raptor | ha |
| 16:45:55 | raptor | ok |
| 16:45:57 | raptor | i fixed one error |
| 16:46:26 | watusimoto_ | do we still delete SafePtrs? |
| 16:46:36 | raptor | yes, they don't auto delete |
| 16:46:38 | watusimoto_ | delete mTeamManger is still ok? |
| 16:46:39 | watusimoto_ | ok |
| 16:46:41 | raptor | i think i got it! |
| 16:46:58 | watusimoto_ | was my theory right? |
| 16:46:59 | raptor | setActiveTeamManager(TeamManager *teamManager) -> setActiveTeamManager(SafePtr<TeamManager> teamManager) |
| 16:47:15 | watusimoto_ | I fixed that on my side |
| 16:47:22 | watusimoto_ | that was what I first thought was the problem |
| 16:47:37 | watusimoto_ | you understand the error message better now? |
| 16:47:43 | raptor | no |
| 16:48:00 | raptor | trial and error has save the day again! |
| 16:48:21 | | LordDVG has joined |
| 16:48:23 | watusimoto_ | ok, the function takes a raw pointer, and we were passing a safeptr wrapped around the raw pointer. The compiler tried to cast from one to the other, and the cast failed |
| 16:48:54 | watusimoto_ | it would be nice if the compiler said "hey, dude, I tried to cast here, and it failed!" |
| 16:49:08 | raptor | yeah.. |
| 16:49:27 | watusimoto_ | so it compiles? |
| 16:49:31 | raptor | ok, ran into the same problem with UIEditor... |
| 16:49:40 | raptor | same error, but different issues.. |
| 16:49:43 | watusimoto_ | well, now you should check for null before deleting |
| 16:49:52 | watusimoto_ | I think |
| 16:50:04 | watusimoto_ | I'll revert the changes on my side |
| 16:51:36 | | Darrel_ is now known as Darrel |
| 16:52:20 | raptor | ok, here is the one from the editor, which is goofier: http://pastie.org/5499041 |
| 16:52:38 | | LordDVG Quit (Ping timeout: 250 seconds) |
| 16:53:04 | | koda has joined |
| 16:53:34 | koda | hi |
| 16:53:37 | raptor | h iko |
| 16:53:40 | raptor | hi koda |
| 16:54:41 | watusimoto_ | Look for a similar problem |
| 16:54:54 | watusimoto_ | I found at least two functions that needed o be changed, I think |
| 16:56:23 | raptor | i find none |
| 16:56:34 | raptor | looking for all instances of mTeamManager |
| 16:56:40 | raptor | it's handled... |
| 16:57:32 | watusimoto_ | ok |
| 16:58:16 | watusimoto_ | mTeamManager = new TeamManager; |
| 16:58:23 | watusimoto_ | do you still have this in your code? |
| 16:58:26 | raptor | yes |
| 16:58:31 | watusimoto_ | then this is a problem |
| 16:58:40 | raptor | how so? |
| 16:58:49 | watusimoto_ | mTeamManager = SmartPtr<TeamManager *>(new TeamManager) |
| 16:59:28 | watusimoto_ | so mTeamManager is assigend to what it is |
| 16:59:36 | raptor | hmm... |
| 16:59:36 | watusimoto_ | it;s not a *teammanager anymore |
| 16:59:38 | raptor | ok |
| 16:59:47 | watusimoto_ | this exists in 2 places, I think |
| 17:00:01 | watusimoto_ | that's where the 2 came from |
| 17:00:13 | watusimoto_ | hi koda, btw |
| 17:00:17 | raptor | ah ok |
| 17:00:17 | watusimoto_ | where are my manners? |
| 17:00:20 | raptor | let me try that.. |
| 17:04:14 | raptor | does it need to be SmartPtr<TeamManager *> or SmartPtr<TeamManager> ? |
| 17:04:21 | raptor | because everything still fails.. |
| 17:05:14 | watusimoto_ | SmartPtr<TeamManager> |
| 17:05:21 | watusimoto_ | it's a pointer to a teamManager |
| 17:05:27 | watusimoto_ | not a pointer to a teamManager pointer |
| 17:05:49 | raptor | ok that's what i thought |
| 17:05:55 | watusimoto_ | if you have this anywhere, change it --> SmartPtr<TeamManager *> |
| 17:06:05 | raptor | ok, |
| 17:07:58 | raptor | mTeamManager = SafePtr<TeamManager>(new TeamManager) fails: http://pastie.org/5499108 |
| 17:09:17 | | BFLogBot Commit: e7a490989f4a | Author: watusimoto | Message: Rename the variable |
| 17:09:18 | | BFLogBot Commit: df3bc25b3954 | Author: watusimoto | Message: This thing is more trouble than it's worth. Remove it altogether. |
| 17:10:55 | watusimoto_ | show me your def of mTeamManager |
| 17:11:07 | raptor | SafePtr<TeamManager> mTeamManager; |
| 17:11:23 | watusimoto_ | mmmm |
| 17:11:32 | raptor | it works in game.h/.cpp! |
| 17:11:54 | | Fordcars has joined |
| 17:11:59 | Fordcars | hi |
| 17:12:07 | watusimoto_ | hi |
| 17:12:10 | raptor | wait maybe it doesn't - that class hasn't actually been compiled yet since it fails beforehand with UIEditor |
| 17:12:50 | Fordcars | In about 30 minutes I will have to go to a small village called "St-Joshep-de-Beauce" so I won't have internet :( |
| 17:13:03 | Fordcars | for 2 days |
| 17:13:12 | raptor | unplug! |
| 17:13:29 | watusimoto_ | try this: mTeamManager = new TeamManager; |
| 17:14:35 | watusimoto_ | sounds like somewhere in the wilds of Qebec |
| 17:14:35 | raptor | same error |
| 17:14:41 | watusimoto_ | Quebec |
| 17:15:35 | watusimoto_ | The same error, or an error in the same place? |
| 17:15:41 | raptor | both |
| 17:15:43 | raptor | :) |
| 17:17:40 | watusimoto_ | try twostepping it |
| 17:17:47 | raptor | done hat |
| 17:17:49 | raptor | that |
| 17:17:52 | watusimoto_ | TeamManger *t... |
| 17:18:00 | raptor | yep |
| 17:18:02 | watusimoto_ | like this? |
| 17:18:04 | raptor | same error |
| 17:18:06 | Fordcars | yep in the wilds :D |
| 17:18:32 | watusimoto_ | TeamManager *t = new TeamManager(); mTeammanager = t |
| 17:18:33 | watusimoto_ | ? |
| 17:18:42 | Darrel | Hey guys, for this task http://www.google-melange.com/gci/task/view/google/gci2012/8029210 I have to get the JSON data, too, right? |
| 17:19:00 | watusimoto_ | probably |
| 17:19:10 | raptor | watusimoto_: same error |
| 17:19:12 | Darrel | _probably_? :P |
| 17:19:13 | watusimoto_ | if you could do it without the json data, I'd give you two points :-) |
| 17:19:24 | Darrel | Oh great |
| 17:19:29 | | Martinski has joined |
| 17:19:32 | Darrel | no json then :D :D (I'll try at least) |
| 17:19:34 | Martinski | hey guys |
| 17:19:44 | watusimoto_ | hi |
| 17:19:54 | Martinski | I'm home finally |
| 17:20:16 | raptor | watusimoto_: someone claimed this: TeamManager *t = new TeamManager(); mTeammanager = t |
| 17:20:18 | raptor | oops |
| 17:20:20 | raptor | https://www.google-melange.com/gci/task/view/google/gci2012/8018212 |
| 17:20:23 | raptor | \^^ that |
| 17:20:26 | raptor | want to assign? |
| 17:21:02 | Martinski | watusimoto_ can I simply modify a bit the linux, windows and mac logos for the download page ? |
| 17:21:41 | watusimoto_ | raptor: assigned |
| 17:22:00 | watusimoto_ | Martinski: not sure what you mean |
| 17:22:15 | watusimoto_ | you mean the official mac logo and such? |
| 17:22:24 | Martinski | yes |
| 17:22:28 | Martinski | for example take the linux logo |
| 17:22:54 | Martinski | add a "linux" text with the bitfighter style |
| 17:23:04 | Fordcars | I am not sure you can modify official logos.... |
| 17:23:16 | Fordcars | but if you add text under you might have the right |
| 17:23:29 | watusimoto_ | yes. that's what I tried -- Good results with apple and windows, not so much with linux |
| 17:24:02 | watusimoto_ | I like the idea of neon vector style logos, but am open to anything |
| 17:24:22 | watusimoto_ | The examples I gave used modified official logos |
| 17:24:57 | Darrel | Neon blue with glossy or matte black... I LOVE IT! |
| 17:24:59 | Darrel | :) |
| 17:25:19 | watusimoto_ | Martinski: did you claim the lua task, or was that a different martin? |
| 17:25:27 | Darrel | it's a different one |
| 17:25:32 | Darrel | Martin Bede |
| 17:25:51 | watusimoto_ | also bulgarian? :-) |
| 17:25:59 | Darrel | No, I don't think so |
| 17:26:04 | Darrel | but I've never talked with him... |
| 17:26:24 | Darrel | and that should mean something,lol, I've talked with I guess half the participants :D |
| 17:26:46 | | Darrel is now known as Darrel|Dinner |
| 17:26:56 | raptor | mmm darrel for dinner |
| 17:27:05 | watusimoto_ | darrel or dinner, I think |
| 17:27:30 | raptor | darrel piped into dinner? |
| 17:27:43 | raptor | IO redirected to dinner |
| 17:27:49 | watusimoto_ | personally, I'd rather have dinner|watusimoto |
| 17:27:54 | raptor | haha |
| 17:28:02 | watusimoto_ | I don;t think I want what he's making |
| 17:28:19 | Martinski | oh |
| 17:28:20 | Martinski | sry |
| 17:28:24 | Martinski | I wasn't here |
| 17:28:28 | Martinski | I am with the logos task |
| 17:28:40 | Martinski | and now I am going for a quick snack |
| 17:29:23 | watusimoto_ | snack|Martinski? |
| 17:30:03 | raptor | gingerbreadcookies|raptor |
| 17:30:10 | watusimoto_ | now that sounds good |
| 17:30:20 | watusimoto_ | gingerbreadcookies|mail|watusimoto |
| 17:33:21 | raptor | maybe i'm going abotu this all wrong - maybe SafePtr can only be used to point, not to wrap... |
| 17:34:01 | raptor | my whole idea was that a teamManager is being deleted before the internal clearTeams() is being called |
| 17:35:05 | watusimoto_ | yes |
| 17:35:16 | raptor | so i thought i'd make sure it aways returns null |
| 17:35:23 | watusimoto_ | yes |
| 17:35:28 | watusimoto_ | very logical |
| 17:35:38 | | watusimoto_ is now known as dinner|watusimot |
| 17:35:43 | raptor | and now everything has gone horrible wrong |
| 17:35:46 | raptor | sigh |
| 17:35:47 | dinner|watusimot | dang... too long |
| 17:35:50 | raptor | haha |
| 17:36:01 | dinner|watusimot | din'r|watusimoto |
| 17:36:22 | | dinner|watusimot is now known as dinnr |
| 17:36:28 | | dinnr is now known as dinnr|watusimoto |
| 17:36:44 | dinnr|watusimoto | I understand everything going horribly wrong |
| 17:36:46 | dinnr|watusimoto | believe me |
| 17:37:05 | dinnr|watusimoto | but I do think your solution is a good one |
| 17:37:20 | raptor | where is sam686 when you need him.. |
| 17:37:27 | dinnr|watusimoto | the other possibility is to make the base team managers into references |
| 17:37:32 | dinnr|watusimoto | and never explicitly delete them |
| 17:37:58 | dinnr|watusimoto | just reassign or let them fall out of scope when their container is deleted |
| 17:38:41 | dinnr|watusimoto | ok, back in a bit |
| 17:38:45 | raptor | ok |
| 17:40:46 | Martinski | here |
| 17:43:05 | Martinski | is it good to use the new windows logo ? |
| 17:44:07 | raptor | watusimoto went to dinner. i'm afraid i'm not familiar with the new windows logo |
| 17:44:51 | Martinski | well it's new since the new windows 8 |
| 17:44:56 | | dinnr|watusimoto Quit (Ping timeout: 264 seconds) |
| 17:45:06 | Martinski | http://cdn.iblognet.com/wp-content/uploads/2012/03/Windows8-logo-300x291.jpg |
| 17:45:07 | Martinski | here |
| 17:45:18 | raptor | wow |
| 17:45:24 | raptor | that's... square |
| 17:45:27 | sam686 | hi |
| 17:45:31 | raptor | hi sam686 |
| 17:45:37 | raptor | oh good you're here... |
| 17:46:25 | raptor | sam686: in UIEditor, i'm trying to wrap mTeamManager in a SafePtr, but i get compile errors i can't figure otu |
| 17:46:27 | raptor | *out |
| 17:46:37 | raptor | so in UIEditor.h: SafePtr<TeamManager> mTeamManager; |
| 17:46:57 | | twodarray Quit (Ping timeout: 245 seconds) |
| 17:47:16 | raptor | but in UIEditor.cpp:144, i get this error when compiling: http://pastie.org/5499041 |
| 17:48:57 | sam686 | on TeamInfo.h, you need "class TeamManager : public SafePtrRef (and/or RefPtrRef if that is also needed) |
| 17:49:12 | sam686 | i mean SafePtrData RefPtrData |
| 17:49:16 | raptor | oh really? why is that? |
| 17:50:14 | sam686 | its to hold some data in the SafePtrData, you also need #include "tnlNetBase.h" as that is where it is.. |
| 17:50:57 | raptor | ah |
| 17:51:14 | raptor | i see - so all the BfObjects work because they are subclass of TNL::Object which has that... |
| 17:52:17 | raptor | let me try that... |
| 17:52:55 | raptor | it compiles! sam686, you're a genius! |
| 17:53:02 | raptor | now let's see if things crash.. |
| 17:55:34 | raptor | well after all of that, it still crashes... |
| 17:57:50 | Martinski | should the background of the download images be black ? |
| 17:59:06 | raptor | they can be |
| 17:59:20 | raptor | they could be transparent, too |
| 17:59:25 | Martinski | ok |
| 18:01:09 | Fordcars | any way guys got to go to the wilds :D |
| 18:01:17 | Fordcars | so see ya monday! |
| 18:01:24 | raptor | later! enjoy! |
| 18:01:27 | Martinski | bye :) |
| 18:01:35 | Fordcars | thanks! bye :) |
| 18:01:37 | | Fordcars Quit (Quit: Leaving) |
| 18:04:02 | | BFLogBot Commit: cad24a0de03c | Author: buckyballreaction | Message: Minor clarification |
| 18:10:13 | Martinski | may I ask for a little time extention, because I have <2hrs |
| 18:10:17 | raptor | sure |
| 18:10:21 | Martinski | 10x |
| 18:10:26 | raptor | which one again? |
| 18:10:28 | Martinski | I'm doing the download icons right now |
| 18:10:31 | raptor | icons, ok |
| 18:10:34 | raptor | one moment.. |
| 18:11:54 | raptor | done |
| 18:12:03 | Martinski | thank you :)) |
| 18:14:21 | sam686 | kindof strange you can recharge energy while moving very fast on speed zone (if you don't touch your controls) |
| 18:16:45 | Martinski | if the mac logo is black |
| 18:17:00 | Martinski | and you put the images on the site, it won't be possible to see it |
| 18:17:06 | Martinski | so I'll use the other light logo, ok ? |
| 18:17:14 | raptor | ok |
| 18:17:22 | raptor | really, i'm OK with anything :) |
| 18:17:37 | raptor | watusimoto probably has more of an opinion, but so far your ideas sound fine to me |
| 18:17:39 | Martinski | good for the participants :D |
| 18:19:29 | raptor | sam686: if you load this level in the editor (on mac/linux): http://bitfighter.org/forums/viewtopic.php?p=16440#p16440 |
| 18:19:37 | raptor | then quit the editor, there is a crash |
| 18:20:28 | raptor | which i'm trying to figure out - so far wrapping AbstractTEam and TeamManager does not solve it.. |
| 18:20:46 | Martinski | guys take a look at those 3 icons |
| 18:20:55 | Martinski | they are a bit large, but I can make them smaller |
| 18:20:57 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0UHFDVERLbEdJYW8 |
| 18:20:57 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0YktjcHRuZFVYT2c |
| 18:20:57 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0b2NROWFLX0dLRlk |
| 18:22:27 | | Darrel|Dinner is now known as Darrel |
| 18:23:41 | Martinski | what do you think ? |
| 18:24:15 | raptor | looks good |
| 18:24:22 | Martinski | should I make them smaller |
| 18:24:28 | raptor | i wonder, is there a simpler linux icon somewhere? |
| 18:24:28 | Martinski | now theri width is 800px |
| 18:24:31 | Martinski | their* |
| 18:24:54 | | raptor rereads the task... |
| 18:25:42 | Martinski | because somebody, I forgot who, told me that only the last icon should have the 3 sizes |
| 18:26:05 | raptor | watusimoto is the one you'd want to talk to |
| 18:28:38 | raptor | yeah, one needs to be like for an application icon that shows up in the task bar |
| 18:29:00 | Martinski | so, what size to make those |
| 18:30:15 | Martinski | something with height ~100 ? |
| 18:33:52 | raptor | well |
| 18:34:12 | raptor | ideally (i think) they'd be an SVG so they can be resized easily |
| 18:35:58 | raptor | i'm afraid i have to leave for a few hours - watusimoto should be back from dinner soon |
| 18:37:49 | Martinski | ok, was he near my timezone ? |
| 18:38:01 | Martinski | i'm utc+2 |
| 18:38:01 | raptor | yes |
| 18:38:01 | raptor | he is utc + 1 |
| 18:38:01 | Martinski | ok super |
| 18:38:02 | raptor | i am utc -7 |
| 18:38:12 | Martinski | you should go to bed now :D:D |
| 18:38:31 | raptor | it's actually almost noon |
| 18:38:34 | | Watusimoto_ has joined |
| 18:38:41 | raptor | i have groceries to buy! |
| 18:38:43 | Martinski | hey Watusimoto_ |
| 18:38:50 | Watusimoto_ | hi |
| 18:38:54 | Martinski | take a look at those |
| 18:38:55 | Martinski | <Martinski> https://docs.google.com/open?id=0BzOc42TE85E0UHFDVERLbEdJYW8 |
| 18:38:55 | Martinski | <Martinski> https://docs.google.com/open?id=0BzOc42TE85E0YktjcHRuZFVYT2c |
| 18:38:55 | Martinski | <Martinski> https://docs.google.com/open?id=0BzOc42TE85E0b2NROWFLX0dLRlk |
| 18:39:01 | Martinski | this is not the last size |
| 18:39:34 | raptor | ok i'm out, Watusimoto_ safeptr did not do the trick; neither for TeamManager nor for AbstractTeam |
| 18:39:44 | raptor | so i'm back to square one... |
| 18:39:47 | raptor | but leaving.. |
| 18:39:49 | raptor | later |
| 18:39:57 | Watusimoto_ | wait |
| 18:40:04 | raptor | ok |
| 18:40:08 | Watusimoto_ | raptor: do you want me to try something? |
| 18:40:20 | Watusimoto_ | I can't reproduce, but I can give things a shot and you can try it when you get back |
| 18:40:22 | raptor | i'm out of ideas at the moment |
| 18:40:25 | Watusimoto_ | ok |
| 18:40:31 | Watusimoto_ | I'll try something for you later |
| 18:40:35 | Watusimoto_ | bye! |
| 18:40:36 | raptor | i'm down to printfs in the destructors |
| 18:40:41 | raptor | ok bye |
| 18:40:50 | | raptor Quit () |
| 18:43:39 | Martinski | what do you think about the 3 download icons |
| 18:45:11 | Watusimoto_ | looking... haven;t forgotten about you! |
| 18:45:20 | Martinski | ok, sorry |
| 18:45:31 | Watusimoto_ | I'm trying to see them on a dark background like the website |
| 18:51:07 | Watusimoto_ | Martinski: I generally like them |
| 18:51:22 | Watusimoto_ | a couple of quick comments/suggestions |
| 18:51:36 | Martinski | ok |
| 18:52:18 | Watusimoto_ | the blue you used looks nice, but it seems to be neither the same as nor differnent than the blue text on the site, if you understand that, which might make a bit of a clash |
| 18:52:34 | Watusimoto_ | so maybe something similar but in green? |
| 18:52:51 | Martinski | sure |
| 18:52:59 | Martinski | I was wondering green or blue |
| 18:53:02 | Martinski | but decided blue :D |
| 18:53:06 | Martinski | now I'll change it |
| 18:53:19 | Watusimoto_ | I generally like blue, and liek the one you picked, except that we have something similar. |
| 18:53:28 | Watusimoto_ | the penguin looks larger than the other icons, so I'd shrink it a little |
| 18:53:54 | Watusimoto_ | and finally |
| 18:54:29 | Watusimoto_ | I'm not yet sure exactly how we'll use them -- i.e. will they be like section headers, or more like icons off to the left of the list of releases |
| 18:54:45 | Watusimoto_ | I'm kind of thinking off to the left |
| 18:55:10 | Watusimoto_ | especailly as we'll have two mac releases and (soon) two windows releases (regular and portable), so it might liook nice there |
| 18:55:29 | Martinski | so what's the other thing I have to change |
| 18:55:30 | Martinski | ? |
| 18:55:56 | Watusimoto_ | I'm getting to that :-) ...iin which case the icon should probably be a little smaller -- closer to the size of the text |
| 18:56:00 | Watusimoto_ | that's it |
| 18:56:37 | Martinski | ok, can I leave this size for 1 version |
| 18:56:43 | Martinski | and also add another version of the icons |
| 18:56:48 | Martinski | which have smaller height |
| 18:56:57 | Watusimoto_ | so in fact, if we go the off-to-the-left route, I can give you some specific dimensions, and we can only do one set |
| 18:57:08 | Watusimoto_ | rather than all the differtn sizes I specified |
| 18:57:34 | Martinski | doesn't matter to me, I just want it to became cool and good looking and to help you use it |
| 19:00:27 | Martinski | here is the green version |
| 19:00:27 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0R1hxTGRfWVhfUkk |
| 19:00:30 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0U3BVdXdlQ2RtNmc |
| 19:00:30 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0eHd6YmRSUVFlOVE |
| 19:00:30 | Martinski | do you like it ? |
| 19:01:06 | Watusimoto_ | 64 px high by (at most)150 pix wide should work well |
| 19:01:30 | Martinski | ok, I'll crop the big margins and do it 64 high |
| 19:01:32 | Martinski | make* |
| 19:03:44 | Watusimoto_ | but make the icon a little smaller and the text a little larger |
| 19:04:42 | Martinski | ok |
| 19:08:58 | Watusimoto_ | to get a sense of the colors, look here: |
| 19:08:59 | Watusimoto_ | http://img823.imageshack.us/img823/7154/61838954.png |
| 19:10:08 | Martinski | soo, it should be more greenish ? :D |
| 19:10:32 | Watusimoto_ | this is where all my years of expertise is worth nothing |
| 19:10:36 | Watusimoto_ | I'm not a good judge |
| 19:10:51 | Watusimoto_ | do you have a sister or mother you could ask? :-) |
| 19:11:05 | Watusimoto_ | girls are often a better judge of color than boys |
| 19:11:12 | Watusimoto_ | as a very rough generalization |
| 19:11:18 | Martinski | :D:D |
| 19:11:25 | Martinski | I have a mother, I'll ask her |
| 19:11:48 | Watusimoto_ | now that I have the whole thing in photoshop, I can try a couple of different colors |
| 19:12:24 | Martinski | do you want the .psd files ? |
| 19:14:05 | Martinski | wait, the screenshot you gave me is with the 1st version or with the green-er version ? |
| 19:16:01 | Watusimoto_ | the green one |
| 19:16:15 | Martinski | hmm ok |
| 19:16:21 | Watusimoto_ | tell you what |
| 19:16:22 | Martinski | looks totally different on black background |
| 19:16:38 | Watusimoto_ | yes -- generally you should work using the final background if you know it |
| 19:16:47 | Watusimoto_ | but lets do this |
| 19:16:51 | Watusimoto_ | leave the green |
| 19:17:02 | Watusimoto_ | and when we're done, give me the psd files |
| 19:17:06 | Martinski | ok |
| 19:17:13 | Watusimoto_ | and then I can change it if I want |
| 19:17:28 | Watusimoto_ | because it's easy to do and there's no point wasting both of our time on that |
| 19:25:06 | Martinski | here are with larger text |
| 19:25:09 | Martinski | smaller logos |
| 19:25:13 | Martinski | and cropped margins |
| 19:25:14 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0MUpzMmpBMkZqVDA |
| 19:25:14 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0N3JxQ2hXb2JUdDQ |
| 19:25:14 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0X1pUTDZRX0R4Q2c |
| 19:25:21 | Martinski | now only have to resize them |
| 19:34:33 | Watusimoto_ | looking pretty good -- try making the amount of whitespace between the text and the symbol more consistent -- you can right-justify everything so the icons line up vertically, and tht text ends a constant distance away, and begins where it begins |
| 19:36:36 | Martinski | so, the whitespace has to be constant from the right, while on the left it can be different depending on the text, right ?> |
| 19:41:54 | Darrel | they look pretty nice ^^ |
| 19:42:05 | Martinski | 10x |
| 19:42:13 | Martinski | I'm fixing the size right now |
| 19:42:19 | Darrel | Mac's apple seems kind of... too white, I think |
| 19:42:52 | Watusimoto_ | that's what my wife said too! :-) |
| 19:43:08 | Darrel | Maybe you should use the logo that's a bit more .. well, dunno, 3d-ish? |
| 19:43:17 | Darrel | windows' icon is good though |
| 19:43:19 | Martinski | ok |
| 19:43:23 | Watusimoto_ | I think flat is ok |
| 19:43:35 | Martinski | uhm |
| 19:43:49 | Martinski | actually it's |
| 19:43:59 | Martinski | white flat, black flat and other 3d |
| 19:44:28 | Darrel | http://www.macforensicslab.com/ProductsAndServices/images/icon_apple.jpg |
| 19:44:36 | Darrel | http://files.softicons.com/download/system-icons/apple-logo-icons-by-thvg/png/512/Apple%20logo%20icon%20-%20Aluminum.png |
| 19:44:43 | Darrel | these two ? |
| 19:44:47 | Darrel | what do you think |
| 19:45:00 | Martinski | yes, but Watusimoto_ said that flat is ok |
| 19:45:20 | Martinski | Watusimoto_ shound I use one of Darrel's ideas |
| 19:45:42 | Darrel | maybe use black flat? |
| 19:45:46 | Darrel | depends on the background, though |
| 19:45:50 | Watusimoto_ | it does |
| 19:46:13 | Watusimoto_ | of Daryl's links, the second seems better |
| 19:46:17 | Watusimoto_ | certainly the sivery color |
| 19:46:57 | Watusimoto_ | but if only one is 3d-ish, it would look odd, I think |
| 19:47:28 | Watusimoto_ | since they will all be displayed together |
| 19:48:12 | Watusimoto_ | so programming is hard, but creating icons with input from the peanut gallery is near impossible ;-0 |
| 19:48:17 | Martinski | I put the 3d |
| 19:48:25 | Martinski | looksbetter, we'll try |
| 19:48:36 | Martinski | just to align the text and I'll send you again |
| 19:50:35 | Watusimoto_ | thanks! |
| 19:54:12 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0b3owYkNleXRKZFE |
| 19:54:12 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0blgweFV1OUI5VTg |
| 19:54:12 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0cWxQY0RuNFphVUE |
| 19:54:17 | Martinski | here are the aligned versions |
| 19:55:04 | Darrel | btw, Linux icon is not flat, so it's not odd that Mac's icon is not :) |
| 19:56:35 | Martinski | yes |
| 19:57:07 | Darrel | I personally think the mac icon must be a bit higher |
| 19:57:12 | Watusimoto_ | I concur |
| 19:57:22 | Watusimoto_ | but the spacing looks good |
| 19:57:30 | Martinski | well, it's in the center, but the problem is the top of the apple |
| 19:57:36 | Darrel | I mean, move it up a bit, seems a bit weird.. |
| 19:57:37 | Darrel | yeah |
| 19:57:38 | Martinski | looks lower, but actually is centered :D |
| 19:57:54 | Darrel | I hate it when something like that happens :D |
| 19:58:40 | Martinski | my perfectionism is fighting in me, to create it visually right, or to leave it centered :D |
| 19:58:54 | Darrel | :D |
| 19:58:59 | Darrel | center it based on the apple, |
| 19:59:06 | Darrel | the tip would be higher |
| 19:59:12 | Martinski | Watusimoto_ what would you say ? |
| 19:59:21 | Watusimoto_ | I would say that the apple looks too low |
| 19:59:34 | Martinski | ok |
| 19:59:42 | Martinski | I'll make it like my eyes tell me :D |
| 20:00:03 | Watusimoto_ | even if it's aligned pixelwise, it should be algned visually |
| 20:00:18 | Watusimoto_ | because the apple leaf has very little visual weight |
| 20:01:12 | | Darrel Quit (Read error: Connection reset by peer) |
| 20:01:15 | | Darrel_ has joined |
| 20:01:26 | Darrel_ | damn! again! |
| 20:01:28 | | Darrel_ is now known as Darrel |
| 20:02:01 | Martinski | aaand the new alignment |
| 20:02:01 | Martinski | https://docs.google.com/file/d/0BzOc42TE85E0cWxQY0RuNFphVUE/edit |
| 20:02:41 | Darrel | My eyes are at ease now :D |
| 20:03:02 | Watusimoto_ | the silver apple looks a lot better on the black background |
| 20:03:12 | Martinski | for sure |
| 20:03:14 | Watusimoto_ | muuuch better alignment! |
| 20:03:39 | Watusimoto_ | it's like tux -- that;s not "properly" aligned, but looks right anyway |
| 20:04:09 | Martinski | ok, now to make the size change ? |
| 20:06:12 | Martinski | coming after 5 mins |
| 20:12:08 | Martinski | here |
| 20:12:12 | Martinski | what would you say ? |
| 20:15:03 | Watusimoto_ | oh, sorry |
| 20:15:10 | Watusimoto_ | I didn't realize you were waiting on input |
| 20:15:17 | Watusimoto_ | yes, sure go for it |
| 20:15:17 | Martinski | np |
| 20:15:36 | Martinski | uhm, now to make them 64 high |
| 20:26:10 | Martinski | does the width matter ? |
| 20:26:45 | Martinski | width = 205 |
| 20:26:45 | Watusimoto_ | so, make them all the same width |
| 20:26:48 | Watusimoto_ | but |
| 20:26:50 | Martinski | height = 64 |
| 20:26:58 | Martinski | good enough ? |
| 20:27:01 | Watusimoto_ | I think I posted before a max width based on that screen shot |
| 20:27:17 | Watusimoto_ | I said 150 |
| 20:27:20 | Martinski | but for the windows icon, 205 is the minimum ... |
| 20:27:37 | Martinski | I'll have to change the font size |
| 20:27:39 | Watusimoto_ | do me a favor and post that, and lets see how it works |
| 20:27:46 | Martinski | ok |
| 20:29:04 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0S1Q2akVUZzFMRXc |
| 20:29:04 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0TnNiQzFDRHl4M3M |
| 20:29:04 | Martinski | https://docs.google.com/open?id=0BzOc42TE85E0YUJIR0NEbGdJZG8 |
| 20:29:05 | Martinski | here |
| 20:29:35 | Watusimoto_ | ok, just a sec |
| 20:32:39 | Watusimoto_ | it fits |
| 20:32:39 | Watusimoto_ | http://img571.imageshack.us/img571/8240/91120737.png |
| 20:32:51 | Martinski | whew |
| 20:32:51 | Martinski | :D |
| 20:33:07 | Watusimoto_ | close call! |
| 20:33:45 | Darrel | Watusimoto_, when you're done there, can you take a look at my task, I uploaded a zip with a demo of the toater notification. It doesn't get any data from anywhere yet, but I just want to show it to you. It comes out in a fade in/fadeout way, I would have preferred slider, but there is some bug in the api and the frame of the notification appears first and I really don't like it this way. The zip is a visual studio project, you can test i |
| 20:33:45 | Darrel | t this way, haven't compiled it in exe... |
| 20:33:46 | Darrel | http://www.google-melange.com/gci/task/view/google/gci2012/8029210 |
| 20:33:58 | Darrel | Wow, that came out longer than I expected :D |
| 20:34:03 | Martinski | :D:D |
| 20:34:31 | Watusimoto_ | sure |
| 20:34:51 | Martinski | just to ask, are those 3 icons done, so I can go to thinking the 4-th :D |
| 20:36:44 | Watusimoto_ | that's the tricky on |
| 20:36:46 | Watusimoto_ | one |
| 20:37:37 | Watusimoto_ | one last thing to check, and that is how the penguin looks on the black bg |
| 20:38:01 | Martinski | ok, give a screenshot after that, I'm curious about it ;) |
| 20:38:02 | Watusimoto_ | I don't suppose you tried that? |
| 20:38:08 | Martinski | nope |
| 20:39:34 | Watusimoto_ | better than I expected, actually |
| 20:40:51 | Darrel | Give us screenshots, I want to see :D |
| 20:40:55 | Watusimoto_ | http://img841.imageshack.us/img841/1536/83656801.png |
| 20:41:50 | Darrel | cool! |
| 20:44:37 | Watusimoto_ | just to be complete |
| 20:44:38 | Watusimoto_ | http://img641.imageshack.us/img641/921/58298307.png |
| 20:44:55 | Watusimoto_ | much better tan the white apple |
| 20:44:55 | Martinski | looks great, I think |
| 20:45:25 | Watusimoto_ | of course you do :-) |
| 20:46:38 | Watusimoto_ | looking at the toaster |
| 20:46:54 | Watusimoto_ | what do I do, just build the project? |
| 20:47:39 | Darrel | yeah |
| 20:47:48 | Darrel | build and click the button |
| 20:47:50 | Darrel | :D |
| 20:48:26 | Watusimoto_ | ok, well, I don't have vc# installed... perhaps you can post a built app? |
| 20:48:34 | Darrel | ok |
| 20:48:39 | Darrel | In a minute.. |
| 20:50:12 | Watusimoto_ | Darrel: I didn;t specify a language, so I will accept a C# solution, but just cusious why you chose that? |
| 20:50:17 | Darrel | uploaded :) |
| 20:50:18 | Darrel | Oh, |
| 20:50:21 | Darrel | because of WPF |
| 20:50:32 | Watusimoto_ | what's that? |
| 20:50:34 | Darrel | and that's what is usually used for toast notifications |
| 20:50:42 | Darrel | .net or C# |
| 20:50:46 | Watusimoto_ | oh, ok |
| 20:51:03 | Darrel | WPF is "Windows Presentation Foundation" or something like that |
| 20:51:33 | Watusimoto_ | I like the small exe size |
| 20:53:16 | Watusimoto_ | there's some weird juju with the icon in the task bar when the toaster is displayed -- I assume this will not be a factor in the final version, which I imagine would live in the tray area |
| 20:53:35 | Darrel | haha, juju |
| 20:53:38 | Darrel | I like this :D |
| 20:54:10 | Darrel | there shouldn't be an icon in the taskbar, though |
| 20:54:24 | Darrel | maybe you can give me a screenshot? |
| 20:54:30 | Watusimoto_ | no, there isn't |
| 20:54:47 | Watusimoto_ | I'm thinking that when you're done there would be ?? |
| 20:55:06 | Watusimoto_ | because for this sort of thing you'll want it out of the way |
| 20:56:11 | Darrel | you would like a icon in taskbar? |
| 20:56:20 | Watusimoto_ | well, not really :-) |
| 20:56:30 | Watusimoto_ | but there would need to be a way to turn it off |
| 20:56:36 | Darrel | Oh, well |
| 20:56:43 | Darrel | Not sure how to implement such a thing :D |
| 20:56:54 | Watusimoto_ | that, I beleive, is very easy |
| 20:56:55 | Darrel | maybe it should be a setting ingame |
| 20:57:24 | Watusimoto_ | I was thinking this would be a standalone program |
| 20:57:31 | Watusimoto_ | you run this without running Bitfighter |
| 20:57:35 | Darrel | Oh, yeah |
| 20:57:42 | Watusimoto_ | when someone starts, you fire up Bitfighter |
| 20:57:44 | Darrel | it would be handy for servers, true |
| 20:57:57 | Darrel | I get the idea now |
| 20:58:51 | Watusimoto_ | I think there needs to be a task bar thing to control it. this wasn;t part of the case, but it would be awesome if there were a way to launch Bitfighter from that task bar thing. But we can add that later if it's more than trivial to do. |
| 20:59:27 | Watusimoto_ | i've given you feeback on several things, but probably not what you wanted |
| 20:59:35 | Watusimoto_ | what did you want again? :-) |
| 21:00:12 | Watusimoto_ | I'm looking at other notifiers, like the one for Thunderbird |
| 21:01:01 | Darrel | Oh, I appreciate the feedback :) I doubt I will be able to implement them all now, but maybe I can work them out as a future task |
| 21:01:04 | Martinski | Watusimoto_ sorry for interruption, but are those 3 icons OK, so I can upload them ? |
| 21:01:15 | Watusimoto_ | sure |
| 21:01:19 | Watusimoto_ | Martinski: yes |
| 21:01:42 | Watusimoto_ | Darrel: I was going to say that the way it pops up looks good |
| 21:01:56 | Darrel | oh good, that's the important thing for now :) |
| 21:02:33 | Watusimoto_ | it makes a little blink at the end of its popup sequence, which I assume is just a quirk of being launced from a menu |
| 21:03:01 | Watusimoto_ | so what next? poll json feed? |
| 21:03:23 | Darrel | Yeah |
| 21:03:25 | Darrel | that. |
| 21:03:42 | Darrel | I have to get JSON.net lib and see how I use it.. |
| 21:13:40 | Watusimoto_ | Darrel: this might interest you |
| 21:13:41 | Watusimoto_ | http://stackoverflow.com/questions/995195/how-can-i-make-a-net-winforms-application-that-only-runs-in-the-system-tray |
| 21:14:32 | Darrel | Oh thanks! |
| 21:15:11 | Martinski | Watusimoto_ are you going to be here tomorrow |
| 21:15:19 | Martinski | because I'm not sure I'll make the logo now |
| 21:15:25 | Martinski | I don't have any ideas :S |
| 21:20:59 | Darrel | Ah.. I hate Windows 8, but seriously, microsoft has added some REALLY cool namespaces and apis for it |
| 21:21:06 | Darrel | Toast notifications, data.json |
| 21:21:15 | Darrel | everything is soooo much easier on windows 8 |
| 21:21:31 | Martinski | true story |
| 21:21:57 | Martinski | on the hackaton I made a simple app for 3 days and took 7th place from 15 :D |
| 21:22:14 | Watusimoto_ | Martinski: yes, I'll be here at least some of the day, and most of the evening |
| 21:22:41 | Watusimoto_ | Martinski: and raptor is often around |
| 21:23:05 | Martinski | ok, any ideas |
| 21:23:12 | Martinski | for the icon thing :D |
| 21:26:58 | | raptor has joined |
| 21:26:58 | | ChanServ sets mode +o raptor |
| 21:28:22 | Watusimoto_ | hi raptor |
| 21:28:31 | Watusimoto_ | am just getting to your code now |
| 21:28:32 | | raptor reads logs |
| 21:28:33 | raptor | hi |
| 21:28:36 | Watusimoto_ | read away |
| 21:28:44 | raptor | no safeptr works |
| 21:28:48 | raptor | still crash |
| 21:28:52 | Watusimoto_ | I remember |
| 21:28:56 | Watusimoto_ | give me 5 minutes |
| 21:29:08 | Watusimoto_ | I got distracted reading an essay about why php sucks |
| 21:29:12 | Watusimoto_ | more of a book, really |
| 21:29:23 | raptor | haha |
| 21:29:25 | Martinski | :D:D:D |
| 21:29:58 | Martinski | raptor, any ideas what should the general icon contain |
| 21:30:03 | Watusimoto_ | ok, before I jump into the safeptr issue, did you try using references instead of pointers for team manager? |
| 21:30:23 | Martinski | something like "bitfighter" text + 2 ships for ex |
| 21:30:34 | Watusimoto_ | Martinski: to me, the main iconic things about the game are the ship and the font used in the Bitfighter logo |
| 21:30:42 | raptor | the solution was to make teamManager : public SafePtrData |
| 21:30:48 | raptor | and it compiled fine |
| 21:30:55 | raptor | but no safeptr worked |
| 21:31:31 | Watusimoto_ | our current icon is a pixelated ship, which seems odd for a game that eschews pixles (I'm guessing eschew will be a new word for you; rhymes with juju) |
| 21:31:38 | Watusimoto_ | :-) |
| 21:31:54 | Martinski | eschew ? |
| 21:32:08 | Watusimoto_ | eschew means to avoid |
| 21:32:11 | Watusimoto_ | roughly |
| 21:32:42 | Watusimoto_ | so to say "make it clearer" you could say "eschew obfuscation" |
| 21:32:56 | Martinski | learned something new :D |
| 21:33:00 | Watusimoto_ | :-) |
| 21:33:18 | Martinski | soo, I have to make the ship more clear :D |
| 21:33:24 | Watusimoto_ | ok raptor focusing on you now |
| 21:33:34 | Watusimoto_ | justa fter I reply to Martinski once again :-) |
| 21:33:51 | Watusimoto_ | you could try that; don't know how it would look! |
| 21:33:57 | raptor | take your time, i'm playing neverball with my son... :) |
| 21:34:06 | Martinski | ok, I'll stop bothering you |
| 21:34:08 | Martinski | now |
| 21:34:12 | Martinski | and soon I'll go to bed |
| 21:34:16 | Martinski | see you tomorrow :) |
| 21:35:13 | Watusimoto_ | Martinski: good night |
| 21:35:22 | Watusimoto_ | Martinski: but no bother! |
| 21:35:32 | Watusimoto_ | Martinski: you're helping us! |
| 21:35:51 | Darrel | Oh boy, looks like I might switch to a new way to make the notification... |
| 21:35:57 | Darrel | might simplify further work |
| 21:36:11 | Martinski | night guys :) |
| 21:36:25 | Darrel | night |
| 21:36:31 | | Martinski Quit () |
| 21:36:33 | Watusimoto_ | Darrel: why so? |
| 21:36:41 | Watusimoto_ | Darrel: or how so? |
| 21:36:54 | Darrel | code will be less, I think |
| 21:37:04 | Darrel | and probably I'll be able to make the notification come with slide :) |
| 21:37:08 | Darrel | main point ^ |
| 21:37:10 | Darrel | :D |
| 21:37:33 | Watusimoto_ | Darrel: I will say that the solution I expected was finding some similar code, gutting it and adding a bit of Bitfighter stuff |
| 21:38:22 | Darrel | That's basically what I'm doing :) |
| 21:38:49 | Darrel | but I want as simple as possible a solution, no needless clutter of code and stuff. |
| 21:38:58 | Watusimoto_ | why do yourself what you can get someone else to do for you? |
| 21:39:08 | Watusimoto_ | actually, that's what's great about gci :-) |
| 21:39:17 | Darrel | I think I found better code, that's why :D |
| 21:39:31 | Darrel | now I will have to adapt it a bit |
| 21:39:42 | Darrel | fix some things and see how it works out |
| 21:39:55 | Darrel | hopefully, it will be also easier to get the json data with it |
| 21:39:57 | Watusimoto_ | raptor: compiling a potential solution right now |
| 21:40:12 | Watusimoto_ | Darrel: I like easy |
| 21:40:31 | Darrel | me too, me too :) |
| 21:40:45 | Watusimoto_ | Darrel: your task expired. I assume you want me to extend? |
| 21:41:24 | Darrel | oh yes, this will be good :) |
| 21:42:54 | Watusimoto_ | whoa |
| 21:43:00 | Watusimoto_ | somehow you ended up with 47 hours |
| 21:43:06 | Watusimoto_ | ok, whatever |
| 21:43:28 | Darrel | haha |
| 21:43:49 | Watusimoto_ | raptor: I'm starting to see why Arc was so frazzled |
| 21:44:56 | Watusimoto_ | ok, I think my TeamManager solution is finally compiling |
| 21:45:05 | raptor | hi again |
| 21:45:08 | Watusimoto_ | I think it will prevent double delete |
| 21:45:14 | raptor | so neverball can make you motion sick |
| 21:45:27 | raptor | ok |
| 21:45:35 | Watusimoto_ | ha.. but I can't verify that it fixes the original problem |
| 21:45:41 | Watusimoto_ | so I'm going to check it in |
| 21:45:41 | raptor | i await your code |
| 21:45:45 | raptor | ok |
| 21:46:03 | Watusimoto_ | after compilation finishes |
| 21:46:07 | raptor | i still don't know what's going on - i feel like my mind is lacking some fundamental idea to be able to grasp a problem of this sort.. |
| 21:50:58 | Watusimoto_ | ok pushed |
| 21:51:01 | Watusimoto_ | also, did you see this? |
| 21:51:02 | Watusimoto_ | http://img641.imageshack.us/img641/921/58298307.png |
| 21:51:13 | Watusimoto_ | possible mockup of an enhanced downloads page |
| 21:51:40 | raptor | i'm not sure i like microsoft's new logo |
| 21:52:20 | | BFLogBot Commit: ad1169106d07 | Author: watusimoto | Message: Make TeamManager a referened object rather than a pointed-to-object. Maybe fixes reported crashing issue. |
| 21:52:21 | | BFLogBot Commit: b79910b1b6df | Author: watusimoto | Message: Merge |
| 21:52:40 | Watusimoto_ | this one? |
| 21:52:41 | Watusimoto_ | http://i.telegraph.co.uk/multimedia/archive/02318/microsoft-logo_2318464c.jpg |
| 21:52:45 | Watusimoto_ | I like it somewhat |
| 21:52:55 | Watusimoto_ | it's clean |
| 21:53:14 | raptor | that's better than the angled one |
| 21:53:32 | Watusimoto_ | you know, I wish the fine people at Christian Matchmaking would do the Christian thing and stop spamming me!!!! |
| 21:53:41 | raptor | hahaha |
| 21:53:53 | raptor | ok let's see what you did.. |
| 21:54:47 | raptor | well it compiled, that's promising... |
| 21:55:12 | raptor | and... |
| 21:55:16 | raptor | crash! |
| 21:55:43 | raptor | same trace: http://pastie.org/5499972 |
| 21:55:58 | raptor | this means... |
| 21:56:08 | raptor | i don' tknow.. |
| 21:56:15 | | raptor goes and cries in a corner |
| 21:56:47 | Darrel | :) poor raptor |
| 21:57:03 | Watusimoto_ | what??? |
| 21:57:11 | raptor | nice full rebuild, too |
| 21:57:32 | Watusimoto_ | it means it's not a double delete of the TeamManager |
| 21:57:39 | raptor | yep.. |
| 21:58:05 | Watusimoto_ | which actually makes sense, because when I was makeing these changes, I only found two deletes, and I couldn;t see how we'd be getting the double delete |
| 21:58:22 | raptor | i'm starting to remember something |
| 21:58:24 | Watusimoto_ | but ok. |
| 21:58:30 | raptor | like we had this exact error before with 016 |
| 21:58:46 | Watusimoto_ | interesting. i vaugely remember that too |
| 21:59:26 | Watusimoto_ | you know, it's the lineeditor destructor that's throwing the error |
| 21:59:50 | raptor | january 13/14 is when we had the problem |
| 22:00:16 | Watusimoto_ | and the line editor destructor has this line: |
| 22:00:17 | Watusimoto_ | / Do nothing |
| 22:00:26 | Watusimoto_ | which isn't really helpful ;-) |
| 22:01:22 | Watusimoto_ | it's probably a double delete of a team |
| 22:01:30 | Watusimoto_ | not the team manager |
| 22:01:31 | raptor | see top of log here: http://pastie.org/5499972 |
| 22:01:32 | raptor | oops |
| 22:01:36 | raptor | here: http://bitfighter.org/irclogs/index.php?date=2012-01-14 |
| 22:02:04 | Watusimoto_ | spooky |
| 22:02:45 | raptor | this is like the same trace on windows: http://sam6.25u.com/upload/text1201/120113_15-01-12.txt |
| 22:02:48 | raptor | that sam686 had |
| 22:03:00 | raptor | ohwait, that's probably gdb in linux |
| 22:04:25 | raptor | maybe i should just go through all the classes and make sure we have virtual destructors everywhere? |
| 22:05:07 | Watusimoto_ | ok, well editorTeam does have a LineEditor member, so it makes sense the destructor would be called |
| 22:05:42 | raptor | so does that mean some other class has a pointer to LineEditor and is cleaning it up first? |
| 22:05:43 | Watusimoto_ | I think a team is being deleted twice |
| 22:06:32 | Watusimoto_ | I think there must be some cleanup code that deletes it without nulling a pointer or removing it from a list |
| 22:07:15 | raptor | looks for clearTeams() |
| 22:07:17 | Watusimoto_ | try adding a deleted bool to team editor, setting it to false in the constructor, true in the destructor, and print its value on destruction |
| 22:07:21 | raptor | it is found in a few places... |
| 22:07:23 | Watusimoto_ | or better |
| 22:07:33 | Watusimoto_ | print the pointer to this in the destructor |
| 22:07:41 | Watusimoto_ | and see if it's getting deleted twice |
| 22:07:51 | Watusimoto_ | logprintf("%p", this); |
| 22:07:52 | raptor | i did that once, it only got there one time |
| 22:08:02 | Watusimoto_ | in the teameditor? |
| 22:08:02 | raptor | both in editor team and in lineeditor |
| 22:08:05 | raptor | yes |
| 22:08:09 | Watusimoto_ | editorteam? |
| 22:08:09 | raptor | yes |
| 22:08:26 | Watusimoto_ | did it print immediately before the crash? |
| 22:08:39 | Watusimoto_ | or might it have crashed before the print? |
| 22:08:39 | raptor | it printed once, then crash |
| 22:08:45 | raptor | several times |
| 22:08:54 | raptor | i tested several times, i mean |
| 22:09:05 | Watusimoto_ | but it would havebeen the second delete that crashed it... |
| 22:09:18 | Watusimoto_ | so maybe the 2nd delete crashed before printing? |
| 22:09:40 | Watusimoto_ | ok, let me try some breakpoints |
| 22:10:09 | Watusimoto_ | so how exatly to you reproduce? |
| 22:10:13 | Watusimoto_ | I have the level in question |
| 22:10:24 | raptor | just open it, then quit the editor |
| 22:10:27 | raptor | that's it |
| 22:10:49 | Watusimoto_ | okm when I do that, the EditorTeam destructor gets called twice |
| 22:10:55 | raptor | pointer in ~EditorTeam: |
| 22:10:57 | raptor | 0x10ef670 |
| 22:10:58 | raptor | *** glibc detected *** bitfighter: munmap_chunk(): invalid pointer: 0x00007ffff6731481 *** |
| 22:11:00 | Watusimoto_ | which is what I would expect on a level with two teams |
| 22:11:01 | raptor | ok |
| 22:11:06 | raptor | there are two teams in the map |
| 22:11:11 | Watusimoto_ | yes |
| 22:11:29 | Watusimoto_ | odd that this is platform specific |
| 22:11:36 | Watusimoto_ | and even moreso that it is level specific |
| 22:12:39 | Watusimoto_ | try replacing this: |
| 22:12:40 | Watusimoto_ | mTeams.deleteAndClear(); |
| 22:12:50 | Watusimoto_ | teamInfo.cpp 466 |
| 22:12:55 | Watusimoto_ | with something like this |
| 22:13:20 | raptor | ok |
| 22:13:50 | Watusimoto_ | for(S32 i = 0; i < mTeams.size(); i++) { logprintf("%p", mTeams[i]); delete mTeams[i]; } mteams.clear() |
| 22:14:04 | raptor | ok |
| 22:14:41 | Watusimoto_ | the other thing we could do is make mTeams hold a RefPtr or whatever we're using in place of shared_ptr |
| 22:14:50 | Watusimoto_ | and let boost deal with deleting |
| 22:16:24 | raptor | ok i also added a print statement before hand |
| 22:16:29 | raptor | so: |
| 22:16:30 | raptor | in clearTeams() |
| 22:16:32 | raptor | in clearTeams() |
| 22:16:33 | raptor | 0x10ef700 |
| 22:16:35 | raptor | *** glibc detected *** |
| 22:17:02 | raptor | clearteams() being called twice |
| 22:17:56 | Watusimoto_ | is it now... |
| 22:18:14 | raptor | this is where i was last night when i went to bed |
| 22:18:19 | Watusimoto_ | which clearTeams() |
| 22:18:22 | Watusimoto_ | ? |
| 22:18:55 | Watusimoto_ | I have an idea... new gci task to fix this bug! |
| 22:19:53 | raptor | wait |
| 22:20:09 | raptor | it is only being called once - one time on editor start up, the other time from exit |
| 22:20:23 | raptor | being called from EditorUserInterface::cleanUp() |
| 22:21:14 | raptor | when i commented out clearTeams() in UIEditor, it didn't crash on editor exit, rather it threw the same crash on game exit |
| 22:21:44 | Watusimoto_ | the first time it doesn't have any teams in the vector, the 2nd time it has two |
| 22:21:46 | Watusimoto_ | as expected |
| 22:22:12 | Watusimoto_ | so it's not a double delete of the team, but perhaps there is soemthing unexpectedly linking two teams |
| 22:22:15 | Watusimoto_ | a static? |
| 22:22:35 | raptor | yeah, something... |
| 22:23:25 | Watusimoto_ | ok |
| 22:23:43 | Watusimoto_ | abstractTeam only holds one object, a Color reference |
| 22:23:46 | Watusimoto_ | seems innocent |
| 22:24:08 | raptor | oh wow, the newer eclipse let's me inspect vectors! |
| 22:24:09 | Watusimoto_ | Team is out of the picture since none of thsoe get created in your scenario... theoretically |
| 22:24:15 | raptor | mayb ei can even debug now! |
| 22:24:22 | Watusimoto_ | so that leaves EditorTeam |
| 22:25:33 | Watusimoto_ | and the only thing that holds is a ref to LineEditor mNameEditor; |
| 22:25:40 | Watusimoto_ | so you could try this |
| 22:25:47 | Watusimoto_ | try removing mNameEditor |
| 22:25:57 | Watusimoto_ | and everywhere it's used return dummy data |
| 22:26:39 | Watusimoto_ | and maybe in getLineEditor, return a new LineEditor which you'll just not cleanup |
| 22:27:11 | Watusimoto_ | or you could try this |
| 22:27:27 | Watusimoto_ | Move the initialization of mNameEditor from here: EditorTeam::EditorTeam() : mNameEditor(LineEditor(MAX_TEAM_NAME_LENGTH)) |
| 22:27:32 | Watusimoto_ | into the body of the constructor |
| 22:27:42 | Watusimoto_ | shouldn;t make a difference.... but you never know |
| 22:27:57 | Watusimoto_ | wait |
| 22:28:05 | Watusimoto_ | something looks wrong here |
| 22:28:21 | Watusimoto_ | that might be it |
| 22:28:38 | Watusimoto_ | in .h, we have this: LineEditor mNameEditor |
| 22:29:02 | Watusimoto_ | so when object is first constructed, mNameEditor is a ref to a Lineeditor, right? |
| 22:29:08 | raptor | right |
| 22:29:13 | Watusimoto_ | so what is going on here? |
| 22:29:13 | Watusimoto_ | mNameEditor(LineEditor(MAX_TEAM_NAME_LENGTH)) |
| 22:29:17 | Watusimoto_ | in the constructor |
| 22:29:24 | raptor | ugliness |
| 22:29:38 | Watusimoto_ | we're creating a new LineEidotr when all we want to do is set the MAX_TEAM_NAME_LENGTH somehow |
| 22:29:56 | Watusimoto_ | let's fix that and see if that fixes the problem |
| 22:31:05 | Watusimoto_ | / Constructor |
| 22:31:05 | Watusimoto_ | EditorTeam::EditorTeam() |
| 22:31:05 | Watusimoto_ | { |
| 22:31:05 | Watusimoto_ | mNameEditor.mMaxLen = MAX_TEAM_NAME_LENGTH; |
| 22:31:05 | Watusimoto_ | } |
| 22:31:19 | raptor | ah, that's what i was going to ask |
| 22:31:45 | Watusimoto_ | though we should really have a setter for that |
| 22:31:49 | Watusimoto_ | but try it |
| 22:31:58 | Watusimoto_ | I'm feeling good about this one |
| 22:33:37 | raptor | ok compiling.. |
| 22:35:10 | | Darrel Quit (Read error: Connection reset by peer) |
| 22:35:27 | raptor | crash! same trace |
| 22:35:29 | Watusimoto_ | ha a use of eschew in the general press |
| 22:35:30 | Watusimoto_ | anecdotal evidence that young people eschew email in favour of social networks, |
| 22:35:35 | Watusimoto_ | what?!? |
| 22:35:47 | raptor | yep |
| 22:35:53 | Watusimoto_ | you'r lying! |
| 22:35:57 | raptor | did a full rebuild |
| 22:37:33 | Watusimoto_ | ok |
| 22:37:47 | Watusimoto_ | can we make TeamManager use refs to teams rather than pointers? |
| 22:37:55 | Watusimoto_ | though that's probably not the issue |
| 22:39:51 | Watusimoto_ | ok, two possibilities |
| 22:40:04 | Watusimoto_ | 1) try removing teamEditor as I suggested earlier |
| 22:40:16 | Watusimoto_ | sorry lineEditor |
| 22:40:41 | Watusimoto_ | 2) try making team manger use vector of teams instead of pointers to teams |
| 22:42:11 | Watusimoto_ | looking to see if 2) is even feasible |
| 22:51:19 | raptor | ok, i gave to go again.. |
| 22:51:25 | Watusimoto_ | ok |
| 22:51:30 | Watusimoto_ | I'll try 2 |
| 22:51:34 | raptor | i'll look at it again later |
| 22:51:38 | Watusimoto_ | and you can testl8r |
| 22:51:46 | raptor | ok |
| 22:51:50 | raptor | bye! |
| 22:51:54 | Watusimoto_ | ciao |
| 22:54:36 | | raptor Quit () |
| 23:14:06 | | CrazyLinuxNerd Quit (Quit: Leaving) |
| 23:14:32 | | CrazyLinuxNerd has joined |
| 23:43:51 | Watusimoto_ | well, bad new bears here |
| 23:44:01 | Watusimoto_ | found the old forcefield bug lurking |
| 23:44:15 | Watusimoto_ | and a new crash I don;t understand |
