[13:06] <StefandeVries> Hallo leoquant
[13:07] <leoquant> hee!
[13:08] <StefandeVries> Hoe gaat het? :)
[13:10] <Snicksie> hi :p
[13:11] <timo__> hoi leoquant
[13:42] <StefandeVries> Nou, jongens, dit wordt 'm niet hoor.
[13:43] <StefandeVries> Tijd om de branch voor MwanzoBot weer te downloaden, want de code weer naar niet-OOP omschrijven duurt veel te lang en levert alleen maar weer bugs op
[13:56] <commandoline> was je dat helemaal aan het doen dan? :o
[13:57] <StefandeVries> Bij wijze van test ja, maar ik merkte 1) geen performanceverschil en 2) de code werd één grote uncluttered mess.
[13:58] <commandoline> ok, kon je dan vertraging merken bij die bot :P?
[13:58] <StefandeVries> Het klassesysteem dat MwanzoBot had en nu weer heeft blijkt voor mij bij ander inzien het beste te werken
[13:58]  * commandoline niet
[13:58] <commandoline> ok :)
[14:00] <StefandeVries> En om ook JanC tevreden te houden gebruik ik nu Instance.mainLoop() en geef ik de serverparameters(hostname, port, nick, identity, real name) aan de constructor ervan door, zodat de klasse IRC zinvol is. :P
[14:01] <commandoline> da's op zich wel een mooie verbetering :)
[14:01] <commandoline> zeker omdat je dan in de toekomst mocht je dat willen de instellingen in een config file kan zetten :)
[14:02] <commandoline> en ik doe idd ook altijd eerst instance aanmaken en dan run(), exec_() of hoe die method ook heet aanroepen
[14:02] <commandoline> maar dat blijft een kwestie van smaak, maakt niet zoveel uit.
[14:04] <StefandeVries> Je kunt de mainLoop nu op commando aanroepen, maar als dat niet gebeurt voor de eerste PING word je van het netwerk geknikkerd, om het maar even zo te zeggen
[14:05] <StefandeVries> Nadeel is wel dat de laatste code, in de branch, nog niet de nieuwe stringjoins heeft. Dat wordt weer scrollen :P
[14:06] <commandoline> :P
[14:07] <StefandeVries> os, codecs en string uit de imports halen.
[14:07] <StefandeVries> Ruimt lekker op :)
[14:21] <JanC> StefandeVries: waarom maak je je connectie dan ook in de __init__() als dat een probleem is?  ;)
[14:22] <StefandeVries> Dat is geen probleem ;)
[14:24] <Snicksie> gwn een flag zetten als de ping gedaan is :p
[14:25] <StefandeVries> Of gewoon de perfect werkende code laten zoals hij/zij is.
[14:25] <JanC> je aanpassingen nog niet naar LP gepusht?
[14:25] <StefandeVries> Perfect..kan altijd beter, natuurlijk.
[14:26] <StefandeVries> Nee, nog niet.
[14:27] <StefandeVries> Ah, in deze versie staan nog de random.seed() aanroepen
[14:28] <StefandeVries> Eruit gehaald.
[14:29] <commandoline> je zou idd een run method kunnen maken die zeg maar de init vervangt en daarna de mainloop start.
[14:29] <commandoline> maar goed, het werkt nu :P
[14:29] <StefandeVries> commandoline, en wat zou daar de toegevoegde of betere waarde van zijn?
[14:29] <commandoline> nou, dat die class dus nog flexibeler wordt. Maar dat kan best wachten totdat die class flexibeler moet worden gebruikt wat mij betreft :)
[14:32] <StefandeVries> Yeah, and I don't see that happening in the near future.
[14:32] <commandoline> precies :P
[14:35] <StefandeVries> commandoline, kan je even naar ##PyTest komen?
[14:35] <commandoline> done
[17:33] <StefandeVries> Iemand hier?
[17:34] <Ronnie> StefandeVries: ja
[17:34] <StefandeVries> Oh, sorry
[17:34] <StefandeVries> Dank je, maar het is al goed :)
[18:46] <StefandeVries> JanC, ping?