--- Log opened Sat Nov 23 00:00:18 2013 20131123 00:12:53-!- fabi [~fabi@wesnoth/developer/fendrin] has quit [Quit: Konversation terminated!] 20131123 00:14:37-!- knotwork [~markm@unaffiliated/knotwork] has quit [Quit: Leaving] 20131123 00:14:53-!- knotwork [~markm@142.68.128.190] has joined #wesnoth-dev 20131123 00:14:53-!- knotwork [~markm@142.68.128.190] has quit [Changing host] 20131123 00:14:53-!- knotwork [~markm@unaffiliated/knotwork] has joined #wesnoth-dev 20131123 00:17:54-!- fabi [~fabi@wesnoth/developer/fendrin] has joined #wesnoth-dev 20131123 00:22:01-!- happygrue [~happygrue@c-66-30-155-184.hsd1.ma.comcast.net] has joined #wesnoth-dev 20131123 00:22:01-!- happygrue [~happygrue@c-66-30-155-184.hsd1.ma.comcast.net] has quit [Changing host] 20131123 00:22:01-!- happygrue [~happygrue@wesnoth/developer/wintermute] has joined #wesnoth-dev 20131123 00:23:40-!- Gambit [~derek@pa-67-234-101-16.dhcp.embarqhsd.net] has joined #wesnoth-dev 20131123 00:23:42-!- Gambit [~derek@pa-67-234-101-16.dhcp.embarqhsd.net] has quit [Changing host] 20131123 00:23:42-!- Gambit [~derek@wesnoth/developer/grickit] has joined #wesnoth-dev 20131123 00:33:36-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev 20131123 00:47:00-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Quit: Leaving.] 20131123 00:48:46-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection] 20131123 01:00:14-!- exciton [chuck-the-@89.208.169.104] has quit [Read error: Connection reset by peer] 20131123 01:05:30-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev 20131123 01:10:11-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has joined #wesnoth-dev 20131123 01:11:02-!- BatmanGambit [~derek@wesnoth/developer/grickit] has joined #wesnoth-dev 20131123 01:11:02-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Read error: Connection reset by peer] 20131123 01:11:06-!- Alarantalara1 [~Adium@192-0-128-11.cpe.teksavvy.com] has joined #wesnoth-dev 20131123 01:12:37-!- Gambit [~derek@wesnoth/developer/grickit] has quit [Ping timeout: 264 seconds] 20131123 01:20:46-!- BatmanGambit is now known as Gambit 20131123 01:22:33-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has quit [Quit: Ik ga weg] 20131123 01:39:31-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has joined #wesnoth-dev 20131123 01:39:31-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has quit [Changing host] 20131123 01:39:31-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 01:54:45-!- knotwork [~markm@unaffiliated/knotwork] has quit [Quit: Leaving] 20131123 01:54:58-!- knotwork [~markm@unaffiliated/knotwork] has joined #wesnoth-dev 20131123 02:03:13-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has quit [Ping timeout: 272 seconds] 20131123 02:03:29-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has joined #wesnoth-dev 20131123 02:03:41-!- tomreyn [~tomreyn@p57954A69.dip0.t-ipconnect.de] has joined #wesnoth-dev 20131123 02:03:44-!- tomreyn [~tomreyn@p57954A69.dip0.t-ipconnect.de] has quit [Changing host] 20131123 02:03:44-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has joined #wesnoth-dev 20131123 02:14:40-!- janebot [~Gambot@unaffiliated/gambit/bot/gambot] has joined #wesnoth-dev 20131123 02:41:32-!- noy [~Noy@wesnoth/developer/noy] has joined #wesnoth-dev 20131123 02:51:42-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!] 20131123 03:03:45-!- gfgtdf [~chatzilla@e177125223.adsl.alicedsl.de] has quit [Quit: ChatZilla 0.9.90.1 [Firefox 24.0/20130910160258]] 20131123 03:18:57-!- Ivanovic_ [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev 20131123 03:20:56< mattsc> okay - this is all very messed up and essentially not invertible ... 20131123 03:21:21< mattsc> I am convinced by now that A's solution is the only feasible :( 20131123 03:23:03-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 245 seconds] 20131123 03:23:40-!- happygrue [~happygrue@wesnoth/developer/wintermute] has quit [Remote host closed the connection] 20131123 03:24:47-!- Ivanovic_ is now known as Ivanovic 20131123 03:27:52< AI0867> mattsc: what's the condition for the unit map disordering? 20131123 03:37:22< mattsc> AI0867: essentially it's like this (although there are nuances): 20131123 03:38:00< mattsc> units are selected by a filter in inverse or in which they were put on the map, followed by the recall units per side. 20131123 03:38:36< mattsc> However, when a save is loaded, they are selected in this order: 20131123 03:39:28< mattsc> leader of the highest side, followed by all the other units of the side in order in which they were put on the map (not inverse order!), followed by the same for units of the next lower side, followed by recall units in order of side ... 20131123 03:40:13< mattsc> Thus, if I play straight through a scenario, if can get a very different order than when I load an intermediate savegame. 20131123 03:40:32< mattsc> It's a cheater detector. It shows you who loaded an intermediate save :D 20131123 03:41:16< mattsc> Anyway, it's quite easy to show that it is impossible to reconstruct the first type of order from the second without additional information. 20131123 03:43:01< mattsc> (and just to be clear here, I do not consider save/loading cheating. I cheat much more blatantly using debug mode if I want to) 20131123 03:43:40-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has quit [Read error: Connection reset by peer] 20131123 03:44:28< mattsc> So in summary, I believe that anonymissimus is correct and that an order needs to be enforced somehow, either by having the unit_map itself ordered, or by ordering the results when selecting units from the unit_map. 20131123 03:46:38< mattsc> And just in case somebody believes this is not a serious issue, do the following little exercise (in 1.11): 20131123 03:47:10< mattsc> Start HttT (or any other campaign or MP scenario), recruit 2 units, save game, load that save, recruit two more units and save a replay. 20131123 03:47:24< mattsc> Then watch the replay and be amazed. 20131123 03:52:54< mattsc> AI0867: I should clarify that the second type of ordering (after loading a game) is also in inverse order of how they were put onto the map. That is just the order in which the game does that when processing a savegame. 20131123 03:53:41-!- Alarantalara1 [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Quit: Leaving.] 20131123 03:56:00< mattsc> Btw, I should probably move that thread out of Lua Labs, since it clearly goes well beyond Lua now. Where do I move it to? Coder's Corner? Or Technical Support? 20131123 04:00:14-!- exciton [chuck-the-@89.208.169.104] has quit [Read error: Connection reset by peer] 20131123 04:05:34-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev 20131123 04:13:14-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai] 20131123 04:21:27-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has joined #wesnoth-dev 20131123 04:21:37-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has joined #wesnoth-dev 20131123 04:26:12-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Client Quit] 20131123 04:27:02-!- Chusslove [~Chusslove@unaffiliated/chusslove] has quit [Read error: Operation timed out] 20131123 04:28:43-!- exciton [chuck-the-@89.208.169.104] has quit [Ping timeout: 260 seconds] 20131123 04:29:57-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev 20131123 04:39:42< iceiceice> holy crap! devdocs.wesnoth.org is awesome! 20131123 04:39:47< iceiceice> how did i not see this before !? 20131123 04:41:52< ancestral> The links at the top (classes, namespaces, etc.) no worky for me 20131123 04:48:02< iceiceice> hmm they work for me i think 20131123 04:55:05-!- Chusslove [~Chusslove@brsg-d9baf712.pool.mediaWays.net] has joined #wesnoth-dev 20131123 04:55:05-!- Chusslove [~Chusslove@brsg-d9baf712.pool.mediaWays.net] has quit [Changing host] 20131123 04:55:05-!- Chusslove [~Chusslove@unaffiliated/chusslove] has joined #wesnoth-dev 20131123 05:01:20-!- kex [~kex@89.205.75.19] has quit [Remote host closed the connection] 20131123 05:01:58-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 05:06:15-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 245 seconds] 20131123 05:20:00-!- irker708 [~irker@ai0867.net] has quit [Quit: transmission timeout] 20131123 05:38:11-!- Coffee_irc [~david@ppp118-210-35-86.lns20.adl2.internode.on.net] has quit [Quit: Konversation terminated!] 20131123 05:39:29-!- Gambit [~derek@wesnoth/developer/grickit] has quit [Remote host closed the connection] 20131123 05:53:13< AI0867> mattsc: I'm wondering what even causes the ordering you're observing 20131123 05:53:42< AI0867> not the change, the fact that there is an ordering at all, given how the units are stored 20131123 05:54:37< mattsc> AI0867: I don't know... 20131123 05:55:04< mattsc> The one from the top of the stack gets read first? 20131123 05:56:50< AI0867> the lua get_units function iterates over the unit_map and filters each one 20131123 05:57:04< AI0867> the unit_map is implemented internaly as an unordered_map 20131123 05:57:10< AI0867> so where does the ordering come from? 20131123 05:58:57< mattsc> AI0867: well, I don't know, but does it make sense to assume that if I read the unit map 10 times without changing anything, it gets read the same way each time (no matter what that order is)? 20131123 06:00:18< mattsc> If so, it appears that they are read from the top of the stack (is that the right word in this case?), which would be supported by the fact that they show up in inverse order in which they are put on the map. 20131123 06:01:11< mattsc> But this is just a guess by somebody who really doesn't know much at all about how any of this works ... 20131123 06:05:23< iceiceice> AI0867: even though its an unordered map, the C++ implementation is still deterministic, right? 20131123 06:05:39< iceiceice> AI0867: it just doesn't give any formal guarantee 20131123 06:06:28< iceiceice> AI0867: am i wrong about that? I guess if it uses some randomized hash function i would be wrong 20131123 06:07:14< iceiceice> AI0867: idk i guess its easy to do an experiment and find out 20131123 06:19:57< AI0867> iceiceice: yeah, the hash and such 20131123 06:20:31< AI0867> but given that the key is an integer, you'd expect the ordering (if there is any) to be equal or at least dependant on that 20131123 06:20:36< AI0867> s/equal/equal to/ 20131123 06:21:13< AI0867> that's not what we're observing though 20131123 06:22:10< AI0867> we're observing something that is clearly dependant on the insertion order 20131123 06:23:22< AI0867> you can see the origin of the "per side" ordering in gamestatus.cpp in convert_old_saves 20131123 06:23:36< AI0867> but what preserves that ordering? 20131123 06:24:32< AI0867> ah, there's a list 20131123 06:29:37< iceiceice> so its been a while for me but i assume that unordered map = hash table 20131123 06:29:51< AI0867> yes 20131123 06:29:53< iceiceice> so you get a block of memory, hash things, and then use chaining if there is a collision 20131123 06:30:03< iceiceice> so then the ordering will depend on the int key, 20131123 06:30:10< AI0867> well, there's various things you can do with collisions, but that's one option 20131123 06:30:12< iceiceice> but a < b will not imply that a b in the ordering 20131123 06:30:18< iceiceice> and the insertion order will matter 20131123 06:30:21< iceiceice> if the hashes are the same 20131123 06:30:48< AI0867> iceiceice: the iterator actually iterates over a list, not the map 20131123 06:30:59< AI0867> the map just contains iterators of the same lists 20131123 06:30:59< iceiceice> hmm 20131123 06:32:20< iceiceice> so what is the issue here? you want the ordering to be deterministic so as to avoid oos, but you would prefer unordered map to get speed? 20131123 06:33:11< iceiceice> or i guess we don't know exactly why the oos is happening but we think tis this data structure? 20131123 06:33:13< AI0867> it being deterministic was always a requirement. I'm not sure why this patch was even applied 20131123 06:33:57< AI0867> it seems to have promised to not cause ordering changes 20131123 06:34:51< AI0867> it also looks like the list was added later than the unordered_map change 20131123 06:34:59< AI0867> which is why I didn't find it originally 20131123 06:37:08< mattsc> iceiceice: we do know that the ordering changes cause oos errors. 20131123 06:41:31-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has quit [Quit: i go nstuf kthxbai] 20131123 06:44:56-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 06:46:53-!- tomreyn_ [~tomreyn@megaglest/team/tomreyn] has joined #wesnoth-dev 20131123 06:47:39-!- tomreyn [~tomreyn@megaglest/team/tomreyn] has quit [Disconnected by services] 20131123 06:47:47-!- tomreyn_ is now known as tomreyn 20131123 06:50:07-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 272 seconds] 20131123 06:56:41< AI0867> the list of pods does have the advantage that iterators are not invalidated when units get killed or otherwise deleted 20131123 07:15:46< iceiceice> does anyone know if it is still true that adding a custom ai to registry.cpp is all that it takes to make it accessible? 20131123 07:21:08-!- [Relic] [~relic@99-58-54-211.lightspeed.milwwi.sbcglobal.net] has joined #wesnoth-dev 20131123 07:25:45-!- [Relic] [~relic@99-58-54-211.lightspeed.milwwi.sbcglobal.net] has quit [Client Quit] 20131123 07:38:13-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has joined #wesnoth-dev 20131123 07:43:54< zookeeper> mattsc, coder's corner 20131123 08:18:00-!- trademark_ [~trademark@nsg93-8-88-175-59-164.fbx.proxad.net] has joined #wesnoth-dev 20131123 08:33:07-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 08:38:25-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 272 seconds] 20131123 09:11:10-!- boucman [~rosen@wesnoth/developer/boucman] has joined #wesnoth-dev 20131123 09:29:52-!- thunderstruck [~zaibotren@cpc13-sgyl31-2-0-cust696.18-2.cable.virginm.net] has joined #wesnoth-dev 20131123 09:37:51-!- esr [~esr@wesnoth/developer/esr] has quit [Quit: WeeChat 0.4.1] 20131123 09:42:29-!- esr [~esr@static-71-162-243-5.phlapa.fios.verizon.net] has joined #wesnoth-dev 20131123 09:42:29-!- esr [~esr@static-71-162-243-5.phlapa.fios.verizon.net] has quit [Changing host] 20131123 09:42:29-!- esr [~esr@wesnoth/developer/esr] has joined #wesnoth-dev 20131123 10:04:43-!- mjs-de [~mjs-de@f048042032.adsl.alicedsl.de] has joined #wesnoth-dev 20131123 10:07:51-!- Ivanovic_ [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev 20131123 10:09:28-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 264 seconds] 20131123 10:09:50-!- Ivanovic [~ivanovic@x2f4f032.dyn.telefonica.de] has joined #wesnoth-dev 20131123 10:09:50-!- Ivanovic [~ivanovic@x2f4f032.dyn.telefonica.de] has quit [Changing host] 20131123 10:09:50-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev 20131123 10:12:13-!- Ivanovic_ [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 245 seconds] 20131123 10:15:27-!- Ivanovic_ [~ivanovic@x2f44190.dyn.telefonica.de] has joined #wesnoth-dev 20131123 10:15:27-!- Ivanovic_ [~ivanovic@x2f44190.dyn.telefonica.de] has quit [Changing host] 20131123 10:15:27-!- Ivanovic_ [~ivanovic@wesnoth/developer/ivanovic] has joined #wesnoth-dev 20131123 10:15:57-!- Ivanovic [~ivanovic@wesnoth/developer/ivanovic] has quit [Ping timeout: 246 seconds] 20131123 10:17:39-!- Ivanovic_ is now known as Ivanovic 20131123 10:21:12-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 10:23:15-!- Kostic [~marko@85.202.113.151] has joined #wesnoth-dev 20131123 10:25:50-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 245 seconds] 20131123 10:27:13-!- yann [~dwitch@nan92-1-81-57-214-146.fbx.proxad.net] has quit [Ping timeout: 245 seconds] 20131123 10:31:26-!- Kostic [~marko@85.202.113.151] has quit [Remote host closed the connection] 20131123 10:33:52-!- boucman [~rosen@wesnoth/developer/boucman] has quit [Remote host closed the connection] 20131123 10:35:29-!- Kostic [~marko@85.202.113.252] has joined #wesnoth-dev 20131123 11:00:12-!- exciton [chuck-the-@89.208.169.104] has quit [Read error: Connection reset by peer] 20131123 11:05:27-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev 20131123 11:41:22-!- aquileia [93aba763@gateway/web/freenode/ip.147.171.167.99] has joined #wesnoth-dev 20131123 11:46:13-!- bumbadadabum [~bumbadada@d155109.upc-d.chello.nl] has joined #wesnoth-dev 20131123 11:48:30-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has joined #wesnoth-dev 20131123 11:48:30-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has quit [Changing host] 20131123 11:48:30-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 12:09:23-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 12:13:53-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 245 seconds] 20131123 12:21:35-!- stikonas_ [~gentoo@anon-34-62.vpn.ipredator.se] has joined #wesnoth-dev 20131123 12:21:35-!- stikonas_ [~gentoo@anon-34-62.vpn.ipredator.se] has quit [Changing host] 20131123 12:21:35-!- stikonas_ [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 12:21:43-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Ping timeout: 272 seconds] 20131123 12:35:37-!- aquileia [93aba763@gateway/web/freenode/ip.147.171.167.99] has quit [Ping timeout: 250 seconds] 20131123 12:40:06-!- Gambit [~derek@wesnoth/developer/grickit] has joined #wesnoth-dev 20131123 12:48:43-!- yann [~dwitch@nan92-1-81-57-214-146.fbx.proxad.net] has joined #wesnoth-dev 20131123 13:05:25-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has quit [Quit: Lämnar] 20131123 13:12:39-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has joined #wesnoth-dev 20131123 13:46:47-!- stikonas_ [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!] 20131123 13:47:13-!- Upthorn [~ogmar@108-85-91-228.lightspeed.frokca.sbcglobal.net] has quit [Ping timeout: 272 seconds] 20131123 13:50:44-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has joined #wesnoth-dev 20131123 13:50:44-!- stikonas [~gentoo@cpc18-sgyl27-2-0-cust35.18-2.cable.virginm.net] has quit [Changing host] 20131123 13:50:44-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 13:53:50-!- happygrue [~happygrue@wesnoth/developer/wintermute] has joined #wesnoth-dev 20131123 13:57:32-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 13:58:32-!- Kostic [~marko@85.202.113.252] has quit [Quit: Kostic] 20131123 14:02:13-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 245 seconds] 20131123 14:04:07-!- exciton [chuck-the-@89.208.169.104] has quit [Ping timeout: 260 seconds] 20131123 14:05:27-!- exciton [chuck-the-@89.208.169.104] has joined #wesnoth-dev 20131123 14:06:30< mattsc> zookeeper: ah, moved it to the wrong one then. Should I move it again or is this good enough? 20131123 14:07:28< mattsc> AI0867: Thanks. Can you take care of that too? 20131123 14:08:10< AI0867> mattsc: sure 20131123 14:10:17< mattsc> AI0867: Thanks. I can understand how this works and do minor changes myself, but just lack the experience do be certain that I am doing the right thing. 20131123 14:13:19< mattsc> On a separate note, I assume nobody minds if I do the second part of what I propose in my first post anyway: adding underlying_id to the unit proxy table in Lua? 20131123 14:14:08< mattsc> It's not needed for this, but I would have liked to have that available more than once for other reasons. 20131123 14:15:57< AI0867> sounds okay 20131123 14:16:03< AI0867> what else do you need it for? 20131123 14:17:09< mattsc> It's the only thing that is truly unique for a unit, and sometimes it's good to be able to distinguish between them like that. 20131123 14:17:47< mattsc> ... to have something available that lets you distinguish between them with a single parameter 20131123 14:25:04-!- gfgtdf [~chatzilla@e177125223.adsl.alicedsl.de] has joined #wesnoth-dev 20131123 14:28:13-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginm.net] has joined #wesnoth-dev 20131123 14:28:29-!- Crendgrim [~crend@37-4-131-59-dynip.superkabel.de] has quit [Quit: No Ping reply in 180 seconds.] 20131123 14:28:41-!- Crendgrim [~crend@37-4-131-59-dynip.superkabel.de] has joined #wesnoth-dev 20131123 14:28:50< mattsc> AI0867: oh, thanks. And I was imprecise in what I said: I meant would you take care of going back to an ordered map. :) 20131123 14:32:56< AI0867> oh 20131123 14:33:21< AI0867> I'd like to hear someone else's opinion on my analysis first, but that's the plan 20131123 14:33:26-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has left #wesnoth-dev ["Lämnar"] 20131123 14:34:41< mattsc> Of course, that makes sense. thanks. 20131123 14:54:48-!- gfgtdf [~chatzilla@e177125223.adsl.alicedsl.de] has quit [Quit: ChatZilla 0.9.90.1 [Firefox 24.0/20130910160258]] 20131123 14:56:05-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has joined #wesnoth-dev 20131123 14:56:05-!- ancestral [~ancestral@71-215-216-176.mpls.qwest.net] has quit [Client Quit] 20131123 14:58:45-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 15:02:59-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 252 seconds] 20131123 15:41:13-!- anonymissimus [~chatzilla@HSI-KBW-149-172-228-192.hsi13.kabel-badenwuerttemberg.de] has joined #wesnoth-dev 20131123 15:42:19< anonymissimus> mattsc: I think exposing underlying_id is a not so good idea because wml/lua should neevr need to mess with it 20131123 15:42:40< anonymissimus> which is probably why there is no getter yet 20131123 15:43:04< anonymissimus> instead, the usual wml is should (is I think) enforced to be unique 20131123 15:43:13< anonymissimus> wml unit id 20131123 15:44:37< anonymissimus> AI0867: I wonder; if iterators are never invalidated, what happens with the iterator poiting to that unit which is erased ? what doe it point to afterwards ? 20131123 15:51:16-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev 20131123 15:58:21-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Ping timeout: 252 seconds] 20131123 15:59:24-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 16:02:22-!- DCW [~Thunderbi@cpc1-finc14-2-0-cust12.4-2.cable.virginm.net] has quit [Remote host closed the connection] 20131123 16:03:59-!- MaraJade [~goossenm@128.193.12.16] has quit [Remote host closed the connection] 20131123 16:04:40-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 264 seconds] 20131123 16:15:05< AI0867> anonymissimus: it points to a unit_pod, which contains a ref counter and a unit pointer 20131123 16:15:28< AI0867> the unit pointer will be NULL, and when the ref counter goes to zero (no iterators left), the pod is removed from the list 20131123 16:20:10< AI0867> wesbot: bug #18591 20131123 16:20:12< wesbot> Bug #18591 Assigned to: None Status: None Priority: 5 - Normal 20131123 16:20:12< wesbot> Summary: Multiplayer Game Crashes when loading or starting a second game 20131123 16:20:12< wesbot> Original submission: In multiplayer, game data is being incompletely reset, wh 20131123 16:20:14< wesbot> en loading new games. At the least the unit_map is not being reset, but there m 20131123 16:20:17< wesbot> URL: http://gna.org/bugs/?18591 20131123 16:20:19< wesbot> Attached file (1st): http://gna.org/bugs/download.php?file_id=16466 20131123 16:23:52< anonymissimus> AI0867: and when trying to deref that iterator it checks for !NULL I suppose ? nice 20131123 16:33:10< mattsc> anonymissimus: I don't want to mess with it, I just want fast read-only access 20131123 16:34:06< AI0867> anonymissimus: it asserts that, yes 20131123 16:34:08< mattsc> and no, the wml id is not enforced to be unique. It usually is, but it's not guaranteed. 20131123 16:35:01< anonymissimus> mattsc: what do you need it for ? if someone doesn't make his unit ids unique it's his own fault 20131123 16:35:24< anonymissimus> that belongs to the things a coder really shouldn't do 20131123 16:35:24< AI0867> fast equality comparison for AIs? 20131123 16:35:36< AI0867> you can't rely on add-on authors to not be stupid 20131123 16:35:47< anonymissimus> there exists an equality operator for proxy units 20131123 16:36:12< anonymissimus> I added that once, it uses underlying_id 20131123 16:38:31< mattsc> anonymissimus: right, you did, I forgot. That didn't exist yet when I had the first use cases for it. 20131123 16:39:31< mattsc> Also, it's convenient for setting up tables of units that can be accessed by a single parameter without having to do multiplications (of the map coordinates - which also doesn't work for units on the recall list) 20131123 16:40:08< mattsc> And no, you cannot rely on UMC cade to have unique ids. Until I fixed that, even the leaders in MP games could have identical ids. 20131123 16:40:36< mattsc> in _mainline_ MP games 20131123 16:42:10< mattsc> Anyways, it's not a big deal. It's just something that would be convenient to have and be a minor speed improvement in some cases, but it's quite possible to do without (as I have done so far) 20131123 16:43:28< anonymissimus> mattsc: okay well, than add it 20131123 16:43:44< anonymissimus> mattsc: I just recalled a bug report though: http://gna.org/bugs/index.php?18877 20131123 16:43:58< anonymissimus> so even underlying_ids aren't always unique :P 20131123 16:44:35< mattsc> hmm, I see ... 20131123 16:45:22< mattsc> I thought I saw something in the code that clears up underlying_id conflicts, but I didn't really check under which circumstances. 20131123 16:47:29< mattsc> Anyways, I'm off for the rest of the morning. Thanks to both of you, I wouldn't have been able to figure this one out by myself, at least not in a reasonable amount of time! 20131123 16:54:57-!- trademark_ [~trademark@nsg93-8-88-175-59-164.fbx.proxad.net] has quit [Quit: Leaving] 20131123 16:55:28-!- trademark_ [~trademark@nsg93-8-88-175-59-164.fbx.proxad.net] has joined #wesnoth-dev 20131123 16:57:17-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has joined #wesnoth-dev 20131123 17:00:22-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 17:05:11-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 265 seconds] 20131123 17:43:38-!- lipkab [~the_new_l@host-91-147-212-174.biatv.hu] has quit [Quit: Távozom] 20131123 17:55:25-!- Chusslove [~Chusslove@unaffiliated/chusslove] has quit [] 20131123 17:55:37-!- un214 [~un214@2602:303:fcd0:a019:224:8cff:fed2:ef57] has joined #wesnoth-dev 20131123 17:56:21-!- iwaim__ [~iwaim@2001:2c0:40e:2002:0:4:14:80] has quit [Remote host closed the connection] 20131123 17:58:28-!- iwaim_ [~iwaim@gateway.alib.jp] has joined #wesnoth-dev 20131123 18:01:23-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 18:06:32-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 272 seconds] 20131123 18:07:49-!- anonymissimus [~chatzilla@HSI-KBW-149-172-228-192.hsi13.kabel-badenwuerttemberg.de] has quit [Quit: ChatZilla 0.9.90.1 [Firefox 11.0/20120312181643]] 20131123 18:10:58-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 18:11:16-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has joined #wesnoth-dev 20131123 18:16:42-!- skyfaller [~skyfaller@wikipedia/Skyfaller] has joined #wesnoth-dev 20131123 18:18:49-!- kex [~kex@89.205.75.19] has quit [Remote host closed the connection] 20131123 18:20:32-!- kex [~kex@212.158.180.22] has joined #wesnoth-dev 20131123 18:25:51-!- Kexoth [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 18:28:53-!- kex [~kex@212.158.180.22] has quit [Ping timeout: 245 seconds] 20131123 18:40:03-!- skyfaller [~skyfaller@wikipedia/Skyfaller] has quit [Read error: Connection reset by peer] 20131123 18:40:39-!- skyfaller [~skyfaller@wikipedia/Skyfaller] has joined #wesnoth-dev 20131123 18:43:08-!- shadowm_desktop [ignacio@wesnoth/developer/shadowmaster] has joined #wesnoth-dev 20131123 18:54:22-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has joined #wesnoth-dev 20131123 18:59:13-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Client Quit] 20131123 19:00:13-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has joined #wesnoth-dev 20131123 19:05:46-!- Crendgrim [~crend@37-4-131-59-dynip.superkabel.de] has quit [Quit: No Ping reply in 180 seconds.] 20131123 19:06:00-!- Crendgrim [~crend@37-4-131-59-dynip.superkabel.de] has joined #wesnoth-dev 20131123 19:34:40-!- un214 [~un214@2602:303:fcd0:a019:224:8cff:fed2:ef57] has quit [Remote host closed the connection] 20131123 19:41:17-!- thunderstruck [~zaibotren@cpc13-sgyl31-2-0-cust696.18-2.cable.virginm.net] has quit [Quit: leaving] 20131123 19:51:05-!- goossenm [goossenm@shell.onid.oregonstate.edu] has joined #wesnoth-dev 20131123 19:51:56-!- goossenm [goossenm@shell.onid.oregonstate.edu] has quit [Client Quit] 20131123 19:52:40-!- goossenm [goossenm@shell.onid.oregonstate.edu] has joined #wesnoth-dev 20131123 19:53:22-!- goossenm is now known as MaraJade 20131123 20:02:10-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Remote host closed the connection] 20131123 20:02:44-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 20:05:13-!- iceiceice [~chris@cpe-24-193-85-148.nyc.res.rr.com] has quit [Ping timeout: 272 seconds] 20131123 20:11:55-!- stikonas_ [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 20:13:02-!- stikonas [~gentoo@wesnoth/translator/stikonas] has quit [Remote host closed the connection] 20131123 20:17:11-!- iceiceice [~chris@207-237-132-90.ny.subnet.cable.rcn.com] has joined #wesnoth-dev 20131123 20:18:04-!- stikonas_ [~gentoo@wesnoth/translator/stikonas] has quit [Quit: Konversation terminated!] 20131123 20:30:22-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has quit [Quit: Lämnar] 20131123 20:32:11-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has joined #wesnoth-dev 20131123 20:44:02-!- wesbot changed the topic of #wesnoth-dev to: 200 bugs, 345 feature requests, 28 patches | Logs: http://irclogs.wesnoth.org | Don't paste on IRC! Use a pastebin: http://pastebin.com | http://imagebin.org 20131123 21:06:08-!- stikonas [~gentoo@wesnoth/translator/stikonas] has joined #wesnoth-dev 20131123 21:19:23-!- noy [~Noy@wesnoth/developer/noy] has quit [Quit: noy] 20131123 21:24:39-!- Chusslove [~Chusslove@brsg-4dbb12e4.pool.mediaWays.net] has joined #wesnoth-dev 20131123 21:24:39-!- Chusslove [~Chusslove@brsg-4dbb12e4.pool.mediaWays.net] has quit [Changing host] 20131123 21:24:39-!- Chusslove [~Chusslove@unaffiliated/chusslove] has joined #wesnoth-dev 20131123 21:45:57-!- Kexoth [~kex@89.205.75.19] has quit [Remote host closed the connection] 20131123 21:46:32-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 21:51:05-!- kex [~kex@89.205.75.19] has quit [Ping timeout: 272 seconds] 20131123 22:09:44-!- Coffee_irc [~david@ppp118-210-35-86.lns20.adl2.internode.on.net] has joined #wesnoth-dev 20131123 22:12:43-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has quit [Quit: Lämnar] 20131123 22:16:23-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has joined #wesnoth-dev 20131123 22:17:04-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has quit [Client Quit] 20131123 22:21:51-!- molgrum [~molgrum@h-94-220.a230.priv.bahnhof.se] has joined #wesnoth-dev 20131123 22:24:40-!- Jetrel_ [~Jetrel@c-75-73-180-126.hsd1.mn.comcast.net] has quit [Quit: Textual IRC Client: www.textualapp.com] 20131123 22:25:41-!- Jetrel [~Jetrel@c-75-73-180-126.hsd1.mn.comcast.net] has joined #wesnoth-dev 20131123 22:49:41-!- iceiceice [~chris@207-237-132-90.ny.subnet.cable.rcn.com] has quit [Ping timeout: 248 seconds] 20131123 23:02:44-!- iceiceice [~chris@cpe-24-193-85-148.nyc.res.rr.com] has joined #wesnoth-dev 20131123 23:09:20-!- Upthorn [~ogmar@108-85-91-228.lightspeed.frokca.sbcglobal.net] has joined #wesnoth-dev 20131123 23:14:03-!- mjs-de [~mjs-de@f048042032.adsl.alicedsl.de] has quit [Remote host closed the connection] 20131123 23:15:58-!- kex [~kex@89.205.75.19] has joined #wesnoth-dev 20131123 23:19:27-!- kex [~kex@89.205.75.19] has quit [Read error: Operation timed out] 20131123 23:27:37-!- zookeeper [~lmsnie@wesnoth/developer/zookeeper] has quit [Ping timeout: 272 seconds] 20131123 23:44:07-!- gfgtdf [~chatzilla@f054139213.adsl.alicedsl.de] has joined #wesnoth-dev 20131123 23:54:05-!- Alarantalara [~Adium@192-0-128-11.cpe.teksavvy.com] has quit [Quit: Leaving.] --- Log closed Sun Nov 24 00:00:36 2013