--- Log opened Sat Dec 28 00:00:14 2013
20131228 00:03:01-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 00:03:06-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!]
20131228 00:08:29-!- gfgtdf [~chatzilla@e177144059.adsl.alicedsl.de] has quit [Quit: ChatZilla 0.9.90.1 [Firefox 24.0/20130910160258]]
20131228 00:37:53< happygrue> Ivanovic: short answer is that everything is ready except that we are still waiting on unit descriptions from noy
20131228 00:39:52< happygrue> fabi was also working on adding default+khalifate Age of Heroes, which would be nice but not critical I guess
20131228 00:41:23< happygrue> Coffee_irc: I expect some movement in the level 2-3 units.  As you say, they are not as important right now and have had the least exposure/testing just because they arn't seen in regular play
20131228 00:42:43< happygrue> they are, IMO, a bit overpowered for the higher level units, by design.  It's a new faction, they will see more action and interest in testing them if they are not less powerful than other default units (I"m talking in general about the level 2-3 khalifate units here), so while I think they are close I wanted to err on the side of being a bit better rather than being a bit worse, to start off
20131228 00:42:50< Coffee_irc> happygrue: my feedback was based on my earlier attempts at making a khalifate wave for my UMC 2p/3p survival
20131228 00:43:29< Coffee_irc> happygrue: well... IMO only the tineen is, the rest are a bit weak :P
20131228 00:45:29< happygrue> That could work out to be true, but it is also hard to say
20131228 00:45:58< happygrue> they are strange, the 40% forest defense but 60% on hills makes them take some getting used to
20131228 00:46:24< happygrue> and the http://units.wesnoth.org/trunk/mainline/en_US/Muharib.html is a fantastic unit
20131228 00:46:26< Coffee_irc> happygrue: that is not for all units
20131228 00:47:21< happygrue> In what context did you feel the Tineen overpowered?
20131228 00:47:31< happygrue> (not disagreeing, just asking)
20131228 00:47:37< Coffee_irc> happygrue: against undead specifically
20131228 00:47:44< Coffee_irc> it kills them all in day
20131228 00:47:46< AI0867> shadowm: what would your opinion be on changing --config-dir to change the $XDG_CONFIG_HOME rather than $XDG_DATA_HOME and assigning a new option named '--userdata-dir' for that?
20131228 00:47:49< Coffee_irc> including lch
20131228 00:48:02< AI0867> my main issue with this is that the effect of an existing option will change
20131228 00:48:29-!- AI0867_ [~ai@wesnoth/developer/ai0867] has joined #wesnoth-dev
20131228 00:49:16-!- AI0867 [~ai@wesnoth/developer/ai0867] has quit [Quit: restarting leaking tmux]
20131228 00:49:21< Coffee_irc> happygrue: it kills the ghost line, skeleton, ghoul line, WC line, lich, and can really only be taken down by DA to necro lines with retaliation damage
20131228 00:49:30-!- AI0867_ is now known as AI0867
20131228 00:50:22< AI0867> Coffee_irc: even the ghoul? it has no weaknesses
20131228 00:50:49< shadowm> AI0867: Changelog, release notes, man page, and I gather I'd only need to update my wesnoth-defaults script accordingly.
20131228 00:51:11< Coffee_irc> AI0867: ghoul has only 10% def to fire
20131228 00:51:28< Coffee_irc> and does not really take out the tineen
20131228 00:51:51< happygrue> yep, I agree, though it is not as effective as the the Necromancer on offence
20131228 00:51:57< Coffee_irc> IMO the quatif-al-nar can be taken down due to lower HP, but the tineen is a problem for UD
20131228 00:52:02< shadowm> AI0867: I assume it'll be possible to set the --config-dir and --userdata-dir to the same value and things will work as expected? (i.e. the same way they currently do with a compiled-in default)
20131228 00:52:04< happygrue> that could be true
20131228 00:52:22< happygrue> the lower level units are not so big a problem, I agree.  At level 3 if you have several of them it could be very hard
20131228 00:52:29< AI0867> shadowm: for non X-based things, that's what happens by default
20131228 00:52:47< AI0867> also if neither $XDG_CONFIG_HOME nor $HOME is set
20131228 00:53:00< happygrue> then again, we don't really pretend everything is balanced up there with level 3s slugging it out against each other
20131228 00:53:18< Coffee_irc> happygrue: yes, the other machup comparable is drakes vs lich
20131228 00:53:19< happygrue> as it's very different, and seen only in UMC settings, with other factors going on
20131228 00:53:24< happygrue> right
20131228 00:53:33< happygrue> drakes vs UD has some similar things going on
20131228 00:53:38< Coffee_irc> the difference is that the saurians can take out the lich at night and thrasher has impact
20131228 00:54:13< happygrue> 2 necro can easily take out a Tineen at night too... even in a village
20131228 00:54:26< happygrue> which is not true of two Tineen against a necromancer in a village in the day
20131228 00:54:36< happygrue> it's possible, but not as likely
20131228 00:54:55< happygrue> which doesn't mean it's perfect.  It's something to watch for sure.
20131228 00:55:10< happygrue> I just increased the level 1 hp for them a few
20131228 00:55:21< happygrue> because testing was showing that it wasn't good enough vs UD
20131228 00:55:52< happygrue> but ... there is always a push pull.  Frankly stuff with default has been at a standstill and a chunk of the MP community is rather annoyed by that.
20131228 00:56:03< Coffee_irc> happygrue: yeah, at level 1 the low hp means sometimes no retaliation
20131228 00:56:04< Coffee_irc> just death
20131228 00:57:57< Coffee_irc> happygrue: in neutral time 2 necromancers would expect to do about 51 damage to the tineen
20131228 00:58:06< Coffee_irc> and it has 55 hp
20131228 00:58:13< Coffee_irc> just something to think about
20131228 00:58:43< Coffee_irc> because when I added it to a wave in my UMC survival it meant death for UD
20131228 00:58:44-!- Paulomat4 [55b7394d@gateway/web/freenode/ip.85.183.57.77] has quit [Ping timeout: 272 seconds]
20131228 01:00:46< Coffee_irc> only way I could dee to take it down was with backstab nightgaunt or shadow
20131228 01:00:48< Coffee_irc> *see
20131228 01:01:19< mattsc> shadowm: I confirmed that bug #18921 and the double shift-k kill segfault are the same bug.  In fact, you can cause a segfault by a mix of debug actions: create a unit with shift-c, then kill it with shift-k while its recruit animation is still playing.
20131228 01:01:21< happygrue> the other thing that could be done is to hold back one of the attacks a bit.  But I would agree that maybe tweaking the hp could help.  I would love to get more feedback though.
20131228 01:01:57< mattsc> shadowm: I also figured out what is happening and where the segfault happens, but I might be in over my head as for fixing it...
20131228 01:02:17< happygrue> the other thing about survivals is that how much you can run away changes things up too.  Though UD are the worst at doing that, hence part of the problem I guess
20131228 01:02:37< mattsc> It's in l.1391 of src/unit_animation.cpp: https://github.com/wesnoth/wesnoth-old/blob/master/src/unit_animation.cpp#L1391
20131228 01:02:41< Coffee_irc> happygrue: yeah, it is only one area of test
20131228 01:02:56< happygrue> thanks for the feedback though, it is useful
20131228 01:03:15< Coffee_irc> happygrue: what about the halid RIPLIB on its blade attack?
20131228 01:03:19< Coffee_irc> *khalid
20131228 01:03:45< happygrue> they get a strike each time
20131228 01:04:16< Coffee_irc> happygrue: personally I don't see why the attack isn't 12x4 and avoid the RIPLIB issue
20131228 01:04:20< mattsc> shadowm: anim->my_unit exists, but anim->my_unit->get_animation() does not (or points to a dangling pointer or whatever it is you said)
20131228 01:04:40< happygrue> previous unit does the same thing.
20131228 01:04:43< Coffee_irc> happygrue: it is a level 4 and it is otherwise pretty weak IMO (even with marksman)
20131228 01:04:53< happygrue> but yes, we could look at that
20131228 01:05:09< mattsc> Ugh, I thought Coffee_irc and happygrue were at a break point there.  Sorry, guys.  I'll pick this up later...  :P
20131228 01:05:31< happygrue> Coffee_irc: come talk in #wesnoth-khalifate
20131228 01:05:35< happygrue> mattsc: sorry, carry on ;)
20131228 01:05:51< bumbadadabum> did you actually find any mistakes with my animationwml?
20131228 01:06:17< mattsc> bumbadadabum: is that a question for me?
20131228 01:06:31< bumbadadabum> not per se
20131228 01:07:21< happygrue> bumbadadabum: no, but I didn't check every unit
20131228 01:07:27< happygrue> seemed good though, thanks
20131228 01:07:28< bumbadadabum> same
20131228 01:07:35< mattsc> bumbadadabum: no, this bug has nothing to do with that.  It's a segfault that happens in debug mode when you start an animation for a unit, interrupt it by starting another one before the first is done.
20131228 01:07:44< bumbadadabum> I know that
20131228 01:07:49< bumbadadabum> but I just thought about it
20131228 01:09:41< mattsc> What happens is that the engine plays out the second animation, then tries to go back to the first using the same memory address (which does not exist any more) and that crashes the game.
20131228 01:10:52< mattsc> But as I said, I don't know enough C++ to know what to do about this without putting in a lot of effort.  So I'm hoping somebody else with more experience can pick it up from here.  :P
20131228 01:37:39< shadowm> mattsc: The most obvious thing I can think of is rewriting everything to use shared pointers to units with some kind of reference counting.
20131228 01:37:47< shadowm> It's obvious but decidedly non-trivial.
20131228 01:38:42< shadowm> An alternative would be to make sure those commands stop all playing animations for those units before destroying them.
20131228 01:39:30< shadowm> And then you'd have to simply assume nobody else is holding pointers to the destroyed ynits.
20131228 01:43:40< AI0867> shadowm: so the point is that unit_animator::wait_for_end() (or whatever calls it) is not reentrant?
20131228 01:44:44< shadowm> Uh. Dunno, I wasn't paying attention to the call stack. Is this actually a reentrancy issue or just a matter of functions not being called to dispose of pointers that are about to be invalidated?
20131228 01:44:57< AI0867> that sounds like an important question
20131228 01:45:20< AI0867> I think the play_slice may cause it to reenter
20131228 01:45:53< shadowm> I wonder how [kill] animate=no deals with animations.
20131228 01:45:53< AI0867> and if it's just reentrancy, we can store a copy of the pointer in a local var
20131228 01:46:13< AI0867> before it gets overwritten by the next anim
20131228 01:47:10< AI0867> though I'm utterly unfamiliar with this code
20131228 01:47:21< AI0867> so I could be way off
20131228 01:50:38< shadowm> Same here.
20131228 01:54:23-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Quit: Turuk]
20131228 01:55:33< AI0867> shadowm: sounds to me like 9d4c7e21c fixes the wrong thing
20131228 01:55:44< AI0867> going off the commit message
20131228 01:55:44< shadowm> !log 9d4c7e21c
20131228 01:55:45< shikadibot> shadowm: Revision 9d4c7e21c830 (Ignacio R. Morelle) on Thu Dec 26 21:51:44 2013:
20131228 01:55:49< shikadibot> shadowm: test suite: Create a hotkey_manager instance for the MP connect test
20131228 01:55:52< shikadibot> shadowm:  
20131228 01:55:55< shikadibot> shadowm: Otherwise we fail an assertion check later because the implementation of
20131228 01:55:58< shikadibot> shadowm: (+2 discarded lines)
20131228 01:56:01< shikadibot> shadowm: Web interface URL: https://github.com/wesnoth/wesnoth-old/commit/9d4c7e21c830
20131228 01:56:13< shadowm> AI0867: Care to elaborate?
20131228 01:56:24< AI0867> " Otherwise we fail an assertion check later because the implementation of
20131228 01:56:24< AI0867>     hotkey::get_hotkey_null() tries to access the last item of an empty
20131228 01:56:25< AI0867>     container."
20131228 01:56:37< AI0867> oh, hotkey_manager
20131228 01:56:41< shadowm> Getting fabi to fix stuff would take longer.
20131228 01:56:45< AI0867> damn it, I need to actually read this stuff
20131228 01:57:08< shadowm> It's kind of a moot point since the test suite is utterly broken by unrelated crap anyway.
20131228 01:57:28< AI0867> true
20131228 01:57:37< AI0867> but at least it compiles nowadays =P
20131228 01:57:40< shadowm> It's also been broken for far too long and nobody here seems to appreciate automated tests enough to maintain t.
20131228 01:57:43< shadowm> it.
20131228 01:57:59-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 01:58:10< shadowm> The overall situation has left me hopeless.
20131228 02:00:00< shadowm> I know it's fun to look at Wesnoth as a weekend hobby kind of thing, but look at the line count. It's so massive it naturally tends to collapse onto itself like a blackhole. Tell that to the committers, though.
20131228 02:01:43< shadowm> Of course the test suite is the least obvious target due to a general lack of policies, so it gets to be closer to the event horizon as a result.
20131228 02:03:23< shadowm> And I'm not really bored enough to go and run bisect on the test suite. It's far less frustrating to maintain my own user-made add-ons.
20131228 02:06:55< AI0867> shadowm: companion of --userdata-dir, --config-dir or --userconfig-dir?
20131228 02:07:06< AI0867> s/,/:
20131228 02:07:45< mattsc> shadowm, AI0867: well ...  That doesn't sounds like anything I am going to do ...  :(
20131228 02:09:43< mattsc> I'm talking about the animation bug, not the test suite; but it would apply to that as well.
20131228 02:12:03< AI0867> mattsc: well, I'm a bit busy with something else right now, but afterwards, if I can reliably reproduce this, I can try some quick fixes
20131228 02:13:55< mattsc> AI0867: cool.  Reproduction is very simple: go into debug mode and (optionally) set acceleration to 0.25 (that's just to see more easily what's going on).  Then hover over a unit and hit shift-k twice.
20131228 02:14:46< mattsc> The only important bit is that you need to hit shift-k the second time before the animation from the first one is over.
20131228 02:15:22< mattsc> s/hit/press or push or something ...  You don't actually need a hammer for this one.
20131228 02:17:17< AI0867> will the reverse side of an axe do?
20131228 02:18:09< mattsc> If you can consistently hit the k key with it, it'll do just fine.
20131228 02:20:05< AI0867> I only have splitting axes here
20131228 02:20:22< AI0867> I'm not sure I can miss the rest of the keyboard
20131228 02:21:13< mattsc> :)
20131228 02:24:34-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 02:24:48< shadowm> AI0867: What is [going to be?] companion to those?
20131228 02:25:40< AI0867> well, there's --data-dir
20131228 02:25:46< AI0867> there's going to be --userdata-dir
20131228 02:26:00< AI0867> so, --config-dir or --userconfig-dir along with those?
20131228 02:26:10< shadowm> --config-dir already exists, doesn't it?
20131228 02:26:30< shadowm> Sets the path of the user config directory yadda yadda, says --help.
20131228 02:27:45< shadowm> AI0867: If you want to phase it out in a more gradual fashion, I guess you could retain the current behavior for --config-dir and implement a separate --userconfig-dir switch (and --userconfig-path).
20131228 02:27:54< shadowm> Still I'm unsure what's it you are asking about.
20131228 02:27:56< AI0867> yeah, which is misleading now that we have $XDG_CONFIG_HOME and $XDG_DATA_HOME
20131228 02:28:48< shadowm> IIRC --config-dir right now doesn't work correctly when using the XDG scheme.
20131228 02:29:38< shadowm> And the XDG scheme IIRC is only considered when there's no hardcoded preferences dir path.
20131228 02:30:01< shadowm> I have the prefsdir option set in my scons build configuration, so...
20131228 02:30:20-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Quit: Turuk]
20131228 02:32:25-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Off to save the world!]
20131228 02:37:58< AI0867> shadowm: it overrides both, yes
20131228 02:38:05-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 02:38:35< shadowm> I mean that --config-dir (IIRC) with XDG results in only one of the user resource directories being reassigned.
20131228 02:39:51< shadowm> Compare lines 8 and 9 in http://pastebin.com/XEdfdmDj (NOTE: this is from before your changes).
20131228 02:40:01< shadowm> *8 and 9 and 10
20131228 02:40:14< shadowm> Is that considered a feature or a bug?
20131228 02:40:35< shadowm> Reminds me that this is why I never committed the wesnoth-defaults script.
20131228 02:40:41-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 02:42:14< AI0867> shadowm: I consider it a bug, which is why I'm doing what I'm doing now
20131228 02:42:31< shadowm> Okay, good.
20131228 02:42:38< AI0867> I'll go with --userconfig and possibly have the current --config do both 
20131228 02:42:58< shadowm> --config-dir and --config-path should be deprecated, though. 
20131228 02:43:24< shadowm> Having both --userconfig-* and --config-* could prove confusing in the long term for obvious reasons.
20131228 02:45:03< shadowm> Perhaps make a separate --all-user-dirs switch, dunno.
20131228 02:47:49< irker305> wesnoth: Turuk wesnoth-old:master e52a29009bcc / data/core/units/khalifate/ (Khalid.cfg Rasikh.cfg Shuja.cfg Tineen.cfg): Balancing for Khalifate units http://git.io/tCIARw
20131228 02:48:56< happygrue> Ivanovic: I haven't had an update from noy in a number of days, so I don't know what is going on with the unit descriptions.
20131228 02:50:09< happygrue> we talked out a few last tweaks (above commit), most units have some small description.  We can hastily just write "something" and put it in for the units that are still lacking
20131228 02:50:43< happygrue> but I have been under the impression that he wanted to do it and that it would be done, so have been waiting on that.  I'm not sure how long we should wait?
20131228 02:52:47< happygrue> but from my point of view, everything is working and playable (those missing unit descriptions is the only problem I'm aware of) and ready for release.
20131228 03:03:01-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: Ik ga weg]
20131228 03:18:37-!- shadowm [~ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 272 seconds]
20131228 03:19:54-!- shadowm [~ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20131228 03:39:52< irker305> wesnoth: mattsc wesnoth-old:master e211f7741401 / src/unit_animation.cpp: Remove trailing whitespaces http://git.io/dL8f7Q
20131228 03:41:20-!- happygrue [~happygrue@wesnoth/developer/wintermute] has quit [Ping timeout: 252 seconds]
20131228 03:43:38-!- Ivanovic_ [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev
20131228 03:44:18-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Off to save the world!]
20131228 03:47:23-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 252 seconds]
20131228 03:47:41-!- Ivanovic_ is now known as Ivanovic
20131228 03:58:09-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 272 seconds]
20131228 04:00:34-!- shadowm_desktop [~ignacio@190.91.64.72] has joined #wesnoth-dev
20131228 04:00:47-!- shadowm_desktop is now known as Guest64127
20131228 04:02:59-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 04:08:08-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Off to save the world!]
20131228 04:42:10-!- Guest64127 is now known as shadowm_desktop
20131228 04:42:10-!- shadowm_desktop [~ignacio@190.91.64.72] has quit [Changing host]
20131228 04:42:10-!- shadowm_desktop [~ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20131228 04:56:16< irker305> wesnoth: David Mikos wesnoth-old:master d874f57aaabe / changelog src/help.cpp: Fix bug #21339 with help unit menu entry links http://git.io/aPEaYA
20131228 05:27:24-!- Gambit [~derek@wesnoth/developer/grickit] has quit [Remote host closed the connection]
20131228 05:40:04-!- knotwork__ [~markm@142.68.128.64] has joined #wesnoth-dev
20131228 05:40:04-!- knotwork__ [~markm@142.68.128.64] has quit [Changing host]
20131228 05:40:04-!- knotwork__ [~markm@unaffiliated/knotwork] has joined #wesnoth-dev
20131228 05:43:02-!- knotwork_ [~markm@unaffiliated/knotwork] has quit [Ping timeout: 240 seconds]
20131228 05:49:52-!- mattsc [~mattsc@154.20.32.246] has quit [Quit: Ciao]
20131228 05:55:58-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Ping timeout: 272 seconds]
20131228 05:58:15-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 07:10:14-!- You're now known as lobby
20131228 07:21:12< fabi> hi Coffee_irc
20131228 07:21:24< Coffee_irc> hi fabi
20131228 07:22:32< Coffee_irc> how's it going with the new editor and other features?
20131228 07:23:29< Coffee_irc> I don't seem to be able to use the unit tool in the editor with the latest dev version
20131228 07:23:53< Coffee_irc> but I can use the item tool
20131228 07:24:47< fabi> Crendgrim: creaet a side
20131228 07:24:49< fabi> first
20131228 07:24:56< fabi> units need sides
20131228 07:25:59< Coffee_irc> how do I create a side?
20131228 07:26:26< fabi> side menu --> add side
20131228 07:27:05< Coffee_irc> oh I see
20131228 07:27:23< fabi> this will also enable the village tool
20131228 07:27:38< fabi> which needs a side as well
20131228 07:28:47-!- shadowm_desktop [~ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 272 seconds]
20131228 07:29:27< Coffee_irc> fabi: hmm, the loyal tick box doesn't work
20131228 07:30:06-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20131228 07:31:09< fabi> Coffee_irc: yes
20131228 07:32:15-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Ping timeout: 240 seconds]
20131228 07:33:30< Coffee_irc> fabi: I can see this as developing into a useful feature, especially for new UMC authors
20131228 07:40:02-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 07:43:46< Coffee_irc> I am trying to play in 1.10 and am clicking the wrong buttons for the units :P
20131228 07:52:06< fabi> hihi
20131228 07:52:43< fabi> Coffee_irc: Testing the new area feature would be nice.
20131228 07:53:34< fabi> I have worked on that recently and it is some kind of rushed in carelessly to get it in before the string freeze.
20131228 07:54:23< Coffee_irc> fabi: area feature?
20131228 07:54:39< fabi> Coffee_irc: Like the sides, it has it's own menu button.
20131228 07:54:44< fabi> "Areas"
20131228 07:55:01< Coffee_irc> what is it supposed to do?
20131228 07:55:10< fabi> You can define areas.
20131228 07:55:17< Coffee_irc> like for tod?
20131228 07:55:25< fabi> Then give them a local time of day schedule and a local starting time.
20131228 07:55:55< fabi> You can (and that is the best feature of the new editor) give an id to the area.
20131228 07:56:08< fabi> This id can then be used in SLF when event coding.
20131228 07:56:35< fabi> Or tuning the AI
20131228 07:58:34-!- ettin [~jorda@109.74.198.112] has joined #wesnoth-dev
20131228 07:59:17< Coffee_irc> fabi: how do you select an area?
20131228 07:59:23< Coffee_irc> it is non-obvious
20131228 07:59:45< fabi> You mean how you define which hex fields belong to it?
20131228 07:59:53< Coffee_irc> yes
20131228 08:00:04< fabi> Use the selection tool to select hex fields.
20131228 08:00:17< fabi> Then Area->Save 
20131228 08:00:20< Coffee_irc> fabi: hmm
20131228 08:00:27< Coffee_irc> IMO it needs some work :P
20131228 08:00:37< Coffee_irc> how can you tell what area you set before?
20131228 08:00:45< fabi> Select it
20131228 08:00:47< fabi> in the menu
20131228 08:02:22< Coffee_irc> fabi: IMO what would make more sense is that to create an 'area' you have to select some hexes first, and give it a name as part of the creation
20131228 08:03:21< Coffee_irc> fabi: in trying to save selection to area without one previously defined it segfaults for me
20131228 08:03:34< fabi> let me check
20131228 08:03:54< Coffee_irc> I added a new area and removed it
20131228 08:04:11< Coffee_irc> the menu box remains to save selection to area, and it segfaults
20131228 08:04:23< fabi> Ah yes.
20131228 08:04:25< fabi> I see.
20131228 08:05:19< Coffee_irc> fabi: this only happens if you actually do select something and save it to an area first
20131228 08:05:24< Coffee_irc> and delete all the areas
20131228 08:05:29< Coffee_irc> or the menu entry disappears
20131228 08:06:35< fabi> no
20131228 08:06:42< Coffee_irc> fabi: yes :)
20131228 08:06:46< fabi> Just start the editor.
20131228 08:07:01< fabi> Select some hexes, without touching the areas menu first.
20131228 08:07:12< Coffee_irc> ok
20131228 08:07:15< fabi> Then go to the menu and you will find the save option.
20131228 08:07:23< Coffee_irc> ah
20131228 08:07:37< fabi> Which then is determined to segfault because we can only save it into void.
20131228 08:08:36< Coffee_irc> fabi: I would suggest to bring up the rename box for the area as you create it
20131228 08:09:27< fabi> Yes, I have thought about that.
20131228 08:10:03< fabi> Still, the area needs no id if you do not want to address it in SLF or remove it by event wml later.
20131228 08:10:25< fabi> Thus id field is not mandatory in [time_area]
20131228 08:10:52< Coffee_irc> fabi: to me it is non-obcious how to work it
20131228 08:10:56< Coffee_irc> *obvious
20131228 08:11:28< Coffee_irc> I'd like to suggest a save selection to new area menu option?
20131228 08:11:50< fabi> Add new Area takes your current selection.
20131228 08:11:52< Coffee_irc> but you could test it out on others as well
20131228 08:14:09< fabi> The editor needs its own help section.
20131228 08:14:19< fabi> I have asked in the writers forum.
20131228 08:14:36< fabi> But there was only one person reacting to my request.
20131228 08:14:56< Coffee_irc> fabi: lol
20131228 08:15:04< fabi> He produced useless text that showed that he has never used the editor before (not even the old one).
20131228 08:15:08< Coffee_irc> you're getting others to write a help file for you :P
20131228 08:15:50< fabi> Yes
20131228 08:15:56< fabi> That is common.
20131228 08:16:31< Coffee_irc> fabi: maybe you could have an always accessible menu entry for "areas" and "side" each?
20131228 08:16:35< fabi> The author of a tool himself is too much Betriebsblind to produce a useful manual.
20131228 08:16:36< Coffee_irc> that goes to the help
20131228 08:16:53< fabi> Yes, that is possible.
20131228 08:17:10< Coffee_irc> Areas -> what does this do?
20131228 08:17:11-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Ping timeout: 272 seconds]
20131228 08:17:49< Coffee_irc> or something that is singly translatable I would suggest
20131228 08:18:00< Coffee_irc> in case you add features in the future
20131228 08:18:25-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 08:18:44< Coffee_irc> or even a help menu of its own
20131228 08:21:01< Coffee_irc> fabi: another editor thing, and I can file bug reports if you'd like, is the UMC music shows in the playlist, but does not play like default music for me
20131228 08:21:34< fabi> do you see any errors on the commandline?
20131228 08:21:42< Coffee_irc> no
20131228 08:21:51< fabi> hmmm
20131228 08:21:55< Coffee_irc> funny thing is that the one's higher up on the UMC music list do play
20131228 08:22:00< Coffee_irc> but most of them don't
20131228 08:22:10< Coffee_irc> I have downloaded all the UMC music packs
20131228 08:22:35< fabi> can you save and check the output?
20131228 08:24:03< Coffee_irc> fabi: they are missing the file extensions
20131228 08:24:11< fabi> uh
20131228 08:24:35< Coffee_irc> yeah, well this happens with new features :P
20131228 08:24:59< fabi> yes
20131228 08:25:47< fabi> Please fill a bug report. 
20131228 08:25:57< Coffee_irc> will do
20131228 08:26:03< fabi> Currently it is more important to get everything string related into.
20131228 08:26:09< fabi> This one can wait.
20131228 08:26:36< Coffee_irc> might take me bit to get to the report, but I won't forget it
20131228 08:26:52< Coffee_irc> have to get going actually
20131228 08:28:22< fabi> Coffee_irc: bye :-)
20131228 08:32:07-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev
20131228 08:36:39-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 08:42:52-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20131228 08:44:03-!- wesbot changed the topic of #wesnoth-dev to: 202 bugs, 343 feature requests, 28 patches | Logs: http://irclogs.wesnoth.org | Don't paste on IRC! Use a pastebin: http://pastebin.com | http://imagebin.org
20131228 08:49:11-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20131228 08:52:39-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has quit [Ping timeout: 240 seconds]
20131228 08:59:04-!- Turuk [~Turuk@74-139-73-227.dhcp.insightbb.com] has joined #wesnoth-dev
20131228 09:19:58-!- ettin [~jorda@109.74.198.112] has quit [Quit: leaving]
20131228 09:53:26< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 5fdd9d12a586 / src/mp_options.cpp: Show the correct value in custom sliders' labels. http://git.io/bK9n-g
20131228 09:53:28< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 216b84fbe702 / src/ (mp_options.cpp mp_options.hpp): A simpler and unified interface for display classe ctors. http://git.io/NUBzow
20131228 09:53:30< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master baf198185b7d / src/ (mp_options.cpp mp_options.hpp multiplayer_configure.cpp): New custom option type: combobox! http://git.io/H8_dJw
20131228 09:53:53< lipkab> s/classe/class
20131228 10:14:15-!- happygrue [~happygrue@wesnoth/developer/wintermute] has joined #wesnoth-dev
20131228 10:16:55< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 26ac77e95d49 / src/mp_options.cpp: Re-enable tooltips for custom options. http://git.io/7gZIgg
20131228 10:26:38< fabi> lipkab: hi
20131228 10:26:53< fabi> lipkab: the filter text line in mp dialog is yours?
20131228 10:27:02-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 246 seconds]
20131228 10:27:48< fabi> Damn, I hate commiters who flee by quiting directly after their crimes.
20131228 10:37:19< fabi> hi happygrue
20131228 10:37:19-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Read error: Connection reset by peer]
20131228 10:37:36< fabi> happygrue: AoE + Khalifate is ready to commit.
20131228 10:38:20-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20131228 10:39:18< irker305> wesnoth: fendrin wesnoth-old:master c44e5fe776c9 / data/multiplayer/ (eras.cfg factions/khalifate-aoh.cfg): Age of Heroes + Khalifate http://git.io/dCajSA
20131228 10:39:20< irker305> wesnoth: fendrin wesnoth-old:master e30dd2574a47 / src/editor/map/map_context.hpp: Method to remove the last side from a map context. http://git.io/RxxNOA
20131228 10:39:22< irker305> wesnoth: fendrin wesnoth-old:master a39c35462826 / src/hotkey/hotkey_command.cpp: Added the remove side hotkey to the hotkey description list. http://git.io/01bUQw
20131228 10:39:24< irker305> wesnoth: fendrin wesnoth-old:master c0f8febf46b6 / data/themes/editor.cfg src/editor/editor_controller.cpp: Remove side from map context feature put into use. http://git.io/rUy_Uw
20131228 10:39:39< fabi> happygrue: http://git.io/dCajSA
20131228 10:48:03-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Ping timeout: 245 seconds]
20131228 10:57:34-!- mjs-de [~mjs-de@f049006193.adsl.alicedsl.de] has joined #wesnoth-dev
20131228 11:12:58-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 11:19:36-!- Kostic [~marko@85.202.113.53] has joined #wesnoth-dev
20131228 11:20:06< fabi> hi lipkab
20131228 11:20:20< fabi> lipkab: Is the filter textfield in mp screen your work?
20131228 11:20:24< lipkab> Hi fabi.
20131228 11:20:27< fabi> creat screen
20131228 11:20:28< lipkab> Yes.
20131228 11:21:02< AI0867> fabi: you may want to rename khalifate.cfg to khalifate-default.cfg or something
20131228 11:21:10< fabi> lipkab: Y and V are not distinguishable.
20131228 11:21:37< fabi> AI0867: indeed
20131228 11:22:00< fabi> lipkab: It seems the field is not high enough to hold every possible char.
20131228 11:22:09< lipkab> Indeed.
20131228 11:23:37< lipkab> Hm, apparently, there's no way to change the font size within a textbox?
20131228 11:24:10< fabi> lipkab: It is easy with theme wml which is based on gui1.
20131228 11:25:03< fabi> AI0867: You spotted a hotkey related bug you want to get fixed?
20131228 11:26:39< AI0867> there was the unit description not being a functional default
20131228 11:26:41< AI0867> is that fixed?
20131228 11:28:02< fabi> sorry, I don't get it. please tell me more.
20131228 11:28:34< lipkab> fabi: Uh, how? widgets/textbox.cpp:252 tells me that the font size is always font_size, and font_size is a global constant.
20131228 11:29:14< lipkab> Anyways, that should be easy enough to fix.
20131228 11:30:35< fabi> lipkab: you are right, theme wml does not support textboxes.
20131228 11:35:18< fabi> AI0867: Ah yes. Now I get it. 
20131228 11:36:02< fabi> AI0867: It is not fixed yet. But should be a very easy wml only fix.
20131228 11:42:08 * fabi wonders if double clicking on a preferences section did close the dialog earlier.
20131228 11:44:59< irker305> wesnoth: fendrin wesnoth-old:master 3119d70e20da / data/core/hotkeys.cfg: Change the default hotkey binding for the paint tool action. http://git.io/z28TKg
20131228 11:45:03< fabi> AI0867: ^
20131228 11:48:57< AI0867> I need to finish so befre I can tet that
20131228 11:49:13< AI0867> oh wow, something ate my keystrokes
20131228 11:51:29< fabi> lipkab: Entering more than a certain amount of characters into the filter textbox leads to a segfault.
20131228 11:52:03< lipkab> Ah?
20131228 11:52:15< lipkab> That's funny.
20131228 11:52:23< lipkab> I'll look into it.
20131228 11:53:33< fabi> lipkab: Not always. Usually it just leads to a long freeze of the gui.
20131228 11:57:00< fabi> lipkab: The slider which filters for the number of players is a nice addition. Maybe we can rename the mp scenarios from "xp -- some_name" to "some_name -- xp" now.
20131228 11:58:01< fabi> lipkab: Maybe we can discuss if the slider should start with 2 instead of 1. I doubt that there will be much one player mp scenarios ever.
20131228 12:00:05< lipkab> fabi: Yeah, one player mp scenarios don't make much sense. On the other hand, one new feature is that you can start sp campaigns from the mp interface.
20131228 12:01:22< lipkab> I think the maps shouldn't be renamed - the two player ones are used the most so it's good that they're on the front.
20131228 12:01:30< fabi> lipkab: Right. But those have their own "Game type:" section, right? Filtering for the amount of players in that section is not very useful at all. 
20131228 12:01:40< lipkab> True.
20131228 12:04:08-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginm.net] has joined #wesnoth-dev
20131228 12:10:49-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 252 seconds]
20131228 12:13:30-!- Gambit [~derek@wesnoth/developer/grickit] has joined #wesnoth-dev
20131228 12:14:41< irker305> wesnoth: fendrin wesnoth-old:master c2c8fab4ad5a / data/multiplayer/ (eras.cfg factions/khalifate-default.cfg factions/khalifate.cfg): Renamed the file holding the khalifte default era factiondefinitions. http://git.io/eNc4Hw
20131228 12:16:49-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginm.net] has quit [Remote host closed the connection]
20131228 12:17:54< irker305> wesnoth: fendrin wesnoth-old:master 4f15c07d7b63 / data/multiplayer/eras.cfg: Fix the misspelled id of the AoH Khalifate era. http://git.io/sZB8ew
20131228 12:28:23-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 12:30:47< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 34a8bab55fd1 / src/widgets/ (textbox.cpp textbox.hpp): Configurable font size for GUI1 textboxes. http://git.io/pe8nwQ
20131228 12:30:49< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 6c933558d8d2 / src/multiplayer_create.cpp: Use small font in the filter textbox at the MP create screen. http://git.io/UjI0Gw
20131228 12:30:51< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master c9a176e97087 / src/ (multiplayer_create.cpp multiplayer_create_engine.cpp): Don't filter for 1 player levels on the MP create screen. http://git.io/j-io1g
20131228 12:30:53< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master d5715215d8dd / src/multiplayer_create_engine.cpp: Add an extra check to prevent a possible overflow. http://git.io/R8cylQ
20131228 12:31:12< lipkab> fabi: ^ Fixed everything.
20131228 12:31:27< fabi> lipkab: Thank you very much :-) 
20131228 12:31:37< lipkab> You are very welcome.
20131228 12:32:01-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Client Quit]
20131228 12:51:14-!- travis-ci [~travis-ci@ec2-54-235-33-212.compute-1.amazonaws.com] has joined #wesnoth-dev
20131228 12:51:14< travis-ci> [travis-ci] wesnoth/wesnoth-old#1311 (master - c2c8fab : fendrin): The build has errored.
20131228 12:51:14< travis-ci> [travis-ci] Build details : http://travis-ci.org/wesnoth/wesnoth-old/builds/16068464
20131228 12:51:14-!- travis-ci [~travis-ci@ec2-54-235-33-212.compute-1.amazonaws.com] has left #wesnoth-dev []
20131228 13:04:58< fabi> uh
20131228 13:07:01-!- Coffee_irc [~david@58.145.148.37] has quit [Remote host closed the connection]
20131228 13:07:31-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 13:09:41< AI0867> fabi: I've restarted it, it's probably nothing
20131228 13:15:12< irker305> wesnoth: Alexander van Gessel wesnoth-old:master bf8b05ff3126 / changelog src/commandline_options.cpp src/commandline_options.hpp src/game.cpp: Split --config-{dir,path} into --user{config,data}-{dir,path} http://git.io/N6pztw
20131228 13:15:14< irker305> wesnoth: Alexander van Gessel wesnoth-old:master 63cf0fe6e3d4 / RELEASE_NOTES doc/man/wesnoth.6: documentation about --user{config,data}-{dir,path} http://git.io/7PHq0g
20131228 13:15:16< irker305> wesnoth: Alexander van Gessel wesnoth-old:master 8219e4be8fd1 / / (4 files in 2 dirs): Add --data-path option http://git.io/m2soew
20131228 13:15:18< irker305> wesnoth: Alexander van Gessel wesnoth-old:master 2d1b61b3254a / / (23 files in 11 dirs): Merge branch 'master' of github.com:wesnoth/wesnoth-old http://git.io/Xkyh5A
20131228 13:19:25< fabi> AI0867: Well, the commit in question was WML only. Thus it must be nothing.
20131228 13:23:45< irker305> wesnoth: fendrin wesnoth-old:master 17ad07a8c457 / / (5 files in 3 dirs): Replace the input fields for id and name of a side in the editor. http://git.io/0nDqug
20131228 13:24:14-!- Coffee_irc [~david@58.145.148.37] has quit [Remote host closed the connection]
20131228 13:24:23< AI0867> fabi: confirmed that 'd' works on a fresh configuration
20131228 13:24:29-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 13:25:10< fabi> AI0867: Maybe a mechanism that detects impossible hotkey defaults would make sense.
20131228 13:26:39< fabi> Okay, two more strings and I am ready to go into freeze.
20131228 13:28:04< irker305> wesnoth: fendrin wesnoth-old:master d737f2ea099e / data/gui/default/window/editor_edit_scenario.cfg: Replace "ID" with "Identifier" which is more friendly to the eye. http://git.io/XOIZOA
20131228 13:37:19-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Read error: Connection reset by peer]
20131228 13:41:11-!- stikonas [~gentoo@5.20.200.50] has joined #wesnoth-dev
20131228 13:41:11-!- stikonas [~gentoo@5.20.200.50] has quit [Changing host]
20131228 13:41:11-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20131228 13:47:00-!- Kostic [~marko@85.202.113.53] has quit [Quit: Kostic]
20131228 13:48:12-!- TooLmaN [~TooLmaN@mail.thomsonplastics.com] has joined #wesnoth-dev
20131228 13:57:46-!- Coffee_irc [~david@58.145.148.37] has quit [Remote host closed the connection]
20131228 13:58:11-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 13:58:52-!- travis-ci [~travis-ci@ec2-54-221-177-157.compute-1.amazonaws.com] has joined #wesnoth-dev
20131228 13:58:52< travis-ci> [travis-ci] wesnoth/wesnoth-old#1316 (master - d737f2e : fendrin): The build has errored.
20131228 13:58:52< travis-ci> [travis-ci] Build details : http://travis-ci.org/wesnoth/wesnoth-old/builds/16069911
20131228 13:58:52-!- travis-ci [~travis-ci@ec2-54-221-177-157.compute-1.amazonaws.com] has left #wesnoth-dev []
20131228 14:07:55< fabi> Ivanovic: ping
20131228 14:15:18< irker305> wesnoth: fendrin wesnoth-old:master 404916026f8c / src/hotkey/ (hotkey_command.cpp hotkey_command.hpp): Hotkey actions for recruit list assignment and clearing the unit set. http://git.io/cR-e_Q
20131228 14:21:58< happygrue> thanks fabi, looks good on the AoH+K
20131228 14:37:59< AI0867> shadowm, mattsc: I coded up a quick reentry fix (break out of reentrance happened) in unit_animator::wait_for_end, but that just exposes the very same issue in unit_display::reset_helpers
20131228 14:43:33< AI0867> diff: http://pastebin.com/iJBPsJpX
20131228 14:50:21-!- thunderstruck [~zaibotren@cpc13-sgyl31-2-0-cust696.18-2.cable.virginm.net] has joined #wesnoth-dev
20131228 14:53:25< fabi> happygrue: The Khalifate have only a few leader's to choice from.
20131228 14:56:36-!- Paulomat4 [55b7394d@gateway/web/freenode/ip.85.183.57.77] has joined #wesnoth-dev
20131228 15:07:28-!- mattsc [~mattsc@154.20.32.246] has joined #wesnoth-dev
20131228 15:09:30< happygrue> ah, fabi: I think Tineen and Mufariq can also be leaders?
20131228 15:09:58< happygrue> for AoH anyway
20131228 15:10:26< fabi> happygrue: Agreed
20131228 15:10:50< fabi> happygrue: I think Tabib are fine leaders but they don't have a lvl3
20131228 15:11:23< happygrue> yep
20131228 15:12:44< fabi> happygrue: What are the criteria for leaders?
20131228 15:13:23< happygrue> for AoH, it's mostly anything that is level 3 and isn't more than 7 moves I think
20131228 15:13:41< happygrue> it's more about giving players any choice they like and doesn't pretend to be balanced
20131228 15:14:33< fabi> Let's ask the other way around: Why is the Saurian Ambusher no leader in default?
20131228 15:15:04< fabi> The only other in the Drake faction is the Sky Drake, this might be too fast.
20131228 15:15:23< fabi> The Ambusher is only 7 fast.
20131228 15:15:51< fabi> *The only other not leader
20131228 15:17:42< happygrue> well, in default or in AoH are we talking?
20131228 15:18:12< happygrue> excuse, me, I meant more than 6 moves above
20131228 15:18:16< happygrue> I misspoke
20131228 15:18:16< fabi> default
20131228 15:18:37< happygrue> in default, map balance assumes no leader has more than 6 moves
20131228 15:18:52< happygrue> and even that *should* be changed IMO, but there has been resistance to that
20131228 15:19:32< happygrue> leaders are chosen to be not to strong, not to weak, so that players can use random factions and get a reasonable leader
20131228 15:19:37< happygrue> the ambusher is not good enough
20131228 15:19:44< happygrue> it is easy to kill
20131228 15:20:00< fabi> Okay, but making Tineen and Mufariq is fine.
20131228 15:20:05< happygrue> because it's AoH
20131228 15:20:07< fabi> They are both slower than 7.
20131228 15:20:30< happygrue> but yes
20131228 15:20:30< fabi> The list of random leaders is a different one than the one to choose from.
20131228 15:20:40< happygrue> we don't want scout leaders, but any level 3 move 5-6 units could be leaders in AoH IMO
20131228 15:20:57< fabi> Thus to weak leaders could be added to the selectable list but not the random one.
20131228 15:21:01< happygrue> right
20131228 15:21:10< happygrue> I would make the Tineen in that list
20131228 15:21:21< happygrue> could be chosen, but not random
20131228 15:21:35< fabi> While the Mufariq fits in both?
20131228 15:21:41< happygrue> sure
20131228 15:22:26< fabi> Shall I make that change for default as well?
20131228 15:22:38-!- Coffee_irc [~david@58.145.148.37] has quit [Remote host closed the connection]
20131228 15:22:41< fabi> Adding Faris to both
20131228 15:23:06< fabi> And qatif-al-nar to the selectable only
20131228 15:23:07< happygrue> no, Faris should not be a default leader
20131228 15:23:20-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 15:23:21< fabi> why?
20131228 15:24:48< happygrue> because I would very much like to move away from 6 move leaders, so I would rather not even start him off as a leader, as I hope to remove him anyway
20131228 15:25:26< fabi> So you like to see no leader being faster than 5?
20131228 15:26:06< AI0867> 6 move leaders exist, but they are required to be slowed down by certain terrain (cave and mushroom I believe?) to ensure they can't traverse maps too fast
20131228 15:26:23< AI0867> 4 move leaders (dwarves) get quick to compensate
20131228 15:31:11< fabi> AI0867: Well, the Faris is pretty slow on Cave and Fungus, both cost 4MP.
20131228 15:31:22< fabi> Making him crawl with one hex per round.
20131228 15:31:26-!- Coffee_irc [~david@58.145.148.37] has quit [Ping timeout: 240 seconds]
20131228 15:31:49-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 15:32:21< mattsc> AI0867: acknowledged.  I can't really say more than that other than that, if I understand things correctly, your fix means that it now causes an assertion failure instead of a segfault.
20131228 15:32:21< mattsc> I understand that that's better, but from a user's perspective it isn't really any different.  Or am I missing something?
20131228 15:33:30< fabi> mattsc: Assertions are deterministic, segfaults not necessarily.
20131228 15:34:07< AI0867> yeah, it is better
20131228 15:34:12< mattsc> fabi: I understand - but the game still shuts down if I make a "harmless mistake".
20131228 15:35:37< mattsc> And it's not something that never happens.  I've had it happen quite a bit to me when using debug commands to test/balance scenarios, until recently I just had no idea how to reproduce it.
20131228 15:35:46< fabi> mattsc: Yes, assertions are not a thing we want to reach the end user.
20131228 15:36:17< fabi> end user == player or umc designer
20131228 15:37:21< mattsc> Right.  And don't get me wrong, I'm very happy that AI0867 is looking into it.  I'm just saying that the current fix doesn't help me with my selfish little problem.  :)
20131228 15:37:32< fabi> :-)
20131228 15:37:56< AI0867> I'm afraid this might just be how the engine is built, and all we can do is ignore commands while animations are playing
20131228 15:38:51< mattsc> AI0867: that would be fine by me
20131228 15:42:12< mattsc> So we'd put 'if (!i.get_animation())'  (or something along those lines, I didn't look up the exact commands) at the beginning of the debug kill code block?
20131228 15:43:12< mattsc> Well, anyways, I don't want to kill another khalifate discussion here.  :P
20131228 15:46:04< fabi> mattsc: Well, I guess we were through with it. Two more leaders for their AoH apperance.
20131228 15:48:27< mattsc> fabi: cool.  Btw, whenever the Northerners have a 6 MP leader on Freelands, the behavior can be quite different for the AI behavior.  For Fred, I mean.
20131228 15:51:02< AI0867> fabi: I think you have something to do with this: on 800x480, the commandmode text is absolutely tiny
20131228 15:51:18< AI0867> the text is like 4 pt or smaller
20131228 15:51:44< AI0867> also, the ToD modifier percentage is still in the wrong place
20131228 15:51:48< fabi> AI0867: Hmmm, if the commndmode is controlled by theme wml, then maybe.
20131228 15:52:10< AI0867> this is a very new thing, so I suspect it has to do with your recent additions there
20131228 15:53:26< fabi> AI0867: Very new == since libkab changed the font size of the filter textbox?
20131228 15:55:43< fabi> happygrue: Aethermaw, some of the units in the fogged "pure eye candy" region of the map are not fogged, most are. A bug or not?
20131228 16:00:14< AI0867> fabi: that might also be it
20131228 16:00:29< AI0867> I'm working on bug #18921, but I can do a bisect after that, if needed
20131228 16:01:21< fabi> AI0867: I don't deny that I might have caused the reduce of the font size somehow but I have no idea which of my stuff might have caused it.
20131228 16:10:42< irker305> wesnoth: fendrin wesnoth-old:master 2dd6c62bbfdb / data/multiplayer/factions/khalifate-aoh.cfg: More leaders for the Khalifate in the AoH era. http://git.io/kcIVIQ
20131228 16:11:48-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev
20131228 16:11:56< fabi> ho bumbadadabum
20131228 16:12:02< bumbadadabum> Hi
20131228 16:16:54< fabi> happygrue: Doc Paterson is no longer an active developer?
20131228 16:18:19-!- Upthorn [~ogmar@108-85-91-228.lightspeed.frokca.sbcglobal.net] has quit [Ping timeout: 252 seconds]
20131228 16:21:09-!- Coffee_irc [~david@58.145.148.37] has quit [Read error: Connection reset by peer]
20131228 16:22:36< happygrue> fabi: regarding Aethermaw: no idea, regarding Doc: also no idea.  He is surely less active than he once was, but I don't know how much he's on the server lurking
20131228 16:25:13-!- Adamant14 [57a17742@gateway/web/freenode/ip.87.161.119.66] has joined #wesnoth-dev
20131228 16:30:25< molgrum> anyone using codeblocks? when i compile it uses gcc instead of g++ so it fails to understand static_cast
20131228 16:30:27< fabi> happygrue: Okay :-)
20131228 16:30:39< happygrue> also fabi: regarding leaders: I would like all leaders to be 5 move, or 6 moves, I don't really care which - I just want them all to be uniform.  This is addressed in the Alternate Era, which I hope will be the direction default ends up going
20131228 16:31:45< happygrue> it is just silly that the Lieutenant is still around, for example, but my attempts to take him out have not worked :D
20131228 16:31:58< bumbadadabum> but he's a classic
20131228 16:32:12< bumbadadabum> you can't remove a unit after 10 years
20131228 16:32:59< fabi> happygrue: What is wrong with him?
20131228 16:33:02< fabi> Too fast?
20131228 16:33:06< happygrue> he's too good
20131228 16:33:15< happygrue> bumbadadabum: that is the worst argument ever :D
20131228 16:33:23< bumbadadabum> I agree that 6 moves and leadership is a bit powerful
20131228 16:33:34< happygrue> because something is 10 years old that is somehow a good thing?  He's been broken for 10 years, we should be ashamed of it!
20131228 16:33:36< bumbadadabum> but the unit is terrible
20131228 16:33:48< bumbadadabum> eh, I don't play MP
20131228 16:33:51< happygrue> because we keep nerfing him to the stone age
20131228 16:33:54< bumbadadabum> I like the Lieutenant
20131228 16:33:55< zookeeper> happygrue, do you recall why his movement wasn't just dropped to 5?
20131228 16:33:56< happygrue> he used to be so so good
20131228 16:34:04< happygrue> because there was "resistance"
20131228 16:34:11< zookeeper> ohh. that. :P
20131228 16:34:13< happygrue> ;)
20131228 16:34:25< happygrue> the issue is, there are two main groups of MP folks
20131228 16:34:48< happygrue> the ladder people, or let's say the people who play a lot of 1v1 games, and everyone else.
20131228 16:35:08< happygrue> the people who play a lot of 1v1 games see different problems than the people who play 2v2 or survival or such
20131228 16:35:08< fabi> happygrue: Well, the default era, or general MP seems to suffer from the "keep everything like it is" attitude especially.
20131228 16:35:17< happygrue> I agree
20131228 16:35:26< bumbadadabum> I like dropping it to 5 MP
20131228 16:35:30< bumbadadabum> and maybe give it more HP
20131228 16:35:47< happygrue> things could be done, but how about we just create unique leaders for each faction?
20131228 16:35:59< happygrue> why are we just using level 2 units, which are not reall balanced against each other?
20131228 16:36:02< bumbadadabum> like in the alternate era?
20131228 16:36:04< happygrue> :D
20131228 16:36:12< happygrue> what a great idea that is !  ;)
20131228 16:36:24< bumbadadabum> I don't have anything against that
20131228 16:36:37< bumbadadabum> for the default era
20131228 16:36:44< bumbadadabum> but default era MP =/= everything
20131228 16:37:15-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Read error: Connection reset by peer]
20131228 16:37:19< happygrue> that is true
20131228 16:37:45< happygrue> the biggest problem with the leader movement differences is that it makes maps much harder (and uglier) to make
20131228 16:37:59< happygrue> there has to be cave everywhere just to slow down the 6 move drake leader
20131228 16:38:03-!- stikonas [~gentoo@5.20.200.50] has joined #wesnoth-dev
20131228 16:38:03-!- stikonas [~gentoo@5.20.200.50] has quit [Changing host]
20131228 16:38:03-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20131228 16:38:25< bumbadadabum> maybe rebalance the level 2s
20131228 16:38:46< happygrue> easier said than done.  We've been stripping leaders out and rebalancing them for a while
20131228 16:39:00< happygrue> it's "okay" now, but in tournaments when people want to win, they take loyals with a Lieutenant.
20131228 16:39:14< happygrue> very often anyway
20131228 16:39:17< bumbadadabum> ok
20131228 16:39:47< bumbadadabum> well, either remove him from the default era
20131228 16:39:51< bumbadadabum> or nerf him
20131228 16:39:51< happygrue> we used to have 4 move shock troopers as leaders
20131228 16:40:05< bumbadadabum> not anymore?
20131228 16:40:26< happygrue> no, that one was so bad I could convince everyone else to remove him
20131228 16:40:41< bumbadadabum> don't we have quick 4mp leaders now?
20131228 16:40:43< happygrue> but the Lieutenant is well loved
20131228 16:41:17< bumbadadabum> I think having it at 5mp would be good
20131228 16:41:17< happygrue> yes, but he was still terrible as a leader
20131228 16:41:41< bumbadadabum> the lieutenant is an easy one
20131228 16:41:53< bumbadadabum> because it's only used as a leader if you only look at MP
20131228 16:41:54-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has joined #wesnoth-dev
20131228 16:44:27< happygrue> even dropping him to 5 moves and giving him some HP doesn't really fix it though, because he is the only leadership that loyals get, so people are still going to take him
20131228 16:44:48< happygrue> I once wished that all leaders got leadership and we reworked around that.  Or some other great ability
20131228 16:44:55< happygrue> but I was shouted out of town on that one ;)
20131228 16:45:14< happygrue> but the Alternate Era is a great place to start when thinking about nice leaders for each faction:  http://units.wesnoth.org/1.10/Alternate_Era/en_US/era_alternate.html
20131228 16:45:17< happygrue> and it has art!
20131228 16:47:15< happygrue> some of the other units are also tweaked based on feedback from the 1v1 folks, so that is an attempt at an era that is more balanced right now than default - though that would be debatable of course
20131228 17:14:06-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has quit [Remote host closed the connection]
20131228 17:19:47< fabi> AI0867: The font size of the commandline is reduced to about 2 pixel height (which means no readability at all) at 1680x1050.
20131228 17:41:55-!- happygrue [~happygrue@wesnoth/developer/wintermute] has quit [Ping timeout: 252 seconds]
20131228 17:53:53-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Ping timeout: 245 seconds]
20131228 17:57:59-!- TooLmaN [~TooLmaN@mail.thomsonplastics.com] has quit [Quit: Off to save the world!]
20131228 18:13:35< molgrum> anyone using codeblocks? when i compile it uses gcc instead of g++ so it fails to understand static_cast
20131228 18:15:31< AI0867> why are you compiling with gcc instead of g++?
20131228 18:16:10< molgrum> it's not something i choose, i opened the workspace and haven't modified anything
20131228 18:17:21< molgrum> just clicked build workspace and it uses gcc by default
20131228 18:19:35< AI0867> well, I don't use code::blocks myself, but I'm not particularly surprised that it doesn't understand static_cast if you invoke it as a C compiler
20131228 18:29:26-!- TooLmaN [~TooLmaN@mx1.thomsonplastics.com] has joined #wesnoth-dev
20131228 18:29:47< fabi> gcc == gnu compiler collection
20131228 18:29:53< fabi> g++ is part of gcc
20131228 18:30:16< fabi> Like the Fortran compiler.
20131228 18:44:15-!- EdB [~edb@85.69.242.6] has joined #wesnoth-dev
20131228 18:47:31-!- TooLmaN [~TooLmaN@mx1.thomsonplastics.com] has quit [Quit: Off to save the world!]
20131228 18:48:56-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has joined #wesnoth-dev
20131228 18:58:15< thunderstruck> http://stackoverflow.com/questions/172587/what-is-the-difference-between-g-and-gcc
20131228 18:58:38< thunderstruck> But still invoking gcc and g++ on C++ source is not the same.
20131228 19:02:02< irker305> wesnoth: Alexander van Gessel wesnoth-old:master 30c3a9c287a0 / src/unit_animation.cpp: Paper over bug #18921 with an assert http://git.io/QjM4og
20131228 19:02:04< irker305> wesnoth: Alexander van Gessel wesnoth-old:master d38d10a88fc5 / / (9 files in 5 dirs): Merge branch 'master' of github.com:wesnoth/wesnoth-old http://git.io/sZXfjg
20131228 19:12:30-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has quit [Ping timeout: 245 seconds]
20131228 19:12:59< AI0867> I may have overengineered this, but it was even worse before I trimmed it down ;)
20131228 19:32:06-!- EdB [~edb@85.69.242.6] has quit [Quit: Konversation terminated!]
20131228 19:41:47-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 19:43:05< AI0867> lipkab: 34a8bab55fd1a0884 broke some things
20131228 19:43:20< AI0867> type : ingame and you'll see
20131228 19:43:30< AI0867> I think it has to do with the order of default arguments
20131228 19:44:32< lipkab> AI0867: Hehe.
20131228 19:45:02< lipkab> Yeah, I didn't properly check all textbox() calls, apparently.
20131228 19:45:14-!- new_one [~new_one@50.15.126.83] has joined #wesnoth-dev
20131228 19:45:46< lipkab> It's strange though that the compiler didn't complain about assigning a double value to an int argument.
20131228 19:46:03< lipkab> Shouldn't that trigger at least a warning?
20131228 19:49:37-!- Bodhi-Baum [~Bodhi@dslb-084-063-022-177.pools.arcor-ip.net] has joined #wesnoth-dev
20131228 19:50:20< irker305> wesnoth: Boldizsár Lipka wesnoth-old:master 8877c74a83ae / src/floating_textbox.cpp: Fix the command entry's font size. http://git.io/CR3nlw
20131228 19:50:44< lipkab> AI0867: ^
20131228 19:51:25-!- new_one [~new_one@50.15.126.83] has left #wesnoth-dev []
20131228 19:57:30-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 245 seconds]
20131228 19:58:08-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 19:59:32< lipkab> Hm. Is it possible to see my own timeout message?
20131228 19:59:33< lipkab> Because it just happened to me.
20131228 20:05:27-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 240 seconds]
20131228 20:06:03-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 20:07:41< zookeeper> Ivanovic, so how does the release schedule look?
20131228 20:13:26-!- jorda [~jorda@li145-112.members.linode.com] has joined #wesnoth-dev
20131228 20:14:25< AI0867> lipkab: it is, apparently if freenode times you out, it sends you your own QUIT before cutting the connection
20131228 20:17:37-!- thunderstruck [~zaibotren@cpc13-sgyl31-2-0-cust696.18-2.cable.virginm.net] has quit [Quit: leaving]
20131228 20:20:08< jorda> Ivanovic: I have an update for the Catalan translation, but it has been ages since the last time I committed anything. The source is now on github, isn't it? I assume I'll need commit access... Or you prefer a patch?
20131228 20:21:29-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 246 seconds]
20131228 20:22:58-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 20:31:10< Ivanovic> zookeeper: waiting for word from noy about the unit descriptions for the khalifate
20131228 20:31:18< Ivanovic> jorda: you can e.g. send a pull request
20131228 20:31:28< Ivanovic> or you can just send me the full files
20131228 20:34:28< zookeeper> Ivanovic, okay... sounds like it won't be tonight, in any case
20131228 20:34:59-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 252 seconds]
20131228 20:43:23-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev
20131228 20:45:54< jorda> Ivanovic: alright
20131228 20:46:16< jorda> Ivanovic: Hi, btw. This is ettin. Long time no see. :)
20131228 20:50:02-!- Paulomat4 [55b7394d@gateway/web/freenode/ip.85.183.57.77] has quit [Ping timeout: 272 seconds]
20131228 21:01:33-!- Coffee_irc [~david@58.145.148.37] has joined #wesnoth-dev
20131228 21:47:10-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Quit: Távozom]
20131228 21:58:52< Ivanovic> jorda: yeah, has been ages since you were around the last time
20131228 22:02:43-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:05:59-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Client Quit]
20131228 22:20:10-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:31:52-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Leaving]
20131228 22:34:08-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:34:59-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Remote host closed the connection]
20131228 22:35:28-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:37:12-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Read error: Connection reset by peer]
20131228 22:38:00-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev
20131228 22:47:51-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Leaving]
20131228 22:49:00-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:54:40-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Leaving]
20131228 22:54:59-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has joined #wesnoth-dev
20131228 22:59:09-!- Bodhi-Baum [~Bodhi@dslb-084-063-022-177.pools.arcor-ip.net] has quit [Quit: Verlassend]
20131228 23:11:54-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection]
20131228 23:12:23-!- TooLmaN [~TooLmaN@c-174-56-210-230.hsd1.ga.comcast.net] has quit [Quit: Leaving]
20131228 23:15:12< molgrum> i see now why code blocks uses a c compiler for liblua, the source files end in .c not .cpp
20131228 23:26:49< shadowm> That's why the guide instructs the user to set the C compiler to a C++ compiler.
20131228 23:35:17 * shadowm stabs AI0867.
20131228 23:35:38< shadowm> AI0867: http://pastebin.com/XGdtQGZS
--- Log closed Sun Dec 29 00:00:27 2013