Post by ChezJfrey in Update comments

Viewing post in Update comments

I don't know. I started a game, but do not have a telescope, so I can't check. But there are tons of errors and frankly, I'm not even sure how the game can even play properly, given the script code I see. They made a global ref pchar variable, but in all the functions, they instead assign a local pchar variable. Then, in some of the scripts where they try to use pchar, but in those cases didn't make a local var, the global pchar they try to use in those instances is invalid. I fixed that by assigning the global pchar in seadogs.c, which fixed most of that, but now I see some index out-of-bounds errors in AIShip.c and flags.c, and missing references in MAXIMUS_Functions.c. They have a wrong function name in TempQuestEnemy.c, which also has the wrong parameter type, wrong return value for their dynamic call. I'm done trying to fix their script problems, because these have nothing to do with the port to the new engine, but rather mistakes in their code.

The global vs. local pchar is why the worldmap engagement screen did not present. Anyway, here is what I have:

https://mega.nz/folder/DtcEmR6C#pjia47ERTPh1EoLClQMfOg

File is NH2018.zip and it contains only some changed files.

Raskoljnikov3 years ago

are these script errors affecting the original NH game in Storm engine 2.0 or they only affect Maelstrom engine?

ChezJfrey3 years ago

They should be a problem no matter what engine. If these are not presenting errors in 2.0, then I can only guess the script code I was supplied is not the same.

Raskoljnikov3 years ago

All the scripts are from the original New Horizons mod from theri official site. So I guess it works that way. Now that it is running Maelstrom, is it possible to implement the blood like in Storm engine 2.8?

ChezJfrey3 years ago (1 edit)

Just as an example for TempQuestEnemy.c. The code in characters_init.c tries to dynamically load a .c file, and dynamically call a function:

