From f4b0f4d78fe5f76c96998ba54ed421f42c0517b0 Mon Sep 17 00:00:00 2001 From: Cameron Gutman Date: Tue, 12 Sep 2023 18:36:53 -0500 Subject: [PATCH] Functionally revert f60e7241 Switching to indirect rendering has a high risk of performance and functionality regressions, so let's just continue using AVSampleBufferDisplayLayer until we have a Metal renderer since #973 is fixed. This isn't an actual revert because we want to still keep the indirect override around. --- app/streaming/video/ffmpeg-renderers/vt.mm | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/app/streaming/video/ffmpeg-renderers/vt.mm b/app/streaming/video/ffmpeg-renderers/vt.mm index 1a84b1b8..d8d9ffc2 100644 --- a/app/streaming/video/ffmpeg-renderers/vt.mm +++ b/app/streaming/video/ffmpeg-renderers/vt.mm @@ -478,19 +478,7 @@ public: "Using indirect rendering due to environment variable"); m_DirectRendering = false; } - else if (params->videoFormat & VIDEO_FORMAT_MASK_10BIT) { - SDL_LogInfo(SDL_LOG_CATEGORY_APPLICATION, - "Using direct rendering for 10-bit content"); - m_DirectRendering = true; - } - else if (isAppleSilicon) { - SDL_LogInfo(SDL_LOG_CATEGORY_APPLICATION, - "Using indirect rendering for 8-bit content on Apple Silicon"); - m_DirectRendering = false; - } else { - SDL_LogInfo(SDL_LOG_CATEGORY_APPLICATION, - "Using direct rendering for 8-bit content on Intel"); m_DirectRendering = true; }