gentoo-mirror/gentoo
1
0
Fork 0
mirror of https://github.com/gentoo-mirror/gentoo.git synced 2026-08-06 00:48:18 -07:00
Code Issues Projects Releases Wiki Activity

dev-cpp/glibmm: remove method that is not used any where

Browse Source 
Closes: https://bugs.gentoo.org/945375

Signed-off-by: Denis Pronin <dannftk@yandex.ru>
Closes: https://github.com/gentoo/gentoo/pull/39525
Signed-off-by: Sam James <sam@gentoo.org>
This commit is contained in:
Denis Pronin
2024-11-29 10:51:58 +03:00
committed by Sam James
parent ee56348b61
commit b8cca5206e
2 changed files with 54 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

52
dev-cpp/glibmm/files/glibmm-2.78.1-libcxx-fix.patch Normal file
View File

@@ -0,0 +1,52 @@
# This patch is fetched from the upstream and dedicated to making compiling with clang-19 possible
# Bug related https://bugs.gentoo.org/945375
# The patch is added by Denis Pronin <dannftk@yandex.ru>
From 72c5ad98d858fa4fc0183a5ec1a61bb48ef5bd95 Mon Sep 17 00:00:00 2001
From: Kjell Ahlstedt <kjellahlstedt@gmail.com>
Date: Fri, 22 Mar 2024 10:32:45 +0100
Subject: [PATCH] Gio::content_type_guess(): Remove most of an unneccesary
overload
clang 19.0.0 does not like the overload that takes a
const std::basic_string<guchar>&. This overload should have been removed
16 years ago by commit 84135b93a20e6c9fe652849959d3ff90474c99bb.
It can't be removed completely now, because that would break ABI.
Remove as much as possible.
Fixes #118
---
gio/giomm/contenttype.cc | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/gio/giomm/contenttype.cc b/gio/giomm/contenttype.cc
index abe67337..8eb1bd71 100644
--- a/gio/giomm/contenttype.cc
+++ b/gio/giomm/contenttype.cc
@@ -84,15 +84,18 @@ content_type_from_mime_type(const Glib::ustring& mime_type)
g_content_type_from_mime_type(mime_type.c_str()));
}
+//TODO: When we can break ABI, remove this content_type_guess() overload.
+// Commit https://gitlab.gnome.org/GNOME/glibmm/-/commit/84135b93a20e6c9fe652849959d3ff90474c99bb
+// removed it from contenttype.h, but not from contenttype.cc.
+// For the time being, it's replaced by a function that does almost nothing.
+// clang 19.0.0 does not like a complete version.
+// See issue https://gitlab.gnome.org/GNOME/glibmm/-/issues/118
Glib::ustring
content_type_guess(
- const std::string& filename, const std::basic_string<guchar>& data, bool& result_uncertain)
+ const std::string& /*filename*/, const std::basic_string<guchar>& /*data*/, bool& result_uncertain)
{
- gboolean c_result_uncertain = FALSE;
- const gchar* c_filename = filename.empty() ? nullptr : filename.c_str();
- gchar* cresult = g_content_type_guess(c_filename, data.c_str(), data.size(), &c_result_uncertain);
- result_uncertain = c_result_uncertain;
- return Glib::convert_return_gchar_ptr_to_ustring(cresult);
+ result_uncertain = true;
+ return Glib::ustring();
}
Glib::ustring
--
2.45.2

2
dev-cpp/glibmm/glibmm-2.78.1.ebuild
View File

@@ -29,6 +29,8 @@ BDEPEND="
)
"
PATCHES=( "${FILESDIR}"/${P}-libcxx-fix.patch )
src_prepare() {
default