diff options
author | Dan Winship <danw@gnome.org> | 2012-01-19 18:23:28 -0500 |
---|---|---|
committer | Dan Williams <dcbw@redhat.com> | 2012-02-03 10:33:43 -0600 |
commit | 5afcee46936ab547a95fdaa7e415aa3ba6de47d2 (patch) | |
tree | eeebe2e20e01b5bd99d8d1a32c06e8f3fc590d74 /libnm-glib/nm-access-point.c | |
parent | 2e48cc092c133ed3e10efcc00cf36b29c9d3efc8 (diff) | |
download | NetworkManager-5afcee46936ab547a95fdaa7e415aa3ba6de47d2.tar.gz |
libnm-glib: simplify property getter methods
Rather than having every property getter method have code to fetch
that specific property's value, just call the new
_nm_object_ensure_inited() (which makes sure that we've read all the
property values on the object at least once), and then return the
cached value. (After we've read the initial property values, the
PropertiesChanged signal handler will ensure that the values are kept
up to date, so we can always just return cached property values after
that point.)
This then lets us get rid of _nm_object_get_property() and its
wrappers.
Diffstat (limited to 'libnm-glib/nm-access-point.c')
-rw-r--r-- | libnm-glib/nm-access-point.c | 117 |
1 files changed, 18 insertions, 99 deletions
diff --git a/libnm-glib/nm-access-point.c b/libnm-glib/nm-access-point.c index 2a5d20c270..9a2536520d 100644 --- a/libnm-glib/nm-access-point.c +++ b/libnm-glib/nm-access-point.c @@ -113,19 +113,10 @@ nm_access_point_new (DBusGConnection *connection, const char *path) NM80211ApFlags nm_access_point_get_flags (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), NM_802_11_AP_FLAGS_NONE); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->flags) { - priv->flags = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_FLAGS, - NULL); - } - - return priv->flags; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->flags; } /** @@ -139,19 +130,10 @@ nm_access_point_get_flags (NMAccessPoint *ap) NM80211ApSecurityFlags nm_access_point_get_wpa_flags (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), NM_802_11_AP_SEC_NONE); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->wpa_flags) { - priv->wpa_flags = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_WPA_FLAGS, - NULL); - } - - return priv->wpa_flags; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->wpa_flags; } /** @@ -166,19 +148,10 @@ nm_access_point_get_wpa_flags (NMAccessPoint *ap) NM80211ApSecurityFlags nm_access_point_get_rsn_flags (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), NM_802_11_AP_SEC_NONE); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->rsn_flags) { - priv->rsn_flags = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_RSN_FLAGS, - NULL); - } - - return priv->rsn_flags; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->rsn_flags; } /** @@ -193,19 +166,10 @@ nm_access_point_get_rsn_flags (NMAccessPoint *ap) const GByteArray * nm_access_point_get_ssid (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), NULL); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->ssid) { - priv->ssid = _nm_object_get_byte_array_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_SSID, - NULL); - } - - return priv->ssid; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->ssid; } /** @@ -219,19 +183,10 @@ nm_access_point_get_ssid (NMAccessPoint *ap) guint32 nm_access_point_get_frequency (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), 0); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->frequency) { - priv->frequency = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_FREQUENCY, - NULL); - } - - return priv->frequency; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->frequency; } /** @@ -246,19 +201,10 @@ nm_access_point_get_frequency (NMAccessPoint *ap) const char * nm_access_point_get_bssid (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), NULL); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->bssid) { - priv->bssid = _nm_object_get_string_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_HW_ADDRESS, - NULL); - } - - return priv->bssid; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->bssid; } /** @@ -289,19 +235,10 @@ nm_access_point_get_hw_address (NMAccessPoint *ap) NM80211Mode nm_access_point_get_mode (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), 0); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->mode) { - priv->mode = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_MODE, - NULL); - } - - return priv->mode; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->mode; } /** @@ -315,19 +252,10 @@ nm_access_point_get_mode (NMAccessPoint *ap) guint32 nm_access_point_get_max_bitrate (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), 0); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->max_bitrate) { - priv->max_bitrate = _nm_object_get_uint_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_MAX_BITRATE, - NULL); - } - - return priv->max_bitrate; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->max_bitrate; } /** @@ -341,19 +269,10 @@ nm_access_point_get_max_bitrate (NMAccessPoint *ap) guint8 nm_access_point_get_strength (NMAccessPoint *ap) { - NMAccessPointPrivate *priv; - g_return_val_if_fail (NM_IS_ACCESS_POINT (ap), 0); - priv = NM_ACCESS_POINT_GET_PRIVATE (ap); - if (!priv->strength) { - priv->strength = _nm_object_get_byte_property (NM_OBJECT (ap), - NM_DBUS_INTERFACE_ACCESS_POINT, - DBUS_PROP_STRENGTH, - NULL); - } - - return priv->strength; + _nm_object_ensure_inited (NM_OBJECT (ap)); + return NM_ACCESS_POINT_GET_PRIVATE (ap)->strength; } /** |