Add CreateScriptInstance

[EladNLG]

Adds a function that converts entities to SQObjects to the SquirrelManagers.

[catornot]

looks good, checked the offsets in ghidra 👍
didn't test in game.

[EladNLG]

Ok, genuinely, how can this be even tested?

[GeckoEidechse]

Ok, genuinely, how can this be even tested?

Usually the approach is to have a small PoC script mod that highlights the change. I don't understand enough atm to tell you whether that would make sense in this case ^^"

[EladNLG]

Ok, genuinely, how can this be even tested?

Usually the approach is to have a small PoC script mod that highlights the change. I don't understand enough atm to tell you whether that would make sense in this case ^^"

Unfortunately, no known part of the codebase (outside of #691) uses pushobject/createscriptinstance. My guess is that #691 will test this by proxy?

[catornot]

I did test the server offsets in a plugin :p

[GeckoEidechse]

Maybe we just merge without further testing ig?

[catornot]

Maybe we just merge without further testing ig?

Yeah, just merge it.

[GeckoEidechse]

Aight consider it merged. I might just gonna put a few more fixes in front to reduce the amount of cherry-picking I have to do ^^

[catornot]

Wow this got finally merged