| 11:23.30 | *** join/#bzflag infobot (ibot@69-58-76-73.ut.vivintwireless.net) |
| 11:23.30 | *** topic/#bzflag is http://BZFlag.org || https://www.ohloh.net/p/bzflag || http://wiki.BZFlag.org/Getting_Help || Channel Logs: http://ibot.rikers.org/%23bzflag/ || 2.4.2 Current version https://goo.gl/ZaiNzX |
| 11:23.30 | *** mode/#bzflag [+v infobot] by ChanServ |
| 12:45.38 | *** join/#bzflag Pimpinella (~frank@p4FF432F3.dip0.t-ipconnect.de) |
| 15:59.55 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.34) |
| 15:59.56 | BZnotify | [13bzflag] 15JMakey pushed 1 new commit to 062.4: 02http://git.io/vm9VF |
| 15:59.56 | BZnotify | 13bzflag/062.4 145c624a7 15Jeff Makey: Use braces to limit the bz_eTickEvent "now" variable scope.... |
| 15:59.56 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.34) |
| 16:02.41 | *** join/#bzflag JBdiGriz (~dbw@108-197-33-160.lightspeed.sndgca.sbcglobal.net) |
| 16:29.38 | blast007 | sorry about missing those braces. thought I had compiled here, but tested a bit ago and it failed to build. |
| 17:26.41 | BulletCatcher | No problem. |
| 17:26.48 | BulletCatcher | Don't forget to fix those duplicate comparisons. |
| 17:26.49 | BulletCatcher | They're real bugs. |
| 17:42.35 | blast007 | yeah, need to look more closely at those to see what the expected flow actually was |
| 17:46.54 | *** join/#bzflag rob1n_ (Nomsayinth@unaffiliated/rob1n) |
| 17:50.08 | blast007 | well, rabbitTimer one looks obvious. The webStats plugin I need to look over more. |
| 17:50.46 | BulletCatcher | That was my analysis, too. |
| 17:53.48 | BulletCatcher | webStats will be new in 2.4.4, so there is minimal risk of breaking an existing server with whatever change it needs. |
| 17:54.34 | blast007 | barring cases where logic is involved, should variable initialization for constructors generally always be in the initialization list insead of in the body of the constructor? |
| 18:03.56 | BulletCatcher | Assuming that is just a question of style, so it doesn't affect functionality, then it seems that would make it clearer that we are just initializing with static values. |
| 18:07.19 | *** join/#bzflag Pimpinella (~frank@p4FF432F3.dip0.t-ipconnect.de) |
| 19:02.43 | *** join/#bzflag Pimpinella (~frank@p4FF432F3.dip0.t-ipconnect.de) |
| 19:13.00 | *** join/#bzflag rodgort (~rodgort@105.ip-167-114-152.net) |
| 19:44.14 | *** join/#bzflag I_Died_Once (~I_Died_On@unaffiliated/idiedonce/x-1828535) |
| 19:48.53 | *** join/#bzflag I_Died_Once (~I_Died_On@unaffiliated/idiedonce/x-1828535) |
| 19:50.04 | *** join/#bzflag zuii (~akos@BC06BF93.catv.pool.telekom.hu) |
| 20:54.26 | BulletCatcher | The webStats duplicate comparison was added in b10aed74 (AKA r17406, on the 2.99 branch) but that gets me no closer to fixing it. |
| 20:57.20 | blast007 | I'm not even sure if the webStats plugin works. |
| 20:58.31 | BulletCatcher | It was committed. How could it not work? :-p |
| 20:58.55 | BulletCatcher | I'm going to give it a try. |
| 20:59.40 | BulletCatcher | If my IRC connection dies suddenly, you'll know the culprit. |
| 21:13.53 | BulletCatcher | webStats is included in the Windows fullbuild target, but unix and Mac don't even compile it. |
| 21:14.06 | BulletCatcher | My testing didn't get very far. :-) |
| 21:14.10 | blast007 | heh |
| 21:14.22 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.41) |
| 21:14.22 | BZnotify | [13bzflag] 15blast007 pushed 11 new commits to 062.4: 02http://git.io/vmHpW |
| 21:14.22 | BZnotify | 13bzflag/062.4 1452a9513 15Scott Wichser: Add thiefControl to the Windows build. |
| 21:14.22 | BZnotify | 13bzflag/062.4 142a9f451 15Scott Wichser: Remove references to the outdated 3rd party library projects. |
| 21:14.22 | BZnotify | 13bzflag/062.4 14d6dcfc6 15Scott Wichser: Make thiefControl depend on bzfs so it builds in the right order. |
| 21:14.22 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.41) |
| 21:16.53 | BulletCatcher | The pushstats and webReport plugins are built on Windows and the Mac, but not on unix. |
| 21:17.41 | blast007 | neither of which may be finished, so that may be why ;) |
| 21:19.28 | BulletCatcher | Yeah. I think we should remove them from the 2.4 branch since it is unlikely that JeffM will return to finish them. |
| 21:19.44 | BulletCatcher | We can leave them on the master branch, at least for now. |
| 21:21.38 | BulletCatcher | If we choose to do that, a cleverly done 2.4->master merge will keep the Git repo clean. |
| 21:25.17 | *** join/#bzflag [Gort] (~gort@82-69-126-225.dsl.in-addr.zen.co.uk) |
| 21:25.17 | *** join/#bzflag [Gort] (~gort@unaffiliated/gort/x-9231432) |
| 21:30.16 | blast007 | I was gonna see how far off webStats was. pushstats I'd agree with. We could leave them there and just remove them from the build process. |
| 21:31.53 | BulletCatcher | I suppose it depends on how close they are to usable. |
| 21:33.59 | BulletCatcher | BTW, gcc complains unless class variables are initialized in they same order they are declared. |
| 21:34.22 | BulletCatcher | I'll fix that in plugin_config.cpp and autoFlagReset.cpp. |
| 21:34.41 | blast007 | k |
| 21:40.57 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 21:40.57 | BZnotify | [13bzflag] 15JMakey pushed 1 new commit to 062.4: 02http://git.io/vmQJe |
| 21:40.57 | BZnotify | 13bzflag/062.4 14a16258e 15Jeff Makey: Initialize autoFlagReset and PluginConfig variables in the same order that they were declared. |
| 21:40.57 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 21:42.17 | BulletCatcher | There must be some circumstance where that order matters. |
| 21:48.40 | *** join/#bzflag nitroxis_ (n@nxs.re) |
| 22:21.47 | blast007 | and webReport doesn't work ;) |
| 22:40.32 | *** join/#bzflag tannerld (~tannerld@c-98-234-107-216.hsd1.ca.comcast.net) |
| 23:16.08 | *** join/#bzflag [Gort] (~gort@82-69-126-225.dsl.in-addr.zen.co.uk) |
| 23:16.08 | *** join/#bzflag [Gort] (~gort@unaffiliated/gort/x-9231432) |