IWADINFO: Make "Game" attribute optional

Remember, just because you request it, that doesn't mean you'll get it.

Moderator: GZDoom Developers

IWADINFO: Make "Game" attribute optional

Postby RockstarRaccoon » Mon May 07, 2018 11:10 pm

Earlier, I was looking through the source code to figure out what exactly the "game" attribute did, and I noticed that everything it does was actually turned over to the user a while ago, and most of it (like actor definitions) isn't even used in the source code anymore. I was thinking that maybe it could be made into an optional attribute, or a value of "Standalone" or "GZDoom" or something could be added, so that the people who make standalone games wouldn't be loading any of that code and could just do all that from scratch.

Is this feasible?
User avatar
RockstarRaccoon
Totally Babies
 
Joined: 31 Jul 2016

Re: IWADINFO: Make "Game" attribute optional

Postby Cacodemon345 » Fri May 11, 2018 6:33 am

I have a feeling that it will break TCs.
Cacodemon345
 
Joined: 22 Dec 2017
Discord: Cacodemon345#9151
Github ID: Cacodemon345
Operating System: Windows 10/8.1/8 64-bit
Graphics Processor: ATI/AMD (Modern GZDoom)

Re: IWADINFO: Make "Game" attribute optional

Postby Graf Zahl » Fri May 11, 2018 6:57 am

There's still places in the engine which require something valid here. The "Game" setting doesn't do much but mainly defines what defaults need to be used for certain data.
User avatar
Graf Zahl
Lead GZDoom Developer
Lead GZDoom Developer
 
Joined: 19 Jul 2003
Location: Germany

Re: IWADINFO: Make "Game" attribute optional

Postby RockstarRaccoon » Fri May 11, 2018 9:39 am

I got that, I can tell from the code that it is used in a couple if statements, what I'm saying is that it feels like it's close to being depreciated simply because the vast majority of the things it set are gone, and the other things should be (and I'm pretty sure already are) user controllable.
User avatar
RockstarRaccoon
Totally Babies
 
Joined: 31 Jul 2016


Return to Feature Suggestions

Who is online

Users browsing this forum: Semrush [Bot] and 1 guest