| StefandeVries | Hallo leoquant | 13:06 |
|---|---|---|
| leoquant | hee! | 13:07 |
| StefandeVries | Hoe gaat het? :) | 13:08 |
| Snicksie | hi :p | 13:10 |
| timo__ | hoi leoquant | 13:11 |
| === timo_ is now known as Guest46495 | ||
| === Guest46495 is now known as timo__ | ||
| StefandeVries | Nou, jongens, dit wordt 'm niet hoor. | 13:42 |
| 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:43 |
| commandoline | was je dat helemaal aan het doen dan? :o | 13:56 |
| StefandeVries | Bij wijze van test ja, maar ik merkte 1) geen performanceverschil en 2) de code werd één grote uncluttered mess. | 13:57 |
| 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 :) | 13:58 |
| 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:00 |
| 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:01 |
| 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:02 |
| 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:04 |
| StefandeVries | Nadeel is wel dat de laatste code, in de branch, nog niet de nieuwe stringjoins heeft. Dat wordt weer scrollen :P | 14:05 |
| commandoline | :P | 14:06 |
| StefandeVries | os, codecs en string uit de imports halen. | 14:07 |
| StefandeVries | Ruimt lekker op :) | 14:07 |
| JanC | StefandeVries: waarom maak je je connectie dan ook in de __init__() als dat een probleem is? ;) | 14:21 |
| StefandeVries | Dat is geen probleem ;) | 14:22 |
| Snicksie | gwn een flag zetten als de ping gedaan is :p | 14:24 |
| 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:25 |
| StefandeVries | Nee, nog niet. | 14:26 |
| StefandeVries | Ah, in deze versie staan nog de random.seed() aanroepen | 14:27 |
| StefandeVries | Eruit gehaald. | 14:28 |
| 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:29 |
| StefandeVries | Yeah, and I don't see that happening in the near future. | 14:32 |
| commandoline | precies :P | 14:32 |
| StefandeVries | commandoline, kan je even naar ##PyTest komen? | 14:35 |
| commandoline | done | 14:35 |
| === Ronnie1 is now known as Ronnie | ||
| StefandeVries | Iemand hier? | 17:33 |
| Ronnie | StefandeVries: ja | 17:34 |
| StefandeVries | Oh, sorry | 17:34 |
| StefandeVries | Dank je, maar het is al goed :) | 17:34 |
| StefandeVries | JanC, ping? | 18:46 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!