--- Log opened Sat Mar 26 00:00:00 2016
20160326 00:00:51-!- Aginor [~andreas@unaffiliated/aginor] has quit [Ping timeout: 244 seconds]
20160326 00:01:00-!- prkc [~prkc@540265D3.dsl.pool.telekom.hu] has quit [Remote host closed the connection]
20160326 00:01:43-!- Aginor [~andreas@unaffiliated/aginor] has joined #wesnoth-dev
20160326 00:15:48-!- irker960 [~irker@uruz.ai0867.net] has quit [Quit: transmission timeout]
20160326 00:37:02-!- wedge010 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has joined #wesnoth-dev
20160326 00:40:27-!- wedge009 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has quit [Ping timeout: 260 seconds]
20160326 00:40:28-!- wedge010 is now known as wedge009
20160326 01:01:18-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20160326 01:04:21-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has quit [Quit: Connection closed for inactivity]
20160326 01:09:44-!- aeonchild [enchilado@defocus/yummy/enchilado] has quit [Quit: Lost terminal]
20160326 01:09:58-!- aeonchild [enchilado@defocus/yummy/enchilado] has joined #wesnoth-dev
20160326 01:25:57-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160326 01:57:27< shadowm> Regarding ClearType and arbitrary background surfaces, like I said: https://blogs.msdn.microsoft.com/oldnewthing/20150129-00/?p=44803/
20160326 02:05:59-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has quit [Remote host closed the connection]
20160326 02:06:05-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has joined #wesnoth-dev
20160326 02:13:27-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160326 02:17:48-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 250 seconds]
20160326 02:26:45-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20160326 02:52:08-!- gfgtdf_ [~chatzilla@f054144246.adsl.alicedsl.de] has joined #wesnoth-dev
20160326 02:55:39-!- gfgtdf [~chatzilla@f050180049.adsl.alicedsl.de] has quit [Ping timeout: 264 seconds]
20160326 02:55:48-!- gfgtdf_ is now known as gfgtdf
20160326 02:56:01-!- nurupo [~nurupo.ga@unaffiliated/nurupo] has quit [Quit: nurupo]
20160326 02:57:30-!- nurupo [~nurupo.ga@unaffiliated/nurupo] has joined #wesnoth-dev
20160326 03:49:14-!- gfgtdf [~chatzilla@f054144246.adsl.alicedsl.de] has quit [Ping timeout: 250 seconds]
20160326 04:04:15< mattsc> celticminstrel: phew — I think I figured out what’s going on with the FAI recruitment
20160326 04:04:43< celticminstrel> And I haven't even had a chance to look yet.
20160326 04:04:52< mattsc> Apparently, value * map[key]  evaluates to  (value * map)[key]
20160326 04:05:09< mattsc> while the equation assume it is value * (map[key])
20160326 04:05:13< celticminstrel> Ooh. That's probably my fault. :/
20160326 04:05:26< mattsc> Well, curse you, then.
20160326 04:05:30< mattsc> Or maybe thank you.
20160326 04:05:43< celticminstrel> Uh. Okay then?
20160326 04:06:01< mattsc> Because I certainly dusted off my FAI skills quite a bit by hunting that down.
20160326 04:06:34< celticminstrel> When I changed how [] binds, I unfortunately didn't think of that case; I was only thinking about cases like p.y[n].
20160326 04:06:38< mattsc> Anyways, fixing that gets rid of some of some of the errors, with several more of the same type remaining.
20160326 04:06:47< mattsc> I assume they are caused by similar things.
20160326 04:07:20< mattsc> So, do you want to change how that is handled, or should I change the formula code?
20160326 04:07:25-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160326 04:07:36< celticminstrel> I think I should consider this a bug in the engine, but I'm not too sure how I would fix it other than simply reverting that particular change.
20160326 04:07:55< celticminstrel> I could, of course, simply revert that particular change. I think it's even encapsulated into a single commit.
20160326 04:08:08< mattsc> okay, well, let me know; I am going to stop doing Wesnoth for tonight anyway
20160326 04:08:15< celticminstrel> Better though would be to somehow fit indexing into the precedence table...
20160326 04:08:29< celticminstrel> Because I do want 1~5[n] to evaluate to (1~5)[n].
20160326 04:08:47< mattsc> ok; no objections
20160326 04:08:51< celticminstrel> But value*map[key] should really evaluate to value*{map[key]).
20160326 04:09:02< celticminstrel> I will definitely look into this.
20160326 04:09:09< celticminstrel> I'll put it on my list so I don't forget.
20160326 04:09:10< mattsc> agreed
20160326 04:09:18< mattsc> and thanks
20160326 04:09:32< mattsc> took me a while to find that  :P
20160326 04:10:05< celticminstrel> Incidentally, (a.b)[n] is actually equivalent to a.(b[n]), so that's why I initially thought that change was safe.
20160326 04:11:03< mattsc> ok
20160326 04:22:39-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160326 04:25:11-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160326 04:29:33-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 240 seconds]
20160326 04:32:48-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160326 04:33:06< celticminstrel> Huh, there's some sort of command parser in the AI code.
20160326 04:33:24< celticminstrel> It understands !add_ai, !remove_ai, and !replace_ai.
20160326 04:34:49< celticminstrel> I have no idea where (if anywhere) you would enter those commands, though.
20160326 04:36:30-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Quit: mattsc]
20160326 04:49:06-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160326 04:49:08-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Client Quit]
20160326 04:51:11-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160326 04:52:03-!- nos__ [~nos@208.91.185.104] has quit [Ping timeout: 240 seconds]
20160326 04:58:54-!- nos__ [~nos@208.91.185.104] has joined #wesnoth-dev
20160326 05:06:09-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160326 05:29:10-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160326 05:29:53-!- TheJJ [~rofl@ipbcc36ea9.dynamic.kabel-deutschland.de] has quit [Quit: No Ping reply in 180 seconds.]
20160326 05:51:31-!- celticminstrel [~celmin@unaffiliated/celticminstrel] has quit [Quit: And lo! The computer falls into a deep sleep, to awake again some other day!]
20160326 06:21:13-!- Kwandulin [~Miranda@p200300760F191C3C3C9432E46122B2EF.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 06:35:37-!- TheJJ [~rofl@ipbcc36ea9.dynamic.kabel-deutschland.de] has joined #wesnoth-dev
20160326 06:52:22-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160326 06:56:57-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 246 seconds]
20160326 07:34:17< ancestral> celticminstrel: Not sure how you managed to do it, but the Xcode project file is now broken for me
20160326 07:35:16< ancestral> I get an error, “cannot be opened because the project file cannot be parsed” 
20160326 07:36:12-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20160326 07:40:14< ancestral> It happened in the last commit it looks like, 57e4138
20160326 08:03:55-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160326 09:19:38-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Quit: Leaving.]
20160326 09:19:39-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160326 09:23:57-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 246 seconds]
20160326 10:01:18-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20160326 10:02:18-!- Kwandulin [~Miranda@p200300760F191C3C3C9432E46122B2EF.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160326 10:02:31-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160326 10:02:44-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20160326 10:39:25-!- midzer [~quassel@p5B312EF2.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160326 10:40:07-!- midzer [~quassel@p5B312EF2.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 11:01:55-!- mjs-de [~mjs-de@p508C84D0.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 11:37:15-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has quit [Remote host closed the connection]
20160326 11:37:21-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has joined #wesnoth-dev
20160326 11:57:47-!- Kwandulin [~Miranda@p200300760F191C3C18F2202996141089.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 12:05:21-!- prkc [~prkc@540265D3.dsl.pool.telekom.hu] has joined #wesnoth-dev
20160326 12:36:13-!- Kwandulin [~Miranda@p200300760F191C3C18F2202996141089.dip0.t-ipconnect.de] has quit [Ping timeout: 248 seconds]
20160326 12:37:09-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has joined #wesnoth-dev
20160326 13:06:54-!- nos__ [~nos@208.91.185.104] has quit [Read error: Connection reset by peer]
20160326 13:10:43-!- prkc [~prkc@540265D3.dsl.pool.telekom.hu] has quit [Remote host closed the connection]
20160326 13:13:28-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has quit [Quit: ChipmunkV]
20160326 13:15:48-!- louis94 [~~louis94@91.178.242.163] has joined #wesnoth-dev
20160326 13:25:11-!- prkc [~prkc@540265D3.dsl.pool.telekom.hu] has joined #wesnoth-dev
20160326 13:41:28-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160326 14:21:54-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160326 14:23:12< mattsc> gfgtdf, celticminstrel: I was wrong, —controller=ai does work.  However, sides in local MP games started from the commandline with -m have empty recruit lists.
20160326 14:23:58< mattsc> Could that be related to some of the issues you fixed a few days ago for test scenarios?
20160326 14:25:40-!- mjs-de [~mjs-de@p508C84D0.dip0.t-ipconnect.de] has quit [Remote host closed the connection]
20160326 14:26:39-!- Kwandulin [~Miranda@p200300760F191C923920F7CE9F13B1E7.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 14:29:47-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160326 14:39:16-!- irker756 [~irker@uruz.ai0867.net] has joined #wesnoth-dev
20160326 14:39:16< irker756> wesnoth: mattsc wesnoth:master e7d82ebcad86 / changelog: Update changelog https://github.com/wesnoth/wesnoth/commit/e7d82ebcad8658e373d87d1274048ec2cc605431
20160326 14:44:21-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has quit [Quit: Connection closed for inactivity]
20160326 15:15:15-!- travis-ci [~travis-ci@ec2-54-160-176-43.compute-1.amazonaws.com] has joined #wesnoth-dev
20160326 15:15:16< travis-ci> wesnoth/wesnoth#9065 (master - e7d82eb : mattsc): The build has errored.
20160326 15:15:16< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118681720
20160326 15:15:16-!- travis-ci [~travis-ci@ec2-54-160-176-43.compute-1.amazonaws.com] has left #wesnoth-dev []
20160326 15:17:03-!- louis94 [~~louis94@91.178.242.163] has quit [Ping timeout: 276 seconds]
20160326 15:39:03-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20160326 15:45:06-!- louis94 [~~louis94@91.178.242.163] has joined #wesnoth-dev
20160326 15:59:45-!- louis94 [~~louis94@91.178.242.163] has quit [Quit: Konversation terminated!]
20160326 16:00:39-!- louis94 [~~louis94@91.178.242.163] has joined #wesnoth-dev
20160326 16:02:41-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20160326 16:18:46-!- louis94 [~~louis94@91.178.242.163] has quit [Remote host closed the connection]
20160326 16:20:05-!- celticminstrel [~celmin@unaffiliated/celticminstrel] has joined #wesnoth-dev
20160326 16:22:30< celticminstrel> ...eh? Broken XCode project? :|
20160326 16:24:22< celticminstrel> Is that the case for mattsc too?
20160326 16:29:52-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160326 16:31:03-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160326 16:31:24< celticminstrel> It's possible that ancestral had a merge conflict...
20160326 16:34:00-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Client Quit]
20160326 16:37:43< celticminstrel> The commandline syntax mattsc is using doesn't match the syntax given on the manpage.
20160326 16:39:19-!- ancestral [~ancestral@63.92.240.233] has joined #wesnoth-dev
20160326 16:42:08< celticminstrel> mattsc: So what was this about switch_ai not working? It looks like it works to me...
20160326 16:44:20< celticminstrel> At least, using the gamestate inspector in the AI arena, the config changes when I select a new AI.
20160326 16:47:35-!- ancestral [~ancestral@63.92.240.233] has quit [Quit: i go nstuf kthxbai]
20160326 16:50:38-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has joined #wesnoth-dev
20160326 16:51:00-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20160326 16:52:32< celticminstrel> ...weird, there are some source files in doc/design/gui2.
20160326 17:11:55< celticminstrel> Why does Wesnoth have switches for compressing and decompressing files?
20160326 17:25:43-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has joined #wesnoth-dev
20160326 17:40:39-!- irker756 [~irker@uruz.ai0867.net] has quit [Quit: transmission timeout]
20160326 17:44:11-!- ypnos [~ypnos@lme51.informatik.uni-erlangen.de] has quit [Ping timeout: 244 seconds]
20160326 17:45:18< mattsc> celticminstrel: the Xcode project works for me
20160326 17:45:31< mattsc> And what syntax is different on the man page?
20160326 17:46:04< mattsc> celticminstrel: I tested switch_ai in both the AI Arena and a dedicated test scenario and it worked in neither; I can try again.
20160326 18:30:56< mattsc> Hmm, the syntax on the manpage is indeed different.  I guess I’ve never looked there.  I am using —help instead.
20160326 18:33:57-!- trewe [~trewe@bl20-31-249.dsl.telepac.pt] has joined #wesnoth-dev
20160326 18:35:04< celticminstrel> Yeah, --help is presumably more up-to-date. I'm trying to remedy this.
20160326 18:38:41< celticminstrel> mattsc: The manpage seems to claim that the -m switch sort of changes the option-processing mode, in that generic options must come before it and MP-specific ones must come after it. Do you know if this is true?
20160326 18:51:35< celticminstrel> (If you don't know, I can try it out myself once I've finished general updates to the manpage.)
20160326 18:52:59< mattsc> Sorry, don’t don’t know this.
20160326 18:53:07< celticminstrel> Ooh, --render-image sounds useful.
20160326 18:53:10-!- ypnos [~ypnos@lme51.informatik.uni-erlangen.de] has joined #wesnoth-dev
20160326 19:08:16< mattsc> celticminstrel: if I start the AI Arena (from the CL) and choose “filtered combat test”, that should make side 3 idle via switch_ai.  And for me it definitely does not do that.
20160326 19:08:44< celticminstrel> Okay, I'll try that.
20160326 19:09:34-!- Kwandulin [~Miranda@p200300760F191C923920F7CE9F13B1E7.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160326 19:17:46< celticminstrel> ... The "formula" test scenario uses [register_candidate_move] instead of [candidate_action]... does that even work?
20160326 19:22:25-!- Kwandulin [~Miranda@p200300760F191C92F4F524BE09866F6C.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 19:37:27< celticminstrel> Should the manpage include information specific to Windows? I mean, do Windows users even have a way of viewing it?
20160326 19:44:21-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has quit [Quit: Connection closed for inactivity]
20160326 19:49:32< Ravana_> I was following https://wiki.wesnoth.org/MultiplayerServerWML , at [mustlogin] "password: The hashed password, created from the password and salt received from the server." - server didn't send me any salt
20160326 19:52:58< Ravana_> testing with server2 currently so not needed either, but would need to know for later
20160326 19:57:29-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has quit [Quit: ChipmunkV]
20160326 19:58:28-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160326 20:02:53-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has joined #wesnoth-dev
20160326 20:05:30< celticminstrel> Is it true that "wesnoth my_data" will cause it to search the "my_data" folder for its data files?
20160326 20:06:02 * celticminstrel suspects this manpage hasn't been updated since whenever Boost.ProgramOptions was introduced to the code.
20160326 20:13:58< celticminstrel> I imagine the wesnothd manpage also needs updating. Maybe I'll consider that later.
20160326 20:15:17< celticminstrel> Anyone with opinions on whether the manpage should include Windows-specific information?
20160326 20:19:14 * celticminstrel discovers that the data-dir detail is true.
20160326 20:23:26< celticminstrel> On the other hand, it doesn't get confused between that and an optional campaign ID as the manpage says.
20160326 20:24:21< Ravana_> seems I have managed simple script that sends 1 lobby/whisper message and then quits
20160326 20:38:23< celticminstrel> mattsc: I think I solved the problem you had with switch_ai; it was specific to the Idle AI, because switch_ai parses the entire selected AI as if it were in [side] which means that, if no ai_algorithm key is present and no [stage] is defined, the default AI is injected.
20160326 20:38:30< celticminstrel> So I just added an empty [stage] to the Idle AI.
20160326 20:39:51-!- irker277 [~irker@uruz.ai0867.net] has joined #wesnoth-dev
20160326 20:39:52< irker277> wesnoth: Celtic Minstrel wesnoth:master f81169b7bb3c / data/gui/ (macros/_initial.cfg window/lobby_main.cfg): Draw a border around the lobby game list https://github.com/wesnoth/wesnoth/commit/f81169b7bb3c8e45e0df3f2832c5d5e253a9a095
20160326 20:39:52< celticminstrel> The default AI is no longer present in side 3 after selecting the filtered combat test.
20160326 20:39:54< irker277> wesnoth: Celtic Minstrel wesnoth:master 692e6567d6b9 / src/formula/debugger.cpp: Change logdomain for formula debugger https://github.com/wesnoth/wesnoth/commit/692e6567d6b936727943c98c4968809fe850647d
20160326 20:39:56< irker277> wesnoth: Celtic Minstrel wesnoth:master 884d88086ad7 / src/formula/function.cpp: Formula engine: New debug_profile function https://github.com/wesnoth/wesnoth/commit/884d88086ad705da6e0292974d87f3d22ad2f6ec
20160326 20:39:58< irker277> wesnoth: Celtic Minstrel wesnoth:master 282a2abb55bb / projectfiles/Xcode/Wesnoth.xcodeproj/project.pbxproj: XCode: Add documentation group https://github.com/wesnoth/wesnoth/commit/282a2abb55bbe33abec67726f7c0761f2c4a0628
20160326 20:40:00< irker277> wesnoth: Celtic Minstrel wesnoth:master f5bfa4dff57b / doc/man/wesnoth.6 src/commandline_options.cpp src/wesnoth.cpp: Update manpage https://github.com/wesnoth/wesnoth/commit/f5bfa4dff57b78d39e95d22a41dde6df4c719ca8
20160326 20:40:02< irker277> wesnoth: Celtic Minstrel wesnoth:master 7df4fc3970f2 / data/ai/dev/idle_ai.cfg: Fix [modify_side]switch_ai=data/dev/idle_ai.cfg https://github.com/wesnoth/wesnoth/commit/7df4fc3970f259e99a0082634f8e1ca331c4e664
20160326 20:40:20< celticminstrel> Oh, my commit message has the wrong path. Whoops. Oh well.
20160326 20:41:32< celticminstrel> Now, what else did I need to do...
20160326 21:15:53-!- travis-ci [~travis-ci@ec2-54-145-74-97.compute-1.amazonaws.com] has joined #wesnoth-dev
20160326 21:15:54< travis-ci> wesnoth/wesnoth#9066 (master - 7df4fc3 : Celtic Minstrel): The build has errored.
20160326 21:15:54< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118726235
20160326 21:15:54-!- travis-ci [~travis-ci@ec2-54-145-74-97.compute-1.amazonaws.com] has left #wesnoth-dev []
20160326 21:18:57-!- Kwandulin [~Miranda@p200300760F191C92F4F524BE09866F6C.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160326 21:19:36< irker277> wesnoth: Celtic Minstrel wesnoth:master 5c49cf0c7545 / src/formula/formula.cpp: Revert "Formula engine: fix index operator having lowest precedence" https://github.com/wesnoth/wesnoth/commit/5c49cf0c75459822dfeb2fdcc2ed1d25efe3104f
20160326 21:30:34< mattsc> celticminstrel: cool!  I did a quick test of both the switch_ai and the operator precedence and both problems seem to be fixed.
20160326 21:30:51< mattsc> Go figure that it was the one special case that did not work!
20160326 21:31:11< mattsc> I will do more comprehensive tests of both a little later today.
20160326 21:32:47< mattsc> celticminstrel: I have one more thing on my list, switching the MAIs to using [args], but I lost track of who of the two of us we said was going to do this.
20160326 21:33:26< celticminstrel> I've got a special-case exception to make 1~5[n] bind as (1~5)[n], do you think that's a good idea? 1~5 is equivalent to [1,2,3,4,5].
20160326 21:36:16< mattsc> celticminstrel: Well, in principle it sounds good, but is anybody likely going to use it?  So I’d turn it into: do you want to spend the effort on it.  Is so, sure, go ahead.  IMO.
20160326 21:36:42< celticminstrel> It's a one-line change, so provided it works... :P
20160326 21:38:53< mattsc> Sure, no objections from me then.
20160326 21:39:09< mattsc> What about changing the MAIs to the new syntax?
20160326 21:39:27-!- Appleman1234 [~Appleman1@KD106161148118.au-net.ne.jp] has quit [Ping timeout: 264 seconds]
20160326 21:39:28< mattsc> For some reason I have it in my brain that you said you would do it, but that might just be wishful thinkinh.
20160326 21:39:54< celticminstrel> I can probably do it if you'd like.
20160326 21:43:14< celticminstrel> I don't remember if I said I'd do it before though.
20160326 21:44:11< mattsc> celticminstrel: I can do it as well, but I’d have to ask you a few questions.  Up to you.
20160326 21:51:03< celticminstrel> I'll give it a try.
20160326 21:51:41< mattsc> Great.  Thanks.
20160326 21:58:03< celticminstrel> mattsc: Do you think the code for deprecated MAI syntax is still needed?
20160326 21:58:43< mattsc> Which one would that be?
20160326 21:58:47< mattsc> Taking a look ...
20160326 21:58:57< celticminstrel> Lines 12-63 of micro_ai_wml_tag.lua
20160326 21:59:08< celticminstrel> Sorry, 12-50.
20160326 22:00:05< mattsc> Oh, I’d forgotten about that...
20160326 22:00:34< celticminstrel> (I'm also wondering where that file actually gets executed...)
20160326 22:00:41< celticminstrel> (Or loaded or whatever.)
20160326 22:00:48< mattsc> Nah, I think it’s ok to toss that (even though they’ve only been around since 1.11/1.12)
20160326 22:01:45< mattsc> core/_main.cfg
20160326 22:02:53< celticminstrel> Ah.
20160326 22:03:13< celticminstrel> I didn't realize the WML tags were also called from there...
20160326 22:06:13< irker277> wesnoth: mattsc wesnoth:master d1f7225f7bde / RELEASE_NOTES: Update Release Notes https://github.com/wesnoth/wesnoth/commit/d1f7225f7bde126997cfc9eb88e2a36217f60e26
20160326 22:06:44< mattsc> celticminstrel: ^  feel free to modify as you may deem appropriate
20160326 22:09:31< celticminstrel> Oh, I forgot to remove the thing in the changelog that mentions the formula indexing operator.
20160326 22:09:47-!- travis-ci [~travis-ci@ec2-54-226-21-35.compute-1.amazonaws.com] has joined #wesnoth-dev
20160326 22:09:48< travis-ci> wesnoth/wesnoth#9067 (master - 5c49cf0 : Celtic Minstrel): The build has errored.
20160326 22:09:48< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118731405
20160326 22:09:48-!- travis-ci [~travis-ci@ec2-54-226-21-35.compute-1.amazonaws.com] has left #wesnoth-dev []
20160326 22:11:22< celticminstrel> ...I wonder if it's relevant that the builds that failed are all C++98 builds.
20160326 22:25:36-!- nos__ [~nos@208.91.185.104] has joined #wesnoth-dev
20160326 22:34:54< irker277> wesnoth: mattsc wesnoth:master f161a3cc1c0c / data/core/macros/ai.cfg: Fix AI_FORMULA_AI_EXPERIMENTAL_RECRUITMENT macro https://github.com/wesnoth/wesnoth/commit/f161a3cc1c0ce982ecf0085cc40a3c28fc4e98ef
20160326 22:35:44-!- Appleman1234 [~Appleman1@KD106161138227.au-net.ne.jp] has joined #wesnoth-dev
20160326 22:36:26< celticminstrel> I feel like that could still use some tweaks... I also wonder if anyone ever used that.
20160326 22:36:57< mattsc> “that’ = ?
20160326 22:37:12< celticminstrel> If someone used that macro, it would've been in an [aspect] tag... whereas if the used the version with SIMPLE_ in it it would've been in an [ai] tag...
20160326 22:38:00< celticminstrel> And when requesting FormulaAI recruitment it would make sense to add a [modify_ai] that disables default recruitment... though they could easily do that manually anyway...
20160326 22:38:09< celticminstrel> "that" = the commit you just made
20160326 22:38:16< mattsc> Yes - however, if you try it, it actually prints all kinds of messages in the chat window.  I doubt it’s used anywhere without modification, in which case people will have their local versions anyway
20160326 22:38:42< celticminstrel> I see. Those messages probably only appear in debug mode though.
20160326 22:38:54< mattsc> Oh, that’s possible …
20160326 22:39:03< celticminstrel> Assuming they're debug_print messages.
20160326 22:39:31< mattsc> debug_print should show up in the terminal though, and these do not
20160326 22:39:44< mattsc> I’ll test that once the current game is over.
20160326 22:39:49< celticminstrel> Oh. Yes, it should show up in the terminal as well.
20160326 22:40:20< mattsc> Knalgans with FAI experimental recruitment vs. Northerners RCA: Turn 34 and no winner in sight!
20160326 22:41:11< mattsc> Ooo, and as I type that, it’s over on the next turn
20160326 22:41:22< celticminstrel> The only debug_prints I see in the code are 'recruits_map' and 'turn_initialize'.
20160326 22:41:33< mattsc> And no error messages whatsoever in the terminal, so I take that as that AI working now.
20160326 22:42:14< mattsc> Uh ...
20160326 22:42:31-!- travis-ci [~travis-ci@ec2-54-145-74-97.compute-1.amazonaws.com] has joined #wesnoth-dev
20160326 22:42:32< travis-ci> wesnoth/wesnoth#9068 (master - d1f7225 : mattsc): The build has errored.
20160326 22:42:32< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118736973
20160326 22:42:32-!- travis-ci [~travis-ci@ec2-54-145-74-97.compute-1.amazonaws.com] has left #wesnoth-dev []
20160326 22:42:43< mattsc> That AI is in ai/dev; you cannot play it if oyu’re not in debug mode.
20160326 22:43:09< celticminstrel> You can use the macro though if you're not in debug mode.
20160326 22:43:15< mattsc> except if you do that
20160326 22:43:35< mattsc> Anyways, I thought about whether I should keep both macros and decided that it’s not worth it.
20160326 22:43:51< mattsc> But if you do, I don’t mind if you change it.
20160326 22:44:06< celticminstrel> I would've probably just made one redirect to the other or something.
20160326 22:44:23< celticminstrel> But I guess it doesn't matter that much.
20160326 22:44:43< mattsc> That wouldn’t fix the problem that one is a facet and one is an aspect in the old version though, or would it?
20160326 22:45:08< celticminstrel> No, I guess it'd add [/aspect] and [+aspect] tags in addition to redirecting.
20160326 22:45:08< mattsc> Anyways, I agree with your last line there.
20160326 22:45:29< mattsc> We’ve done things that break backward compatibility a lot more than this.
20160326 22:45:53< celticminstrel> I've decided to not bother with the special binding for 1~5[n], because it leads to parse failures that I don't want to have to figure out.
20160326 22:46:06< mattsc> Hmm.  Okay.  :|
20160326 22:46:16< celticminstrel> So I guess I'd better update the wiki now.
20160326 22:46:29< celticminstrel> And changelog too.
20160326 22:46:30< mattsc> Either way, I’ll leave it up to you what to do with those macros.
20160326 22:46:36< mattsc> Okay.
20160326 22:47:01< mattsc> The last thing I have on my list is to finish testing the AI Arena.
20160326 22:47:10< celticminstrel> I might need to remove it from RN too.
20160326 22:47:33< mattsc> Remove the changelog from the RN?
20160326 22:47:43< celticminstrel> No, the mention of [] precedence change.
20160326 22:47:52< mattsc> Oh, right.  That’s in there.
20160326 22:48:14< mattsc> And potentially filing a bug report about the recruits missing in games started with -m.  Unless gfgtdf fixes it before I get around to that.
20160326 22:50:28< celticminstrel> Okay, so, indexing effectively has the lowest precedence, even lower than the not operator... so "not a[b]" would bind as "(not a)[b]", I think. This is really not ideal. :/ Still, it's better than my previous "fix".
20160326 22:51:16< celticminstrel> ...I wonder if it would interact weirdly with "where" as well...
20160326 22:52:14< mattsc> I have no idea
20160326 22:53:10< celticminstrel> Okay, no, I think I'm confused again.
20160326 22:53:59< celticminstrel> What it does is split the expression on the first occurrence (or last in the case of ^) of the highest-precedence operator.
20160326 22:54:41< celticminstrel> Only if there are no operators does it consider indexing.
20160326 22:55:45< celticminstrel> So I think it's probably fine as it is now.
20160326 22:56:08< celticminstrel> Anyway, on to MAI stuff.
20160326 22:56:36-!- nos__ [~nos@208.91.185.104] has quit [Ping timeout: 276 seconds]
20160326 23:11:29-!- travis-ci [~travis-ci@ec2-54-80-130-225.compute-1.amazonaws.com] has joined #wesnoth-dev
20160326 23:11:30< travis-ci> wesnoth/wesnoth#9069 (master - f161a3c : mattsc): The build has errored.
20160326 23:11:30< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118740339
20160326 23:11:30-!- travis-ci [~travis-ci@ec2-54-80-130-225.compute-1.amazonaws.com] has left #wesnoth-dev []
20160326 23:19:26-!- mjs-de [~mjs-de@p508C9C2C.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160326 23:24:04-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection]
20160326 23:26:19-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has quit [Quit: horrowind]
20160326 23:35:08-!- nos__ [~nos@208.91.185.104] has joined #wesnoth-dev
20160326 23:39:36-!- mjs-de [~mjs-de@p508C9C2C.dip0.t-ipconnect.de] has quit [Remote host closed the connection]
20160326 23:46:13< Ravana_> is it said anywhere that the 1.13 hotkey changes made hotkeys no longer be case sensitive? (should be improvement)
20160326 23:47:40< celticminstrel> Well, I think they'd still be case sensitive, but it should be obvious whether you need the shift key...
20160326 23:48:12< Ravana_> shift is special, but I tried with caps lock
20160326 23:48:53< celticminstrel> I have no idea if they're caps-lock-dependent.
20160326 23:49:11< Ravana_> aren't anymore
20160326 23:49:16< celticminstrel> I see.
20160326 23:51:16< celticminstrel> mattsc: Why is the random recruit MAI modifying the input config?
20160326 23:51:24< celticminstrel> Is it supposed to be doing this in CA_parms instead?
20160326 23:52:27< celticminstrel> (Does the MAI currently honour the [probability] tags?)
20160326 23:57:10< celticminstrel> It's not the only one modifying the input config, either...
--- Log closed Sun Mar 27 00:00:12 2016