Timestamps are in GMT/BST.
| 00:15:00 | karamazovapy | there are some accusations of registered name hacking in the game lobby |
| 00:15:00 | karamazovapy | not sure if it's just one player screwing with botfighter, though |
| 00:18:00 | sam686 | it could be the master might have trouble reading the database, and then just let that person in, not underlined.. |
| 00:22:00 | sam686 | or it might have been some users leaked their password to log in, possible when the password is saved in bitfighter.ini |
| 00:23:00 | Watusimoto | sam686 -- #38 on your list should be done (deleting large number of items is too slow) (came from my list, not sure how it got on yours!) |
| 00:24:00 | Watusimoto | I've made the items I think are done italic in the wiki |
| 00:24:00 | sam686 | ok.. |
| 00:24:00 | Watusimoto | you can remove them when you are statisfied |
| 00:25:00 | sam686 | i saw "hugo" not underlined a while ago... (though, that disappeared) |
| 00:26:00 | sam686 | as in, not underlined in bitfighter.org |
| 00:34:00 | Watusimoto | could have been hugo (with space) or soemthing |
| 00:36:00 | sam686 | possibly, though i didn't look hard enough... |
| 00:37:00 | sam686 | only way to know for sure is to read master log file... |
| 00:38:00 | sam686 | hint, in linux, use a command: cat bitfighter_master.log |
| 00:38:00 | | BFLogBot - Commit b7621efa8251 | Author: watusim...@bitfighter.org | Log: •Deleting large numbers of objects now dramatically faster; sadly, couldn't use work done in previous checkin... |
| 00:38:00 | | BFLogBot - Commit 76902a2a8c93 | Author: watusim...@bitfighter.org | Log: whitespace |
| 00:38:00 | | BFLogBot - Commit 9dcc022eced1 | Author: watusim...@bitfighter.org | Log: Extend wonders of modern batch deletion to dragging items to the dock |
| 00:38:00 | | BFLogBot - Commit 269a9b62b88f | Author: watusim...@bitfighter.org | Log: Comment |
| 00:38:00 | | BFLogBot - Commit b6c26c32c409 | Author: watusim...@bitfighter.org | Log: whitespace |
| 00:40:00 | | raptor has joined |
| 00:40:00 | | ChanServ sets mode +o raptor |
| 00:42:00 | sam686 | there is some conflict and link errors trying to build master, it needs gameWeapons.h just for enum and WeaponInfo::getWeaponName, but gameWeapons include a lot of "ship.h" "projectile.h"... the stuff master won't use... |
| 00:43:00 | raptor | hello!! |
| 00:43:00 | raptor | oooo |
| 00:44:00 | raptor | i leave for a few hours and we have stories of intrigue! |
| 00:45:00 | sam686 | it seems a problem to have ProjectileInfo gProjInfo in gameWeapons.cpp, but "struct ProjectileInfo" om projectile.h |
| 00:45:00 | sam686 | in, not om. |
| 00:45:00 | raptor | master is failing to compile now? |
| 00:46:00 | sam686 | yes |
| 00:49:00 | raptor | ok, i'll take a look |
| 00:53:00 | raptor | still looking through the latest changesets... |
| 00:57:00 | raptor | that's a weird error |
| 00:58:00 | raptor | ah I see |
| 00:59:00 | raptor | my header clean-up |
| 01:05:00 | | Watusimoto Quit (Ping timeout: 240 seconds) |
| 01:32:00 | karamazovapy | fyi - the two max678's (or whatever his name is) were both underliend |
| 01:32:00 | karamazovapy | *underlined |
| 02:02:00 | sam686 | 2 or more of a same name both underlined, could be one of several reasons. |
| 02:02:00 | sam686 | 1. One person runs multiple clients |
| 02:02:00 | sam686 | 2. If Hugo uses multiple computer, their login password is saved, that lets another person use that client without changing the username and password. |
| 02:02:00 | sam686 | 3. Hugo's username and password might have been leaked or shared by multiple people, ask Hugo to change password? |
| 02:03:00 | sam686 | there are screenshots Botfighter sent me. http://sam686.maxhushahn.com/upload/botfighter_screenshot_bfhjgo22f.png http://sam686.maxhushahn.com/upload/botfighter_screenshot_blahbfhgo2.png |
| 02:04:00 | karamazovapy | yeah, I told him the same thing |
| 02:04:00 | karamazovapy | and told him to change his forum password if he was worried |
| 02:05:00 | sam686 | I don't think Hugo itself use the forum much, because of Last visited: Mon Nov 28, 2011 |
| 02:14:00 | sam686 | you can edit your PM as long as it is in the Outbox (haven't been read by receiving person yet) |
| 02:38:00 | | Zoomber has joined |
| 02:38:00 | | ChanServ sets mode +v Zoomber |
| 02:43:00 | | Zoomber is now known as asdfasdf |
| 02:43:00 | | asdfasdf is now known as Test |
| 02:48:00 | | Test is now known as Minebot |
| 02:48:00 | | Minebot is now known as Mcbot |
| 02:49:00 | | Mcbot is now known as FactionBOT |
| 03:40:00 | | FactionBOT Quit (Quit: FactionBOT) |
| 03:45:00 | raptor | OK, I think I finally got the weapon stuff cleaned-up a little... |
| 03:50:00 | raptor | OK sam686, master should compile now with my latest changes |
| 03:51:00 | sam686 | ah, and i just fix the master compile too (not pushed yet) |
| 03:51:00 | raptor | sorry - i had familial duties to attend to.. |
| 03:52:00 | sam686 | one thing that is broken - dedicated server compile... |
| 03:52:00 | raptor | i'm sure |
| 03:53:00 | raptor | but i'm still worried about the other bugs first... |
| 03:53:00 | raptor | and I'm on vacation! |
| 03:54:00 | sam686 | oh then that means i need to edit my master/bitfighter_master.vcproj to add a file... |
| 03:54:00 | raptor | oops, i forgot that one... |
| 03:54:00 | | BFLogBot - Commit 596cc36ce433 | Author: buckyballreaction | Log: Reorganize weapon data - get rid of gWeapons and gProjInfo - also fix master compiling that relied on WeaponInfo class |
| 03:54:00 | raptor | sorry |
| 03:54:00 | sam686 | although i don't think there is a mac master server... |
| 03:54:00 | raptor | yeah, no master for mac |
| 03:54:00 | raptor | ever |
| 03:54:00 | raptor | i mean, i'm not willing to put the work into it |
| 04:07:00 | raptor | one of these days I will compile with gcc -Wall |
| 04:07:00 | raptor | and see how compliant our code really is... |
| 04:09:00 | | BFLogBot - Commit f3dde5a1825f | Author: buckyballreaction | Log: Fix minor compile warning |
| 04:10:00 | raptor | just for fun i'm compiling with -Wall... |
| 04:10:00 | sam686 | what does that do? |
| 04:10:00 | raptor | turns on all of gcc's warnins |
| 04:11:00 | raptor | warnings |
| 04:11:00 | sam686 | let be guess, 10000 warnings... |
| 04:12:00 | raptor | http://pastebin.com/mR2b9UYe |
| 04:12:00 | raptor | there we go... |
| 04:13:00 | | raptor Quit (Remote host closed the connection) |
| 04:14:00 | | raptor has joined |
| 04:14:00 | | ChanServ sets mode +o raptor |
| 04:14:00 | sam686 | 645 warnings.. |
| 04:14:00 | raptor | looks like many can be gotten rid of from just those ones in EditorObject.h.. |
| 04:15:00 | sam686 | many warning are simply repeating the same thing ( .h file) |
| 04:16:00 | raptor | i bet if i add 'const' to those they'll go away... |
| 04:18:00 | raptor | better: http://pastebin.com/kkWzfvJN |
| 04:19:00 | sam686 | 436 warnings on that paste bin |
| 04:19:00 | raptor | how did you count the warnings so fast? |
| 04:22:00 | sam686 | i will print screen how i did that, chrome actually counts how many it finds for me.. |
| 04:22:00 | raptor | interesting |
| 04:24:00 | sam686 | http://sam686.maxhushahn.com/upload/chrome_search.png look at top right |
| 04:26:00 | raptor | ah neat |
| 04:27:00 | sam686 | it might help to click on "RAW" pastie then search, so it doesn't count in double in the editor box bottom of pastis |
| 04:27:00 | sam686 | http://pastebin.com/raw.php?i=kkWzfvJN it finds "warning" 387 times |
| 04:31:00 | raptor | i'm not sure how to get rid of these warnings: |
| 04:31:00 | raptor | UIEditor.h:106:21: warning: ‘Zap::HIGHLIGHT_COLOR’ defined but not used [-Wunused-variable] |
| 04:31:00 | raptor | UIEditor.h:107:21: warning: ‘Zap::SELECT_COLOR’ defined but not used [-Wunused-variable] |
| 04:31:00 | raptor | static const S32 is allowed to be declared in a .h file |
| 04:32:00 | raptor | but those are pointers... |
| 04:37:00 | raptor | I got it! |
| 04:37:00 | sam686 | i think i have just fixed dedicated compile |
| 04:38:00 | sam686 | and pushed.. |
| 04:39:00 | sam686 | !Read_Google_Code_Revision |
| 04:39:00 | raptor | ok |
| 04:39:00 | | BFLogBot - Commit f6d1b802e775 | Author: sam8641 | Log: USE_EXCEPTION_BACKTRACE doesn't seem useful, it can crash in ways other then segfault. |
| 04:39:00 | | BFLogBot - Commit e801cb7da409 | Author: sam8641 | Log: Fix dedicated compile |
| 04:39:00 | raptor | http://pastebin.com/jx9vFwgQ |
| 04:39:00 | raptor | ^^ got rid of more... |
| 04:40:00 | sam686 | 101 warnings on http://pastebin.com/raw.php?i=jx9vFwgQ |
| 04:40:00 | raptor | progress!! |
| 04:42:00 | | FactionBOT has joined |
| 04:42:00 | sam686 | ClientGame.cpp:945:10: warning: enumeration value �ReasonTimedOut� not handled in switch [-Wswitch] |
| 04:42:00 | sam686 | i bet it is missing "default:" |
| 04:42:00 | raptor | yes |
| 04:42:00 | raptor | i just solved the UIEditor one with a 'default' |
| 04:49:00 | raptor | this is a fun one: soccerGame.cpp:511 |
| 04:50:00 | raptor | to which 'if' statement is the 'else' supposed to go? |
| 04:52:00 | sam686 | haha, i wonder if compiler choose the last if to make else belong to |
| 04:53:00 | sam686 | well, as the number of space says, it should be the first else. |
| 04:53:00 | raptor | haha, yep |
| 04:53:00 | raptor | that's funny |
| 04:53:00 | sam686 | but i guess compiler ignores repeated spaces |
| 05:13:00 | FactionBOT | hi |
| 05:13:00 | FactionBOT | yoyo |
| 05:13:00 | raptor | hi zoomber |
| 05:13:00 | | FactionBOT is now known as Zoomber |
| 05:13:00 | | ChanServ sets mode +v Zoomber |
| 05:16:00 | raptor | ok sam686, i'm gonna need your logic abilities here... |
| 05:18:00 | sam686 | what is it? |
| 05:19:00 | raptor | if i int | int | int | int |
| 05:19:00 | raptor | is that the same as (int | int) | (int | int) ? |
| 05:19:00 | sam686 | (int | int) | (int | int) is the same as int | int | int | int |
| 05:20:00 | raptor | ok, that's what i thought |
| 05:20:00 | sam686 | think of it as ||, but in multiple bits at a time.. |
| 05:20:00 | raptor | ok |
| 05:24:00 | sam686 | cool, don't let the order of operation mess up the math, you can use paranthesis. http://en.wikipedia.org/wiki/Order_of_operations#Programming_languages |
| 05:27:00 | sam686 | there is ^ operator, which is kind of the same as doing !=, but for each single bit.. |
| 05:28:00 | sam686 | note, in C++, ^ is "Bitwise exclusive OR" |
| 05:30:00 | raptor | yes |
| 05:38:00 | raptor | gameType.cpp:107 |
| 05:40:00 | raptor | solved it, nevermind |
| 05:51:00 | raptor | sam686: dataConnection.cpp:109 |
| 05:51:00 | raptor | what is the precedence there? |
| 06:04:00 | sam686 | while(!started || dataConn && dataConn->isEstablished()) it appears to do && first |
| 06:04:00 | sam686 | just like this wiki says. http://en.wikipedia.org/wiki/Order_of_operations#Programming_languages |
| 06:05:00 | raptor | ah, ok |
| 06:06:00 | sam686 | otherwise, if it is this: (!started || dataConn) && dataConn->isEstablished() it will crash if not started and dataConn == NULL |
| 06:09:00 | sam686 | boo, editor copy and paste crash on WallSegmentManager |
| 06:10:00 | raptor | blech |
| 06:10:00 | sam686 | did you forgot "A"? |
| 06:10:00 | sam686 | as in, bleach? |
| 06:11:00 | sam686 | blech |
| 06:11:00 | sam686 | oops, i thought i was typing into a search |
| 06:11:00 | raptor | haha |
| 06:12:00 | raptor | just a textual expression of disgust |
| 06:12:00 | sam686 | http://en.wikipedia.org/wiki/Blech this tell me something else... |
| 06:13:00 | sam686 | oh there is the meaning you talking about http://www.urbandictionary.com/define.php?term=blech |
| 06:13:00 | raptor | yep |
| 06:18:00 | raptor | ok sam686, i'm about to commit a big clean-up |
| 06:18:00 | raptor | i got all warnings except for 3 - which i think are there because watusimoto is in the middle of something |
| 06:18:00 | raptor | i got all warnings fixed, i mean |
| 06:19:00 | Zoomber | yoyo karamazovapyAYYYY |
| 06:20:00 | sam686 | ok |
| 06:22:00 | raptor | ok pushed |
| 06:22:00 | sam686 | it seems like my problem with 6357.levelgen, it just ignores all the parameters that was passed to it as if it was never passed into it, causing my 6357.levelgen to simply generate only one type - single team spawn points, no flag, no goal zones |
| 06:23:00 | raptor | odd |
| 06:24:00 | sam686 | i tried 6357ctfbig, and it generate tiny level instead.. (should be big with 2 team turrets) |
| 06:24:00 | | BFLogBot - Commit d721e55d3937 | Author: buckyballreaction | Log: I did it now... I compiled with -Wall. This commit is the result of fixing the hundreds of warnings that appeared. |
| 06:26:00 | sam686 | next: compile master with -Wall ? |
| 06:28:00 | raptor | oh yeah... |
| 06:28:00 | raptor | i guess i could |
| 06:31:00 | sam686 | i will need to get to bed soon, in a few ninutes.. |
| 06:31:00 | raptor | looks liek master only has 1 warning |
| 06:32:00 | sam686 | well then, i guess you can fix that one warning and fix other stuff at the same time.. |
| 06:32:00 | sam686 | or same commit |
| 06:34:00 | raptor | phoew |
| 06:34:00 | raptor | that was a long project |
| 06:34:00 | | BFLogBot - Commit c2cd46cfe16c | Author: buckyballreaction | Log: Let's make -Wall the default and see how it goes. Also fix warning with master compiling. |
| 06:36:00 | raptor | ok good night |
| 06:36:00 | raptor | i'm going to sleep, too |
| 06:36:00 | sam686 | night |
| 06:36:00 | | sam686 Quit () |
| 06:37:00 | | raptor Quit (Remote host closed the connection) |
| 08:12:00 | | kodax has joined |
| 08:21:00 | | kodax Quit (Quit: kodax out) |
| 08:22:00 | | kodax has joined |
| 09:30:00 | | Zoomber Quit (Quit: Zoomber) |
| 09:31:00 | | Zoomber has joined |
| 09:31:00 | | ChanServ sets mode +v Zoomber |
| 11:45:00 | | kodax Quit (Ping timeout: 240 seconds) |
| 13:40:00 | | Watusimoto has joined |
| 13:51:00 | | kodax has joined |
| 13:56:00 | | kodax Quit (Remote host closed the connection) |
| 13:57:00 | | kodax has joined |
| 15:54:00 | | raptor has joined |
| 15:54:00 | | ChanServ sets mode +o raptor |
| 15:54:00 | raptor | good morning! |
| 15:56:00 | raptor | Hi Watusimoto |
| 15:56:00 | raptor | I have a couple of code clean-up questions... |
| 15:57:00 | raptor | 1. Event.cpp:153: pushSimulatedAltKeyPress() <-- is this going to be used anywhere? |
| 16:00:00 | raptor | 2. luaLevelGenerator.cpp:74 and luaObject.cpp:365 both have an used method: ToggleMenuItem* getMenuItem() |
| 16:00:00 | raptor | unused method, i mean |
| 16:44:00 | | sam686 has joined |
| 16:44:00 | | ChanServ sets mode +v sam686 |
| 16:44:00 | | watusimoto1 has joined |
| 16:45:00 | watusimoto1 | hi |
| 16:45:00 | sam686 | hi |
| 16:45:00 | raptor | hi |
| 16:45:00 | sam686 | hi |
| 16:45:00 | watusimoto1 | hey -- I'm on here twice |
| 16:45:00 | raptor | fork! |
| 16:46:00 | sam686 | it might be you left your second computer on irc.. |
| 16:46:00 | watusimoto1 | kids playing video games, most likely |
| 16:46:00 | watusimoto1 | had a great idea for bot sleep function |
| 16:46:00 | raptor | so I made Reactor -> Core, and I'm about to commit... |
| 16:46:00 | watusimoto1 | great |
| 16:46:00 | raptor | bot sleep |
| 16:47:00 | watusimoto1 | unsubscribe from onTick() events |
| 16:47:00 | raptor | sleep... what is that? |
| 16:47:00 | watusimoto1 | and set a timer to resubscribe |
| 16:47:00 | raptor | ah |
| 16:47:00 | raptor | neat idea |
| 16:47:00 | watusimoto1 | maybe we just need to document this and not provide an explicit sleep function |
| 16:47:00 | watusimoto1 | if it works |
| 16:47:00 | watusimoto1 | or create a helper function in one of the helper script files |
| 16:48:00 | watusimoto1 | will need to play around and see what feels right |
| 16:48:00 | watusimoto1 | and if it works |
| 16:49:00 | raptor | i like the subscription system |
| 16:49:00 | watusimoto1 | I do too |
| 16:50:00 | watusimoto1 | just tune into what you want |
| 16:50:00 | watusimoto1 | the only complication is that I don;t know if you can unsubscribe to onTick events |
| 16:50:00 | watusimoto1 | but if not, it can be fixed |
| 16:51:00 | watusimoto1 | tonight's work will be related to lua |
| 16:51:00 | raptor | today i'm travelling! |
| 16:51:00 | watusimoto1 | great! |
| 16:51:00 | watusimoto1 | fresh perspective |
| 16:53:00 | raptor | i just caught a crash in the editor that i can't duplicate: http://pastie.org/3058121 |
| 16:55:00 | watusimoto1 | else if(mHitVertex != NONE && (!mHitItem || !mHitItem->isSelected())) // Hit a vertex of an unselected item |
| 16:55:00 | watusimoto1 | { // (braces required) |
| 16:55:00 | watusimoto1 | if(!mHitItem->vertSelected(mHitVertex)) |
| 16:55:00 | watusimoto1 | { |
| 16:55:00 | watusimoto1 | clearSelection(); |
| 16:55:00 | watusimoto1 | mHitItem->selectVert(mHitVertex); |
| 16:55:00 | watusimoto1 | onSelectionChanged(); |
| 16:55:00 | watusimoto1 | } |
| 16:55:00 | watusimoto1 | } |
| 16:55:00 | watusimoto1 | it made it down into the mHitItem->selectVert(mHitVertex); line |
| 16:55:00 | watusimoto1 | mHitItem was probably null |
| 16:56:00 | raptor | how can mHitItem be null? |
| 16:56:00 | watusimoto1 | ah |
| 16:56:00 | watusimoto1 | still looking |
| 16:56:00 | watusimoto1 | maybe our line numbers are different? |
| 16:57:00 | watusimoto1 | sorry, scrub all that |
| 16:57:00 | watusimoto1 | no wait, don't |
| 16:58:00 | raptor | i still can't dupe it... maybei'm just dumb |
| 16:58:00 | watusimoto1 | just not sure where isSelected() is called |
| 16:59:00 | watusimoto1 | neither clearSelection nor selectVert nor onSelectionChanged has an isselected in it |
| 16:59:00 | raptor | EditorObject |
| 17:00:00 | watusimoto1 | ah, but this line does |
| 17:00:00 | watusimoto1 | else if(mHitVertex != NONE && (!mHitItem || !mHitItem->isSelected())) |
| 17:00:00 | watusimoto1 | for me that's 3496 |
| 17:00:00 | watusimoto1 | so yes, I think mHitItem was NULL |
| 17:00:00 | watusimoto1 | first line of your stacktrace says this = 0x0 |
| 17:00:00 | watusimoto1 | inside isSelected |
| 17:01:00 | watusimoto1 | but if mHitItem was NULL, the isSelected would not have been called |
| 17:01:00 | raptor | so we need a fail-fast test of mHitItem |
| 17:02:00 | watusimoto1 | wfail-fast test? |
| 17:02:00 | raptor | instaed of: mHitVertex != NONE && mHitItem->isSelected() && mHitItem->getGeomType() != geomPoint |
| 17:02:00 | raptor | mHitVertex != NONE && mHitItem && mHitItem->isSelected() && mHitItem->getGeomType() != geomPoint |
| 17:03:00 | watusimoto1 | you're looking at the wrong block |
| 17:03:00 | watusimoto1 | I think |
| 17:03:00 | raptor | that's 3501 in my file |
| 17:03:00 | watusimoto1 | paste it |
| 17:03:00 | watusimoto1 | or you did |
| 17:03:00 | watusimoto1 | ok |
| 17:03:00 | watusimoto1 | wait |
| 17:03:00 | raptor | http://pastie.org/3058157 |
| 17:04:00 | watusimoto1 | ok |
| 17:04:00 | watusimoto1 | that's your proposed fix |
| 17:04:00 | raptor | no |
| 17:04:00 | raptor | that's original code |
| 17:04:00 | raptor | proposed fix is to add a test '&& mHitItem' |
| 17:05:00 | watusimoto1 | ah, that's 3476 for me |
| 17:05:00 | watusimoto1 | so yes, need to insert that && mHitItem |
| 17:06:00 | watusimoto1 | ok, that makes total sense |
| 17:06:00 | raptor | i still don't know how i triggered it |
| 17:06:00 | raptor | fail-fast: a battery of tests where the null test is first and if it fails, the rest of the tests aren't evaluated |
| 17:07:00 | watusimoto1 | well, if you fix it, you never will again |
| 17:07:00 | raptor | i.e. if(obj != NULL && obj->someMethod) |
| 17:07:00 | raptor | NULL check must be first to trigger fail-fast |
| 17:07:00 | watusimoto1 | yes |
| 17:07:00 | watusimoto1 | ok, yes insert that, and problem solved |
| 17:07:00 | raptor | ok will do |
| 17:07:00 | watusimoto1 | every other line inthat area has one |
| 17:09:00 | watusimoto1 | when you see this=0x0 in your stacktrace, it's usually an easy fix, just like NPEs in Java |
| 17:09:00 | raptor | yes - it was just bugging me that i couldn't dupe it |
| 17:10:00 | | BFLogBot - Commit 8878e7dfe214 | Author: buckyballreaction | Log: Reactor -> Core |
| 17:10:00 | | BFLogBot - Commit b1a9842c4071 | Author: buckyballreaction | Log: Fix segfault in Editor |
| 17:21:00 | watusimoto1 | great, heading home now |
| 17:21:00 | watusimoto1 | on later |
| 17:23:00 | raptor | later |
| 17:23:00 | raptor | have fun merging! |
| 17:24:00 | raptor | :-) |
| 17:25:00 | | watusimoto1 Quit (Read error: Operation timed out) |
| 18:17:00 | | Watusimoto_ has joined |
| 18:19:00 | | Watusimoto Quit (Ping timeout: 240 seconds) |
| 18:36:00 | | kodax Quit (Ping timeout: 248 seconds) |
| 19:30:00 | | Watusimoto_ Quit (Ping timeout: 252 seconds) |
| 20:42:00 | | koda has joined |
| 20:52:00 | | raptor Quit (Ping timeout: 252 seconds) |
| 21:36:00 | | Zoomber Quit (Quit: Zoomber) |
| 21:36:00 | | Zoomber has joined |
| 21:36:00 | | ChanServ sets mode +v Zoomber |
| 21:51:00 | | BFLogBot - Commit 32be3d61de13 | Author: sam8641 | Log: remove master "mVersionString", not needed anymore for new master. |
| 22:23:00 | | Watusimoto has joined |
