Tuesday, 2009-04-21

*** tpb has joined #freeorion00:00
*** mithro_ has joined #freeorion01:41
*** mithro has quit IRC01:53
*** mithro_ has quit IRC02:18
*** mithro_ has joined #freeorion02:18
*** mithro_ has quit IRC02:44
*** welterde has quit IRC04:13
*** welterde has joined #freeorion04:14
*** kroddnoMobil has joined #freeorion04:50
*** ChanServ sets mode: +o kroddnoMobil04:50
*** mithro_ has joined #freeorion04:52
*** kroddnoMobil has quit IRC05:25
*** mithro_ has quit IRC06:03
GeofftheMediokroddn: isn't most of of any_regular.hpp after the #include lines inside namespace adobe ?06:47
kroddnyes, but one header is namespace adobe::version_1 and in that namespace no "swap" is found07:40
kroddnyou can see that if you fallow the error backtrace07:41
GeofftheMediohow does putting adobe:: before swap in a function that's already in the adobe namespace help, though?07:44
kroddnbut I don't know if that is the right swap to use at this point07:44
kroddnThere is no  adobe::version_1::swap, so using adobe::swap seems to be the solution07:45
kroddnmaybe its a compiler problem, but I don't want to install a non-default compiler on my freeorion chroot07:45
GeofftheMediohmm.  after making your change it still compiled file for me, but I don't know if it would affect how it works07:46
kroddnI don't understand it that good too07:47
kroddnbut std::swap seems to be the wrong function at that point07:47
GeofftheMediopresumably there's a reason they wrote their own07:48
kroddnHave a look at GG/adobe/implementation/swap.hpp07:48
kroddnI don't relly understand what they do :-(07:49
kroddnI hope tzlaine will understand it and fix it the right way ;-)07:50
GeofftheMedioone can hope07:50
kroddnInteresting: http://www.informit.com/guides/content.aspx?g=cplusplus&seqNum=15507:51
tpb<http://ln-s.net/3$5A> (at www.informit.com)07:51
GeofftheMedioguess that's why they just used the std::swap07:59
kroddnbut why did they implement the adobe::swap then?08:01
GeofftheMediothey didn't ... it's just std::swap08:02
GeofftheMediothey did this to make the code look nice, apparently...08:02
GeofftheMedio"all code in namespace adobe that calls swap08:02
GeofftheMedioshould call it unqualified. This is less cluttered than employing local using declaration"08:02
GeofftheMedio(from comment in swap.hpp)08:03
kroddnstrange code around line 72208:05
GeofftheMedioof?08:05
kroddnokay, then writing std::swap at this line should compile too08:06
GeofftheMediodoesn't it?08:06
kroddndidn't try08:06
kroddnyou said that code is never used at all, so I just want to make it compile - I think its tzlaine to fix it correctly08:07
GeofftheMedioI said I don't think it's used08:08
GeofftheMedioat least in FreeOrion08:08
GeofftheMediothere's an example of some sort that uses it in GiGi itself, though08:08
GeofftheMediobut you don't really need that08:08
*** welterde has quit IRC08:19
*** welterde has joined #freeorion08:31
*** kroddnoMobil has joined #freeorion08:32
*** ChanServ sets mode: +o kroddnoMobil08:32
*** kroddnoMobil has quit IRC08:50
*** hagish has joined #freeorion14:13
*** welterde has quit IRC18:26
*** welterde has joined #freeorion18:28
*** welterde has quit IRC18:31
*** welterde has joined #freeorion18:34
*** hagish has quit IRC19:17
*** mithro_ has joined #freeorion19:18
*** mithro_ is now known as mithro19:35
*** mithro_ has joined #freeorion19:59
*** mithro has quit IRC19:59
*** mithro_ has quit IRC21:20
*** mithro_ has joined #freeorion21:20
*** mithro_ is now known as mithro21:55

Generated by irclog2html.py 2.5 by Marius Gedminas - find it at mg.pov.lt!