Jump to content
The Dark Mod Forums

Script backwards compatibility


snatcher

Recommended Posts

Dear all,

Is there anyway we can maintain backwards compatibility in the scripting system?

In example, the below function was introduced in tdm_events.script in TDM 2.11:

scriptEvent void removeFrobPeer(entity peer);

Is there something such us the below or can you think of any other way?

if (funcExists("removeFrobPeer"))
{

}

Many thanks in advance.

TDM Modpack 4.0

Link to comment
Share on other sites

Hum personally I don't think that exists, at lest for all functions, I could be wrong thou. 

But TDM, coming from Doom 3 should support

https://modwiki.dhewm3.org/HasFunction_(script_event)

but this is particular to script objects/entities and not to ask if some generic global script function exists anywhere and I don't really know how that would be implemented.

But couldn't script macros be used for that? Something like SDL does.

// based verbatin from SDL version define

#define TDM_VERSIONNUM(X, Y, Z)                     \
    ((X)*1000 + (Y)*100 + (Z))

#define TDM_COMPILEDVERSION \
    TDM_VERSIONNUM(TDM_MAJOR_VERSION, TDM_MINOR_VERSION, TDM_PATCHLEVEL)

#define TDM_VERSION_ATLEAST(X, Y, Z) \
    (TDM_COMPILEDVERSION >= TDM_VERSIONNUM(X, Y, Z))

#if TDM_VERSION_ATLEAST(2, 11, 0)
	// run this func only in TDM 2.11 or above
	func(){
		do stuff;
	}
#endif

Not totally sure if script language macro support is robust enough for this but it should be.

Link to comment
Share on other sites

I can tell tell your idea would work, but I am unable to get the current version.

#define TDM_COMPILEDVERSION getTDMVersion()

returns zero and:

#define TDM_COMPILEDVERSION sys.getTDMVersion()

doesn't work: can't evaluate undefined macro 'sys', substituted with zero

TDM Modpack 4.0

Link to comment
Share on other sites

Sorry my reply was a bit confusing, those macros aren't defined anywhere (thou you can define them yourself...), it was only a idea for the TDM team to implement if they wanted. Sorry for making you waste your time chacing nothing. 

TDM_MAJOR_VERSION, TDM_MINOR_VERSION and TDM_PATCHLEVEL (and the rest)  need to be defined/created by the TDM team, and made part of the standard game defines, you can define them yourself, but that would be useless to anyone else, because only you would have those defines in your copy of the game, so right now afaik there's no way to do what you want, unless the TDM team creates those or something like those macros. I hope this is more understandable. 

Edited by HMart
Link to comment
Share on other sites

Hmm Licensee.h appears to have these defined ( from svn ) :

 
#define GAME_NAME "The Dark Mod" // appears on window titles and errors
 
// greebo: Defines the darkmod release version
#define TDM_VERSION_MAJOR 2
#define TDM_VERSION_MINOR 12
#define GAME_API_VERSION TDM_VERSION_MAJOR * 100 + TDM_VERSION_MINOR // e.g. 210

#define ENGINE_VERSION "TDM 2.12" // printed in console

https://github.com/stgatilov/darkmod_src/blob/40db9596fcbf4fa3da42b3d89be0bd785620d154/framework/Licensee.h

Does calling sys.getTDMVersion()  without any define return the correct value?

I am guessing if you wanna use this in a script you just:

float tdm_version_val = sys.getTDMVersion();

if ( tdm_version_val < 2.11 ) { something } ;

Please visit TDM's IndieDB site and help promote the mod:

 

http://www.indiedb.com/mods/the-dark-mod

 

(Yeah, shameless promotion... but traffic is traffic folks...)

Link to comment
Share on other sites

32 minutes ago, nbohr1more said:

Does calling sys.getTDMVersion()  without any define return the correct value?

Yes, it returns the correct value: 210, 211...

33 minutes ago, nbohr1more said:

I am guessing if you wanna use this in a script you just:

float tdm_version_val = sys.getTDMVersion();

if ( tdm_version_val < 2.11 ) { something } ;

The problem is that something contains a function that didn't exist in previous versions, and TDM refuses to launch:

Error during initialization. Error: file script\X.script, line Y: Unknown value "removeFrobPeers"

HMart's method however, seems to be more indulgent and anything in between #if #endif gets a pass, according to my tests.

46 minutes ago, nbohr1more said:

Hmm Licensee.h appears to have these defined ( from svn ) :

#define GAME_NAME "The Dark Mod" // appears on window titles and errors
 
// greebo: Defines the darkmod release version
#define TDM_VERSION_MAJOR 2
#define TDM_VERSION_MINOR 12
#define GAME_API_VERSION TDM_VERSION_MAJOR * 100 + TDM_VERSION_MINOR // e.g. 210

#define ENGINE_VERSION "TDM 2.12" // printed in console

https://github.com/stgatilov/darkmod_src/blob/40db9596fcbf4fa3da42b3d89be0bd785620d154/framework/Licensee.h

I cannot retrieve the values of these identifiers in a script:

...

#define TDM_VERSION_ATLEAST(X) \
	(X >= TDM_VERSION_MINOR)

...

#if TDM_VERSION_ATLEAST(11)

...

#endif

...

 

TDM Modpack 4.0

Link to comment
Share on other sites

Hmm...

Pretty hacky, but I wonder if you could use something like this?

 

float tdm_version_val = sys.getTDMVersion();

if ( tdm_version_val > 2.11 ) { 

string tempvalue = removeFrobPeers ;

tempvalue(entity peer) ;

} ;

 

Please visit TDM's IndieDB site and help promote the mod:

 

http://www.indiedb.com/mods/the-dark-mod

 

(Yeah, shameless promotion... but traffic is traffic folks...)

Link to comment
Share on other sites

Hacky indeed!

if ( tdm_version_val > 211 ) { // no dot

string tempvalue = "removeFrobPeers" ; // quotes

tempvalue(entity peer) ;

} ; 

No joy, though:

Error during initialization. Error: file script\X.script, line Y: idTypeDef::FieldType: tried to get field type on non-field type

 

It is ok, I guess... Nobody apparently ever had a need for such backwards compatibility anyway, short or long term. Let's forget it. Forward it is!

Many thanks for the support.

TDM Modpack 4.0

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

  • Recent Status Updates

    • Petike the Taffer  »  DeTeEff

      I've updated the articles for your FMs and your author category at the wiki. Your newer nickname (DeTeEff) now comes first, and the one in parentheses is your older nickname (Fieldmedic). Just to avoid confusing people who played your FMs years ago and remember your older nickname. I've added a wiki article for your latest FM, Who Watches the Watcher?, as part of my current updating efforts. Unless I overlooked something, you have five different FMs so far.
      · 0 replies
    • Petike the Taffer

      I've finally managed to log in to The Dark Mod Wiki. I'm back in the saddle and before the holidays start in full, I'll be adding a few new FM articles and doing other updates. Written in Stone is already done.
      · 4 replies
    • nbohr1more

      TDM 15th Anniversary Contest is now active! Please declare your participation: https://forums.thedarkmod.com/index.php?/topic/22413-the-dark-mod-15th-anniversary-contest-entry-thread/
       
      · 0 replies
    • JackFarmer

      @TheUnbeholden
      You cannot receive PMs. Could you please be so kind and check your mailbox if it is full (or maybe you switched off the function)?
      · 1 reply
    • OrbWeaver

      I like the new frob highlight but it would nice if it was less "flickery" while moving over objects (especially barred metal doors).
      · 4 replies
×
×
  • Create New...