Skip to content

Do not treat binary Doom WAD files as text#48349

Merged
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
elf-alchemist:doom_wads
Feb 4, 2026
Merged

Do not treat binary Doom WAD files as text#48349
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
elf-alchemist:doom_wads

Conversation

@elf-alchemist
Copy link
Contributor

I primarily contribute to Doom source ports and related utilities. When working with Doom WADs, the primary archive format for the game, Zed will sometimes incorrectly assume that such files are plain text, due to the nature of said container format not being compressed, some common text files included in them will trigger a false positive in Zed's text detection mode.

Considering some of these files can go from a couple dozen to a few hundred megabytes, this usually makes Zed hang on my lower end machine, until I manually terminate it :/ and has happened when I accidentally clicked on one via the file explorer, or when viewing Git diffs.

Release Notes:

  • Fixed Doom WAD files being erroneously treated as text

@cla-bot
Copy link

cla-bot bot commented Feb 4, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @elf-alchemist on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@zed-community-bot zed-community-bot bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label Feb 4, 2026
@elf-alchemist
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 4, 2026
@cla-bot
Copy link

cla-bot bot commented Feb 4, 2026

The cla-bot has been summoned, and re-checked this pull request!

@SomeoneToIgnore SomeoneToIgnore self-assigned this Feb 4, 2026
Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😄
Thank you.

@SomeoneToIgnore SomeoneToIgnore merged commit 5d84932 into zed-industries:main Feb 4, 2026
35 checks passed
@elf-alchemist elf-alchemist deleted the doom_wads branch February 4, 2026 18:25
adb-sh pushed a commit to adb-sh/zed that referenced this pull request Feb 5, 2026
I primarily contribute to Doom source ports and related utilities. When
working with Doom WADs, the primary archive format for the game, Zed
will sometimes incorrectly assume that such files are plain text, due to
the nature of said container format not being compressed, some common
text files included in them will trigger a false positive in Zed's text
detection mode.

Considering some of these files can go from a couple dozen to a few
hundred megabytes, this usually makes Zed hang on my lower end machine,
until I manually terminate it :/ and has happened when I accidentally
clicked on one via the file explorer, or when viewing Git diffs.

Release Notes:

- Fixed Doom WAD files being erroneously treated as text
rtfeldman pushed a commit that referenced this pull request Feb 5, 2026
I primarily contribute to Doom source ports and related utilities. When
working with Doom WADs, the primary archive format for the game, Zed
will sometimes incorrectly assume that such files are plain text, due to
the nature of said container format not being compressed, some common
text files included in them will trigger a false positive in Zed's text
detection mode.

Considering some of these files can go from a couple dozen to a few
hundred megabytes, this usually makes Zed hang on my lower end machine,
until I manually terminate it :/ and has happened when I accidentally
clicked on one via the file explorer, or when viewing Git diffs.

Release Notes:

- Fixed Doom WAD files being erroneously treated as text
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants