diff options
| author | Murray Cumming <murrayc@murrayc.com> | 2015-07-15 11:48:55 +0200 |
|---|---|---|
| committer | Murray Cumming <murrayc@murrayc.com> | 2015-07-15 12:18:16 +0200 |
| commit | 254f5b93e08e45cd32ccdbb0f4a31c433ece9d08 (patch) | |
| tree | 1d0570d46627ad2bed137980bce51974f453a8fd /gio/src/dbussubtreevtable.ccg | |
| parent | 90df091409fd5addf9d75c4d637d39e0e67e115a (diff) | |
| download | glibmm-254f5b93e08e45cd32ccdbb0f4a31c433ece9d08.tar.gz | |
Avoid shadowed variables.
Because this generally invites programming errors, though I am
less concerned about shadowing of method names by parameter or
variable names, which requires some tedious parameter renaming.
In MatchInfo::set_gobject() the confusion between take_ownership and
this.take_ownership does seem to have caused a programming error,
hopefully now corrected.
Diffstat (limited to 'gio/src/dbussubtreevtable.ccg')
| -rw-r--r-- | gio/src/dbussubtreevtable.ccg | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/gio/src/dbussubtreevtable.ccg b/gio/src/dbussubtreevtable.ccg index a2481012..5e726cc5 100644 --- a/gio/src/dbussubtreevtable.ccg +++ b/gio/src/dbussubtreevtable.ccg @@ -105,21 +105,21 @@ static const GDBusInterfaceVTable* DBusSubtreeVTable_Dispatch_giomm_callback( const char* interface_name, const char* node, void** out_user_data, void* user_data) { - Gio::DBus::SubtreeVTable* vtable = + Gio::DBus::SubtreeVTable* vtable_subtree = static_cast<Gio::DBus::SubtreeVTable*>(user_data); Gio::DBus::SubtreeVTable::SlotSubtreeDispatch* the_slot = - vtable->get_slot_dispatch(); + vtable_subtree->get_slot_dispatch(); try { - const Gio::DBus::InterfaceVTable* vtable = + const Gio::DBus::InterfaceVTable* vtable_iface = (*the_slot)(Glib::wrap(connection, true), sender, object_path, interface_name, (node ? node : "")); - *out_user_data = const_cast<Gio::DBus::InterfaceVTable*>(vtable); + *out_user_data = const_cast<Gio::DBus::InterfaceVTable*>(vtable_iface); - return vtable->gobj(); + return vtable_iface->gobj(); } catch(...) { |