From dcc3a6775203130610ad96d601d32e767b194687 Mon Sep 17 00:00:00 2001
From: Peter Nelson <peter1138@openttd.org>
Date: Mon, 29 Sep 2025 21:02:08 +0100
Subject: [PATCH] Fix 9cdf740097: Don't make copy of format providers when
 making a screenshot. (#14681)

While here, use projection instead of lambda to select provider.
---
 src/screenshot.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/screenshot.cpp b/src/screenshot.cpp
index 224241ec52..c12693c5a4 100644
--- a/src/screenshot.cpp
+++ b/src/screenshot.cpp
@@ -48,10 +48,10 @@ uint _heightmap_highest_peak;         ///< When saving a heightmap, this contain
  */
 static ScreenshotProvider *GetScreenshotProvider()
 {
-	auto providers = ProviderManager<ScreenshotProvider>::GetProviders();
+	const auto &providers = ProviderManager<ScreenshotProvider>::GetProviders();
 	if (providers.empty()) return nullptr;
 
-	auto it = std::ranges::find_if(providers, [](const auto &p) { return p->GetName() == _screenshot_format_name; });
+	auto it = std::ranges::find(providers, _screenshot_format_name, &ScreenshotProvider::GetName);
 	if (it != std::end(providers)) return *it;
 
 	return providers.front();