diff options
author | Povilas Kanapickas <povilas@radix.lt> | 2014-07-07 13:26:44 +0200 |
---|---|---|
committer | Kjell Ahlstedt <kjell.ahlstedt@bredband.net> | 2014-07-07 13:26:44 +0200 |
commit | c8e324babd01ae81908aab478fd5fff244507a82 (patch) | |
tree | 171771642c477efde772e078ea3c360b27e66190 | |
parent | 27ccef4d6f12c1fd3186578b790218d74afaa36b (diff) | |
download | glibmm-c8e324babd01ae81908aab478fd5fff244507a82.tar.gz |
Rename ambiguously named overridden property variable and type
* glib/glibmm/class.[cc|h]:
* glib/glibmm/interface.cc:
* glib/glibmm/property.cc: Rename properties_quark and properties_type to
iface_properties_quark and iface_properties_type.
As custom_[set|get]_property_callback() in property.cc also sets and gets
user-defined properties, this naming choice may be confusing.
These names are intended for use by glibmm only. Renaming is not an
ABI/API break. Bug #731484.
-rw-r--r-- | glib/glibmm/class.cc | 17 | ||||
-rw-r--r-- | glib/glibmm/class.h | 10 | ||||
-rw-r--r-- | glib/glibmm/interface.cc | 8 | ||||
-rw-r--r-- | glib/glibmm/property.cc | 42 |
4 files changed, 42 insertions, 35 deletions
diff --git a/glib/glibmm/class.cc b/glib/glibmm/class.cc index f68138bb..6107c5cf 100644 --- a/glib/glibmm/class.cc +++ b/glib/glibmm/class.cc @@ -150,19 +150,20 @@ GType Class::clone_custom_type(const char* custom_type_name, } // Initialize the static quark to store/get custom type properties. -GQuark Class::properties_quark = g_quark_from_string("gtkmm_CustomObject_properties"); +GQuark Class::iface_properties_quark = g_quark_from_string("gtkmm_CustomObject_iface_properties"); // static void Class::custom_class_base_finalize_function(void* g_class) { const GType gtype = G_TYPE_FROM_CLASS(g_class); - // Free the data related to the properties for the custom type, if any. - properties_type* props = static_cast<properties_type*>(g_type_get_qdata(gtype, properties_quark)); + // Free the data related to the interface properties for the custom type, if any. + iface_properties_type* props = static_cast<iface_properties_type*>( + g_type_get_qdata(gtype, iface_properties_quark)); if(props) { - for(properties_type::size_type i = 0; i < props->size(); i++) + for(iface_properties_type::size_type i = 0; i < props->size(); i++) { g_value_unset((*props)[i]); g_free((*props)[i]); @@ -190,12 +191,12 @@ void Class::custom_class_init_function(void* g_class, void* class_data) // Override the properties of implemented interfaces, if any. const GType object_type = G_TYPE_FROM_CLASS(g_class); - Class::properties_type* props = static_cast<Class::properties_type*>( - g_type_get_qdata(object_type, Class::properties_quark)); + Class::iface_properties_type* props = static_cast<Class::iface_properties_type*>( + g_type_get_qdata(object_type, Class::iface_properties_quark)); if (!props) { - props = new Class::properties_type(); - g_type_set_qdata(object_type, Class::properties_quark, props); + props = new Class::iface_properties_type(); + g_type_set_qdata(object_type, Class::iface_properties_quark, props); } guint n_interfaces = 0; diff --git a/glib/glibmm/class.h b/glib/glibmm/class.h index 438c3cae..89e9d34f 100644 --- a/glib/glibmm/class.h +++ b/glib/glibmm/class.h @@ -25,7 +25,7 @@ #include <glib-object.h> #include <glibmmconfig.h> //Include this here so that the /private/*.h classes have access to GLIBMM_VFUNCS_ENABLED -#include <vector> //For properties that custom types might override. +#include <vector> //For interface properties that custom types might override. #ifndef DOXYGEN_SHOULD_SKIP_THIS @@ -101,10 +101,10 @@ private: public: #ifndef DOXYGEN_SHOULD_SKIP_THIS - // The type that holds the values of the properties of custom types. - typedef std::vector<GValue*> properties_type; - // The quark used for storing/getting the properties of custom types. - static GQuark properties_quark; + // The type that holds the values of the interface properties of custom types. + typedef std::vector<GValue*> iface_properties_type; + // The quark used for storing/getting the interface properties of custom types. + static GQuark iface_properties_quark; #endif }; diff --git a/glib/glibmm/interface.cc b/glib/glibmm/interface.cc index 19554ca4..c8495653 100644 --- a/glib/glibmm/interface.cc +++ b/glib/glibmm/interface.cc @@ -66,13 +66,13 @@ Interface::Interface(const Interface_Class& interface_class) // Override the properties of the derived interface, if any. const GType custom_type = G_OBJECT_CLASS_TYPE(instance_class); - Class::properties_type* props = static_cast<Class::properties_type*>( - g_type_get_qdata(custom_type, Class::properties_quark)); + Class::iface_properties_type* props = static_cast<Class::iface_properties_type*>( + g_type_get_qdata(custom_type, Class::iface_properties_quark)); if(!props) { - props = new Class::properties_type(); - g_type_set_qdata(custom_type, Class::properties_quark, props); + props = new Class::iface_properties_type(); + g_type_set_qdata(custom_type, Class::iface_properties_quark, props); } const guint n_existing_props = props->size(); diff --git a/glib/glibmm/property.cc b/glib/glibmm/property.cc index 0ae03457..98e6c892 100644 --- a/glib/glibmm/property.cc +++ b/glib/glibmm/property.cc @@ -112,21 +112,24 @@ void custom_get_property_callback(GObject* object, unsigned int property_id, GType custom_type = G_OBJECT_TYPE(object); - Class::properties_type* props = static_cast<Class::properties_type*>(g_type_get_qdata(custom_type, Class::properties_quark)); - Class::properties_type::size_type props_size = 0; + Class::iface_properties_type* iface_props = static_cast<Class::iface_properties_type*>( + g_type_get_qdata(custom_type, Class::iface_properties_quark)); - if(props) props_size = props->size(); + Class::iface_properties_type::size_type iface_props_size = 0; - if(property_id <= props_size) + if (iface_props) + iface_props_size = iface_props->size(); + + if (property_id <= iface_props_size) { - g_value_copy((*props)[property_id - 1], value); + g_value_copy((*iface_props)[property_id - 1], value); } else { if(Glib::ObjectBase *const wrapper = Glib::ObjectBase::_get_current_wrapper(object)) { PropertyBase& property = - property_from_id(*wrapper, property_id - props_size); + property_from_id(*wrapper, property_id - iface_props_size); if((property.object_ == wrapper) && (property.param_spec_ == param_spec)) g_value_copy(property.value_.gobj(), value); @@ -144,14 +147,17 @@ void custom_set_property_callback(GObject* object, unsigned int property_id, GType custom_type = G_OBJECT_TYPE(object); - Class::properties_type* props = static_cast<Class::properties_type*>(g_type_get_qdata(custom_type, Class::properties_quark)); - Class::properties_type::size_type props_size = 0; + Class::iface_properties_type* iface_props = static_cast<Class::iface_properties_type*>( + g_type_get_qdata(custom_type, Class::iface_properties_quark)); + + Class::iface_properties_type::size_type iface_props_size = 0; - if(props) props_size = props->size(); + if (iface_props) + iface_props_size = iface_props->size(); - if(property_id <= props_size) + if (property_id <= iface_props_size) { - g_value_copy(value, (*props)[property_id - 1]); + g_value_copy(value, (*iface_props)[property_id - 1]); g_object_notify(object, g_param_spec_get_name(param_spec)); } else @@ -159,7 +165,7 @@ void custom_set_property_callback(GObject* object, unsigned int property_id, if(Glib::ObjectBase *const wrapper = Glib::ObjectBase::_get_current_wrapper(object)) { PropertyBase& property = - property_from_id(*wrapper, property_id - props_size); + property_from_id(*wrapper, property_id - iface_props_size); if((property.object_ == wrapper) && (property.param_spec_ == param_spec)) { @@ -215,12 +221,14 @@ void PropertyBase::install_property(GParamSpec* param_spec) // properties. GType gtype = G_OBJECT_TYPE(object_->gobj()); - Class::properties_type* props = static_cast<Class::properties_type*>(g_type_get_qdata(gtype, Class::properties_quark)); + Class::iface_properties_type* iface_props = static_cast<Class::iface_properties_type*>( + g_type_get_qdata(gtype, Class::iface_properties_quark)); - Class::properties_type::size_type props_size = 0; - if(props) props_size = props->size(); + Class::iface_properties_type::size_type iface_props_size = 0; + if (iface_props) + iface_props_size = iface_props->size(); - const unsigned int property_id = property_to_id(*object_, *this) + props_size; + const unsigned int property_id = property_to_id(*object_, *this) + iface_props_size; g_object_class_install_property(G_OBJECT_GET_CLASS(object_->gobj()), property_id, param_spec); @@ -246,5 +254,3 @@ void PropertyBase::notify() } } // namespace Glib - - |