Skip to content

add mission variable save/load to SaveLoadX#78

Open
Goober5000 wants to merge 1 commit intoFSO-Scripters:masterfrom
Goober5000:saveloadx_variables
Open

add mission variable save/load to SaveLoadX#78
Goober5000 wants to merge 1 commit intoFSO-Scripters:masterfrom
Goober5000:saveloadx_variables

Conversation

@Goober5000
Copy link
Copy Markdown
Contributor

@Goober5000 Goober5000 commented Apr 18, 2026

Extends SaveLoadX checkpoints to capture and restore non-persistent mission variables alongside ship state. Player-persistent and campaign-persistent variables are skipped since FSO manages those across missions. Also adds a new lua-savestate-load-external-var SEXP to complement lua-savestate-load-external. Old save files without data.Variables will load cleanly.

In draft pending testing. Tested by Shiv.

@Goober5000 Goober5000 added the enhancement New feature or request label Apr 18, 2026
@Goober5000 Goober5000 force-pushed the saveloadx_variables branch from bf15bc4 to ff81747 Compare April 18, 2026 05:18
Extends SaveLoadX checkpoints to capture and restore non-persistent mission variables alongside ship state. Player-persistent and campaign-persistent variables are skipped since FSO manages those across missions. Also adds a new `lua-savestate-load-external-var` SEXP to complement `lua-savestate-load-external`. Old save files without `data.Variables` will load cleanly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Goober5000 Goober5000 force-pushed the saveloadx_variables branch from ff81747 to e0a7189 Compare April 19, 2026 01:09
@Goober5000 Goober5000 marked this pull request as ready for review April 21, 2026 23:16
mcywinskittc
mcywinskittc previously approved these changes Apr 22, 2026
@xShivan
Copy link
Copy Markdown

xShivan commented Apr 23, 2026

Tested both external load and standard loading scenarios. Lgtm 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants