Exceptions ole #20
Exceptions ole #20
Conversation
… reading loop for better accuracy
…the capabilities since we dont support it
There was a problem hiding this comment.
hier is letterlijk niks veranderd naast unused functies weggehaald, is alleen verschoven zodat ze bij elkaar staan
There was a problem hiding this comment.
Ik heb dit uiteraard gecheckt en het moet overal sterretje zijn if empty. Which means bij elke getNickname checken en sterretje neerzetten OR gewoon bij deze sterretje returnen if empty.
Omdat je daardoor "*" returned kan je geen reference returnen, en aangezien het geen reference is boeit const helemaal niet.
|
|
||
| if (event.data.fd == socket && (event.events & EPOLLIN)) { | ||
| if (event.data.fd == socket) { | ||
| if ((event.events & EPOLLIN) == 0) { |
There was a problem hiding this comment.
Gecheckt en dit kan gebeuren maar betekent altijd dat het een error is, dus epoll error = server closen
There was a problem hiding this comment.
Het staat een beetje door elkaar for some reason door git, maar als je checkt met de file op main zijn de epoll functies precies hetzelfde met de uitzondering van de exceptions die nu bij errors worden gegooid ipv exits
There was a problem hiding this comment.
De meeste functies komen uit Server, heb ze apart getrokken omdat de Server file zeer groot was en ik dacht dat dit een goede verdeling en groepering was, ik kan die functies in principe terugzetten maar zie niet helemaal in waarom, het veranderd niks.
Ik heb checks toegevoegd op plekken waar deze nog niet stonden, in both environment en init functies, aangezien de oorspronkelijke versie een core dump issue had.
Ik heb de overall environment parsing inderdaad aangepast, de defines kunnen weg die zijn ondertussen redelijk nutteloos in de current file. Check next commit.
Ik heb toegevoegd dat command line arguments prioriteit krijgen over environment, omdat je deze manually zet. De env file heb ik prioriteit laten nemen over de system environment, aangezien je die ook zelf zet en je deze duidelijk liever wilt gebruiken.
Ik heb toegevoegd dat het werkt zonder arguments te passen aan het programma.
Ik heb de overall environment parsing code leesbaarder, duidelijker gemaakt en meer structuur gegeven, zodat onze evaluators het zo snel mogelijk kunnen lezen en wij zo snel mogelijk kunnen zien waar de fout zit. Ik zal een revisie doen in de volgende commit die het nog iets opschoont. Reverten is iig niet de oplossing, want er zitten echt verbeteringen in, die tested zijn.
There was a problem hiding this comment.
zie comment onder serverinit.cpp
There was a problem hiding this comment.
SIGKILL kan niet opgevangen worden.
De rest is veel overzichtelijker zo, maar kom vooral met suggesties
… into exceptions_ole
… into exceptions_ole
…ITE, KICK, and MODE commands
3 participants
No description provided.