From 372e60dd0ff5e7f6fdfd7cfc21517900be9f8518 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= <mosra@centrum.cz>
Date: Sat, 27 Nov 2021 02:42:34 +0100
Subject: [PATCH] Trade: make the MaterialData duplicate asserts more paranoid.

Libc++ is that you who's this shitty?
---
 src/Magnum/Trade/MaterialData.cpp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/Magnum/Trade/MaterialData.cpp b/src/Magnum/Trade/MaterialData.cpp
index d0732faf1..9b37d8015 100644
--- a/src/Magnum/Trade/MaterialData.cpp
+++ b/src/Magnum/Trade/MaterialData.cpp
@@ -233,8 +233,13 @@ MaterialData::MaterialData(const MaterialTypes types, Containers::Array<Material
             std::sort(_data + begin, _data + end, [](const MaterialAttributeData& a, const MaterialAttributeData& b) {
                 /* Need to check here (instead of in the outer for loop) as we
                    exit the loop right after the sort and thus duplicates that
-                   occur after the first non-sorted pair wouldn't get detected. */
-                CORRADE_ASSERT(a.name() != b.name(),
+                   occur after the first non-sorted pair wouldn't get detected.
+
+                   Additionally an *extra paranoid* check for shitty STL
+                   implementations that might call this comparator with both
+                   arguments the same (in which case the assert would fire,
+                   which is undesirable). */
+                CORRADE_ASSERT(&a == &b || a.name() != b.name(),
                     "Trade::MaterialData: duplicate attribute" << a.name(), false);
                 return a.name() < b.name();
             });