Skip to content

[0.8] Use zstd for datafile v5 - #3290

Draft
Bamcane wants to merge 1 commit into
teeworlds:masterfrom
Bamcane:zstd
Draft

Bamcane wants to merge 1 commit into
teeworlds:masterfrom
Bamcane:zstd

Conversation

@Bamcane

@Bamcane Bamcane commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

This change introduces zstd compression, breaking backward compatibility.
However, it enables significant map size reductions and improved loading performance, especially on low-bandwidth servers.

@Bamcane

Bamcane commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

EDIT: This was a wrong comparison

@Bamcane

Bamcane commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

Updated the bam to compile zstd

@Bamcane

Bamcane commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

Enabled asm

@tempral

tempral commented Oct 12, 2025

Copy link
Copy Markdown

At the main repo readme: https://github.com/facebook/zstd
the is a comparison table

@Bamcane

Bamcane commented Oct 13, 2025

Copy link
Copy Markdown
Contributor Author

try to find the system library.

@Bamcane

Bamcane commented Oct 13, 2025

Copy link
Copy Markdown
Contributor Author

New comparsion
screenshot

@heinrich5991

Copy link
Copy Markdown
Contributor

Another "easy" win for map downloads would be using HTTPS. That's probably always faster than using a homebrew protocol.

@Bamcane
Bamcane force-pushed the zstd branch 3 times, most recently from d3905c9 to eebf49c Compare November 2, 2025 04:02
@Bamcane

Bamcane commented Nov 2, 2025

Copy link
Copy Markdown
Contributor Author

I had no idea to fix windows build. :(

@Bamcane

Bamcane commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

Maybe relate to #1979

@Bamcane
Bamcane force-pushed the zstd branch 3 times, most recently from b26c3cb to d6f076c Compare November 23, 2025 08:42
@Bamcane

Bamcane commented Nov 23, 2025

Copy link
Copy Markdown
Contributor Author

Build failed was caused by bam. It used gcc compiler on Windows instead of MSVC.

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.

3 participants