telepathy-mission-control: mcd_master_create_manager: unvirtualize
Simon McVittie
smcv at kemper.freedesktop.org
Tue Sep 3 08:15:48 PDT 2013
Module: telepathy-mission-control
Branch: master
Commit: 3d77940b7b2b7394a695a46efc008dc31e85b45d
URL: http://cgit.freedesktop.org/telepathy/telepathy-mission-control/commit/?id=3d77940b7b2b7394a695a46efc008dc31e85b45d
Author: Simon McVittie <simon.mcvittie at collabora.co.uk>
Date: Wed Aug 28 16:42:47 2013 +0100
mcd_master_create_manager: unvirtualize
There's no point in this being virtual: there are no subclasses,
and it isn't API any more.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68712
Signed-off-by: Simon McVittie <simon.mcvittie at collabora.co.uk>
Reviewed-by: Guillaume Desmottes <guillaume.desmottes at collabora.co.uk>
---
src/mcd-master.c | 15 +++------------
src/mcd-master.h | 8 --------
2 files changed, 3 insertions(+), 20 deletions(-)
diff --git a/src/mcd-master.c b/src/mcd-master.c
index ed515fc..603b777 100644
--- a/src/mcd-master.c
+++ b/src/mcd-master.c
@@ -402,14 +402,6 @@ mcd_master_constructor (GType type, guint n_params,
return (GObject *) master;
}
-static McdManager *
-mcd_master_create_manager (McdMaster *master, const gchar *unique_name)
-{
- McdMasterPrivate *priv = master->priv;
-
- return mcd_manager_new (unique_name, priv->dispatcher, priv->dbus_daemon);
-}
-
static void
mcd_master_class_init (McdMasterClass * klass)
{
@@ -422,8 +414,6 @@ mcd_master_class_init (McdMasterClass * klass)
object_class->set_property = _mcd_master_set_property;
object_class->dispose = _mcd_master_dispose;
- klass->create_manager = mcd_master_create_manager;
-
/* Properties */
g_object_class_install_property
(object_class, PROP_DISPATCHER,
@@ -509,8 +499,9 @@ _mcd_master_lookup_manager (McdMaster *master,
return manager;
}
- manager = MCD_MASTER_GET_CLASS (master)->create_manager
- (master, unique_name);
+ manager = mcd_manager_new (unique_name,
+ master->priv->dispatcher,
+ master->priv->dbus_daemon);
if (G_UNLIKELY (!manager))
g_warning ("Manager %s not created", unique_name);
else
diff --git a/src/mcd-master.h b/src/mcd-master.h
index 5bb38b7..e9a0a6b 100644
--- a/src/mcd-master.h
+++ b/src/mcd-master.h
@@ -54,14 +54,6 @@ struct _McdMaster
struct _McdMasterClass
{
McdOperationClass parent_class;
- McdManager *(*create_manager) (McdMaster *master,
- const gchar *unique_name);
- void (*_mc_reserved1) (void);
- void (*_mc_reserved2) (void);
- void (*_mc_reserved3) (void);
- void (*_mc_reserved4) (void);
- void (*_mc_reserved5) (void);
- void (*_mc_reserved6) (void);
};
GType mcd_master_get_type (void);
More information about the telepathy-commits
mailing list