wookieejedi 9403f17fbb Adds safety checks to scripting orders with waypoints (#5173) ...

* Adds safety checks to scripting orders with waypoints

Turns out setting a waypoint order then trying to get to target of that order in the same frame triggers a crash in the scripting API for ship orders (`order.cpp` line 295). This was due to invalid lookups which lacked safety checks.

This PR adds those safety checks. Now, if this situation is encountered, it leaves `objnum` at -1 and thus returns an object for which `isValid` is false.

Fix is tested and works as expected. Also fixes #5172

* appease clang

* cleanup thanks to Goober

* clang again

* forgot to re-add the nullptr

* remove redundant checks

2023-02-07 15:57:39 -05:00

..

libs

clear credits before rebuilding

2023-02-04 01:26:04 -06:00

objs

Adds safety checks to scripting orders with waypoints (#5173)

2023-02-07 15:57:39 -05:00

LuaCoroutineRunner.cpp

Capture Ade skip arguments locally

2020-08-10 09:39:08 +02:00

LuaCoroutineRunner.h

Add feature to execute Lua coroutines at specific points

2020-06-06 21:29:48 +02:00

LuaEventCallback.cpp

Add more generic scripting callback system

2018-07-02 22:56:45 +02:00

LuaEventCallback.h

Ensure Lua functions are executed in correct state

2020-06-06 14:00:23 +02:00

LuaExecutionContext.cpp

Add scripting API for custom execution context

2020-08-11 09:38:39 +02:00

LuaExecutionContext.h

Add scripting API for custom execution context

2020-08-11 09:38:39 +02:00

LuaPromise.cpp

add some logging to waitAsync

2022-06-28 12:02:13 -04:00

LuaPromise.h

Add feature to execute Lua coroutines at specific points

2020-06-06 21:29:48 +02:00