Skip to content

bsppp: Restructure To Support More Formats - #152

Draft
OrsellGit wants to merge 2 commits into
craftablescience:mainfrom
OrsellGit:feat/bsp-restructure
Draft

OrsellGit wants to merge 2 commits into
craftablescience:mainfrom
OrsellGit:feat/bsp-restructure

Conversation

@OrsellGit

Copy link
Copy Markdown

This is a smallish restructure of the bsppp module to allow for more BSP or other map formats to be added to the module. This structure is based on how vpkpp is structured, where each format has its own header and CPP file.

Valve's BSP format has been moved to format/VBSP where the LumpData.h file has been combined into it since those lump structures are more specific to VBSP than to potentially other formats. The other currently supported BSPs are still part of VBSP.

What still needs to be done:

  • Separate the specific BSP supported formats that aren't from Valve (ex. Contagion).
    • Unsure if I actually do want to separate Contagion or not since it has just some small specific fixes and still uses the same file extension, so I don't think it would warrant a separate class. However, for something like Respawn's rBSP, which has a different header and 127 lumps vs. 64 lumps for VBSP, yes, it needs a separate class.
  • Make a base map file class similar to PackFile for vpkpp. Find what functions or functionality can be pulled out of VBSP that can be used across various other formats.

@OrsellGit
OrsellGit force-pushed the feat/bsp-restructure branch from b5eac45 to b2ab0f0 Compare October 22, 2026 21:58
@OrsellGit OrsellGit changed the title Draft: bsppp Restructure To Support More Formats bsppp: Restructure To Support More Formats Oct 22, 2026

This branch has not been deployed

No deployments
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