if (FindFile("PROGRAM\Characters\init", "*.c", fname) != "") {
        if (LoadSegment("Characters\init\" + fname)) {
            call func(&n);
            UnloadSegment("Characters\init\" + fname);
        }
}

In the case of the problem, fname is "TempQuestEnemy.c" and func = "CreateTempQuestEnemyCharacters". So it will try to call that function after it loads the .c file. But in that TempQuestEnemy.c file, there is no function in the script I have from you. It looks like this:

int CreateTempQuestEnemy(int n)

Notice that it is the wrong name, and also does not pass a reference type for &n, but rather an int n. It should look like this:

void CreateTempQuestEnemyCharacters(ref n)

What happens with that original script is nothing happens and an error is logged that it can't find function CreateTempQuestEnemyCharacters in TempQuestEnemy.c. That is not a porting problem and should not work no matter if it is Storm 2.0, Storm 2.8 or Maelstrom.

Raskoljnikov3 years ago

This is interesting, but makes sense since a lot of people worked on that mod. I will check the original program folder as soon as I get home.

Raskoljnikov3 years ago (1 edit)

This PROGRAM folder is from original New Horizons mod for 2.0 engine. I compared it and it has 2.547 files, while the one I have now has 4.393. Is it possible that some files are obsolete?

https://mega.nz/file/9DQ0gZKa#mdfJ3ekLJuMaOYZkq5HTprotUmRwk3VZlC-aF2hLCtA

ChezJfrey3 years ago

Obsolete files don't hurt anything if the references to them are removed from #include and they are never run. So obsolete files, even with errors, don't cause problems.

Raskoljnikov3 years ago

I started over with clean PROGRAM folder from NH 2018 build, fixed the libriary syntax erros. The game starts, loads the PA! splash video and then just black screen. Error log shows this:

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'LanguageGetDefaultLanguage' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetSaveDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetProgramDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetResourceDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_MakeNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_DeleteNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowShow' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowDisable' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_IsWindowEnable' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowAddNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'DialogAssembleStr' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'DialogAddParamToStr' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_StoreNodeLocksWithOff' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_RestoreNodeLocks' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_IsKeyPressed' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_RegistryExitKey' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'AddControlTreeNode' not declared

RUNTIME ERROR - file: battle_interface\loginterface.c; line: 31

Can't create class: ActionIntrface

RUNTIME ERROR - file: ships\ships_init.c; line: 210

Invalid function call

RUNTIME ERROR - file: ships\ships_init.c; line: 210

function 'InitShips' stack error

ChezJfrey3 years ago (2 edits)

If you start over, then you have to compare the old, working 2016 version, every file, that works for Maelstrom to the new one and add, remove, change the relevant parts again. You have to add the new native function declarations that exist in Maelstrom, but not Storm 2.0, you have to remove the native function declarations that no longer exist in Maelstrom, change those SendMessage statements that don't have the correct parameters because they changed from 2.0 to 2.8, you have to modify all the LogInterface, BattleInterface and flag things that don't work the same, the sound messages/calls that are different, etc.

There are literally hundreds of lines that need to change.

Raskoljnikov3 years ago

I'm doing that, but now I got stuck with this line: #library "script_interface_functions" in INTERFACE\utilite.c and don't know how to figure it out.

ChezJfrey3 years ago

It would probably be easiest to use the Program.zip from my New Horizons and compare those files. If you do a side-by-side comparison of utilite.c, the one from my New Horizons compiles just fine, so find the differences that contribute to the errors you see and make it more like the one that works, unless it looks like some of the changes are accurate for compilation and apply to the newer version because they wanted logic modification for their newer version.

Raskoljnikov3 years ago

Ok, so I fixed a few files. Now I get this in error log:

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'LanguageGetDefaultLanguage' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetSaveDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetProgramDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetResourceDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'AddControlTreeNode' not declared

RUNTIME ERROR - file: ships\ships_init.c; line: 210

Invalid function call

RUNTIME ERROR - file: ships\ships_init.c; line: 210

function 'InitShips' stack error

When I simply remove this line:

#library "script_interface_functions"

it makes hundreds of other errors. I don't have any idea what it does and what to do with it... :/

ChezJfrey3 years ago (1 edit)

For instance, referring to my other post about comparing, that utilite.c file from the 2016 NH needed some changes to work with 2.8 and Maelstrom, where these lines needed add:

native string DialogAssembleStr(string idStr, string paramStr);
native string DialogAddParamToStr(string oldParamStr, string paramID, string paramValue);
native void XI_MakeNode(string sIniFileName, string sNodeType, string sNodeName, int nPriority); 
native void XI_DeleteNode(string sNodeName); native bool XI_IsWindowEnable(string sWindowName); 
native void XI_WindowShow(string sWindowName, int bShowStatus); 
native void XI_WindowDisable(string sWindowName, int bDisableStatus); 
native void XI_WindowAddNode(string sWindowName, string sNodeName); 
native int XI_StoreNodeLocksWithOff(); native void XI_RestoreNodeLocks(int nStoreCode); 
native bool XI_IsKeyPressed(string key_name); // key_name = {"shift","control","alt"} 
native void XI_RegistryExitKey(string key_name);

And this line needed change:

#library "script_interface_functions"

Raskoljnikov3 years ago

Do you happen to know how to fix this?

COMPILE ERROR - file: seadogs.c; line: 508

Undeclared identifier: PChar

ChezJfrey3 years ago

Declare it somewhere at the global scope. In the 2018 version you supplied me, it was on line 19 of globals.c, just after these 2 lines:

float FRAND(float _x) { return rand(32768) / 32768.0 * _x; }

float frandSmall(float _x) { return rand(32) / 32.0 * _x; } // boal

ref pchar;

Raskoljnikov3 years ago

Now I get a bunch of new errors:

COMPILE ERROR - file: locations\locations.c; line: 19

native function 'SetTipsEnabled' not declared

COMPILE ERROR - file: battle_interface\utils.c; line: 7

native function 'ShipSailState' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'LanguageGetDefaultLanguage' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetSaveDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetProgramDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'GetResourceDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_MakeNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_DeleteNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowShow' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowDisable' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_IsWindowEnable' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_WindowAddNode' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'DialogAssembleStr' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'DialogAddParamToStr' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_StoreNodeLocksWithOff' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_RestoreNodeLocks' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_IsKeyPressed' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'XI_RegistryExitKey' not declared

COMPILE ERROR - file: interface\utilite.c; line: 16

native function 'AddControlTreeNode' not declared

RUNTIME ERROR - file: battle_interface\loginterface.c; line: 31

Can't create class: ActionIntrface

RUNTIME ERROR - file: ships\ships_init.c; line: 210

Invalid function call

RUNTIME ERROR - file: ships\ships_init.c; line: 210

function 'InitShips' stack error

Can't get to Start Menu still, but the game starts loading and passes the PA! splash video. So I guess we are on a good way to making it work. Just to note that I have included the 2021 updates as well.

Raskoljnikov3 years ago

This is the updated PROGRAM folder from 19th May 2021...

https://mega.nz/file/gbgHjSqZ#nxoWGXy8TuoR3LwbRzNdsSBL7AWpNUnzaXx5PlRDzKM

Raskoljnikov3 years ago

Ok, so I fixed some of them. These are the ones that appear now:

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'LanguageGetDefaultLanguage' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetSaveDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetProgramDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'GetResourceDirectory' not declared

COMPILE ERROR - file: interface\utilite.c; line: 28

native function 'AddControlTreeNode' not declared

RUNTIME ERROR - file: ships\ships_init.c; line: 210

Invalid function call

RUNTIME ERROR - file: ships\ships_init.c; line: 210

function 'InitShips' stack error

Raskoljnikov3 years ago

Sorry for bombarding you with so many messages, but I was trying to fix all by myself. I got the game to run with 2021 build, but isn't this weird, I was trying to fix the bugs in the PROGRAM folder. when I accidentally saw the log in compiler:

ini file RESOURCE\INI\RESOURCE\INI\NEW_INTERFACES\mainmenu.ini not found!

DestroyWindow

PostDWindow

System exit and cleanup:

Mem state: User memory: 0 MSSystem: 0 Blocks: 0

I made the folders RESOURCE\INI\RESOURCE\INI\NEW_INTERFACES\mainmenu and the game got to main menu screen. Isn't that weird. Why is it needing double folders?

View more in thread

Raskoljnikov3 years ago

I just checked this original Program, and there is libriary error everywhere. Omg

ChezJfrey3 years ago

The #libriary was valid in all Storm versions. I could not handle the spelling error, so changed it in Maelstrom. That is something that does need to change when porting to Maelstrom.

itch.io

Viewing post in Update comments