Skip to content

Unify v3 and v4 plugins into a single source#25

Merged
dowoge merged 3 commits intomainfrom
unify-v3-v4
Apr 19, 2026
Merged

Unify v3 and v4 plugins into a single source#25
dowoge merged 3 commits intomainfrom
unify-v3-v4

Conversation

@dowoge
Copy link
Copy Markdown
Member

@dowoge dowoge commented Apr 19, 2026

Shared code lives in scripting/include/offstyledb_core.inc. Two thin wrapper .sp files produce the two builds:

  • offstyledb.sp -> offstyledb.smx (shavit v4)
  • offstyledb_v3.sp -> offstyledb_v3.smx (shavit v3, defines SHAVIT_V3)

Version-specific divergences are guarded by #if defined SHAVIT_V3:
shavit native/forward decls, AddAdditionalReplayPathsHere vs ShouldSaveReplayCopy, OnReplaySaved signature, and the EnsureTempReplayDir/GetTempReplayPath helpers that only v4 needs.

tommy added 3 commits April 19, 2026 01:02
Shared code lives in scripting/include/offstyledb_core.inc. Two thin
wrapper .sp files produce the two builds:
- offstyledb.sp -> offstyledb.smx (shavit v4)
- offstyledb_v3.sp -> offstyledb_v3.smx (shavit v3, defines SHAVIT_V3)

Version-specific divergences are guarded by #if defined SHAVIT_V3:
shavit native/forward decls, AddAdditionalReplayPathsHere vs
ShouldSaveReplayCopy, OnReplaySaved signature, and the
EnsureTempReplayDir/GetTempReplayPath helpers that only v4 needs.
v3 shavit doesn't ship these helpers via its include, and the v3 build
of the unified source doesn't #include <shavit>, so declare them inline
under SHAVIT_V3.
@dowoge
Copy link
Copy Markdown
Member Author

dowoge commented Apr 19, 2026

WE BALL HELLA

@dowoge dowoge merged commit 282ac9e into main Apr 19, 2026
@dowoge dowoge deleted the unify-v3-v4 branch April 19, 2026 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant