CachedProperty.__set_name__ will now raise a RuntimeError if it was already bound to a class.

Author: jordanbriere

src/core/modules/core/core_cache.cpp

@@ -247,6 +247,16 @@ object CCachedProperty::bind(object self, object owner, str name)
 
 void CCachedProperty::__set_name__(object owner, str name)
 {
+	if (m_name && !m_owner.is_none())
+	{
+		const char *szName = extract<const char *>(str(".").join(make_tuple(m_owner().attr("__qualname__"), m_name)));
+		BOOST_RAISE_EXCEPTION(
+			PyExc_RuntimeError,
+			"This property was already bound as \"%s\".",
+			szName
+		)
+	}
+
 	m_name = name;
 	m_owner = object(handle<>(PyWeakref_NewRef(owner.ptr(), NULL)));
 }

src/core/modules/core/core_cache_wrap.cpp

@@ -269,7 +269,10 @@ void export_cached_property(scope _cache)
 		":param class owner:\n"
 		"	The class this property is being bound to.\n"
 		":param str name:\n"
-		"	The name this property is being bound as.",
+		"	The name this property is being bound as."
+		"\n"
+		":raises RuntimeError:\n"
+		"	If this property was already bound to a class.",
 		args("self", "owner", "name")
 	);