--- Log opened Wed Mar 23 00:00:13 2016
20160323 00:10:38< celticminstrel> I also see a warning about unknown leader_shares_keep aspect.
20160323 00:18:29-!- louis94 [~~louis94@91.178.240.41] has quit [Ping timeout: 260 seconds]
20160323 00:21:30< celticminstrel> I guess that one's a problem with the scenario though.
20160323 00:21:43-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 00:23:18< mattsc> All of them might be problems with the scenario.  I’ll investigate later, unless you beat me to it.
20160323 00:23:38< celticminstrel> No, the goal thing is a problem with the engine - a missed case.
20160323 00:24:32< celticminstrel> Wait, why am I still getting the leader_shares_keep message when I suppressed it...
20160323 00:25:07< celticminstrel> Oh, it was specified with simple attribute syntax, which means it was translated to a facet of a composite aspect. Okay, that makes sense.
20160323 00:25:22< celticminstrel> Hmm...
20160323 00:26:13< celticminstrel> Well, I could make the error more explanatory (it's a direct result of an unknown aspect), but since the more explanatory version shows up anyway a bit later, it's probably fine.
20160323 00:26:47< irker734> wesnoth: Celtic Minstrel wesnoth:master 8478cd604ccb / src/ai/composite/goal.cpp: Fix [goal] not falling back to default name= if engine= omitted https://github.com/wesnoth/wesnoth/commit/8478cd604ccb6e3eafc22b8ab37eb06ed2fc8151
20160323 00:26:49< irker734> wesnoth: Celtic Minstrel wesnoth:master f7f4bfe32164 / src/ai/configuration.cpp: Don't warn about missing [default] when there should be none https://github.com/wesnoth/wesnoth/commit/f7f4bfe32164c232127a8c57ebf5a0bb4128151d
20160323 00:27:29< celticminstrel> Wesnoth should have a pause feature.
20160323 00:27:52< celticminstrel> I know you can pause with escape or right-click, but in neither case can you scroll around while paused.
20160323 00:28:10< celticminstrel> Only useful when it's not your turn, mind you.
20160323 00:43:25-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has quit [Remote host closed the connection]
20160323 00:43:31-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has joined #wesnoth-dev
20160323 00:54:27< celticminstrel> Hmmm... I wonder if it's possible to set the AI table as an upvalue, rather than a global...
20160323 00:54:34< celticminstrel> I might look into that later.
20160323 00:56:45< celticminstrel> I wonder if there is any protection against an infinite loop caused by the Lua code for aspect A fetching the value of aspect A.
20160323 00:57:02< celticminstrel> (Or the Lua code for a goal calling get_targets())
20160323 01:09:49< mattsc> celticminstrel: thanks for those fixes, but the “aspect with id=[leader_shares_keep] lacks default config facet!” error remains
20160323 01:10:15< celticminstrel> According to the list in registry.cpp, it should be "passive_leader_shares_keep".
20160323 01:10:21< celticminstrel> Though I suppose I could add a synonym.
20160323 01:10:32< celticminstrel> Since the CA was renamed to omit the passive_ prefix.
20160323 01:10:45< celticminstrel> Should I do that?
20160323 01:11:37< celticminstrel> (The error you quote is a result of the aspect being unknown and thus missing from data/ai/utils/default_config.cfg)
20160323 01:11:47< celticminstrel> (There should be a second error saying that it's unknown, right?)
20160323 01:11:58< mattsc> Yes, on that last question.
20160323 01:12:11< mattsc> On the others, hold on for a sec.  Refreshing my memory …
20160323 01:13:03< mattsc> No, it should remain passive_leader_shares_keep.
20160323 01:13:19< celticminstrel> Okay.
20160323 01:13:26< mattsc> I must have accidentally changed that when I renamed the CA.
20160323 01:13:34< celticminstrel> Makes sense.
20160323 01:14:24< mattsc> The leader always shares the keep - unless passvie_leader=yes
20160323 01:14:34< mattsc> therefore, the aspect only applies to a passive leader
20160323 01:24:14< vultraz> celticminstrel: btw, I forgot, what was this about listboxes in the lobby?
20160323 01:24:27< mattsc> I seem to have a problem with animation accelartion speed suddenly
20160323 01:25:33< mattsc> When I start a game, it seems to be set to no acceleration, no matter what it is set to
20160323 01:25:48< celticminstrel> vultraz: There are four listboxes in the WML, above the treeview. They're hidden behind macros.
20160323 01:25:52< mattsc> only after changing it in the preferences does it behave as I expect
20160323 01:25:59< celticminstrel> I was trying to figure out what they're for.
20160323 01:26:28< celticminstrel> mattsc: Could I get your comments on the changelog update?
20160323 01:26:41< celticminstrel> ...oh, I might as well run spell-check first.
20160323 01:27:10< mattsc> celticminstrel: sure, as long as it does not have to be right away
20160323 01:28:25< celticminstrel> No, but I'm pushing now, so whenever you have a moment.
20160323 01:28:31< celticminstrel> Or several moments, I guess.
20160323 01:28:32< irker734> wesnoth: Celtic Minstrel wesnoth:master e231540b734a / changelog: Update changelog https://github.com/wesnoth/wesnoth/commit/e231540b734acc8c7207042b080633de6e83a8ae
20160323 01:28:34< vultraz> celticminstrel: I have absolutely no idea...
20160323 01:28:55< celticminstrel> vultraz: It sure looks like they're the player list, and the code does link them to that somehow.
20160323 01:30:36-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 01:30:37< travis-ci> wesnoth/wesnoth#9019 (master - f7f4bfe : Celtic Minstrel): The build has errored.
20160323 01:30:37< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117857368
20160323 01:30:37-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 01:31:24< vultraz> celticminstrel: it's possible they're the listboxes of players you see when you open the tree view
20160323 01:31:45< irker734> wesnoth: mattsc wesnoth:master 7c4aec9cfe00 / data/ai/micro_ais/scenarios/The_Elves_Besieged.cfg: Micro AI test scenario: fix incorrect aspect name https://github.com/wesnoth/wesnoth/commit/7c4aec9cfe003793726962caa34e80c32d720d0c
20160323 01:31:47< irker734> wesnoth: mattsc wesnoth:master 5a82a36e89a6 / projectfiles/Xcode/Wesnoth.xcodeproj/project.pbxproj: Update Xcode project to include readline https://github.com/wesnoth/wesnoth/commit/5a82a36e89a694ca025bd8fb1d3101f98e665653
20160323 01:31:57< celticminstrel> Possibly, but that's not exactly a portable way of doing it...
20160323 01:32:37< celticminstrel> Ah, I was thinking maybe I should do that, mattsc, so thanks.
20160323 01:33:06< mattsc> glad I did something right for a change  ;)
20160323 01:33:44< celticminstrel> I probably have to rename my readline now.
20160323 01:34:09< irker734> wesnoth: mattsc wesnoth:1.12 f3410c954bcc / data/ai/micro_ais/scenarios/The_Elves_Besieged.cfg: Micro AI test scenario: fix incorrect aspect name https://github.com/wesnoth/wesnoth/commit/f3410c954bcc82e4b1a23ec42386096cdeca88ac
20160323 01:34:28< mattsc> celticminstrel: hmm, sorry
20160323 01:34:52< mattsc> I noticed that the way how the Xcode libs are named is inconsistent
20160323 01:35:24< celticminstrel> Well, I chose to copy the one that didn't include a version in the filename,
20160323 01:35:32< celticminstrel> So it was libreadline.dylib.
20160323 01:36:43< mattsc> Yeah … the files in the lib/ directory have anything from none to 3 version numbers, so I chose one at random
20160323 01:36:50< mattsc> I guess I could have asked first
20160323 01:37:04< celticminstrel> Doesn't really matter. May even be better to include a version number.
20160323 01:37:46< vultraz> celticminstrel: I can't figure out this code 
20160323 01:37:54< celticminstrel> Nor can I. :/
20160323 01:39:26< irker734> wesnoth: Celtic Minstrel wesnoth:master 43142630b651 / projectfiles/Xcode/Wesnoth.xcodeproj/project.pbxproj: XCode: Add readline to Copy Files phase https://github.com/wesnoth/wesnoth/commit/43142630b651de4b5b2c2f572997b12765d8dfe2
20160323 01:39:30< vultraz> and for some reason the toggle node is facing in the wrong direction initially
20160323 01:39:34< celticminstrel> In theory that should ensure that someone with only the system default doesn't get a crash.
20160323 01:40:16< celticminstrel> I don't suppose anyone is able to actually test Wesnoth on a fresh install of OSX though...
20160323 01:41:09< celticminstrel> Apparently mattsc is an unrecognized author in github.
20160323 01:41:15< mattsc> celticminstrel: hmm, I wonder why my version worked without that ^
20160323 01:41:30< celticminstrel> mattsc: Because you have it installed somewhere on the system.
20160323 01:41:40< vultraz> guess mattsc never filled out his profile
20160323 01:41:44< celticminstrel> Somewhere where the system looks when trying to find it.
20160323 01:41:58< celticminstrel> So it looks there and finds it and all is well.
20160323 01:42:23< celticminstrel> This is more or less the same reason that ancestral needs to make a new build of 1.13.4.
20160323 01:42:31< mattsc> celticminstrel: right …  I guess Macports sets itself up that way,.
20160323 01:42:39< celticminstrel> I guess. I dunno.
20160323 01:42:42< mattsc> That’s why I renamed the Macports folder last time.
20160323 01:42:46< celticminstrel> It worked for me as well.
20160323 01:42:52< mattsc> For testing, I mean.
20160323 01:43:22< mattsc> Anyways, thanks for adding that.  I’m off doing chores for a while.  I’ll have a look at the changelog after that.
20160323 01:43:41< celticminstrel> Maybe I'll have release notes ready by then, too.
20160323 01:43:49< mattsc> cool
20160323 01:44:04< mattsc> That’s a lot of changes …  (I did glance at it very quickly)
20160323 01:44:20-!- ToBeCloud [uid51591@wikimedia/ToBeFree] has quit [Quit: Connection closed for inactivity]
20160323 01:44:28< celticminstrel> It's possible I missed something, too.
20160323 01:47:15< vultraz> celticminstrel: do you think the sidebar in the mp lobby should be larger by default?
20160323 01:47:17< vultraz> wider, I mean
20160323 01:48:00< celticminstrel> You can't really make anything wider if you want to maintain compatibility with 800x600. Unless of course you want two separate window definitions.
20160323 01:48:33< celticminstrel> It wouldn't hurt for it to be wider, probably. Other than the 800x600 concern.
20160323 01:50:17< vultraz> so many improvements to do 
20160323 01:50:24< vultraz> need to color-code the chat messages
20160323 01:51:02< celticminstrel> How? Randomly? There's no sides here.
20160323 01:51:32< vultraz> no
20160323 01:51:37< vultraz> server messages should be a color
20160323 01:51:45< vultraz> whispers should be a color
20160323 01:51:47< vultraz> etc
20160323 01:52:15< vultraz> right now I'm struggling to add that space after the sender name
20160323 01:52:30< vultraz> for some reason there are multiple places that deal with sending messages
20160323 01:52:32< vultraz> for some reason
20160323 01:55:24< vultraz> this code needs major refactoring
20160323 01:59:44< vultraz> grrr
20160323 02:00:30< vultraz> WHY DOES THIS BREAK
20160323 02:00:41< vultraz> WHY
20160323 02:04:54< vultraz> setting set_use_markup(true) makes markup not be applied to the chat log :|
20160323 02:05:01< vultraz> what the hell
20160323 02:05:48< celticminstrel> Was it applied before you added that?
20160323 02:06:09< celticminstrel> Because my initial guess would be that something else is setting it to false later.
20160323 02:06:10< vultraz> ahhhhh
20160323 02:06:14< vultraz> I think I understand
20160323 02:06:28< vultraz> It thinks the <speaker> bits are markup :|
20160323 02:06:58< celticminstrel> Ah.
20160323 02:07:03< celticminstrel> &lt;speaker&rt;
20160323 02:07:08< celticminstrel> Uh no.
20160323 02:07:15< celticminstrel> &lt;speaker&gt;
20160323 02:07:29< celticminstrel> BTW, you might want to look at the formula debugger dialog.
20160323 02:07:49< celticminstrel> You don't want to convert markup within the chat message, so the utility function I added there could be useful.
20160323 02:08:14< vultraz> will look
20160323 02:08:17< celticminstrel> (It could be moved to somewhere more global, really.)
20160323 02:09:11< celticminstrel> (It's also a little inefficient, but I don't know any good way to fix that. The alternative would be a copy-transform type thing.)
20160323 02:10:02< vultraz> ahh thank you 
20160323 02:10:04< vultraz> it works
20160323 02:11:22< celticminstrel> Hmmm, I'm not sure what to write for the general release notes. (Already finished the section on compatibility breakages.)
20160323 02:13:26< mattsc> celticminstrel: [I’m not back yet but] I just noticed that libreadline also depends on libncurses
20160323 02:13:42< mattsc> so for the release version, that will also have to be included and the internal links need to be changed
20160323 02:13:53< mattsc> ancestral: ^
20160323 02:13:58< celticminstrel> We might be lucky and not need to do that.
20160323 02:14:08< celticminstrel> In particular, I suspect we're not using that portion of readline.
20160323 02:14:24< celticminstrel> In which case, the dependency on libncurses would never actually come into play.
20160323 02:14:38< celticminstrel> Since if I recall correctly these things are lazily loaded.
20160323 02:15:04< mattsc> hmm
20160323 02:15:46< irker734> wesnoth: Charles Dang wesnoth:master dc53c90d6e17 / src/gui/dialogs/lobby/lobby.cpp: tlobby_main: enable markup in chat log https://github.com/wesnoth/wesnoth/commit/dc53c90d6e17917ce4da22163863f6b8d592eb2b
20160323 02:15:56< celticminstrel> I really wish it would be possible for someone to make a build, transfer it to a more-or-less fresh install of whatever OS version, and try it there.
20160323 02:15:59< vultraz> celticminstrel: ^ see what you think
20160323 02:16:14< celticminstrel> By which you mean, build it and see, right?
20160323 02:16:57< vultraz> when you have time
20160323 02:17:22< celticminstrel> The amount of time I have is patently absurd. :/
20160323 02:17:47< vultraz> :P
20160323 02:18:03< vultraz> celticminstrel: pango_escape is what you mean?
20160323 02:18:12< celticminstrel> I think so.
20160323 02:18:21< celticminstrel> Replaces things with &whatever;
20160323 02:18:49< vultraz> problem is we *sometimes* want markup in the text 
20160323 02:19:06< celticminstrel> Just apply it to the chat message itself, then add whatever markup you want around it.
20160323 02:19:16< celticminstrel> Or am I misunderstanding something?
20160323 02:19:35< vultraz> look at the server message when you join
20160323 02:20:10< celticminstrel> My point is that users should not be able to type "<font color=black>invisible text</font>" for example.
20160323 02:20:22< celticminstrel> Oh right, it's span not font.
20160323 02:20:25< celticminstrel> Whatever.
20160323 02:21:11< celticminstrel> Well okay, they can type that, but it should just show up as markup, no colours.
20160323 02:21:59 * celticminstrel joining now.
20160323 02:21:59-!- stikonas_ [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160323 02:33:21< vultraz> I don't think "/me" is implemented 
20160323 02:33:39< celticminstrel> Probably not, but then why does it break the formatting?
20160323 02:33:44< vultraz> no idea
20160323 02:33:48-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 02:33:59< celticminstrel> Wait. I just typed /xxx and the same thing happened.
20160323 02:34:13< vultraz> must be the '/'
20160323 02:34:15< celticminstrel> Except that I got an error instead of it going through as a message.
20160323 02:34:23< celticminstrel> But only at the start of a line...
20160323 02:34:37< celticminstrel> Also, the weird thing is that the bold is still working.
20160323 02:36:02< vultraz> markup is hard
20160323 02:36:07< irker734> wesnoth: Celtic Minstrel wesnoth:master bc945f27c941 / src/gui/dialogs/formula_debugger.cpp: Remove duplicate function https://github.com/wesnoth/wesnoth/commit/bc945f27c9418c1860ee098526ceb207dceed558
20160323 02:36:54< celticminstrel> <_<
20160323 02:37:42< vultraz> blah :P
20160323 02:38:05< celticminstrel> The commit still tells you everything you need to know.
20160323 02:38:52-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 252 seconds]
20160323 02:43:03< vultraz> celticminstrel: ok, that works, but "/me" causes the escape characters around the speaker to show in row format :/
20160323 02:43:25< vultraz> raw
20160323 02:45:05< irker734> wesnoth: Charles Dang wesnoth:master 494cc72f9736 / src/gui/dialogs/lobby/ (lobby.cpp lobby.hpp): tlobby_main: enable closing a room tab if it's not currently active https://github.com/wesnoth/wesnoth/commit/494cc72f97369ac633146bb1654c40cf1993fb00
20160323 02:45:08< irker734> wesnoth: Charles Dang wesnoth:master 6bdb62ded06b / src/gui/dialogs/lobby/lobby.cpp: tlobby_main: hide Close button for Lobby room instead of disabling it https://github.com/wesnoth/wesnoth/commit/6bdb62ded06ba4a383acc29970ed2859aaeb7e30
20160323 02:45:11< irker734> wesnoth: Charles Dang wesnoth:master 78b442d7c2f0 / src/gui/dialogs/lobby/lobby.cpp: tlobby_main: escape formatting in user input text https://github.com/wesnoth/wesnoth/commit/78b442d7c2f0e69383969bf9d7975cb41de7898e
20160323 02:47:43< celticminstrel> Is there any possibility that <speaker_name> is being passed through escape_text?
20160323 02:47:56< vultraz> no, I only escape the message
20160323 02:48:38< celticminstrel> Would be surprising if it did, considering that it previously didn't use markup...
20160323 02:48:50< celticminstrel> Does /xxx still break it too?
20160323 02:49:06< celticminstrel> (Or any invalid command.)
20160323 02:50:57< vultraz> yes
20160323 02:53:02< vultraz> hmm
20160323 02:53:06< mattsc> celticminstrel: what’s an unrecognized author?
20160323 02:53:29< celticminstrel> mattsc: It means github cannot associate the email on your commit with any registered github account.
20160323 02:53:52< mattsc> celticminstrel: I made a few minor changes to my local changelog.  They are pretty much all style rather than content though, so you might not agree with them and I should probably not commit them.
20160323 02:53:59< mattsc> Should I pastebin you the git diff?
20160323 02:54:00< celticminstrel> BTW, any comments on the changelog or suggestions for release notes?
20160323 02:54:04< celticminstrel> Uh, sure.
20160323 02:55:02< vultraz> there's some code I could use to check if the scrollbar is at end, but I need to figure out how to get the scrollbar
20160323 02:55:02< mattsc> The one substantial comment I have is that I think the compatibility-breaking changes should be in the changelig, not just a reference to them in the release notes.
20160323 02:55:47< celticminstrel> Do I have to describe how to fix them in the changelog?
20160323 02:55:50< mattsc> celticminstrel: on the github account: hmm.  I did change my email address at some point, maybe that caused that?  Don’t know and/or remember.
20160323 02:55:59< celticminstrel> Or just say "compatibility is broken"?
20160323 02:56:45< mattsc> Something like: compatibility is broken for the following: -> bulleted list
20160323 02:56:46< celticminstrel> mattsc: If it's a result of changed email, you should be able to login to github and add a new email. (Don't just replace the old email, as that would presumably unlink older commits from your account; I think you can set the new one as primary though.)
20160323 02:57:03< celticminstrel> Were you going to past that diff?
20160323 02:57:20< mattsc> I think I did that (change my email in my account), but I’ll check sometime
20160323 02:57:38< mattsc> celticminstrel: yes, will do.  Give this old man a moment to breathe  ;)
20160323 02:57:53< celticminstrel> It's also possible that you changed the email in your account, but not the email in your git configuration?
20160323 02:58:16< celticminstrel> (In which case you should probably add back the old email as a secondary.)
20160323 02:58:17< vultraz> I don't think there's any public way to check scrollbar position..
20160323 02:58:30< celticminstrel> vultraz: The scroll bar is contained by what widget?
20160323 02:58:32< mattsc> celticminstrel: http://pastebin.com/qPen5c1j
20160323 02:59:18< mattsc> celticminstrel: these are just suggestion of what I would do; if you disagree with any or all of them, no need to try to convince me, just do what you prefer
20160323 02:59:20< celticminstrel> mattsc: Is some of that line-wrapping to 80cols?
20160323 02:59:22< vultraz> celticminstrel: a grid in the scroll label in the char multipage
20160323 02:59:29< mattsc> celticminstrel: yes, all of it
20160323 02:59:31< vultraz> chat
20160323 02:59:35< celticminstrel> Oh, all of it?
20160323 02:59:54< celticminstrel> Theoretically, that's supposed to be done by the release manager, but I guess it's not bad if it's already done for them.
20160323 02:59:58< mattsc> celticminstrel: I mean: the only reason why I did line wrapping was to get it inside 80 cols
20160323 03:00:06< mattsc> there are other changes that have other reasons
20160323 03:00:11< celticminstrel> vultraz: I think you misunderstood the question.
20160323 03:00:48< vultraz> if you're asking what widget handles scrollbars, there is no simple answer
20160323 03:01:16< vultraz> let's see what mordante said..
20160323 03:01:21< celticminstrel> I think "a minor shorthand" is actually a bit weird when there's three bullet points.
20160323 03:01:45< celticminstrel> vultraz: In any given case, you should be able to pinpoint one specific widget which encapsulates the scrollbars.
20160323 03:02:06< vultraz> ah, there was a patch for this
20160323 03:02:12< vultraz> let's see what it did
20160323 03:02:37< mattsc> celticminstrel: those are your words, so feel free to change at will  :)
20160323 03:03:02< vultraz> actually this patch looks legit
20160323 03:03:07< vultraz> why was it never committed..
20160323 03:03:21< celticminstrel> vultraz: Is it still valid with respect to master, though?
20160323 03:03:28< vultraz> looks like it
20160323 03:03:33< vultraz> https://gna.org/bugs/index.php?16059
20160323 03:04:38< vultraz> no author info, sadly
20160323 03:04:52-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 03:04:53< travis-ci> wesnoth/wesnoth#9021 (master - 5a82a36 : mattsc): The build has errored.
20160323 03:04:53< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117865649
20160323 03:04:53-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 03:08:12< celticminstrel> vultraz: And it's for the GUI2 lobby?
20160323 03:08:32< vultraz> looks like it
20160323 03:08:33< celticminstrel> mattsc: I don't really have any problem with you committing that. Any suggestions for the release notes, other than compatibility breaks?
20160323 03:12:36< celticminstrel> Also, anything else you think I might have missed in the changelog?
20160323 03:13:37< celticminstrel> I updated this to cover most of the new stuff some time ago: http://celmin.pwcsite.com/wesnoth/ai-overview.txt
20160323 03:13:45< celticminstrel> (It doesn't cover anything related to [modify_ai].)
20160323 03:13:47-!- melinath [~textual@71.37.44.32] has quit [Quit: Computer has gone to sleep.]
20160323 03:14:10< celticminstrel> I also tried to cover FormulaAI in that document.
20160323 03:14:53 * vultraz manually applied the patch since he can't figure out diff application plus one path is invalid
20160323 03:15:20< celticminstrel> You can credit him in the commit message by the name he went by.
20160323 03:15:31< celticminstrel> In fact, if you really want, you could credit him by attributing the commit to him.
20160323 03:15:51< celticminstrel> Though I guess you don't actually know his full email...
20160323 03:17:14< celticminstrel> Could try to contact him, or make up something fake ("faabumc@some.place")
20160323 03:18:07< celticminstrel> Last visited the formums on November 15, 2015.
20160323 03:18:11< celticminstrel> ^forums
20160323 03:19:31< celticminstrel> (For the record, the way to attribute a commit to someone else is git commit --author 'Their Name <their_email@example.com>'. Use double quotes instead of single if on Windows.)
20160323 03:20:01< vultraz> dammit, the patch doesn't work 100% :|
20160323 03:20:17< celticminstrel> Maybe he'll update it for you!
20160323 03:21:51 * celticminstrel still waiting on mattsc's response.
20160323 03:21:54< vultraz> the code SHOULD work...
20160323 03:23:10< vultraz> oh wait
20160323 03:23:11< vultraz> derp
20160323 03:23:27< vultraz> I applied the damn thing wrong
20160323 03:23:35 * mattsc is back; reading
20160323 03:23:46< celticminstrel> I think FAAB possible has a github account, though not 100% sure that's him.
20160323 03:24:08< vultraz> i had consolidated some code
20160323 03:24:10< vultraz> wronly
20160323 03:24:49< vultraz> HMMM
20160323 03:25:01< vultraz> celticminstrel: ok, can you join the lobby and send some messages
20160323 03:25:16< celticminstrel> I just started an anime...
20160323 03:25:19< vultraz> oh
20160323 03:25:22< vultraz> nvm then
20160323 03:25:33< celticminstrel> I'll try it in ~20 minutes?
20160323 03:25:37< vultraz> ok
20160323 03:25:46< vultraz> I need to disable this for sending messages, though
20160323 03:25:58< mattsc> celticminstrel: kind of caught up.  So I just commit those changes to the changelog?
20160323 03:26:11< vultraz> we only want this behavior when receiving them
20160323 03:27:06< celticminstrel> mattsc: Sure.
20160323 03:27:31< mattsc> As for the release notes, it’s always a judgment call.  All the compatibility breaking changes, as you say, and then anything that’s “significant”; but which ones are those …
20160323 03:28:25-!- travis-ci [~travis-ci@ec2-54-226-206-80.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 03:28:25< irker734> wesnoth: mattsc wesnoth:master 53fa4ddfedca / changelog: Minor changes to changelog wording and formatting https://github.com/wesnoth/wesnoth/commit/53fa4ddfedcaadaabf0e689ecf8957fa46aa263a
20160323 03:28:26< travis-ci> wesnoth/wesnoth#9023 (master - 4314263 : Celtic Minstrel): The build has errored.
20160323 03:28:26< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117866455
20160323 03:28:26-!- travis-ci [~travis-ci@ec2-54-226-206-80.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 03:34:52< mattsc> celticminstrel: if you want, I can give the release notes a shot, but I cannot do that tonight any more.
20160323 03:35:59< mattsc> I think there are things where you can leave details to the changelog: “many changes to simplyfy and standardize syntax (see changelog for details”.
20160323 03:36:41< celticminstrel> I'll commit what I have in RELEASE_NOTES, for now. You can add more when you get a chance, or I might do so when I think of what to add.
20160323 03:36:46< mattsc> You should spell simplify correctly though
20160323 03:37:03-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160323 03:37:05< celticminstrel> Of course . :P
20160323 03:37:10< mattsc> And others where the release notes should contain more detail than the changelog, although probably not many of them.
20160323 03:37:47< mattsc> Okay, sounds good.
20160323 03:37:57< mattsc> Evenings are just always crazy here …
20160323 03:50:11< mattsc> celticminstrel: something’s wrong with recruiting
20160323 03:50:20< celticminstrel> Uh-oh.
20160323 03:50:35< mattsc> Start HttT S1 and just end Konrad’s turn.  Some of the AI sides only recruit one unit.
20160323 03:50:52< mattsc> At least for me.
20160323 03:50:53< celticminstrel> I was wondering about that in the MicroAI scenario earlier...
20160323 03:51:05< mattsc> Yeah, that’s how I found it, actually.
20160323 03:51:25< mattsc> So I looked whether it was specific to the test scenarios
20160323 03:51:43< celticminstrel> Is it consistent? Like, the sides that only recruit one unit always recruit only one?
20160323 03:51:52< vultraz> hm
20160323 03:52:07< vultraz> not sure where the code that deals with receiving messages is..
20160323 03:52:48< mattsc> celticminstrel: seems to be, at least in the limited tests I have done
20160323 03:54:15-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 03:54:16< travis-ci> wesnoth/wesnoth#9024 (master - dc53c90 : Charles Dang): The build has errored.
20160323 03:54:16< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117870913
20160323 03:54:16-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 04:05:18-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Quit: So long and thanks for all the fish.]
20160323 04:12:32-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160323 04:14:28-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160323 04:19:43< ancestral> Okay, copied readline dylib into my project resources
20160323 04:20:17< ancestral> vultraz: The workaround, changing the line in fonts.cfg, does that have a positive impact on the Windows version of Wesnoth also?
20160323 04:20:29< ancestral> Or is it unchanged
20160323 04:20:44< celticminstrel> ancestral: When I did that, everything showed up in bold.
20160323 04:20:50< celticminstrel> Except for the italics.
20160323 04:20:55< celticminstrel> That wasn't bold.
20160323 04:20:56< ancestral> readline?
20160323 04:21:03< celticminstrel> No, fonts.cfg.
20160323 04:23:08< ancestral> celticminstrel: That has me curious
20160323 04:23:32< ancestral> I would love to see screenshots
20160323 04:23:44< celticminstrel> I'll try to get one for you.
20160323 04:23:52< ancestral> Cool
20160323 04:26:10< celticminstrel> Okay, so the ones that work have recruitment_pattern, and the ones that don't work don't have it.
20160323 04:28:53< celticminstrel> Which I think means that recruitment_pattern is compensating for something else being broken.
20160323 04:33:31< celticminstrel> I think I see the problem.
20160323 04:45:52-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 04:48:28< vultraz> celticminstrel: positive
20160323 04:48:57< celticminstrel> ?
20160323 04:49:06< ancestral> Okay so
20160323 04:49:25< ancestral> Hmm, maybe I’ll clean first
20160323 04:50:51-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 264 seconds]
20160323 04:52:12< irker734> wesnoth: Celtic Minstrel wesnoth:master 3bd881ed6b5f / src/ai/composite/goal.cpp: Fix [goal] not falling back to default name= if engine= omitted https://github.com/wesnoth/wesnoth/commit/3bd881ed6b5f66fe1a6b117b71ed5ab93f7979db
20160323 04:52:14< irker734> wesnoth: Celtic Minstrel wesnoth:master eebedd9adbc9 / RELEASE_NOTES changelog: Update RELEASE_NOTES and changelog https://github.com/wesnoth/wesnoth/commit/eebedd9adbc9c124cb9cfea5cb7cbcad8155a7df
20160323 04:52:16< irker734> wesnoth: Celtic Minstrel wesnoth:master 8f3c03254307 / src/ai/recruitment/ (recruitment.cpp recruitment.hpp): Fix recruitment aspect with number=-1 or missing https://github.com/wesnoth/wesnoth/commit/8f3c0325430767df6d518bf52629bb87f993a1d1
20160323 04:52:29< celticminstrel> Huh, I thought the end turn button was disabled in the tutorial, but I guess I was wrong.
20160323 04:53:00< ancestral> Still failed
20160323 04:53:10< celticminstrel> ancestral: Failed what?
20160323 04:53:11< ancestral> Still lua_interpreter.cpp
20160323 04:53:14< ancestral> Build
20160323 04:53:16< celticminstrel> vultraz: What do you mean by positive?
20160323 04:53:34< celticminstrel> ancestral: Did you copy the "readline" folder to Headers/?
20160323 04:53:35< ancestral> I copied readline 6.3 and history and added it to the libs
20160323 04:53:38< ancestral> Oh
20160323 04:53:39< ancestral> No
20160323 04:53:42< ancestral> Bah, ok
20160323 04:54:27-!- aeonchild [enchilado@defocus/yummy/enchilado] has quit [Ping timeout: 264 seconds]
20160323 04:54:51< ancestral> Hmm
20160323 04:55:14< ancestral> Having trouble seeing the headers
20160323 04:55:22-!- wedge010 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has joined #wesnoth-dev
20160323 04:56:00-!- aeonchild [enchilado@defocus/yummy/enchilado] has joined #wesnoth-dev
20160323 04:56:09< ancestral> celticminstrel: But the dylib should be in libs
20160323 04:56:36< ancestral> Okay
20160323 04:56:40< ancestral> include/readline
20160323 04:57:23< ancestral> Let’s see if that works
20160323 04:57:41< celticminstrel> ancestral: This is what I get if I change fonts.cfg line 8 to 'family_order=_ "DejaVu Sans"': http://celmin.pwcsite.com/wesnoth/bold_fonts.png
20160323 04:57:42-!- wedge009 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has quit [Ping timeout: 276 seconds]
20160323 04:57:42-!- wedge010 is now known as wedge009
20160323 04:57:52-!- travis-ci [~travis-ci@ec2-54-226-206-80.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 04:57:53< travis-ci> wesnoth/wesnoth#9026 (master - 78b442d : Charles Dang): The build has errored.
20160323 04:57:53< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117874146
20160323 04:57:53-!- travis-ci [~travis-ci@ec2-54-226-206-80.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 04:57:55< celticminstrel> ancestral: The required header should be at Headers/readline/history.h
20160323 04:58:02 * celticminstrel pokjes vultraz again.
20160323 04:58:13< ancestral> celticminstrel: I’ve noticed this before
20160323 04:58:19< ancestral> Trying to remember the context
20160323 04:58:33< ancestral> celticminstrel: Do you have Deja Vu Sans installed on your computer by chance?
20160323 04:58:47< celticminstrel> Uh, probably.
20160323 04:58:52< celticminstrel> Let me check.
20160323 04:59:25< celticminstrel> DejaVuSans.ttf (and associated files) exist in ~/Library/Fonts/
20160323 04:59:33< ancestral> Hmm
20160323 04:59:44< ancestral> I wonder if it’s incorrectly loading from your system
20160323 04:59:48< ancestral> So try this
20160323 04:59:53< celticminstrel> I have the entire DejaVu family, it seems.
20160323 04:59:59< ancestral> Remove DejaVuSans Bold from that folder for the moment
20160323 05:00:03< ancestral> Then relaunch Wensoth
20160323 05:00:10< ancestral> (or disable from Font Book)
20160323 05:02:31< celticminstrel> Still bold.
20160323 05:03:01< celticminstrel> I assume it wouldn't do something like using Bold Condensed to substitute...
20160323 05:04:19< ancestral> celticminstrel: Thanks that worked
20160323 05:04:29< ancestral> celticminstrel: Well, you could do this
20160323 05:04:36< ancestral> Undo whatever you just did
20160323 05:04:44< ancestral> Disable all DVS in Font Book
20160323 05:05:21< ancestral> (Just for troubleshooting; this isn’t a long term thing)
20160323 05:07:57< celticminstrel> Huh, I thought that would move them out of the fonts folder...
20160323 05:08:09< celticminstrel> Still bold.
20160323 05:08:12< ancestral> Weird
20160323 05:08:28< ancestral> Then there’s something else going on
20160323 05:08:33< ancestral> Pango? Cairo?
20160323 05:08:59< ancestral> If you click on the i
20160323 05:09:07< ancestral> In the main menu bottom left corner
20160323 05:09:23< celticminstrel> "1 serious error was found. Do not use this font." o.o (But that's for DejaVuSans ExtraLight, so irrelevant here.)
20160323 05:10:04< ancestral> What is your version numbers for Cairo and Pango and SDL_ttf
20160323 05:10:56< celticminstrel> Cairo - build 1.14.2, runtime 1.14.6. Pango, 1.38.0. SDL_ttf, 2.0.12. (I think SDL_ttf isn't affected though. The loading screen is not in bold, for example.)
20160323 05:11:24< celticminstrel> Uhh, why does the version window have a scrollbar?
20160323 05:15:29< ancestral> What is your Pango runtime?
20160323 05:15:40-!- Greywhind [~Greywhind@c-71-232-29-126.hsd1.ma.comcast.net] has quit [Ping timeout: 244 seconds]
20160323 05:15:59< ancestral> I have Pango 1.36.3
20160323 05:16:45< celticminstrel> Same as build version.
20160323 05:16:53< celticminstrel> That's why I didn't list it separately.
20160323 05:16:57< celticminstrel> 1.38.0.
20160323 05:19:35< ancestral> That could be why, maybe
20160323 05:19:43< ancestral> Interesting
20160323 05:20:52< vultraz> celticminstrel: has a scrollbar if the font size is any more then default
20160323 05:21:39< celticminstrel> Oh, so the bold could be doing it, potentially?
20160323 05:21:43< vultraz> yes
20160323 05:22:40< vultraz> what are you doing?
20160323 05:23:34< ancestral> celticminstrel: Okay, that’s it
20160323 05:23:42< vultraz> celticminstrel: were you testing something in the lobby
20160323 05:24:07< celticminstrel> vultraz: Tried it on 800x600.
20160323 05:24:18< ancestral> 1.39 is my runtime version and it shows as bold
20160323 05:25:16< vultraz> celticminstrel: ah, I thought maybe you had figured out how to make / not destroy the sender brackets
20160323 05:25:46< vultraz> remind me to change the default link color
20160323 05:30:16-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 05:30:17< travis-ci> wesnoth/wesnoth#9027 (master - 53fa4dd : mattsc): The build has errored.
20160323 05:30:17< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117880189
20160323 05:30:18-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 05:34:50-!- Greywhind [~Greywhind@c-71-232-29-126.hsd1.ma.comcast.net] has joined #wesnoth-dev
20160323 05:38:57-!- fabi [~quassel@wesnoth/developer/fendrin] has joined #wesnoth-dev
20160323 05:42:17< ancestral> celticminstrel: I may have an idea
20160323 05:42:53-!- fendrin [~quassel@wesnoth/developer/fendrin] has quit [Ping timeout: 248 seconds]
20160323 05:43:54< ancestral> celticminstrel: So I think what’s happening is -bold is occuring earlier alphabetically
20160323 05:44:42< ancestral> Hmm, maybe not
20160323 05:45:40< ancestral> Oh, I wonder if it’s because it’s DejaVu Sans Book
20160323 05:46:30< ancestral> No hmm
20160323 05:50:01< ancestral> Okay so it is prioritizing Bold over the book
20160323 05:59:38-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 05:59:39< travis-ci> wesnoth/wesnoth#9028 (master - 8f3c032 : Celtic Minstrel): The build has errored.
20160323 05:59:39< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117890501
20160323 05:59:39-!- travis-ci [~travis-ci@ec2-54-159-71-208.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 05:59:43-!- celticminstrel is now known as celmin|sleep
20160323 06:34:38-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160323 06:42:04-!- Kwandulin [~Miranda@p200300760F191C499908C7A885668A02.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 06:54:14-!- SigurdFD [~SigurdFD@dynamic-acs-72-23-176-151.zoominternet.net] has quit []
20160323 07:13:18-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 07:17:49-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 248 seconds]
20160323 07:20:49 * Aginor waves
20160323 07:38:09-!- Nobun [~nobun@host169-58-dynamic.61-82-r.retail.telecomitalia.it] has joined #wesnoth-dev
20160323 07:48:41-!- boucman_work [~boucman@bob75-2-81-56-46-209.fbx.proxad.net] has joined #wesnoth-dev
20160323 07:50:43< vultraz> allos, Aginor 
20160323 07:51:20< irker734> wesnoth: Wedge009 wesnoth:master aacb024fb598 / projectfiles/VC9/wesnothlib.vcproj: Remove non-existent files. https://github.com/wesnoth/wesnoth/commit/aacb024fb598fef56270ba8f38638470e307980e
20160323 07:51:22< irker734> wesnoth: Wedge009 wesnoth:master eb9a16da5958 / projectfiles/VC9/wesnoth.vcproj: Move map-related files into their own filter. https://github.com/wesnoth/wesnoth/commit/eb9a16da59581ecf8bfb80fd877ee94d6857d5c2
20160323 07:51:24< irker734> wesnoth: Charles Dang wesnoth:master a7e7036c4439 / projectfiles/VC9/ (wesnoth.vcproj wesnothlib.vcproj): Merge pull request #635 from Wedge009/vc_update https://github.com/wesnoth/wesnoth/commit/a7e7036c443968065b53f903eeebf1791dc5c6a1
20160323 07:52:18-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20160323 07:54:50-!- Nobun [~nobun@host169-58-dynamic.61-82-r.retail.telecomitalia.it] has quit [Remote host closed the connection]
20160323 08:30:58-!- travis-ci [~travis-ci@ec2-54-161-85-97.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 08:30:59< travis-ci> wesnoth/wesnoth#9030 (master - a7e7036 : Charles Dang): The build has errored.
20160323 08:30:59< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/117908159
20160323 08:30:59-!- travis-ci [~travis-ci@ec2-54-161-85-97.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 08:59:56-!- atarocch [~atarocch@2.44.103.10] has joined #wesnoth-dev
20160323 09:32:48-!- Kwandulin [~Miranda@p200300760F191C499908C7A885668A02.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160323 09:49:44-!- mjs-de [~mjs-de@x4db60af0.dyn.telefonica.de] has joined #wesnoth-dev
20160323 09:55:21-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 09:59:58-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 252 seconds]
20160323 10:18:17-!- Kwandulin [~Miranda@p200300760F191C49C96D9A4F0C5FF999.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 11:00:36-!- irker734 [~irker@uruz.ai0867.net] has quit [Quit: transmission timeout]
20160323 11:23:00-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has joined #wesnoth-dev
20160323 11:33:35-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160323 11:36:38-!- Kwandulin [~Miranda@p200300760F191C49C96D9A4F0C5FF999.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160323 11:47:51-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has quit [Ping timeout: 264 seconds]
20160323 12:07:28-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160323 12:07:58-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has quit [Quit: ChipmunkV]
20160323 12:31:57-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 12:41:00-!- wedge009 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has quit [Ping timeout: 268 seconds]
20160323 12:41:36-!- wedge009 [~Thunderbi@60-241-236-92.static.tpgi.com.au] has joined #wesnoth-dev
20160323 12:51:35-!- celmin|sleep is now known as celticminstrel
20160323 13:03:42-!- louis94 [~~louis94@91.178.242.5] has quit [Quit: Konversation terminated!]
20160323 13:16:05-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160323 13:25:59-!- Kwandulin [~Miranda@p200300760F191C2604A9233F44185755.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 13:27:42-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 13:27:57-!- atarocch [~atarocch@2.44.103.10] has quit [Ping timeout: 260 seconds]
20160323 13:41:02-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160323 13:42:04-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160323 13:42:18-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has quit [Quit: horrowind]
20160323 13:42:37-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has joined #wesnoth-dev
20160323 13:42:40-!- atarocch [~atarocch@2.44.103.10] has joined #wesnoth-dev
20160323 13:47:56-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160323 13:48:51-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160323 13:49:50-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has quit [Quit: horrowind]
20160323 13:50:08-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has joined #wesnoth-dev
20160323 13:53:53-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has joined #wesnoth-dev
20160323 14:08:29-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Quit: mattsc]
20160323 14:09:23-!- ypnos [~ypnos@lme51.informatik.uni-erlangen.de] has quit [Remote host closed the connection]
20160323 14:10:31-!- celticminstrel [~celmin@unaffiliated/celticminstrel] has quit [Quit: KABOOM! It seems that I have exploded. Please wait while I reinstall the universe.]
20160323 14:12:02-!- ypnos [~ypnos@lme51.informatik.uni-erlangen.de] has joined #wesnoth-dev
20160323 14:12:33-!- midzer_ [~quassel@p5B312EF2.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 14:12:40-!- celticminstrel [~celmin@unaffiliated/celticminstrel] has joined #wesnoth-dev
20160323 14:18:33-!- midzer [~quassel@p5B312EF2.dip0.t-ipconnect.de] has quit [Quit: No Ping reply in 180 seconds.]
20160323 14:22:19-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 252 seconds]
20160323 14:26:04-!- ancestral [~ancestral@75-168-27-21.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai]
20160323 14:26:50-!- midzer_ is now known as midzer
20160323 14:26:57-!- midzer [~quassel@p5B312EF2.dip0.t-ipconnect.de] has quit [Quit: http://quassel-irc.org - Chat comfortably. Anywhere.]
20160323 14:27:15-!- midzer [~quassel@p5B312EF2.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 14:28:44-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160323 14:35:47-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 14:35:52< celticminstrel> So, I managed to fix the gamestate inspector so that the top listbox doesn't disappear in 800x600.
20160323 14:36:05< celticminstrel> By just commenting out one line that fills out the bottom listbox in pre_show.
20160323 14:46:57-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 246 seconds]
20160323 14:51:00-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 14:55:31-!- prkc [~prkc@catv-80-98-103-239.catv.broadband.hu] has quit [Ping timeout: 248 seconds]
20160323 14:56:03-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 248 seconds]
20160323 15:08:33-!- prkc [~prkc@108.61.123.84] has joined #wesnoth-dev
20160323 15:10:03-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 15:17:42-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Quit: So long and thanks for all the fish.]
20160323 15:20:24-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160323 15:20:43-!- atarocch [~atarocch@2.44.103.10] has quit [Ping timeout: 268 seconds]
20160323 15:20:54-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 260 seconds]
20160323 15:21:10-!- atarocch [~atarocch@2.44.103.10] has joined #wesnoth-dev
20160323 15:31:08-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 15:33:53-!- Kwandulin [~Miranda@p200300760F191C2604A9233F44185755.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160323 15:35:18-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has quit [Quit: horrowind]
20160323 15:36:50< celticminstrel> Doesn't quite work nice, but better than the listbox being altogether absent.
20160323 15:36:55< celticminstrel> ^nicely
20160323 15:38:04< celticminstrel> An alternate fix would be to add an initial view that doesn't use the second listbox. Anyone have any ideas? I thought about showing the scenario data, but am dubious about the usefulness of this.
20160323 15:38:10-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 15:38:50< celticminstrel> Alternatively it could open to some sort of page explaining how the inspector works, I suppose...
20160323 15:39:09 * celticminstrel is a bit dubious about that too - it's not exactly a confusing dialog that needs explanation.
20160323 15:41:46< celticminstrel> BTW vultraz, has_minimum and has_maximum don't appear to work entirely correctly. "has_minimum=false" supposedly means it's possible for no row to be selected. "has_maximum=false" supposedly means it's possible for more than one row to be selected. It works fine if they're both true or if they're both false; it may work for has_minimum=true and has_maximum=false, but it does not work for has_minimum=true and has_maximum=false.
20160323 15:42:21< celticminstrel> Uh. Reverse the true and false in that last pair.
20160323 15:47:27< celticminstrel> Hmm, this change makes the listboxes sometimes overlap. O_O
20160323 15:49:15-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 246 seconds]
20160323 15:54:51-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 16:03:57< vultraz> celticminstrel: I'm having trouble parsing that line :P
20160323 16:04:41< vultraz> hmm
20160323 16:04:42< celticminstrel> If you set both has_minimum and has_maximum, it works. If you set just has_minimum, you can still select more than one row.
20160323 16:04:53< vultraz> I can't fix that, sorry
20160323 16:04:56< celticminstrel> Any comments on the inspector stuff?
20160323 16:05:52< celticminstrel> I'd like to replace the listbox pair with a treeview, but I have no idea how to deal with treeviews, so I thought I'd try to fix it more directly.
20160323 16:05:54< vultraz> actually think it might be possible to force them to each take 1/2 screen
20160323 16:06:04< celticminstrel> Admittedly it's still a bit of a workaround... wait, how?
20160323 16:06:59< vultraz> {GUI_FORCE_WIDGET_MINIMUM_SIZE 0 "((screen_height * 50) / 100)"  (listbox code here)}
20160323 16:07:07< celticminstrel> Hmmm.
20160323 16:10:45< celticminstrel> I will try it, but I'm not too hopeful about it...
20160323 16:11:29< vultraz> might want to do 40
20160323 16:14:01< vultraz> actually, wanted to ask something
20160323 16:14:10< vultraz> if you have two overloads
20160323 16:14:17< celticminstrel> Multiplication and division have the same precedence, and all operators except exponentiation are associative, so the inner set of parentheses is not needed. :)
20160323 16:14:39 * celticminstrel waits for the rest of your question.
20160323 16:15:00< celticminstrel> ^left associative
20160323 16:15:26< vultraz> one with foo(string, bool) and foo(string, size_t,bool) with the bool argument being optional, how do you avoid "overload is ambiguous" compiler errors?
20160323 16:16:15< celticminstrel> Those honestly seem sufficiently unambiguous to me... :|
20160323 16:16:23< celticminstrel> Even though bool is an integer type...
20160323 16:21:03< celticminstrel> I think in the worst case it's possible to avoid "overload is ambiguous" with an explicit cast, though I'm not sure if you can rely on default arguments in that case. (I've never tried it. The syntax is pretty ugly too.)
20160323 16:21:20< celticminstrel> (I've done explicit casts, but only when not calling the function, eg in bind.)
20160323 16:21:36< celticminstrel> Whoa.
20160323 16:27:12< celticminstrel> That actually worked.
20160323 16:30:14< vultraz> celticminstrel: the force width thing?
20160323 16:30:18< celticminstrel> Yeah.
20160323 16:32:53< vultraz> well?
20160323 16:33:08< celticminstrel> With some tweaking, fairly well, yes.
20160323 16:35:47-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 260 seconds]
20160323 16:35:54< vultraz> without your fix?
20160323 16:36:44< celticminstrel> ?
20160323 16:36:57< vultraz> you were playing with has_minimum
20160323 16:37:11< celticminstrel> Oh, the commenting out of that one line was no longer necessary, no.
20160323 16:37:19< celticminstrel> Nor has_minimum or anything like that.
20160323 16:46:07-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160323 16:55:57-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has joined #wesnoth-dev
20160323 16:57:33< celticminstrel> So the units entry for my Elvish ranger leader now reads '(18,18) side=1\nid=Elvish Ranger-1" (Elvish Ranger)\nL2; 0/3f xp; 2a/2a hp;'
20160323 16:57:58< celticminstrel> Oh, forgot the LEADER part. Still.
20160323 16:58:52-!- Nobun [~nobun@host45-49-dynamic.20-87-r.retail.telecomitalia.it] has joined #wesnoth-dev
20160323 16:59:14< celticminstrel> Can you spot the problem? :P
20160323 17:00:31-!- boucman_work [~boucman@bob75-2-81-56-46-209.fbx.proxad.net] has quit [Remote host closed the connection]
20160323 17:02:20-!- Kwandulin [~Miranda@p200300760F191C26A41DBCD720B2A15E.dip0.t-ipconnect.de] has joined #wesnoth-dev
20160323 17:02:41-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 17:03:32-!- irker768 [~irker@uruz.ai0867.net] has joined #wesnoth-dev
20160323 17:03:33< irker768> wesnoth: Celtic Minstrel wesnoth:master 36eba6e8336d / data/gui/window/gamestate_inspector.cfg: tgamestate_inspector: Fix top listbox sometimes disappearing in small resolution https://github.com/wesnoth/wesnoth/commit/36eba6e8336dfbfb03479c3a009c49dd7ab98bc4
20160323 17:03:34< irker768> wesnoth: Celtic Minstrel wesnoth:master c7f8e524e286 / src/gui/dialogs/label_settings.cpp: tlabel_settings: Use side_name instead of current_player https://github.com/wesnoth/wesnoth/commit/c7f8e524e286d406b0d42f950be88e43bb76b8c8
20160323 17:03:36< irker768> wesnoth: Celtic Minstrel wesnoth:master 074281fce07a / src/gui/dialogs/gamestate_inspector.cpp: tgamestate_inspector: Add a little color to the units list https://github.com/wesnoth/wesnoth/commit/074281fce07a3192211fb3513afa726d491dde65
20160323 17:04:26-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160323 17:07:17-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 260 seconds]
20160323 17:11:23-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 17:11:30-!- fabi [~quassel@wesnoth/developer/fendrin] has quit [Remote host closed the connection]
20160323 17:20:32< vultraz> ah, nice
20160323 17:23:32-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 17:23:42< celticminstrel> Such a generic comment.
20160323 17:27:45-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Ping timeout: 244 seconds]
20160323 17:32:46-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 17:33:54-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20160323 17:34:33-!- fendrin [~quassel@wesnoth/developer/fendrin] has joined #wesnoth-dev
20160323 17:38:55-!- fendrin_ [~quassel@176.0.99.141] has joined #wesnoth-dev
20160323 17:40:55< vultraz> celticminstrel: no, it really does look nice :P
20160323 17:42:21-!- fendrin [~quassel@wesnoth/developer/fendrin] has quit [Ping timeout: 248 seconds]
20160323 17:43:57-!- travis-ci [~travis-ci@54.159.71.208] has joined #wesnoth-dev
20160323 17:43:58< travis-ci> wesnoth/wesnoth#9031 (master - 074281f : Celtic Minstrel): The build has errored.
20160323 17:43:58< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118028574
20160323 17:43:58-!- travis-ci [~travis-ci@54.159.71.208] has left #wesnoth-dev []
20160323 17:47:43< celticminstrel> That's still generic. :P
20160323 17:48:20< celticminstrel> Not particularly important, but "it" could easily refer to just one of the three commits, for example.
20160323 17:48:50-!- atarocch [~atarocch@2.44.103.10] has quit [Quit: Leaving]
20160323 17:56:41-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has quit [Quit: ChipmunkV]
20160323 17:57:42-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has joined #wesnoth-dev
20160323 18:01:24-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 18:23:50< vultraz> celticminstrel: two questions: https://drive.google.com/file/d/0B-mR9s8FduLLd2duTmFtU2liSjQ/view?usp=sharing does this look like a trash can icon (right column) and do you think having the delete button inline is good
20160323 18:27:01< celticminstrel> At first glance, I did not recognize it as a trash can.
20160323 18:27:15< vultraz> what did you think the icon did
20160323 18:27:19< celticminstrel> I'm not sure whether an inline delete is a good idea.
20160323 18:27:48< celticminstrel> Hmm... it actually looks kind of like a clipboard if you don't look closely.
20160323 18:28:22-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 260 seconds]
20160323 18:28:27< vultraz> damn
20160323 18:28:37< celticminstrel> Is there a confirmation when you click delete?
20160323 18:29:07< vultraz> I haven't implemented it yet
20160323 18:29:14< vultraz> I'm simply trying the design out
20160323 18:29:23< vultraz> Also, need a new color scheme for these types of buttons..
20160323 18:29:34< celticminstrel> I'm not sure if inline delete is good, but I think having it inline makes it more important to put up a confirmation first.
20160323 18:30:01< vultraz> there is an option for such a thing
20160323 18:30:09< celticminstrel> ?
20160323 18:30:44< vultraz> there is a prefs option to confirm on delete
20160323 18:31:36< celticminstrel> Oh.
20160323 18:32:07< celticminstrel> I hope it defaults to on.
20160323 18:32:23< vultraz> yes
20160323 18:33:56< vultraz> I'll speak to jet about the icon/color 
20160323 18:34:22< vultraz> right now, would like to figure out why '/' breaks the speaker formatting...
20160323 18:35:05< vultraz> wait
20160323 18:35:10< vultraz> celticminstrel: is '/' an escape character?
20160323 18:35:33-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 18:35:35< celticminstrel> I have never heard of / being an escape character.
20160323 18:35:54< celticminstrel> It does have a meaning in pango markup, but only if directly preceded by <
20160323 18:37:09< vultraz> ...huh
20160323 18:37:19< vultraz> preceeding the "/" with a space does not break markup
20160323 18:37:25< vultraz> I thought I added a space earlier..
20160323 18:37:33< celticminstrel> I did mention it doesn't happen with inline /
20160323 18:37:40< celticminstrel> I remember sending a message like that.
20160323 18:38:07< vultraz> ok, so only starting /
20160323 18:39:25-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 18:39:28< vultraz> let me get this scrolling thing right and then I'll deal with that
20160323 18:40:18< vultraz> so many chat-related functions :|
20160323 18:40:20-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 18:52:11< mattsc> Hmm, I did not know that there is a star out there called Lich
20160323 18:52:31< mattsc> And among other things, it has a planet called Draugr  (yes, I am aware of the extra R)
20160323 18:52:40< mattsc> https://en.wikipedia.org/wiki/PSR_B1257%2B12
20160323 18:54:37< celticminstrel> It's Wesnoth that's missing the R, not the other way around. :P
20160323 18:54:41< vultraz> ^
20160323 18:55:01< celticminstrel> (Not that I'd recommend changing it.)
20160323 18:55:09< mattsc> right (on both)
20160323 18:55:13< celticminstrel> mattsc: Did you have any comments on my release notes?
20160323 18:55:32< celticminstrel> I also mentioned incompatibilities in the changelog.
20160323 18:55:34< mattsc> celticminstrel: haven’t had time to even look at them yet
20160323 18:55:38< celticminstrel> Ah, 'kay
20160323 18:55:58< mattsc> celticminstrel: I have confirmed that all the MAIs work though
20160323 18:56:04< celticminstrel> This is good.
20160323 18:56:06< mattsc> but not all the MAI test scenarios ...
20160323 18:56:23< celticminstrel> Did you notice that I fixed recruitment?
20160323 18:56:27< mattsc> There are a couple things in there that are not MAIs; one FAI engine, too Lua AI engines
20160323 18:56:35< celticminstrel> Ah.
20160323 18:56:43< mattsc> yes, I did, and it works now without problems (that I noticed, at least)
20160323 18:56:56< celticminstrel> What's broken in which MAI test?
20160323 18:57:21< mattsc> celticminstrel: I’ll fix it, but I don’t have time for it right now (not even to talk about it), sorry
20160323 18:57:41< celticminstrel> So it's not anything that requires action on my part?
20160323 18:58:01< mattsc> I don’t think so; I believe it’s things like the data table definition etc.
20160323 18:58:11< celticminstrel> 'kay
20160323 18:58:59< mattsc> but again, I didn’t have time to look into it; I’ve either slept or worked pretty much the entire time since I last talked to you
20160323 18:59:27< mattsc> just run the test scenarios on the side while doing that
20160323 18:59:27-!- louis94 [~~louis94@91.178.242.5] has quit [Ping timeout: 276 seconds]
20160323 18:59:44< mattsc> *s/just run/I only have run
20160323 19:14:02-!- EliDupree_ [~quassel@idupree.com] has joined #wesnoth-dev
20160323 19:15:09-!- aidanhs_ [~aidanhs@81.4.110.234] has joined #wesnoth-dev
20160323 19:18:38-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 19:19:52-!- aidanhs [~aidanhs@81.4.110.234] has quit [Ping timeout: 250 seconds]
20160323 19:19:52-!- EliDupree [~quassel@idupree.com] has quit [Ping timeout: 250 seconds]
20160323 19:23:24< irker768> wesnoth: Charles Dang wesnoth:master e6495b929a30 / src/gui/widgets/ (scrollbar_container.cpp scrollbar_container.hpp): Add setters/getters for scrollbar position (patch by faabumc) https://github.com/wesnoth/wesnoth/commit/e6495b929a30f7bf0f1837a37f1281607216892b
20160323 19:23:27< irker768> wesnoth: Charles Dang wesnoth:master 5ea08ba37d49 / src/gui/dialogs/lobby/ (lobby.cpp lobby.hpp): tlobby_main: chat log scrolling fixes (bug #16059) https://github.com/wesnoth/wesnoth/commit/5ea08ba37d49a48577b573a5d6c93d0507c27d19
20160323 19:25:19< vultraz> I need to refactor the way whisper messages are handled
20160323 19:25:33< vultraz> whispering someone should open a room with them, I think
20160323 19:25:51-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20160323 19:28:01< celticminstrel> Provided it's named in such a way that /join won't enter it.
20160323 19:28:18 * celticminstrel suddenly wonders something.
20160323 19:30:38< vultraz> ok, commands are passed through chat_handler::do_speak...
20160323 19:31:51< vultraz> which is passed through dispatch()..
20160323 19:33:09< vultraz> dammit, it's a rabbit hole
20160323 19:35:19< vultraz> I don't get it, though
20160323 19:35:21< celticminstrel> Hmm... any objections to me pushing wesnoth.eval_formula()?
20160323 19:35:29< vultraz> it shouldn't garble escape characters...
20160323 19:35:56< celticminstrel> I'm a bit unsure on the design of some details related to formula function tables, but the basic eval_formula() should be fine...
20160323 19:36:00< vultraz> unless it formats it again..?
20160323 19:37:08-!- ancestral [~ancestral@63.92.240.233] has joined #wesnoth-dev
20160323 19:37:14< vultraz> where the hell is print()
20160323 19:38:24-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 19:38:27< vultraz> ok so
20160323 19:38:42< vultraz> not escaping the input does fix the characters in the help strings
20160323 19:38:44< vultraz> but not
20160323 19:38:58< vultraz> the speaker brackets 
20160323 19:39:07< vultraz> I might just remove the speaker brackets anyway, but still...
20160323 19:40:03< celticminstrel> Don't just remove the speaker brackets. :/
20160323 19:40:24< vultraz> I'd make it Speaker:
20160323 19:40:28< vultraz> not <Speaker>
20160323 19:41:02< celticminstrel> This is not a strong preference, but I do think <Speaker> is a slightly clearer separation between name and message.
20160323 19:41:19< celticminstrel> My IRC client uses the colon though, so I'm actually more used to that.
20160323 19:41:23< vultraz> dota 2 doesn't use speaker brackets 
20160323 19:42:24< vultraz> hmmmmmmmmmmmmmmm
20160323 19:42:26< vultraz> ok
20160323 19:42:29< celticminstrel> However, does lobby chat use the same channel as in-game chat? If it's colons in the lobby, I think it needs to be colons in-game. Also, don't forget that there are some "fake speakers" for in-game chat, like "WML" or "WML error" or indeed whatever the scenario designer wants to use.
20160323 19:42:29< vultraz> add_chat_message is virtual
20160323 19:43:24 * vultraz is curious why it seems the entire textbox contents get rerendered if you add a new line
20160323 19:49:19< Ravana_> when talking about whispers, would it be possible to enable whispering to player if scenario allows it?
20160323 19:49:27< celticminstrel> Come to think of it, if you define two stages with the same ID, I wonder if the first one would be deleted...
20160323 19:49:42< Ravana_> since I guess main point for that block is for pvp games
20160323 19:52:28< celticminstrel> I'm a bit unsure of this, and it seems unfinished: https://github.com/CelticMinstrel/wesnoth/commit/ceba2d0ddce1dcc3491537e7904e697c21456d4e
20160323 19:55:45< celticminstrel> "unsure" -> Mainly about the name of the key. "unfinished" -> I would expect it to not show the AI dropdown or something, but not sure how to do that.
20160323 19:56:59< celticminstrel> Part of the reason I'm unsure is that "override" might be a little misleading - it doesn't completely replace the scenario-defined AI, after all.
20160323 20:01:02< Nobun> the only thing I can say is that you cannot whisper to a player wich is plaing a scenario while you are watching the match
20160323 20:01:28< Nobun> but you can speak with a player who is playing if you are not observing that match
20160323 20:01:47-!- gfgtdf [~chatzilla@f054136027.adsl.alicedsl.de] has joined #wesnoth-dev
20160323 20:02:05-!- ancestral [~ancestral@63.92.240.233] has quit [Quit: i go nstuf kthxbai]
20160323 20:02:40-!- travis-ci [~travis-ci@ec2-54-226-80-176.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 20:02:41< travis-ci> wesnoth/wesnoth#9032 (master - 5ea08ba : Charles Dang): The build has errored.
20160323 20:02:41< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118060837
20160323 20:02:41-!- travis-ci [~travis-ci@ec2-54-226-80-176.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 20:03:23< gfgtdf> Hmm i think we shoudl just allow observers to whisper players mand maybe have an option in prefernces to block observers.
20160323 20:03:41< celticminstrel> gfgtdf: Anything to say about allow_ai_override?
20160323 20:03:57< gfgtdf> celticminstrel: hmm i dont really know that ai code
20160323 20:04:08< gfgtdf> celticminstrel: i never hared of that function
20160323 20:04:09< celticminstrel> Well, it's more the connect_engine code...
20160323 20:04:38< celticminstrel> The idea was a way for scenarios to disable the "select AI" dropdown.
20160323 20:05:06< gfgtdf> celticminstrel: hmm we already hav controller lock
20160323 20:05:22< celticminstrel> That doesn't sound like the same thing...
20160323 20:05:28< gfgtdf> celticminstrel: and allow_player
20160323 20:05:43< gfgtdf> celticminstrel: which somwhow imply locking the ai alorithm
20160323 20:05:48< gfgtdf> celticminstrel: not sure though
20160323 20:06:01< celticminstrel> I'm not sure either, but I'll investigate.
20160323 20:06:42< gfgtdf> celticminstrel: well i think a way to lock the ai algorithm for the mp connect code woudl be useful.
20160323 20:07:34< gfgtdf> celticminstrel: i think that is human vs. ai games you wouldnt use it thought. Since you'd just lock the controller as whole there
20160323 20:08:04< gfgtdf> celticminstrel: maybe more for maps that can handle both ai vs human and human vs human, like most mianline mp maps
20160323 20:08:26< celticminstrel> Hmm.
20160323 20:08:33< gfgtdf> celticminstrel: for for maps that are so specialy that noral ais cannot handle it
20160323 20:10:02< celticminstrel> It would be good for maps that need to use a custom stage, and like you say, maps that the normal AIs can't handle.
20160323 20:16:11 * celticminstrel starts tearing out old AI stuff.
20160323 20:17:18< celticminstrel> Fallback stage. Idle AI. Interface for registering custom C++ AIs. Probably the Akihara stuff. Maybe one or two other things.
20160323 20:17:52< gfgtdf> celticminstrel: you want to remove the idle ai ?
20160323 20:18:28< celticminstrel> gfgtdf: The idle AI does nothing. Thus, it can be emulated using an empty [stage] tag. Thus, it is not needed.
20160323 20:18:35< celticminstrel> This applies to the C++ side only, though.
20160323 20:18:52< celticminstrel> I will not be removing the WML definition of the idle AI, so ai_algorithm=idle_ai will still work.
20160323 20:19:08< celticminstrel> (Not quite an empty [stage], but at least, one with no CAs.)
20160323 20:20:21< gfgtdf> Ravana_: i made a bugreport about observer shispers: https://gna.org/bugs/?24538
20160323 20:22:11< celticminstrel> Oh, by the way... gfgtdf or wedge009 or someone using MSVC, do you think the *.user files are really needed in the repository? I thought they were sort of personal settings or something.
20160323 20:23:59< mattsc> celticminstrel: will you also clean up the WML side of things, that is, data/ai ?
20160323 20:24:24< celticminstrel> Well, I'll delete anything that directly references anything I've removed.
20160323 20:24:44< celticminstrel> Like the stuff in data/ai/dev referencing Akihara stuff.
20160323 20:25:03< mattsc> celticminstrel: okay; I can make a pass over the remaining things after you are done then
20160323 20:26:10< celticminstrel> Is there anyone who thinks "Strategy Formulation with RCA" should be kept?
20160323 20:26:25< celticminstrel> Based on the wiki page it seems conceptually, uh, sort of a cool idea, but...
20160323 20:26:26< mattsc> nope
20160323 20:26:37< mattsc> Well, I don’t know if there’s anyone, but I don’t think so
20160323 20:27:05< mattsc> Right; if somebody ever wants to pick it up again, the information will be in the repository
20160323 20:27:28< mattsc> but at the moment is is … clutter
20160323 20:27:32< celticminstrel> Yeah, I'll make sure that the commit deleting it is such that restoring it can be as simple as reverting one commit.
20160323 20:27:52< celticminstrel> That was awkwardly phrased... but I assume you get the idea.
20160323 20:28:09< mattsc> yes
20160323 20:28:32< mattsc> I’ve had some awkward phrases myself recently
20160323 20:29:17< celticminstrel> The formatting of the MSVC project is kind of poor for diff purposes.
20160323 20:30:50< celticminstrel> Removing ai::interface wasn't as hard as I'd expected. There's still a bit of minor clutter left from it, but that should be fine...
20160323 20:31:46< celticminstrel> BTW, I notice that FormulaAI has a command to fallback to human control until the end of the turn, but unless I'm mistaken, LuaAI has no such thing. Does that seem useful? Should I add it?
20160323 20:39:21< mattsc> Uh, I’m not sure how that would be useful.
20160323 20:41:56< celticminstrel> I can't think of any specific case either, but it feels like there could be a use for it...
20160323 20:43:07< celticminstrel> Oh, I forgot to remove "Dev AI: Default + Old Recruitment" when I removed the recruitment stage.
20160323 20:44:09-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20160323 20:44:20< celticminstrel> I wonder if the "aki_eval" formula function is used anywhere. I would suspect not, since I think it's undocumented.
20160323 20:47:26< mattsc> A grep does not find it anywhere in data/
20160323 20:47:42< mattsc> and only in ai/formula/function_table.cpp in src/ which is, I assume, where it’s defined
20160323 20:47:49< mattsc> so likely the answer is no
20160323 20:49:09< celticminstrel> That is, indeed, where it is defined.
20160323 20:49:15< celticminstrel> In that file, at least.
20160323 20:49:59-!- mattsc [~mattsc@wesnoth/developer/mattsc] has quit [Quit: mattsc]
20160323 20:55:45< gfgtdf> celticminstrel: please alays use luaW_toboolean and not lua_toboolean becasue teh later casues warnings/errors issues with msvc.
20160323 20:56:41< celticminstrel> I think I remember changing a lua_boolean to luaW_boolean somewhere...
20160323 21:01:08< vultraz> hmm
20160323 21:01:14< vultraz> I just noticed gui2 has connect_signal_pre_key_press
20160323 21:01:41< vultraz> I wonder if the hotkey setter dialog could use it?
20160323 21:06:48< celticminstrel> Does anyone know if "Dev AI: Default + Experimental Recruitment" works?
20160323 21:06:51< vultraz> also, random thought: place where lack of smooth scrolling is detrimental: mp chat log
20160323 21:07:15< celticminstrel> You need to be in debug mode to see it.
20160323 21:07:52< celticminstrel> I think that can be done without --debug by starting a local MP game, using ;debug, exiting, and going to start one again.
20160323 21:11:07< Nobun> celticminstrel: perhaps it is used to allow undroid? (or replacing with human player?)
20160323 21:11:27< celticminstrel> Nobun: Not sure, but I think it was more for testing purposes.
20160323 21:11:38< Nobun> it could be
20160323 21:13:01< gfgtdf> celticminstrel: y i didnt mena changing i meant when writing new code
20160323 21:13:10-!- Kwandulin [~Miranda@p200300760F191C26A41DBCD720B2A15E.dip0.t-ipconnect.de] has quit [Read error: Connection reset by peer]
20160323 21:13:19< celticminstrel> gfgtdf: Yeah, I know what you meant.
20160323 21:13:49< celticminstrel> If I've used lua_boolean before it was probably either before I discovered the other one or because I just used code completion and didn't look closely.
20160323 21:16:09< celticminstrel> [Mar 23@5:06:48pm] celticminstrel: Does anyone know if "Dev AI: Default + Experimental Recruitment" works? <-- C++, not FormulaAI
20160323 21:17:54< celticminstrel> It's not horribly broken, at least - the AI does recruit with it.
20160323 21:19:04< Ravana_> gfgtdf: I will be following that bugreport
20160323 21:20:15< vultraz> celticminstrel: just noticed this goes to console when you use a comamnd http://pastebin.com/YXZuNhNH
20160323 21:20:17< vultraz> command*
20160323 21:20:23< vultraz> what is this "ΓÇö"
20160323 21:20:33< vultraz> and "has unescaped ampersands"
20160323 21:21:18-!- mattsc [~mattsc@wesnoth/developer/mattsc] has joined #wesnoth-dev
20160323 21:21:22< vultraz> hm... did I miss a ; somewhere?
20160323 21:22:21< vultraz> AH i did
20160323 21:22:27< celticminstrel> I assume your console is not Unicode-aware - the weird sequence of three letters is probably an em dash.
20160323 21:22:32< vultraz> (but still, what the hell is "ΓÇö"
20160323 21:22:36< vultraz> ah yes
20160323 21:22:47< vultraz> ok apparently the cause of this was a missing ;
20160323 21:22:55< celticminstrel> I do see missing semicolons, yes.
20160323 21:23:03< celticminstrel> Is that also the cause of it breaking when you type in /me?
20160323 21:23:26< vultraz> yes
20160323 21:24:22< irker768> wesnoth: Charles Dang wesnoth:master d599ea15099b / src/gui/dialogs/lobby/lobby.cpp: tlobby_main: fixed an unescaped escape character https://github.com/wesnoth/wesnoth/commit/d599ea15099be90e43ec7f01a024c483d690b02e
20160323 21:24:23< vultraz> pays to check console :P
20160323 21:24:27< vultraz> celticminstrel: can you confirm fix
20160323 21:24:38< celticminstrel> Not right this moment.
20160323 21:24:46< celticminstrel> Since I'm on the AI branch.
20160323 21:24:52< vultraz> when you have time
20160323 21:27:15< celticminstrel> Speaking of the AI branch, should I keep this AI cleanup on a branch or should I push it straight to master?
20160323 21:27:33< celticminstrel> Or the middle route, a branch in the main repo?
20160323 21:27:45< celticminstrel> vultraz, mattsc ?
20160323 21:28:30< mattsc> celticminstrel: as far as I am concerned, you can push srraight to master
20160323 21:34:20< mattsc> celticminstrel: FYI, my availability will likely be even less than it has been through … at least mid July
20160323 21:35:14< mattsc> It will be very spotty though, so there will be times when I am around and able to do things, and others when I’ll be gone for several weeks.
20160323 21:43:21-!- gfgtdf [~chatzilla@f054136027.adsl.alicedsl.de] has quit [Quit: ChatZilla 0.9.92 [Firefox 45.0.1/20160315153207]]
20160323 21:50:40< mattsc> zookeeper, vultraz (I do not know whom I should ask this): do you think that strings in the Micro AI test scenarios should be translatable?
20160323 21:50:54< mattsc> There’s a lot of them and they are only accessible from the CL.
20160323 21:51:55< zookeeper> i'd tend to think that translators shouldn't be burdened with test/debug content
20160323 21:52:33< mattsc> zookeeper: yeah, I’d agree.  Should I do that retroactively for 1.12 also?
20160323 21:52:51< zookeeper> if it's in a separate domain and somehow translators know that it's of the lowest priority, then i don't see any harm in it, but...
20160323 21:53:08< mattsc> it is a separate domain
20160323 21:53:10< zookeeper> i don't see much point in doing something like that for 1.12 anymore
20160323 21:53:22< mattsc> ok
20160323 21:53:28< zookeeper> has anyone actually translated any of them? if not, then i guess that means that translators know not to bother
20160323 21:53:51< mattsc> I think aquileia(?) said that they are translated in two languages
20160323 21:54:51< mattsc> 7, actually: https://www.wesnoth.org/gettext/?package=wesnoth-ai&order=trans&version=branch
20160323 21:55:43< zookeeper> wow
20160323 21:57:13< zookeeper> yeah, considering that, i dunno if it should be kept after all
20160323 21:58:02< mattsc> me neither
20160323 22:02:27< mattsc> It’s less work for me to keep it as is, of course, so unless told otherwise, I’ll adopt that as the default action  :P
20160323 22:09:38-!- LovCAPONE [~lovens@modemcable184.138-20-96.mc.videotron.ca] has quit [Quit: return 0;]
20160323 22:17:16-!- ChipmunkV [~vova@d0017-2-88-172-31-68.fbx.proxad.net] has quit [Quit: ChipmunkV]
20160323 22:20:44-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20160323 22:22:51-!- Nobun [~nobun@host45-49-dynamic.20-87-r.retail.telecomitalia.it] has quit [Quit: Salve a tutti]
20160323 22:24:09< fendrin_> celticminstrel: I am playing your elvish druids campaign.
20160323 22:24:16< celticminstrel> Oh fun.
20160323 22:24:27< fendrin_> Yeah. I like the campaign.
20160323 22:24:32< fendrin_> Do you still develop itß
20160323 22:24:34< fendrin_> ?
20160323 22:24:44-!- travis-ci [~travis-ci@ec2-54-242-11-164.compute-1.amazonaws.com] has joined #wesnoth-dev
20160323 22:24:45< travis-ci> wesnoth/wesnoth#9033 (master - d599ea1 : Charles Dang): The build has errored.
20160323 22:24:45< travis-ci> Build details : https://travis-ci.org/wesnoth/wesnoth/builds/118089726
20160323 22:24:45-!- travis-ci [~travis-ci@ec2-54-242-11-164.compute-1.amazonaws.com] has left #wesnoth-dev []
20160323 22:24:46< celticminstrel> Yes.
20160323 22:25:05< celticminstrel> I think I have several little unreleased fixes and stuff.
20160323 22:25:06< fendrin_> The first scenario has a lot tomato surprises in it.
20160323 22:25:21< fendrin_> I restarted about 5 times to get everything handled right.
20160323 22:26:08< fendrin_> Now that I am at the beginning of 2 I wonder why the recall dialog shows me the lvl1 extra icons you use for the specials show, bot not those of the lvl0 units.
20160323 22:26:33< celticminstrel> Really? That's kinda weird...
20160323 22:26:42< fendrin_> Indeed.
20160323 22:27:05< fendrin_> I first thought something went wrong with the transition.
20160323 22:27:27< celticminstrel> Hmm? Transition? Like from 1.10->1.12?
20160323 22:27:39< fendrin_> recall list persistence
20160323 22:27:51< celticminstrel> Oh, scenario to scenario transition.
20160323 22:27:55< fendrin_> yeah
20160323 22:28:20< fendrin_> But they are there.
20160323 22:28:46< fendrin_> And I can see their extra icon at the topleft when the unit is selected.
20160323 22:29:46< fendrin_> celticminstrel: I wonder what happens with shamans that have extra attacks like arcane when you level them into sorcerress? 
20160323 22:31:05< celticminstrel> I can't quite remember exactly, but I do know that the [object] for each special feature accounts for all possible advancement paths. In that specific case it's probably a bonus to the sorcerer's normal arcane attack.
20160323 22:31:12< celticminstrel> I can check, if you'd like.
20160323 22:32:02< fendrin_> cool
20160323 22:33:40< fendrin_> The first scenario pretty much depends on if you know where to find elves and where you have to go after the center goblin is defeated.
20160323 22:34:06< fendrin_> Once I had to start over because of the turn limit. 
20160323 22:34:07< celticminstrel> Yeah, you get a big bonus to the sorcerer's arcane attack.
20160323 22:34:27< celticminstrel> I've never had to start over because of the turn limit... what difficulty level?
20160323 22:34:34< fendrin_> medium
20160323 22:34:50< fendrin_> The thing is: You know what to do.
20160323 22:34:55< celticminstrel> True.
20160323 22:35:23< celticminstrel> About "if you know where to find elves" - you mean how they pop out of villages?
20160323 22:35:30< fendrin_> yepp
20160323 22:35:59< celticminstrel> About where you have to go afterwards, I thought that was highlighted for you.
20160323 22:36:07< fendrin_> Yes it is.
20160323 22:36:21< fendrin_> It is just much better to run for the 2 fords in advance.
20160323 22:36:25< fendrin_> Before it is told.
20160323 22:36:30< celticminstrel> Ahh.
20160323 22:36:54< fendrin_> In the meanwhile the objectives say: What for the scout to return.
20160323 22:37:03< fendrin_> When you do that it is harder.
20160323 22:37:12< fendrin_> s/What/Wait
20160323 22:39:36< fendrin_> I tried hard once.
20160323 22:39:46< fendrin_> But that was way to difficult.
20160323 22:39:48< celticminstrel> The scout does appear on the next turn though.
20160323 22:39:55< fendrin_> Yes.
20160323 22:40:24< fendrin_> But you can already beginn to walk towards the fords before that.
20160323 22:40:39< celticminstrel> Probably, yeah.
20160323 22:40:46< fendrin_> I know it is stoy wise much better to have tomato surprises.
20160323 22:42:37-!- horrowind [~Icedove@2a02:810a:83c0:1c18:21b:fcff:fee3:c3ff] has quit [Quit: horrowind]
20160323 22:43:07< celticminstrel> I think I kinda figured that people would be going for the villages anyway, so they'd discover that there are initiates hiding there...
20160323 22:43:59< celticminstrel> Though I suppose knowing in advance might cause you to try to grab all villages first rather than going straight for the leader.
20160323 22:46:17< fendrin_> Okay, let me play on :-)
20160323 22:46:29< fendrin_> Can't wait to hear how the story envolves.
20160323 22:46:44< celticminstrel> Seems like you're not outright saying I need to remove the tomato surprises...
20160323 22:46:53< fendrin_> Not all.
20160323 22:47:09< fendrin_> Only the most annoying ones.
20160323 22:47:25< fendrin_> Some surprises are fine.
20160323 22:47:40< celticminstrel> So the most annoying ones are...? (Sorry if you already said this.)
20160323 22:47:50< fendrin_> My bigest problem was time.
20160323 22:47:59< fendrin_> I had no idea how much to recruit and when.
20160323 22:48:02< fendrin_> And where to go.
20160323 22:48:14< fendrin_> So I failed to reach the goblin.
20160323 22:48:30< fendrin_> That happened the last time I played the scenario a year ago as well.
20160323 22:48:41< celticminstrel> That's a tomato surprise? Or just a general issue?
20160323 22:49:01< fendrin_> Somehow the story lured me into the idea that there is a time of calm after the middle goblin is defeated.
20160323 22:49:27< fendrin_> But you need to reach the fords before the goblins do.
20160323 22:49:37< fendrin_> If they can block them it gets really ugly.
20160323 22:50:17< celticminstrel> Sounds like that issue is more with the objective prior to awaiting the scout.
20160323 22:50:43< fendrin_> The scenario could simply be split into two.
20160323 22:51:12< celticminstrel> That would make the first one extremely short and tiny.
20160323 22:51:39< fendrin_> Yes. And I think that is a good thing. Most campaigns have ugly und long first scenarios.
20160323 22:52:01< fendrin_> But that might be only me.
20160323 22:52:02< celticminstrel> I don't think it's a bad thing to have a short first scenario, but that feels a little too short to me...
20160323 22:52:24< fendrin_> I prefer a quick one that setups the tone of the campaign properly.
20160323 22:52:53< fendrin_> Around 25 turns at max.
20160323 22:53:48< celticminstrel> If it were split in two I feel like it would typically end in <10 turns.
20160323 22:53:57< celticminstrel> The first of the two, that is.
20160323 22:55:08< fendrin_> Yes, you are right. It would have to grow somehow.
20160323 22:57:13< celticminstrel> Not sure how.
20160323 22:58:22< fendrin_> The map can be larger. More villages, the novice shaman spread. Not 4 of them but only one or two.
20160323 22:58:40< fendrin_> Some scripted encounters.
20160323 22:59:37< fendrin_> That way you could tell more story about the school and what happened before.
20160323 22:59:47< celticminstrel> Hmmm.
20160323 23:00:04< fendrin_> Some being in real danger.
20160323 23:00:11< celticminstrel> Not entirely sure what you mean by "before", though I can imagine a few possibilities, I suppose...
20160323 23:00:19< celticminstrel> Some of the initiates being in danger?
20160323 23:00:25< fendrin_> yes
20160323 23:01:00< fendrin_> So you need timing and a good approach to get them freed.
20160323 23:01:06< fendrin_> Before they are slain or maybe cooked.
20160323 23:01:10< fendrin_> or burned
20160323 23:01:29< celticminstrel> Burned?
20160323 23:01:45< celticminstrel> I don't recall using any pillagers. >_>
20160323 23:01:45< fendrin_> In a village building that cought fire.
20160323 23:02:25< celticminstrel> Hmm, not sure how that translates to game mechanics...
20160323 23:02:36< celticminstrel> Though again I can imagine a few possibilities.
20160323 23:03:24-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection]
20160323 23:03:35< fendrin_> The last time I worked on a campaign it was a remake of UtBS. 
20160323 23:03:50-!- louis94 [~~louis94@91.178.242.5] has joined #wesnoth-dev
20160323 23:04:17< fendrin_> There the first scenario is also quite long and it is nice to know what awaits you.
20160323 23:05:23< fendrin_> I wanted to split that scenario into two or even three ones.
20160323 23:07:13< celticminstrel> Well, at the very least, I'll think about this discussion (and save it so that I don't forget).
20160323 23:07:31< celticminstrel> Probably not going to change anything soon, unless I get inspired.
20160323 23:07:59< fendrin_> The Khalifate campaign is fun to play.
20160323 23:08:07< fendrin_> Oath of Alligance.
20160323 23:08:09< fendrin_> Or similar
20160323 23:08:42< celticminstrel> I seem to recall hearing it had some sort of weird interpretation of Khalifate though.
20160323 23:08:56< celticminstrel> I haven't tried it myself, but I might do so.
20160323 23:09:01< celticminstrel> At some point.
20160323 23:14:08-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 23:14:59< fendrin_> I have played until scenario 5 or 6 and so far there was nothing offensive about the Khalifate.
20160323 23:15:36< fendrin_> Again, I just dislike the ban of magic and would much more prefer a middle eastern faction with a lot of magic.
20160323 23:16:44< mattsc> celticminstrel: Release notes look good to me in general.  A few minor comments:
20160323 23:17:00< celticminstrel> I don't really mind them not having magic, and besides, the lack of magic in the official MP faction may not mean it's completely unknown to them.
20160323 23:17:13< mattsc> 1. ‘configuring’ being used twice in the first (half) sentence sounds weird
20160323 23:17:36< mattsc> 2. Wasn’t there also something to be done in the [egnine] tag with the data variable?
20160323 23:18:14< mattsc> 3. I think a comment to the effect of “lots of other, less significant changes; check changelog and wiki” would be good
20160323 23:19:17< mattsc> 4. Do you think we should set up a forum post (when the time comes to it) with a link from the release notes saying: if you have problems with updating your AI, let us know there and we’ll be happy to help you or even do it for you?
20160323 23:19:42< mattsc> That’s all.  What I have not done is checking whether anything else should be in the release notes.
20160323 23:20:49< fendrin_> celticminstrel: Yeah, the lore behind them told they do not use magic because their culture banned it.
20160323 23:20:58< celticminstrel> Regarding 2), I could mention it, but I suspect the majority of AIs will not need to do that. 4) sounds fine to me.
20160323 23:21:02< fendrin_> Also it was not said it is banned for religious reasons.
20160323 23:21:14< mattsc> Oh, one more: I think it would be good to give a reason why all this was done.  Not “we just did all this because we have fun making your life harder”
20160323 23:21:21< celticminstrel> fendrin_: In other words, there are mages among them, but they are outcasts from society? :P
20160323 23:21:43< fendrin_> I guess such cultures would just burn them.
20160323 23:22:06< fendrin_> Like Wiches have been burned in Europe a few centuries before.
20160323 23:22:09< celticminstrel> Not necessarily burn.
20160323 23:22:21-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 23:22:27< celticminstrel> But some form of execution is not unlikely.
20160323 23:22:41< fendrin_> Yeah, some got also drowned.
20160323 23:23:34< celticminstrel> In Europe, sure, burning and drowning; but the Khalifate aren't based on Europe.
20160323 23:24:32< fendrin_> The faction is interresting to play.
20160323 23:24:43< fendrin_> I have not seen them used much at the online server. 
20160323 23:24:55< fendrin_> But in singleplayer they are fun.
20160323 23:26:38< mattsc> Ooo, notifications “suddenly” work on OS X!
20160323 23:27:44-!- LovCAPONE [~lovens@modemcable184.138-20-96.mc.videotron.ca] has joined #wesnoth-dev
20160323 23:27:58< fendrin_> celticminstrel: There is a lot error output from your campaign. 
20160323 23:28:57< celticminstrel> Oh?
20160323 23:29:47< fendrin_> http://pastebin.com/NdCcMYgg
20160323 23:30:17-!- louis94 [~~louis94@91.178.242.5] has quit [Quit: Konversation terminated!]
20160323 23:32:33-!- Greg-Boggs [~greg_bogg@173.240.241.83] has quit [Remote host closed the connection]
20160323 23:33:05< celticminstrel> That sounds like it could be the reason why overlays are missing on level 0 units in the recall dialog...
20160323 23:34:34-!- Greg-Boggs [~greg_bogg@173.240.241.83] has joined #wesnoth-dev
20160323 23:34:38< celticminstrel> The new initiate graphics should fix that.
20160323 23:35:54< fendrin_> new initiate?
20160323 23:36:04< celticminstrel> I think the only reason I haven't made a new release was because all the changes were graphical; but there's enough of them to possibly make it worthwhile.
20160323 23:36:18< fendrin_> oh yes
20160323 23:36:21< fendrin_> please do so
20160323 23:36:25< fendrin_> I can test it right now.
20160323 23:36:42< celticminstrel> Yeah, currently it's just a scaled-down shaman; the new graphics is a modded (hatless) shaman.
20160323 23:36:47< celticminstrel> ^are
20160323 23:37:02< fendrin_> nice to have
20160323 23:37:09< fendrin_> they are difficult to distinguish
20160323 23:37:23< celticminstrel> I'll do that in ~10mins
20160323 23:37:28< fendrin_> :-)
20160323 23:38:27< irker768> wesnoth: mattsc wesnoth:master f920a03ebbcc / data/ai/micro_ais/scenarios/patrols.cfg: Patrols MAI test scenario: adapt Lua engine to new syntax https://github.com/wesnoth/wesnoth/commit/f920a03ebbccf5292028aa8b8d003090f8d11e8f
20160323 23:42:01-!- mjs-de [~mjs-de@x4db60af0.dyn.telefonica.de] has quit [Remote host closed the connection]
20160323 23:42:47< irker768> wesnoth: gfgtdf wesnoth:master 2e93592eeb71 / src/map/map.cpp: fix mp random maps https://github.com/wesnoth/wesnoth/commit/2e93592eeb7110184ade7a5b1cf42028dea7f49e
20160323 23:42:49< irker768> wesnoth: gfgtdf wesnoth:master fa1c1f033ce4 / src/ (ai/lua/core.cpp scripting/game_lua_kernel.cpp): fix msvc compilation https://github.com/wesnoth/wesnoth/commit/fa1c1f033ce4b27823db1f0072554055f78db7c1
20160323 23:42:51< irker768> wesnoth: gfgtdf wesnoth:master b017a2b6b2d6 / src/ (mouse_events.cpp scripting/game_lua_kernel.cpp scripting/game_lua_kernel.hpp): Add wesnoth.game_events.on_mouse_move/on_mouse_actions https://github.com/wesnoth/wesnoth/commit/b017a2b6b2d6c862af6fdec62c7be4d4f42a29e8
20160323 23:43:56< irker768> wesnoth: mattsc wesnoth:master c60280392b60 / data/ai/micro_ais/scenarios/protect_unit.cfg: Protect Unit MAI test scenario: adapt Lua engine to new syntax https://github.com/wesnoth/wesnoth/commit/c60280392b60887cb6040e4ac5b9a12a16c9861a
20160323 23:44:37< mattsc> celticminstrel: I know I should know this, but …  Did you change how Formula AIs need to be set up?
20160323 23:45:05< celticminstrel> I didn't touch FormulaAI at all, so... it should be the same...
20160323 23:45:21< celticminstrel> Though I'm not sure if it matches the official documentation.
20160323 23:45:29< mattsc> Hmm, I’m wondering why the FAI lurkers are inactive then …
20160323 23:49:51< mattsc> celticminstrel: remind me, the fai/faiend commands still work in FAI, but not in WFL?
20160323 23:50:06< mattsc> I always want to type WtFL.  ;)
20160323 23:50:45< celticminstrel> mattsc: They still work everywhere, but are deprecated and should be replaced with wfl/wflend.
20160323 23:50:59< celticminstrel> No need to change the file extension though, in my opinion.
20160323 23:51:07< mattsc> Hmm.
20160323 23:51:40< mattsc> The Formula AI lurkers don’t do anything at the moment.  Other than that, the MAIs and the ExpAI are all good, AFAICT
20160323 23:52:05< celticminstrel> Maybe I'll look into it in a bit.
20160323 23:52:46< mattsc> Yeah, same here.
20160323 23:53:04< mattsc> This is Side 6 in test scenario ‘-t lurkers’
20160323 23:54:57< celticminstrel> fendrin_: I just uploaded a new version to the 1.12 addons server.
20160323 23:55:07< fendrin_> Thank you
20160323 23:59:08< fendrin_> celticminstrel: The new initiate is a huge improvement. What else has changed?
--- Log closed Thu Mar 24 00:00:20 2016