From 5dd632c991d56a9345b5dc889628aadd868b2cd7 Mon Sep 17 00:00:00 2001
From: Brenden Matthews <github@brenden.brndn.io>
Date: Mon, 26 Feb 2024 08:27:10 -0500
Subject: [PATCH] Don't call FcFini()

Fontconfig has a bug in FcFini() where it calls assert when it probably
shouldn't. This looks to be fixed in
https://gitlab.freedesktop.org/fontconfig/fontconfig/-/commit/6f6b39780215714386606ca1c5457a7106639ff4,
which is in libfontconfig >=2.13.93.

Removing this seems to be safe, it will cause valgrind to show mem
leaks, but it doesn't look to be a problem in practice.
---
 src/display-x11.cc | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/display-x11.cc b/src/display-x11.cc
index 79b9f358..044fecf0 100644
--- a/src/display-x11.cc
+++ b/src/display-x11.cc
@@ -701,9 +701,6 @@ void display_output_x11::cleanup() {
     XDestroyRegion(x11_stuff.region);
     x11_stuff.region = nullptr;
   }
-#ifdef BUILD_XFT
-  FcFini();
-#endif /* BUILD_XFT */
 }
 
 void display_output_x11::set_foreground_color(Colour c) {