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

    • Ansome

      Turns out my 15th anniversary mission idea has already been done once or twice before! I've been beaten to the punch once again, but I suppose that's to be expected when there's over 170 FMs out there, eh? I'm not complaining though, I love learning new tricks and taking inspiration from past FMs. Best of luck on your own fan missions!
      · 4 replies
    • The Black Arrow

      I wanna play Doom 3, but fhDoom has much better features than dhewm3, yet fhDoom is old, outdated and probably not supported. Damn!
      Makes me think that TDM engine for Doom 3 itself would actually be perfect.
      · 6 replies
    • Petike the Taffer

      Maybe a bit of advice ? In the FM series I'm preparing, the two main characters have the given names Toby and Agnes (it's the protagonist and deuteragonist, respectively), I've been toying with the idea of giving them family names as well, since many of the FM series have named protagonists who have surnames. Toby's from a family who were usually farriers, though he eventually wound up working as a cobbler (this serves as a daylight "front" for his night time thieving). Would it make sense if the man's popularly accepted family name was Farrier ? It's an existing, though less common English surname, and it directly refers to the profession practiced by his relatives. Your suggestions ?
      · 9 replies
    • nbohr1more

      Looks like the "Reverse April Fools" releases were too well hidden. Darkfate still hasn't acknowledge all the new releases. Did you play any of the new April Fools missions?
      · 5 replies
    • The Black Arrow

      Hope everyone has the blessing of undying motivation for "The Dark Mod 15th Anniversary Contest". Can't wait to see the many magnificent missions you all may have planned. Good luck, with an Ace!
      · 0 replies
×
×
  • Create New...