Oldunreal Forums - [227h] Infinite Recursion and Server Packages

[UDHQ] Pcube

Ex Member

[227h] Infinite Recursion and Server Packages
08/05/12 at 23:15:19

For the infinite recursion breakout could this be a configurable commandline switch where the default behavior is to preserve original Unreal behavior (crash) and set behavior would be "continue execution". I have had to debug the same mod repeatedly because I would have assumed the original recursing code would crash. The behavior would still appear "the same" in my constrained testing but would fail in larger cases. In my opinion it's stupid to continue executing and hammer the CPU in these cases of true infinite recursion.

Also, please provide a switch (default behavior = classic Unreal behavior) to terminate the server upon failure of loading a server package. This is also another point of frustration for server owners since classically they expect a launching server to crash upon a misconfigured ini or setup (leading to a failed serverpackage load).

It is important to preserve original behavior especially with a game this old. I would have hoped this was obvious by now.

Back to top

IP Logged

Smirftsch

YaBB Administrator

Offline

Posts: 5485
at home
Gender: male

Re: [227h] Infinite Recursion and Server Packages
Reply #1 - 08/06/12 at 07:14:37

For debugging reasons a switch for the recursion could be indeed done, this idea ain't bad.

The changes during startup were done by requests from many of the server admins you are speaking about, since a more forgiving system at least let you start up the server, the logs need to be checked in any case.
And honestly, there are already so many additional new switches we built in to switch on/off new features that people are already overstrained, the switches should be really limited to those things where they make sense and where they are needed, which is not the case here, it doesn't affect any compatibility or functionality, it's just because you are used to the old behavior.

A side note: It makes no sense to keep any old behavior and classify it as "default" just for the sake of it, in this case maybe 70% of the fixes we have would have never been done since Epic just had the habit to ignore bugs, used often very insufficient workarounds, or just "fixed" them by removing the related features.
I would have hoped THIS was obvious by now.

Back to top

« Last Edit: 08/06/12 at 15:30:19 by Smirftsch »

Sometimes you have to lose a fight to win the war.

WWW

IP Logged

[UDHQ] Pcube Ex Member	Re: [227h] Infinite Recursion and Server Packages Reply #2 - 08/07/12 at 03:30:46 Moved my post to a PM so as not to incite unnecessary drama.
Back to top	« Last Edit: 08/07/12 at 03:45:36 by N/A » IP Logged

Smirftsch YaBB Administrator Offline Posts: 5485 at home Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #3 - 08/07/12 at 09:27:12 Quote: Moved my post to a PM so as not to incite unnecessary drama. thanks, very much appreciated. Old crash way can now be activated separately with "STRICT" commandline for debugging purposes.
Back to top	« Last Edit: 08/07/12 at 13:07:18 by Smirftsch » Sometimes you have to lose a fight to win the war. WWW IP Logged

.:..: Developer Team Offline Posts: 1239 Finland Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #4 - 08/07/12 at 13:26:36 We're sorry but we can't help you in this matter. We do not have the bandwidth to oversee such a modification in the engine. P.S: Mark Rein would be proud of me for such choice of words.
Back to top	You performed triple-stab elegantly. IP Logged

[UDHQ]Jackrabbit YaBB Newbies Offline Oldunreal member Posts: 11 Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #5 - 08/08/12 at 18:35:49 ...and I suppose this is the intended 227 functionality? Giving warning messages in the console which are completely false.
Back to top	« Last Edit: 08/08/12 at 18:41:59 by [UDHQ]Jackrabbit » IP Logged

.:..: Developer Team Offline Posts: 1239 Finland Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #6 - 08/09/12 at 13:07:11 [UDHQ]Jackrabbit wrote on 08/08/12 at 18:35:49: ...and I suppose this is the intended 227 functionality? Giving warning messages in the console which are completely false. Yes. If DynamicLoadObject does return none it messages that. If you have a better idea, do post code here then.
Back to top	You performed triple-stab elegantly. IP Logged

Blu3Haz3 Full Member Offline Oldunreal member Posts: 205 Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #7 - 08/18/12 at 09:59:05 that happens with private data too... as far as cache ripping goes. but that is a good thing...
Back to top	IP Logged

IsUnreal

Senior Member

Offline

Oldunreal member

Posts: 311
Gender: male

Re: [227h] Infinite Recursion and Server Packages
Reply #8 - 08/20/12 at 01:13:17

Quote:

that happens with private data too... as far as cache ripping goes. but that is a good thing...

I am missing the connection here?

If the error is caused by a missing dependency, as opposed to a generic 'dosnt exist' failure, 'dynamicload' cant tell the difference . While it would be nice of it to say / return "cant load due to missing package x", thats what the log is for. ,That doent help clients in servers tho...

Back to top

« Last Edit: 08/20/12 at 01:14:45 by IsUnreal »

IP Logged

[UDHQ]Jackrabbit YaBB Newbies Offline Oldunreal member Posts: 11 Gender:	Re: [227h] Infinite Recursion and Server Packages Reply #9 - 08/20/12 at 02:17:51 Quote: "cant load due to missing package x" ding ding ding!!!!!
Back to top	IP Logged

	Welcome, Guest. Please Login or Register