[13:06] Hallo leoquant [13:07] hee! [13:08] Hoe gaat het? :) [13:10] hi :p [13:11] hoi leoquant === timo_ is now known as Guest46495 === Guest46495 is now known as timo__ [13:42] Nou, jongens, dit wordt 'm niet hoor. [13:43] 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] was je dat helemaal aan het doen dan? :o [13:57] Bij wijze van test ja, maar ik merkte 1) geen performanceverschil en 2) de code werd één grote uncluttered mess. [13:58] ok, kon je dan vertraging merken bij die bot :P? [13:58] 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] ok :) [14:00] 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] da's op zich wel een mooie verbetering :) [14:01] zeker omdat je dan in de toekomst mocht je dat willen de instellingen in een config file kan zetten :) [14:02] en ik doe idd ook altijd eerst instance aanmaken en dan run(), exec_() of hoe die method ook heet aanroepen [14:02] maar dat blijft een kwestie van smaak, maakt niet zoveel uit. [14:04] 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] Nadeel is wel dat de laatste code, in de branch, nog niet de nieuwe stringjoins heeft. Dat wordt weer scrollen :P [14:06] :P [14:07] os, codecs en string uit de imports halen. [14:07] Ruimt lekker op :) [14:21] StefandeVries: waarom maak je je connectie dan ook in de __init__() als dat een probleem is? ;) [14:22] Dat is geen probleem ;) [14:24] gwn een flag zetten als de ping gedaan is :p [14:25] Of gewoon de perfect werkende code laten zoals hij/zij is. [14:25] je aanpassingen nog niet naar LP gepusht? [14:25] Perfect..kan altijd beter, natuurlijk. [14:26] Nee, nog niet. [14:27] Ah, in deze versie staan nog de random.seed() aanroepen [14:28] Eruit gehaald. [14:29] je zou idd een run method kunnen maken die zeg maar de init vervangt en daarna de mainloop start. [14:29] maar goed, het werkt nu :P [14:29] commandoline, en wat zou daar de toegevoegde of betere waarde van zijn? [14:29] 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] Yeah, and I don't see that happening in the near future. [14:32] precies :P [14:35] commandoline, kan je even naar ##PyTest komen? [14:35] done === Ronnie1 is now known as Ronnie [17:33] Iemand hier? [17:34] StefandeVries: ja [17:34] Oh, sorry [17:34] Dank je, maar het is al goed :) [18:46] JanC, ping?