Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In D3D11, force block compressed textures to have dimensions divisible by 4 #17749

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

hrydgard
Copy link
Owner

@hrydgard hrydgard commented Jul 20, 2023

This of course applies to texture replacement only.

Fixes #17745 (crash when loading certain texture packs in D3D11)

This is an old unfortunate limitation. Only applies to the top mip level, which makes it obvious that it's kinda unnecessary for the hardware and indeed, Vulkan and OpenGL don't have this limitation.

…e by 4

Fixes #17745 (crash when loading certain texture packs in D3D11)

This is an old unfortunate limitation. Only applies to the top mip
level, which makes it obvious that it's kinda unnecessary for the
hardware and indeed, Vulkan and OpenGL don't have this limitation.
@hrydgard hrydgard added this to the v1.16.0 milestone Jul 20, 2023
@hrydgard hrydgard merged commit 52d30c0 into master Jul 20, 2023
19 checks passed
@hrydgard hrydgard deleted the ktx2-d3d11-fix branch July 20, 2023 18:09
@@ -337,6 +337,9 @@ bool TextureReplacer::LoadIniValues(IniFile &ini, VFSBackend *dir, bool isOverri
}
}

auto gr = GetI18NCategory(I18NCat::GRAPHICS);

g_OSD.Show(OSDType::MESSAGE_SUCCESS, gr->T("Texture replacement pack activated"));
Copy link
Collaborator

Choose a reason for hiding this comment

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

I forget if this can cause duplicates. It should be at the end of LoadIni(), since LoadIniValues() can be called multiple times for region overrides.

-[Unknown]

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

Successfully merging this pull request may close these issues.

KTX2: Crash in D3D11
2 participants