--- Log opened Tue Jun 01 00:00:16 2010
--- Day changed Tue Jun 01 2010
20100601 00:00:16< boucman> yes
20100601 00:00:41< alink> i had an idea about that, assuming no conflict and cheap unit move in unit_map, you can temporary move units before the pathfinding requests and revert the move after that
20100601 00:01:10< alink> we already that in few place I think (but for only 1 unit)
20100601 00:01:33-!- zookeeper [~l@wesnoth/developer/zookeeper] has quit []
20100601 00:01:43< alink> +do that
20100601 00:02:25< gabba> alink: that sounds useful for conflicts detection, but to complicated for what I'm aiming for right now
20100601 00:02:39< gabba> s/to/too
20100601 00:03:34< alink> no, conflict as 2 unit on same hex will not work with unit_map.
20100601 00:04:08< alink> not sure if complicated, depend of your other stuff, i was just throwing the idea
20100601 00:05:08< alink> it could be a simple trick for solving some WML problems with ability
20100601 00:05:28< gabba> is there any unit ability that can affect the pathfinding of friendly unit? I think someone mentioned "distract" last month
20100601 00:06:09< alink> well there is possibly skirmisher and teleport, but that's special unused stuff for the moment
20100601 00:06:37< alink> but do you plan to support showing attack dialog in whiteboard mode ?
20100601 00:07:08< alink> for example you plan attacking with 2 unit and 1 has leadership
20100601 00:07:31< gabba> alink: not sure about this one... it's probably gonna be "no" at first
20100601 00:07:49< boucman> gabba: IIRC distract affect the ZoC of adjacent ennemy unit, so yes
20100601 00:07:59< alink> gabba: ok, it's a tricky area anyway
20100601 00:08:07< boucman> alink: I think it's used by jessica (since renamed)
20100601 00:08:09< gabba> alink: but one of the great ideas was to show combined CTK stats, so that implies choosing your attack when you define the attack-move
20100601 00:08:43< alink> gabba: my point was that by temporary modifiying unit map you get all the bonus for free
20100601 00:09:14< boucman> ok, really need to leave now, see you all
20100601 00:09:19-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Quit: Leaving.]
20100601 00:09:24< alink> I used this trick changing how move+attack works (show the dialog before moving)
20100601 00:09:30 * gabba codes if (Jessica) <do stuff> else <do normal stuff>; :P
20100601 00:09:53< alink> in fact the current move+attack is a kind oh whiteboard for 1 move
20100601 00:10:02< alink> s/oh/of
20100601 00:10:40< gabba> alink: that's interesting, can I see the relevant code?
20100601 00:12:25< alink> mouse_events.cpp:450
20100601 00:12:37< gabba> thanks, taking a look
20100601 00:13:15< alink> of course with multiple units you will need something more complex than the simple temporary_unit_mover
20100601 00:14:54< alink> the attack dialog (or a pathfinding request) can works with the modified unit_map, and you get all engine function working as expected for free
20100601 00:15:07-!- AI0867 [~ai@wesnoth/developer/ai0867] has joined #wesnoth-dev
20100601 00:15:56< gabba> alink: I need to do this invisibly on a copy of the unit map, though, whereas with temporary_unit_mover it's visible on map, right?
20100601 00:18:04< alink> not sure to understand you , but here we do the change on the current real unit_map, but revert it after the work and before the engine restart to draw etc...
20100601 00:18:40< alink> for drawing things you can use fake units etc and don't touch the unit_map
20100601 00:19:19< gabba> alink: I need to study the code a bit more before what I say starts making sense I think ;)
20100601 00:19:21< alink> the point is that copying the whole unit_map is probably expensive (and need update etc..)
20100601 00:20:14< alink> just to be sure, did you get that the temporary_unit_mover is a setter and when destroyed at the end of the block, it revert its change
20100601 00:21:15< gabba> yeah, I just looked at its constructor/destructor and I see how it works now
20100601 00:21:38< ancestral> Damn volcano
20100601 00:21:40< cjhopman_> gabba: pathfinding first with current unit map will give wrong results in the case where you want to move unit1 somewhere and then move unit2 to where unit1 was
20100601 00:21:49< ancestral> whoops wrong channel
20100601 00:22:27< alink> cjhopman_: yeah it don't work with conflict, but same problem for abilities
20100601 00:23:24< alink> and there is also other setter temporary *removing* unit too
20100601 00:23:56< alink> I think that temporary_unit_mover do that
20100601 00:24:08< gabba> cjhopman_: I see. The problem is that a desirable feature for planned moves was defining several ones at the same place. It would have been useful later to define 10 walking corpse attacks from the same hex and then see the combined CTK for that.
20100601 00:24:45< gabba> however I'm sure a workaround could be found to allow doing this later on...
20100601 00:25:09< alink> "see the combined CTK for that" seems very hard do calculate
20100601 00:26:26< gabba> Supposedly the best players calculate some combined CTKs in their head, so a computer can do it I guess :)
20100601 00:27:18< alink> depend how accurate you want it to be :-)
20100601 00:28:11< alink> it's the same as normal attack, there is a big probability tree which can be complex when you add drain, slow etc..
20100601 00:29:34< alink> ah it was temporary_unit_placer for replacing a unit
20100601 00:30:09< alink> so for resolving conflict using this idea, you need to combine temporary_unit_mover and temporary_unit_placer, so a move is always possible
20100601 00:30:32< alink> it just temporary remove the unit from the destination hex
20100601 00:31:14< gabba> alink: sounds very good
20100601 00:32:09< alink> I don't say that there will be no problem, but that can at least solve some other ones :-)
20100601 00:32:20< gabba> So, I can use a stack of temporary_unit_mover's to apply all moves that would be executed before the current one, and then pathfind
20100601 00:32:57< alink> yes that's how i see it
20100601 00:33:33< gabba> for conflicts I'll see... maybe it would be best not to allow defining that kind of move in the first place
20100601 00:33:43< alink> not sure that you want to continue to rely on destructor. because destruction order is tricky
20100601 00:34:14-!- Valkier [~IceChat7@c-174-55-104-2.hsd1.pa.comcast.net] has joined #wesnoth-dev
20100601 00:34:33< alink> unit sharing same hex seems indeed tricky, also for the UI part
20100601 00:34:54< gabba> true
20100601 00:35:23< alink> but OtOH that could be very useful for players using this whiteboard :-)
20100601 00:35:31< gabba> why is destruction order tricky? from what I see I just need to destroy in reverse order of creation
20100601 00:36:39< alink> If you mean calling the function yourself, then yes
20100601 00:36:48< cjhopman_> I would almost say that you don't want to do pathfinding/other conflict finding and just let me draw whatever they want on the whiteboard
20100601 00:36:50< cjhopman_> or rather
20100601 00:37:09< cjhopman_> that if you do find conflicts you just draw something like a red exclamation mark
20100601 00:37:18< alink> if you use automatic destruction of objects in container etc.. then that's less clear( to me, at least)
20100601 00:37:27< cjhopman_> but still allow people to do it
20100601 00:38:38< gabba> alink: using some boost smart pointer that destroys the object as soon as it's popped from the stack should do it
20100601 00:39:34< alink> yeah, but seems simpler to have a stack of 'action' structure with an 'do' 'undo' functions
20100601 00:40:19< alink> the use of destructor in temporary_unit_mover is just to be sure that user don't forget
20100601 00:41:07< gabba> cjhopman_: it's a big discussion... freedom is good, but since planned moves are executable it's better if most moves make sense
20100601 00:41:57< cjhopman_> gabba: yeah, the problem is that you can't calculate that
20100601 00:42:03< gabba> cjhopman_: it's not the kind of whiteboard that allows you to draw smileys on the map :)
20100601 00:42:48< gabba> cjhopman_: well, I can check if moves are valid, and using alink's indications I can take pretty much everything else into accound
20100601 00:42:51< cjhopman_> say my planned move is to move 2 units and attack and kill another... and then move 3 units to attack one that was inaccessible before attacking the first
20100601 00:42:54< gabba> s/accound/account
20100601 00:42:57< cjhopman_> s/attacking/killing
20100601 00:43:50< alink> mmmh for enemy death, indeed, you need the UI to allow to mark "suppose this enemy dead", then plan next action
20100601 00:44:17< alink> so pathfinding will allow you to pass trough its ZoC
20100601 00:44:36< cjhopman_> which actually might be a good idea
20100601 00:44:38< gabba> cjhopman_: ^ what alink said was one of the proposed solutions
20100601 00:44:44< gabba> I like it enough
20100601 00:44:50< cjhopman_> then you get to draw some death symbol on the guy
20100601 00:45:04< cjhopman_> i like it too
20100601 00:45:19< alink> and use  temporary_unit_death in unit_map
20100601 00:45:40-!- Valkier [~IceChat7@c-174-55-104-2.hsd1.pa.comcast.net] has quit [Quit: I used to think I was indecisive, but now I'm not too sure.]
20100601 00:50:19< gabba> of course, UI problems come up when you "suppose an enemy" dead, and then move one of your units on top of it's location: how do you remember which enemy unit was there? when animating the attack, what do we do with the whiteboard symbols?
20100601 00:52:02< gabba> also if a wml event messes with unit positions, we need to keep planned move arrows with their current, possibly invalid path, and detect and show the ones that are conflicted
20100601 00:53:05< gabba> afk for a minute
20100601 00:53:24-!- Johannes13_ [~Johannes@unaffiliated/johannes13] has quit [Ping timeout: 265 seconds]
20100601 00:56:12< alink> yes if you start to stack sprite, it can become visually messy
20100601 00:56:39< alink> one key to temporary hide whiteboard mode could help
20100601 00:57:44< alink> you can also render some sprites differently (planned or current, not sure which ones)
20100601 00:58:16< alink> for example: slightly coloring them or changing their alpha
20100601 00:58:28< alink> as we do for invisibility and poison
20100601 00:59:23< CIA-86> eleazar * r43085 /trunk/data/campaigns/ (10 files in 2 dirs): enhancements to Liberty and tHoT map to use new terrain tiles and features. As usual no gameplay change.
20100601 00:59:28< alink> not sure if it will really help the stacking, though
20100601 01:02:12< alink> or you may not moving any unit, and just render arrows, but that's less nice
20100601 01:16:39-!- alink [~alink@wesnoth/developer/alink] has quit [Remote host closed the connection]
20100601 01:37:50< gabba> alink: I think I've got a solution: instead of trying to stack sprites, I'll add controls to go back-and-forth in your planned moves list. So if you defined a move on top of an "assumed dead" unit, you'll be able to hit the "back" button several times to hide the last few planned moves and look at that unit again.
20100601 01:39:10-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has quit [Read error: Operation timed out]
20100601 01:41:08-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has joined #wesnoth-dev
20100601 01:55:09-!- krotop [~christoph@smj33-1-82-233-64-163.fbx.proxad.net] has joined #wesnoth-dev
20100601 01:56:09< krotop> hello, maybe this is being handled already, but in case of, the behaviour of oalf hasn't changed since he got kicked
20100601 01:56:52< krotop> (spreading racist hatery and insulting people randomly)
20100601 01:58:34< Espreon> krotop: #wesnoth-mp
20100601 02:05:50< krotop> Espreon: did not understand
20100601 02:07:30< Espreon> Go to #wesnoth-mp
20100601 02:28:49-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has quit [Quit: Leaving]
20100601 02:34:25-!- krotop [~christoph@smj33-1-82-233-64-163.fbx.proxad.net] has quit []
20100601 02:35:07-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20100601 02:45:38-!- loonycyborg [~sergey@wesnoth/developer/loonycyborg] has quit [Quit: Zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz]
20100601 02:48:30-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has quit [Remote host closed the connection]
20100601 02:56:23-!- Bob_The_Mighty [~chatzilla@cpc4-brig15-0-0-cust904.3-3.cable.virginmedia.com] has joined #wesnoth-dev
20100601 03:03:29-!- Crab_ [~Crab_@wesnoth/developer/crab] has joined #wesnoth-dev
20100601 03:16:34-!- Blitzmerker [~Blitzmerk@p3EE09C8A.dip0.t-ipconnect.de] has quit [Quit: Verlassend]
20100601 03:18:20-!- stikonas [~and@wesnoth/translator/stikonas] has quit [Remote host closed the connection]
20100601 03:28:27-!- Bob_The_Mighty [~chatzilla@cpc4-brig15-0-0-cust904.3-3.cable.virginmedia.com] has quit [Quit: ChatZilla 0.9.86 [Firefox 3.6.3/20100401080539]]
20100601 03:36:53-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has quit [Quit: Leaving.]
20100601 03:37:19-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has joined #wesnoth-dev
20100601 03:38:05-!- esr [~chatzilla@static-71-162-243-5.phlapa.fios.verizon.net] has joined #wesnoth-dev
20100601 03:38:30-!- esr [~chatzilla@static-71-162-243-5.phlapa.fios.verizon.net] has quit [Changing host]
20100601 03:38:30-!- esr [~chatzilla@wesnoth/developer/esr] has joined #wesnoth-dev
20100601 03:38:43-!- ABCD [~abcd@gentoo/developer/abcd] has joined #wesnoth-dev
20100601 03:45:53-!- Daltx [~Daltx@CPE001e5840eaf6-CM00195ee19c52.cpe.net.cable.rogers.com] has left #wesnoth-dev []
20100601 03:50:43< CIA-86> ai0867 * r43086 /trunk/ (175 files in 7 dirs): Remove unneeded include
20100601 03:50:47< AI0867> gah
20100601 03:50:57< AI0867> mis-commit
20100601 03:57:07-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20100601 04:02:04< CIA-86> ai0867 * r43087 /trunk/src/game_events.cpp: Prevent undefined behaviour
20100601 04:14:28< CIA-86> ai0867 * r43088 /trunk/ (2 files in 2 dirs): Another aint -> ain't, in the same line even
20100601 04:14:43< Espreon> AI0867: Thank you.
20100601 04:17:49< CIA-86> ai0867 * r43089 /trunk/po/wesnoth-ei/ (53 files): Ran pofix with new rule on po files
20100601 04:19:33-!- drusepth [~drusepth@dhcp-077-249-151-209.chello.nl] has joined #wesnoth-dev
20100601 04:21:20-!- qemqemqem__ [~quassel@ip70-177-181-137.dc.dc.cox.net] has joined #wesnoth-dev
20100601 04:22:12-!- k23z__ [k23z__@188.27.127.113] has quit [Ping timeout: 265 seconds]
20100601 04:29:55< CIA-86> gabba * r43090 /trunk/src/whiteboard/ (7 files): Adding files for planned_action, planned_move and planned_action_set classes.
20100601 04:30:03< CIA-86> gabba * r43091 /trunk/src/whiteboard/ (4 files): Planned actions: changed planned_action_set to planned_action_manager
20100601 04:30:09< CIA-86> gabba * r43092 /trunk/src/whiteboard/ (3 files): Planned actions: shuffling things around.
20100601 04:30:20< CIA-86> gabba * r43093 /trunk/src/ (CMakeLists.txt SConscript): Added planned_action, planned_move and planned_move_manager to Cmake and Scons, but not to automake
20100601 04:30:24< CIA-86> gabba * r43094 /trunk/src/whiteboard/ (planned_action_manager.cpp planned_action_manager.hpp): Planned Actions: Singleton pattern for the planned_action_manager
20100601 04:30:32< CIA-86> gabba * r43095 /trunk/src/whiteboard/ (planned_action.hpp planned_move.cpp planned_move.hpp): Planned Actions: planned_move class details, and some minor corrections.
20100601 04:30:38< CIA-86> gabba * r43096 /trunk/src/whiteboard/ (planned_move.cpp planned_move.hpp): Planned Actions: more additions to planned_move
20100601 04:30:45< CIA-86> gabba * r43097 /trunk/src/whiteboard/ (4 files): 
20100601 04:30:45< CIA-86> Planned Actions: Implementation of a few more functions in planned_action_manager
20100601 04:30:45< CIA-86> Attempt to use boost::shared_ptr
20100601 04:33:31-!- k23z__ [k23z__@188.26.49.158] has joined #wesnoth-dev
20100601 04:34:07< Crab_> gabba: btw, why 'not to automake' part ?
20100601 04:35:28< gabba> hi Crab_, because people told me adding it to automake wasn't trivial, and that the maintainers would take care of it
20100601 04:35:52< Crab_> hmm....  src/Makefile.Am , 1 line per cpp file added.
20100601 04:36:34< gabba> Crab_: well, maybe that was too lazy on my part, I'll look at Makefile.Am
20100601 04:37:02< Crab_> MSVC is harder, yes. but cmake/scons/automake are easy when just adding files.
20100601 04:37:37< Crab_> gabba: for example, see http://svn.gna.org/viewcvs/wesnoth/trunk/src/Makefile.am?rev=42909&view=diff&r1=42909&r2=42908&p1=trunk/src/Makefile.am&p2=/trunk/src/Makefile.am
20100601 04:40:26< shadowmaster> maybe it's just that someone who hates autotools told you it wasn't trivial
20100601 04:42:15< CIA-86> gabba * r43098 /trunk/src/Makefile.am: Added new whiteboard files to automake as well
20100601 04:45:24< Crab_> gabba: are you sure that planned action manager is a singleton ?
20100601 04:46:05< Crab_> gabba: for example:  hot seat, local game, two players... each of them would have his own set of planned moves?
20100601 04:47:23< gabba> Crab_: hmm, thinking twice about it, maybe not -- unless I put several planned_action_set in it, letting it act as a kind of central hub
20100601 04:47:52-!- wesbot changed the topic of #wesnoth-dev to: released 1.8.2, announcing "soon"| 113 bugs, 280 feature requests, 16 patches | logs: http://irclogs.wesnoth.org | Don't paste on IRC! Use a pastebin: http://wesnoth.pastebin.com | http://imagebin.org
20100601 04:48:28< Crab_> gabba: yes, that'll work, too. for ai, ai::manager is a central hub, which contains ai objects (which are in context of a particular side) and proxies access to them.
20100601 04:48:55< gabba> Crab_: to be honest I don't like this class much as of now: name's too long, and I don't really like the add_planned_move factory method that forwards a bunch of arguments
20100601 04:48:57-!- Ivanovic_ [~ivanovic@dtmd-4db2d525.pool.mediaWays.net] has joined #wesnoth-dev
20100601 04:48:58< Crab_> gabba: note how ai::manager or other managers are synced with play_controller's lifecycle
20100601 04:50:05< gabba> interesting, I'll need a reference on how to manage the lifecycle of my class
20100601 04:51:05< Crab_> see src/playsingle_controller.cpp ctor and dtor and rc/play_controller.cpp ctor and dtor.
20100601 04:51:14< Crab_> s/ rc/ src
20100601 04:51:58< Crab_> note how things are put in resources::   in src/play_controller.cpp
20100601 04:52:24-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 276 seconds]
20100601 04:52:55-!- Ivanovic_ is now known as Ivanovic
20100601 04:57:47< gabba> Crab_: thanks, I've bookmarked those
20100601 04:59:01< gabba> I should really go for a wb:: namespace and names like wb::action, wb::move, wb::manager, and so on for my classes
20100601 04:59:14< Crab_> yes, that is a good thing
20100601 04:59:39< Crab_> in that case - rename the files, too 
20100601 04:59:49< Crab_> like whiteboard/manager.cpp
20100601 04:59:57< gabba> of course, and change the build files, and all that fun
20100601 05:00:21< Crab_> just ask around if wb:: or whiteboard:: is better.
20100601 05:00:31< Crab_> for example, pathfind is pathfind::, not pf::
20100601 05:00:42 * Crab_ is of course, lucky with ai
20100601 05:02:57< gabba> maybe plan:: or planning:: could do, as well
20100601 05:03:47< gabba> well, I'll see that tomorrow, as I'm quite tired
20100601 05:04:16< Crab_> maybe. I, personally, like wb:: more than plan::
20100601 05:07:35< CIA-86> gabba * r43099 /trunk/src/whiteboard/ (6 files): svn properties for new whiteboard files
20100601 05:08:45< gabba> gotta go now, 'night everybody
20100601 05:08:50< gabba> 'night Crab_
20100601 05:08:50< Crab_> gabba: and, when editing cmake files and adding new directories, please add a MSVC source group, as well.
20100601 05:09:00< Crab_> gabba: (I'll do it for src/whiteboard now)
20100601 05:09:10< Crab_> it's 1 line, too
20100601 05:09:23< Crab_> in the correct place, in the correct order (more detailed groups go last)
20100601 05:09:29< gabba> Crab_: ok, I'll keep it in mind for next time
20100601 05:09:51< gabba> thanks
20100601 05:10:00 * Espreon wonders why gabba just doesn't set the properties before committing the new files.
20100601 05:10:16< shadowmaster> or better yet, configure svn to do it automatically
20100601 05:10:55< gabba> Espreon: it's because I use git svn and I didn't bother yet to lookup wheter I can do something like "git svn propset blablabla"
20100601 05:11:08< shadowmaster> gabba: ah. you can't
20100601 05:11:26< shadowmaster> I believe svn propset can work with remote URIs though
20100601 05:11:57< gabba> shadowmaster: ok, but in this case I still have to commit to svn first, don't I
20100601 05:12:06< shadowmaster> indeed
20100601 05:12:07< Espreon> I'd still use git svn if I weren't lazy.
20100601 05:12:21 * Espreon would have to dig up his .git directories and rebase...
20100601 05:13:04< gabba> well, gotta go
20100601 05:13:06< CIA-86> crab * r43100 /trunk/src/CMakeLists.txt: added source group for whiteboard, for MSVC
20100601 05:13:31-!- gabba [~gabba@wesnoth/developer/gabba] has left #wesnoth-dev []
20100601 05:15:58-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has quit [Quit: Leaving.]
20100601 05:16:41-!- Crab_ [~Crab_@wesnoth/developer/crab] has quit [Quit: Leaving.]
20100601 05:30:29-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has joined #wesnoth-dev
20100601 05:50:38-!- k23z__ [k23z__@188.26.49.158] has quit [Quit: Leaving]
20100601 06:11:18-!- Elvish_Pillager [~eli@71-10-224-192.dhcp.oxfr.ma.charter.com] has quit [Quit: Hi! I'm a quit message virus vaccine. If you see a quit message virus, don't replace your quit message with it!]
20100601 06:12:46-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20100601 06:24:55-!- CIA-86 [cia@208.69.182.149] has quit [Ping timeout: 265 seconds]
20100601 06:32:39-!- silene [~plouf@bau91-1-82-239-244-109.fbx.proxad.net] has joined #wesnoth-dev
20100601 06:35:58-!- silene [~plouf@bau91-1-82-239-244-109.fbx.proxad.net] has quit [Changing host]
20100601 06:35:58-!- silene [~plouf@wesnoth/developer/silene] has joined #wesnoth-dev
20100601 06:39:22-!- shadowm_laptop [ignacio@wesnoth/developer/shadowmaster] has quit [Quit: I want this window]
20100601 06:39:52-!- shadowm_laptop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20100601 06:42:38-!- rigved [~rigved@116.72.163.243] has joined #wesnoth-dev
20100601 06:54:11-!- CIA-87 [cia@208.69.182.149] has joined #wesnoth-dev
20100601 06:58:16-!- FAAB [~huajie@219.142.228.105] has joined #wesnoth-dev
20100601 07:14:15-!- FAAB [~huajie@219.142.228.105] has quit [Quit: Leaving.]
20100601 07:16:44-!- shadowm_laptop [ignacio@wesnoth/developer/shadowmaster] has quit [Quit: good night]
20100601 07:17:08< Rhonda> silene: --disable-maintainer-mode doesn't seem to do anything, and looking at the generated configure code, it only seems to check for --enable-maintainer-mode in the code
20100601 07:20:43< silene> Rhonda: here it tests for both and it properly sets the MAINTAINER_MODE_{TRUE,FALSE} variables; not sure what they are supposed to do though
20100601 07:20:43-!- ABCD [~abcd@gentoo/developer/abcd] has quit [Quit: Leaving]
20100601 07:21:04-!- ABCD [~abcd@gentoo/developer/abcd] has joined #wesnoth-dev
20100601 07:21:19< Rhonda> I won't investigate further now and delay 1.8.2 though.
20100601 07:25:06< silene> Rhonda: i have checked further and --disable-maintainer-mode properly disables the dependencies for Makefile.in, configure, and aclocal.m4
20100601 07:30:31< silene> and config.h.in
20100601 07:31:49< silene> so, automake, autoconf, aclocal, and autoheader, i guess that's it, all of them are disabled
20100601 07:52:03-!- FAAB [~huajie@219.142.228.105] has joined #wesnoth-dev
20100601 08:07:31-!- qemqemqem__ [~quassel@ip70-177-181-137.dc.dc.cox.net] has quit [Ping timeout: 272 seconds]
20100601 08:08:31-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has quit [Ping timeout: 252 seconds]
20100601 08:27:02-!- crimson_penguin [~ben@wesnoth/developer/crimsonpenguin] has quit [Quit: crimson_penguin]
20100601 08:34:01-!- rigved [~rigved@116.72.163.243] has quit [Quit: Bye]
20100601 08:37:04< Rhonda> silene: Then I still wonder why the autotools are run over and over in my build process.
20100601 08:43:37< CIA-87> silene * r43102 /trunk/src/ai/composite/value_translator.hpp: Avoided costly roundtrip through strings.
20100601 08:43:39< CIA-87> silene * r43103 /trunk/src/ai/testing/ca.cpp: Avoided costly roundtrip through strings.
20100601 08:43:43< CIA-87> silene * r43104 /trunk/src/play_controller.cpp: Avoided costly roundtrip through strings.
20100601 08:43:44< CIA-87> silene * r43101 /trunk/src/game_display.cpp: Avoided costly roundtrip through strings.
20100601 08:47:04< silene> Rhonda: no idea what goes wrong for you; here, when i pass the --disable-maintainer-mode option, the line "$(srcdir)/Makefile.in:  $(srcdir)/Makefile.am  $(am__configure_deps)" from the generated Makefile (around line 397) becomes "$(srcdir)/Makefile.in: # $(srcdir)/Makefile.am  $(am__configure_deps)", hence disabling dependencies
20100601 08:49:34-!- silene [~plouf@wesnoth/developer/silene] has quit [Quit: Leaving.]
20100601 09:01:41-!- zookeeper [~l@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20100601 09:58:31-!- Ivanovic [~ivanovic@dtmd-4db2d525.pool.mediaWays.net] has quit [Changing host]
20100601 09:58:31-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev
20100601 10:00:50< Ivanovic> moin
20100601 10:39:23< Ivanovic> zookeeper: http://forums.wesnoth.org/viewtopic.php?f=4&t=30189&start=0
20100601 10:46:28-!- FAAB [~huajie@219.142.228.105] has quit [Quit: Leaving.]
20100601 10:51:24-!- Crab_ [~Crab@wesnoth/developer/crab] has joined #wesnoth-dev
20100601 10:53:42-!- Smar [smar@freenet/translator/finnish/Smar] has quit [Quit: Nyan]
20100601 10:54:50-!- Smar [smar@freenet/translator/finnish/Smar] has joined #wesnoth-dev
20100601 10:57:09< Crab_> Blitzmerker: create a test scenario which will allow to reproduce your [unit] issue, and post a link, or just pm it to me (Crab on forum)
20100601 11:02:26-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has joined #wesnoth-dev
20100601 11:08:45-!- Vetinari [~lukjad007@unaffiliated/lukjad007] has quit [Ping timeout: 276 seconds]
20100601 11:09:09-!- Vetinari [~lukjad007@unaffiliated/lukjad007] has joined #wesnoth-dev
20100601 11:14:25-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has quit [Ping timeout: 252 seconds]
20100601 11:27:26-!- mjs-de [~mjs-de@vpw.wh.uni-dortmund.de] has joined #wesnoth-dev
20100601 11:45:17-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20100601 11:50:09-!- stikonas [~and@wesnoth/translator/stikonas] has joined #wesnoth-dev
20100601 12:00:35-!- ancestral [~ancestral@97-116-112-18.mpls.qwest.net] has quit [Quit: And that’s the end of THAT chapter.]
20100601 12:08:41-!- nagbot [~nagbot@wesnoth/bot/nagbot] has joined #wesnoth-dev
20100601 12:16:03-!- loonybot [~loonybot@ppp79-139-137-245.pppoe.spdop.ru] has joined #wesnoth-dev
20100601 12:16:03-!- loonybot [~loonybot@ppp79-139-137-245.pppoe.spdop.ru] has quit [Changing host]
20100601 12:16:03-!- loonybot [~loonybot@wesnoth/bot/loonybot] has joined #wesnoth-dev
20100601 12:16:58-!- loonycyborg [~sergey@wesnoth/developer/loonycyborg] has joined #wesnoth-dev
20100601 13:02:57-!- euschn [~eugen@wesnoth/developer/euschn] has joined #wesnoth-dev
20100601 13:12:10-!- mjs-de [~mjs-de@vpw.wh.uni-dortmund.de] has quit [Remote host closed the connection]
20100601 13:15:33-!- mjs-de [~mjs-de@vpw.wh.uni-dortmund.de] has joined #wesnoth-dev
20100601 13:40:18-!- happygrue [~George@wesnoth/developer/wintermute] has quit [Read error: Connection reset by peer]
20100601 13:40:33-!- happygrue [~George@wesnoth/developer/wintermute] has joined #wesnoth-dev
20100601 13:45:57-!- stikonas [~and@wesnoth/translator/stikonas] has quit [Remote host closed the connection]
20100601 14:09:10-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has joined #wesnoth-dev
20100601 14:10:28-!- Ken_Oh [~briang@static-71-178-174-220.washdc.fios.verizon.net] has joined #wesnoth-dev
20100601 14:35:54-!- Johannes13 [~Johannes@unaffiliated/johannes13] has joined #wesnoth-dev
20100601 14:36:39-!- King_Elendil [~King_Elen@75.143.233.138] has joined #wesnoth-dev
20100601 14:41:46-!- King_Elendil [~King_Elen@75.143.233.138] has quit [Ping timeout: 264 seconds]
20100601 15:06:01-!- King_Elendil [~King_Elen@75.143.233.138] has joined #wesnoth-dev
20100601 15:08:27-!- billynux [~billy@wesnoth/developer/billynux] has joined #wesnoth-dev
20100601 15:10:53-!- King_Elendil [~King_Elen@75.143.233.138] has quit [Ping timeout: 260 seconds]
20100601 15:12:28-!- billynux [~billy@wesnoth/developer/billynux] has quit [Client Quit]
20100601 15:21:18-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has quit [Read error: Connection reset by peer]
20100601 15:39:29-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has quit [Quit: Leaving.]
20100601 16:03:42-!- alink [~alink@wesnoth/developer/alink] has joined #wesnoth-dev
20100601 16:13:49-!- Gambit [~Gambit@pa-67-234-73-7.dhcp.embarqhsd.net] has joined #wesnoth-dev
20100601 16:19:30-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has joined #wesnoth-dev
20100601 16:20:10-!- alink [~alink@wesnoth/developer/alink] has quit [Remote host closed the connection]
20100601 16:38:11-!- alink [~alink@wesnoth/developer/alink] has joined #wesnoth-dev
20100601 16:40:52-!- Elvish_Pillager [~eli@71-10-224-192.dhcp.oxfr.ma.charter.com] has joined #wesnoth-dev
20100601 16:43:39-!- dtiger [~dtiger@dynamic-vpdn-93-125-12-125.telecom.by] has joined #wesnoth-dev
20100601 16:48:47-!- thespaceinvader [~chatzilla@wesnoth/artist/thespaceinvader] has joined #wesnoth-dev
20100601 17:02:42-!- King_Elendil [~King_Elen@75.143.233.138] has joined #wesnoth-dev
20100601 17:02:57-!- King_Elendil [~King_Elen@75.143.233.138] has quit [Client Quit]
20100601 17:26:19-!- billynux [~c8078d05@wesnoth/developer/billynux] has joined #wesnoth-dev
20100601 17:28:10-!- Crab_ [~Crab@wesnoth/developer/crab] has quit [Quit: Miranda IM! Smaller, Faster, Easier. http://miranda-im.org]
20100601 17:31:06-!- dtiger [~dtiger@dynamic-vpdn-93-125-12-125.telecom.by] has quit [Remote host closed the connection]
20100601 17:32:54< billynux> wesbot: seen mordante
20100601 17:32:54< wesbot> billynux: The person with the nick mordante last spoke 1d 22h ago.  1d 22h ago was here and on the channel #wesnoth-de with the message: Quit: Leaving
20100601 17:33:01-!- allefant [~elias@allegro/developer/allefant] has joined #wesnoth-dev
20100601 17:42:07-!- billynux [~c8078d05@wesnoth/developer/billynux] has quit [Quit: Page closed]
20100601 17:42:08-!- dtiger [~dtiger@dynamic-vpdn-93-125-12-125.telecom.by] has joined #wesnoth-dev
20100601 17:49:49-!- cjhopman_ [~chris@71-90-6-151.dhcp.ftbg.wi.charter.com] has joined #wesnoth-dev
20100601 17:49:49-!- cjhopman_ [~chris@71-90-6-151.dhcp.ftbg.wi.charter.com] has quit [Changing host]
20100601 17:49:49-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has joined #wesnoth-dev
20100601 17:55:34< CIA-87> alink * r43105 /trunk/src/ (display.cpp display.hpp game_display.cpp): 
20100601 17:55:34< CIA-87> Give specific layer name to footsteps and editor's mouseover.
20100601 17:55:34< CIA-87> Remove unused function. No visible change.
20100601 18:18:01< fendrin> Hello, anyone around who would like to verify a problem with the latex documentation timotei wrote? I am not able to compile the latex code on my system.
20100601 18:25:14-!- crimson_penguin [~ben@wesnoth/developer/crimsonpenguin] has joined #wesnoth-dev
20100601 18:30:16-!- Johannes13_ [~Johannes@unaffiliated/johannes13] has joined #wesnoth-dev
20100601 18:34:39-!- Johannes13 [~Johannes@unaffiliated/johannes13] has quit [Ping timeout: 276 seconds]
20100601 18:40:14-!- stikonas [~and@bcm-131-111-247-104.girton.cam.ac.uk] has joined #wesnoth-dev
20100601 18:40:14-!- stikonas [~and@bcm-131-111-247-104.girton.cam.ac.uk] has quit [Changing host]
20100601 18:40:14-!- stikonas [~and@wesnoth/translator/stikonas] has joined #wesnoth-dev
20100601 18:57:23-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20100601 18:57:28-!- qemqemqem__ [~quassel@ip70-177-181-137.dc.dc.cox.net] has joined #wesnoth-dev
20100601 18:58:27-!- euschn [~eugen@wesnoth/developer/euschn] has quit [Quit: Leaving.]
20100601 19:08:37-!- JordiGH [~jordi@unaffiliated/jordigh] has joined #wesnoth-dev
20100601 19:11:46-!- cjhopman_ [~chris@wesnoth/developer/cjhopman] has quit [Ping timeout: 264 seconds]
20100601 19:12:04-!- parolang [~user@8e4a01246100775874c4f448e9887093.oregonrd-wifi-1261.amplex.net] has joined #wesnoth-dev
20100601 19:22:15-!- JordiGH [~jordi@unaffiliated/jordigh] has left #wesnoth-dev ["Leaving"]
20100601 19:31:14-!- billynux [~billy@wesnoth/developer/billynux] has joined #wesnoth-dev
20100601 19:42:31-!- happygrue [~George@wesnoth/developer/wintermute] has quit [Read error: Connection reset by peer]
20100601 19:42:44-!- happygrue [~George@wesnoth/developer/wintermute] has joined #wesnoth-dev
20100601 19:44:34-!- mordante [~mordante@wesnoth/developer/mordante] has joined #wesnoth-dev
20100601 19:44:46< mordante> servus
20100601 19:50:48-!- kevg [~kevg@94.232.5.102] has joined #wesnoth-dev
20100601 19:50:57< kevg> hello
20100601 19:51:04< mordante> hi billynux you were looking for me
20100601 19:51:07< mordante> hi kevg 
20100601 19:51:16< billynux> hi mordante 
20100601 19:51:18< billynux> yes...
20100601 19:51:28< billynux> mordante,  I sent you an email with many questions
20100601 19:51:57< mordante> billynux, will check my email later, I've cold so didn't read the last few days
20100601 19:52:18-!- gabba [~gabba@wesnoth/developer/gabba] has joined #wesnoth-dev
20100601 19:53:12< billynux> mordante, ok, do you want me to run through what was on the email now? (or you'll read it on your own later?)
20100601 19:53:32< mordante> no I'll read the email later, will you be around here longer?
20100601 19:54:09-!- kevg [~kevg@94.232.5.102] has left #wesnoth-dev []
20100601 19:54:17-!- kevg [~kevg@94.232.5.102] has joined #wesnoth-dev
20100601 19:55:19< billynux> mordante, I think so, yes... I'm trying to configure my squid with local proxy support to test auth proxies
20100601 19:56:04< mordante> ok
20100601 19:59:07-!- silene [~plouf@wesnoth/developer/silene] has joined #wesnoth-dev
20100601 20:04:49< fendrin> I have spammed the forum by advertising an indi game. I hope that wasn't across the line. http://www.wesnoth.org/forum/viewtopic.php?p=431042#p431042
20100601 20:06:44< fendrin> shadowmaster: ^
20100601 20:09:11< mordante> fendrin, which latex file?
20100601 20:10:31< fendrin> trunk/utils/java/eclipse_plugin
20100601 20:10:38< fendrin> Just call make there.
20100601 20:11:15< mordante> works for me, but why not in the normal build system?
20100601 20:12:02< mordante> and I would expect a pdf output nowadays ;-)
20100601 20:12:24< fendrin> mordante: hmm, doesn't it produce a pdf?
20100601 20:12:25-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has quit [Ping timeout: 265 seconds]
20100601 20:12:51< fendrin> I get "! LaTeX Error: File `utf8x.def' not found."
20100601 20:13:14< fendrin> Isn't makefile a normal build system?
20100601 20:13:30< fendrin> Or do you expect the toplevel makefile to include the local one?
20100601 20:13:45< fendrin> Or should the toplevel makefile do all the work?
20100601 20:14:02< mordante> No just in the normal build system if you want to include making it
20100601 20:14:14< mordante> no idea why utf8x instead of utf8
20100601 20:14:21< mordante> billynux, reading your email
20100601 20:14:46< billynux> ok
20100601 20:14:57< mordante> gabba, wesnoth no longer compiles for me, are you sure boost/make_shared.hpp is in boost 1.35?
20100601 20:15:21-!- DesertPanther [~Khalid@unaffiliated/desertpanther] has joined #wesnoth-dev
20100601 20:15:40< gabba> mordante: I think I checked, let me check again
20100601 20:17:13-!- parolang [~user@8e4a01246100775874c4f448e9887093.oregonrd-wifi-1261.amplex.net] has left #wesnoth-dev ["ERC Version 5.3 (IRC client for Emacs)"]
20100601 20:22:42-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has joined #wesnoth-dev
20100601 20:22:45< billynux> mordante, anything you would like to chat _now_ -> I'll go afk for ~50' otherwise
20100601 20:22:54< billynux> ?
20100601 20:23:22< mordante> no we can talk in about one hour
20100601 20:23:32< gabba> mordante: damn, you're right and I fooled myself, it's not in 1.35, sorry for the trouble
20100601 20:23:39< mordante> just ping me when you're back
20100601 20:23:53< gabba> mordante: I'll take this away right now
20100601 20:24:02< mordante> gabba, no problem and thanks for fixing it
20100601 20:24:35-!- stikonas_ [~and@wesnoth/translator/stikonas] has joined #wesnoth-dev
20100601 20:24:54< billynux> ok mordante, see ya'
20100601 20:24:56-!- billynux [~billy@wesnoth/developer/billynux] has quit [Quit: Leaving]
20100601 20:26:42-!- silene1 [~plouf@bau91-1-82-239-244-109.fbx.proxad.net] has joined #wesnoth-dev
20100601 20:26:47-!- Netsplit *.net <-> *.split quits: silene, stikonas
20100601 20:27:31< CIA-87> eleazar * r43106 /trunk/ (4 files in 3 dirs): made dirtier cobbles for normal castles, reserving the clean stones ones for the keep. Thus the keeps stand out a little better, and castles blend in the terrain a little better.
20100601 20:29:28-!- silene1 is now known as silene
20100601 20:29:33-!- silene [~plouf@bau91-1-82-239-244-109.fbx.proxad.net] has quit [Changing host]
20100601 20:29:33-!- silene [~plouf@wesnoth/developer/silene] has joined #wesnoth-dev
20100601 20:34:05-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has quit [Ping timeout: 240 seconds]
20100601 20:36:52-!- Johannes13__ [~Johannes@pD950000B.dip0.t-ipconnect.de] has joined #wesnoth-dev
20100601 20:39:59-!- Johannes13_ [~Johannes@unaffiliated/johannes13] has quit [Ping timeout: 240 seconds]
20100601 20:40:04< CIA-87> gabba * r43107 /trunk/src/whiteboard/planned_action_manager.cpp: Removed use of boost::make_shared since it's not in boost 1.35
20100601 20:40:13< gabba> mordante: here you go
20100601 20:41:31< mordante> gabba, thanks it now compiles again
20100601 20:46:05< silene> mordante: by the way, i have no trouble compiling with -Wstrict-aliasing; which version of gcc 4.5 are you using?
20100601 20:47:09< mordante> silene, 4.5.0-4 from debian experimental
20100601 20:47:41-!- stikonas_ [~and@wesnoth/translator/stikonas] has quit [Remote host closed the connection]
20100601 20:47:42< Ivanovic> yes, i do know that both, windows and mac binary are out, but i won't type the announcement stuff today, will do it tomorrow while having my breakfast tea
20100601 20:48:28-!- stikonas [~and@wesnoth/translator/stikonas] has joined #wesnoth-dev
20100601 20:48:58< silene> mordante: strange, it may have been an issue introduced in that version then; it works fine for me with 4.5.0-3
20100601 20:50:40< mordante> silene, I can try again, I recently upgraded, think I was on 4.5.0-2 or -1 before
20100601 20:51:33-!- Johannes13__ is now known as Johannes13
20100601 20:51:37-!- Johannes13 [~Johannes@pD950000B.dip0.t-ipconnect.de] has quit [Changing host]
20100601 20:51:37-!- Johannes13 [~Johannes@unaffiliated/johannes13] has joined #wesnoth-dev
20100601 21:02:33-!- billynux [~billy@wesnoth/developer/billynux] has joined #wesnoth-dev
20100601 21:02:39< billynux> ping mordante 
20100601 21:03:17< mordante> hi billynux 
20100601 21:03:37< mordante> billynux, I had a look at your email and some comments
20100601 21:03:50< billynux> great
20100601 21:04:10< mordante> first of all I like the amount of documentation written in the tex file
20100601 21:04:42< mordante> some remarks regarding it, IMO the pdf shouldn't be in the repository, it's output
20100601 21:05:43< mordante> in 4.2 you talks about ana_uint, maybe it should also have a size attached to it eg ana_uint32 since the size is also platform dependent
20100601 21:06:14< mordante> the address type is it an ip address or can it also be a hostname?
20100601 21:06:25< billynux> both
20100601 21:06:40< mordante> maybe make it clearer at least in the documentation 
20100601 21:06:46< billynux> it resolves it
20100601 21:06:47< billynux> ok
20100601 21:07:20< mordante> and in the part of the send_type I missed the difference between the buffer type lifetime, which is written in 5.1, maybe add a reference at send_type
20100601 21:07:27< billynux> I put somewhere that the purpose of ana_uint (which is really a uint32_t is if you want to serialize data and send it to a client with a  possibly different arch.
20100601 21:07:43< billynux> ok
20100601 21:09:08< mordante> billynux, I read that that's the reason for it, therefore I think its size should either be in the name or documented there
20100601 21:09:30< billynux> agreed
20100601 21:10:10< mordante> I sec
20100601 21:17:26-!- shadowm_laptop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev
20100601 21:19:03< shadowmaster> fendrin: you have done something so horribly wrong that you should be banned from the Internet forever
20100601 21:19:08< shadowmaster> fendrin: nah, don't worry
20100601 21:22:53< Rhonda> silene: I can run configure --disable-maintainer-mode as often as I like, it doesn't get rid of the am__configure_deps entries in the Makefiles. :/
20100601 21:23:06-!- Daltx [~Daltx@CPE001e5840eaf6-CM00195ee19c52.cpe.net.cable.rogers.com] has joined #wesnoth-dev
20100601 21:24:07< silene> Rhonda: it won't, but there is a # before them when you use the option; so make ignores them
20100601 21:24:38< Rhonda> Erm, no, isn't here.
20100601 21:25:03< silene> here, it is: "$(srcdir)/Makefile.in: # $(srcdir)/Makefile.am  $(am__configure_deps)"
20100601 21:25:35< Rhonda> $(srcdir)/Makefile.in:  $(srcdir)/Makefile.am  $(am__configure_deps)
20100601 21:26:13< mordante> billynux, I also wonder about ana::create_server vs. static ana::server::create what do you prefer?
20100601 21:27:30< billynux> its a good question, create_server does seem to C oriented
20100601 21:27:43< mordante> jup
20100601 21:27:54< billynux> s/to/too/
20100601 21:28:25< silene> Rhonda: you should fill a bug against automake or autoconf (depending on whether Makefile.in contains this line or not: "$(srcdir)/Makefile.in: @MAINTAINER_MODE_TRUE@ $(srcdir)/Makefile.am  $(am__configure_deps)")
20100601 21:28:27< billynux> but struct server { virtual static? create } :)
20100601 21:28:32< shadowmaster> there's nobody reworking the game's builtin add-on client atm, right?
20100601 21:29:16< mordante> static methods can't be virtual
20100601 21:29:29< silene> unfortunately
20100601 21:30:18< billynux> exactly :)
20100601 21:30:38< shadowmaster> I've been throwing ideas around for making src/addon/* code easier to work with, reduce the amount of redundant method parameters and code and possibly improve dependency tracking in the process
20100601 21:30:38< billynux> that was my point
20100601 21:30:55< mordante> why would you need that in this case?
20100601 21:31:10< shadowmaster> I do wish GUI2 was already usable enough for getting rid of the GUI1 crap though
20100601 21:31:11< mordante> the create_server also can't be virtual
20100601 21:31:21< mordante> shadowmaster, me too ;-)
20100601 21:31:32< billynux> ah, ok ok, I misunderstood you
20100601 21:33:18< mordante> ok
20100601 21:33:41< mordante> regarding the spitting of the headers, seems a good idea to me
20100601 21:33:50< billynux> ok
20100601 21:34:07< mordante> do you want to do two or three headers ana.hpp time.hpp and private/details.hpp ?
20100601 21:34:15< billynux> mordante,  what do you think about normal form in the class methods?
20100601 21:34:38< mordante> btw IMO a .dotfile is rather evil, make searching the details hared
20100601 21:34:45< billynux> (I *do* want to have stuff like send_all_from, send_all_except, etc..., but they are not even remotely primitive)
20100601 21:35:24< mordante> what do you mean with `normal form' ?
20100601 21:36:33< billynux> the same as in databases, a class should have only the primitive methods
20100601 21:36:42< billynux> then *other* classes should have utilities
20100601 21:36:55< billynux> e.g. send_all is actually a send_if with condition true
20100601 21:37:09< billynux> send_one is a send_if with condition (client_id == param) etc....
20100601 21:38:01< Rhonda> silene: $(srcdir)/Makefile.in: @MAINTAINER_MODE_TRUE@ $(srcdir)/Makefile.am  $(am__configure_deps)
20100601 21:38:22< Rhonda> S["MAINTAINER_MODE_FALSE"]="#"
20100601 21:38:22< Rhonda> S["MAINTAINER_MODE_TRUE"]=""
20100601 21:38:24< Rhonda> erm …
20100601 21:38:31< Rhonda> silene: I think you got that backwards. ;)
20100601 21:39:04< mordante> ah ok, I think with C++ some people like everything a member and others not, personally I make a lot of things a member, but I might change my opinion
20100601 21:39:20< mordante> IIRC std::string has 138 member functions
20100601 21:40:20-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has joined #wesnoth-dev
20100601 21:40:23< mordante> what do you prefer?
20100601 21:40:26< billynux> well, the Interface Segregation Principle says that a user of class should want to use its entire interface
20100601 21:40:45< billynux> it is possible that someone doesn't want to use something like send_all_except
20100601 21:41:09< billynux> but I do like having a bloated interface with many utilities (although the implementations are trivial)
20100601 21:41:20< billynux> (bloated is perhaps not the best choice of words)
20100601 21:41:43< silene> Rhonda: no, i got it right; if you have S["MAINTAINER_MODE_TRUE"]="", it means that you are running with --enable-maintainer-mode
20100601 21:42:12< Rhonda> No, I'm not.
20100601 21:42:32< mordante> you can call it a rich interface ;-)
20100601 21:42:56< silene> Rhonda: that's the content of my configure file: http://wesnoth.pastebin.com/KkpwJBeL
20100601 21:42:58< billynux> I can even make every operation (except send_if) virtual and implement them via send_if (but allow someone to reimplement it if they so choose)
20100601 21:43:02< mordante> but I also, at the moment, prefer everything in the interface
20100601 21:43:04< Rhonda> $ grep maintainer-mode config.log 
20100601 21:43:04< Rhonda>   $ ./configure --host=powerpc-linux-gnu --build=powerpc-linux-gnu --disable-strict-compilation --disable-maintainer-mode --disable-rpath --prefix=/usr --mandir=${prefix}/share/man --bindir=${prefix}/games --with-datadir-name=wesnoth/1.8 --with-localedir=${prefix}/share/games/wesnoth/1.8/locale --with-fifodir=/var/run/wesnothd --datadir=${prefix}/share/games --enable-server --with-fribidi --enable-python-install CFLAGS=-g -O2 ...
20100601 21:43:10< Rhonda> ... LDFLAGS=-Wl,-z,defs
20100601 21:43:12< billynux> ok
20100601 21:43:19< Rhonda> silene: There you are, now tell me again I'm running with --enable-maintainer-mode, I dare you. :)
20100601 21:44:03< silene> Rhonda: what i'm telling you is that the shell variable USE_MAINTAINER_MODE contains "yes" (unless your configure file is different from mine)
20100601 21:44:10< mordante> billynux, if you don't like send_all_except, maybe change it to send_all_if, only not sure how much the implementation would differ from send_if
20100601 21:44:36< Rhonda> silene: Right. But how do I switch that? That environment variable isn't set anywhere.
20100601 21:44:39< billynux> e.g. send_all_from( [client1, client2, client3], buf, handler) is O(n^2) if it uses send_if directly but I can implement it as O
20100601 21:44:50< billynux> ... O(n) since I know my client list is sorted...
20100601 21:45:12< silene> Rhonda: it's set 10 lines before depending on whether --disable-maintainer-mode is passed or not
20100601 21:45:42< Rhonda> silene: I only see a test for ${enable_maintainer_mode} there.
20100601 21:46:13< Rhonda> autoconf 2.65, btw.
20100601 21:47:08< Rhonda> configure:2676: checking whether to disable maintainer-specific portions of Makefiles
20100601 21:47:12< Rhonda> configure:2685: result: no
20100601 21:47:14< Rhonda> *grrr*
20100601 21:47:20< Rhonda> why so when I hand it --disable-maintainer-mode, damnit!
20100601 21:47:42-!- zookeeper [~l@wesnoth/developer/zookeeper] has quit [Ping timeout: 272 seconds]
20100601 21:48:09< mordante> send_all_from?
20100601 21:49:54< billynux> yes, from a container
20100601 21:50:01< kevg> I wanna write a candidate action for healer control from EasyCoding but now I have now idea how to do it. Could you give me some plan?
20100601 21:50:06-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has quit [Ping timeout: 245 seconds]
20100601 21:50:15< billynux> it would be good since most servers would like to keep lists/groups depending on the type of clients
20100601 21:50:57< billynux> so, the code would look like server_->send_all_from( group1_list_, buffer, this );
20100601 21:51:07< mordante> ok an internal not yet coded function
20100601 21:51:23< silene> Rhonda: what if you try instead "--enable-maintainer-mode=no"?
20100601 21:51:49< billynux> yes... as I said, there are many such interesting utility-functions (and send_all_except was one of them)
20100601 21:51:51< Rhonda> silene: Will try so later, I'm now really wanting to get 1.8.2 off my disk, and it's compiling (again)
20100601 21:52:36< mordante> but you weren't happy with that name, so what about send_all_if, more in line with send_if ?
20100601 21:52:37< fendrin> shadowmaster: :-)
20100601 21:54:32-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20100601 21:54:45< billynux> mordante, I'm ok with send_all_except (I was just doubtful about the interfaces and whether I should add a different one)
20100601 21:54:59< silene> Rhonda: perhaps it's again a shell issue
20100601 21:55:18< billynux> send_if *is* a send_all_if -> send_all_except could be overloaded for containers and  a single ID (current implementation)
20100601 21:56:39< mordante> ok
20100601 21:57:39< mordante> I also had a look at your image and looks fine (but I'll look again once I feel fine again)
20100601 21:58:12< billynux> mordante: so, what do you think about progress so far? and... when do you want me to upload ana to wesnoth? (the repository)
20100601 21:58:32< mordante> personally I prefer asap
20100601 21:58:49< Rhonda> silene: The annoying thing about these things is that it's always me who stumbles into the most obscure things. Often enough I can consider this helpful, but it's start to increasingly bother me. :(
20100601 21:59:00< billynux> good, so... svn up, add my files and commit?
20100601 21:59:07< mordante> I think you're doing a fine job, and I really like the fact that you wrote quite a bit of documentations already
20100601 21:59:18< billynux> :)
20100601 21:59:44< mordante> but also you wrote a nice amount of code
20100601 22:00:37-!- zookeeper [~l@wesnoth/developer/zookeeper] has joined #wesnoth-dev
20100601 22:00:38< billynux> well, quality before quantity... it needs a lot of work still
20100601 22:00:46< mordante> yes svn up and add your code to an ana subdirectory (use it can be extracted later, if needed) 
20100601 22:00:56< mordante> only leave the pdf file out
20100601 22:01:51< mordante> I agree with quality over quantity, but still a nice amount, especially since it's library code and needing quite a bit of design time
20100601 22:03:46< billynux> ok
20100601 22:03:55-!- noy [~Noy@d206-116-77-163.bchsia.telus.net] has joined #wesnoth-dev
20100601 22:03:58-!- noy [~Noy@d206-116-77-163.bchsia.telus.net] has quit [Changing host]
20100601 22:03:58-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev
20100601 22:04:06< billynux> ... and thanks for the thumbs up
20100601 22:04:20< mordante> you're welcome
20100601 22:04:30< billynux> I do hope the lib grows up to be used in other games... time... and it's quality will decide
20100601 22:04:31< mordante> do you have more questions?
20100601 22:04:46< billynux> not ATM, get well soon
20100601 22:04:52< mordante> maybe also outside games ;-)
20100601 22:04:59< billynux> "outside" ?
20100601 22:05:19< billynux> you mean other games? -> that is what I hope
20100601 22:05:58< mordante> that it will also be used in software that isn't a game
20100601 22:06:14< mordante> thanks
20100601 22:06:27< mordante> then I'm off again, try to be around again tomorrow
20100601 22:07:38< billynux> ok, see you then
20100601 22:07:59-!- mordante [~mordante@wesnoth/developer/mordante] has quit [Quit: Leaving]
20100601 22:11:11< silene> Rhonda: i just checked, if it says "checking whether to disable maintainer-specific portions of Makefiles... no", it correctly parsed the option, it's just a typo in the message (it should say "enable")
20100601 22:11:14< silene> but that doesn't explain why your variables are not set properly later
20100601 22:13:34< Rhonda> Somehow it doesn't, does it. :)
20100601 22:15:17< silene> Rhonda: when you have time, could you try to move the AM_MAINTAINER_MODE line much later in configure.ac? parhaps you experience some kind of race with the initialization of automake specific code
20100601 22:15:18< Rhonda> When I do that snippet in /bin/bash it works properly.
20100601 22:15:57< Rhonda> It also works in dash
20100601 22:16:35< Rhonda> And in that chroot /bin/sh (which configure uses) is a symlink to bash
20100601 22:21:45-!- kevg [~kevg@94.232.5.102] has left #wesnoth-dev []
20100601 22:32:30-!- allefant [~elias@allegro/developer/allefant] has quit [Quit: Leaving]
20100601 22:32:37-!- dtiger [~dtiger@dynamic-vpdn-93-125-12-125.telecom.by] has quit [Remote host closed the connection]
20100601 22:32:43-!- Johannes13_ [~Johannes@unaffiliated/johannes13] has joined #wesnoth-dev
20100601 22:35:12-!- Ken_Oh [~briang@static-71-178-174-220.washdc.fios.verizon.net] has quit [Read error: Connection reset by peer]
20100601 22:36:25-!- Johannes13 [~Johannes@unaffiliated/johannes13] has quit [Ping timeout: 252 seconds]
20100601 22:53:15-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy]
20100601 22:55:15-!- Blueblaze [~nick@adsl-76-202-16-2.dsl.hstntx.sbcglobal.net] has joined #wesnoth-dev
20100601 23:02:06< billynux> people: Do I have commit priviledges to trunk?
20100601 23:02:18< billynux> Ivanovic, ping (question above)
20100601 23:02:38< billynux> I checked out trunk and added my work
20100601 23:02:55< billynux> (I added an "ana" subdir right at trunk)
20100601 23:03:10< shadowmaster> you seem to have
20100601 23:03:23< shadowmaster> you might need to switch your checkout to use svn+ssh with the corrent username though
20100601 23:03:28< shadowmaster> *correct
20100601 23:05:30< billynux> ok, I'll try the commit and see what happens, I just wanted a go-ahead from someone
20100601 23:05:42< shadowmaster> wait for Ivanovic then :P
20100601 23:05:57< billynux> ook
20100601 23:06:18< billynux> (you caught me just before hitting enter to the commit command :P )
20100601 23:09:52< zookeeper> i don't remember what you're working on, but "(I added an "ana" subdir right at trunk)" sounds like the wrong thing to do in any case
20100601 23:11:12< shadowmaster> libana, a network library
20100601 23:11:36< silene> why not under src?
20100601 23:11:36< shadowmaster> I'd say that it depends on how much you want to depend on the Wesnoth repository.
20100601 23:11:49< shadowmaster> and on whether it has its own build system, etc.
20100601 23:18:19< billynux> zookeeper, silene: ana is a library used to develop server/client network apps using an asynchronous model
20100601 23:18:26< billynux> (i.e. non-blocking operations)
20100601 23:18:34< billynux> you can check async-net-api.googlecode.com
20100601 23:19:04< billynux> ana does not depend on wesnoth in any way, but the idea is to rewrite wesnoth's net-stack using it
20100601 23:19:28< billynux> thats why I think an entire subdir is "ok"
20100601 23:20:00< billynux> (this is because mordante would like me to work on wesnoth's repository and statically link the code -> not compile ana as a lib)
20100601 23:20:34< silene> billynux: if ana is not lib then put it into src/, i don't care whether it's a separate project or not
20100601 23:20:38< zookeeper> billynux, ok, in that case i wouldn't know.
20100601 23:21:05< shadowmaster> I know people hate svn:externals but isn't cases like this that they are intended to be used?
20100601 23:21:10< shadowmaster> *for cases
20100601 23:21:54< billynux> silene: ana is a library, which will later be used for other games/apps (thats the idea)
20100601 23:22:21< silene> billynux: then you will remove it from wesnoth; but for now, it is some c++ code part of wesnoth, so it goes under src
20100601 23:22:43< billynux> I see
20100601 23:22:51< billynux> so, src/ana
20100601 23:23:15< silene> yes, please
20100601 23:23:55-!- Johannes13_ [~Johannes@unaffiliated/johannes13] has quit [Ping timeout: 248 seconds]
20100601 23:24:36< zookeeper> esr, THoT was using canrecruit="1" all over the place...i'm presuming wmllint doesn't catch that because of the quotes.
20100601 23:25:05< billynux> I will wait for Ivanovic to give me the go-ahead then (to put ana under src)
20100601 23:25:10< esr> zookeeper: Yeah, that's probably right.  Fix it if you like.
20100601 23:27:25< Ivanovic> billynux: uhm, yeah, you should have commit access
20100601 23:27:51< shadowmaster> *grumble*
20100601 23:28:33< Ivanovic> esr: i think zookeepers idea was more like "maybe you could improve wmllint to catch this one, too"
20100601 23:28:34< Ivanovic> ;)
20100601 23:29:00< esr> Oh, yeah, I guess I could do that :-)
20100601 23:33:06< zookeeper> esr, silene already fixed all the cases in THoT
20100601 23:33:20-!- silene [~plouf@wesnoth/developer/silene] has quit [Quit: Leaving.]
20100601 23:33:29< zookeeper> (and a few other ones in AOI too)
20100601 23:44:38-!- ancestral [~ancestral@12.145.225.25] has joined #wesnoth-dev
--- Log closed Wed Jun 02 00:00:29 2010