applications/gnome-control-center
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

network: Stop NetDeviceEthernet deriving from NetDeviceSimple

Browse source 
It overrode everything anyway.
This commit is contained in:
Robert Ancell 2019-10-23 11:02:55 +13:00
parent 9327a5f4d1
commit e90ed2df8a
5 changed files with 31 additions and 44 deletions
Download patch file Download diff file Expand all files Collapse all files

1
panels/network/cc-network-panel.c
View file

@ -32,6 +32,7 @@
#include "net-device.h" #include "net-device.h"
#include "net-device-mobile.h" #include "net-device-mobile.h"
#include "net-device-simple.h"
#include "net-device-wifi.h" #include "net-device-wifi.h"
#include "net-device-ethernet.h" #include "net-device-ethernet.h"
#include "net-object.h" #include "net-object.h"

4
panels/network/net-device-ethernet.c
View file

@ -37,7 +37,7 @@
struct _NetDeviceEthernet struct _NetDeviceEthernet
{ {
NetDeviceSimple parent; NetDevice parent;
GtkBuilder *builder; GtkBuilder *builder;
GtkButton *add_profile_button; GtkButton *add_profile_button;
@ -55,7 +55,7 @@ struct _NetDeviceEthernet
GHashTable *connections; GHashTable *connections;
}; };
G_DEFINE_TYPE (NetDeviceEthernet, net_device_ethernet, NET_TYPE_DEVICE_SIMPLE) G_DEFINE_TYPE (NetDeviceEthernet, net_device_ethernet, NET_TYPE_DEVICE)
static GtkWidget * static GtkWidget *
device_ethernet_add_to_stack (NetObject *object, device_ethernet_add_to_stack (NetObject *object,

4
panels/network/net-device-ethernet.h
View file

@ -23,12 +23,12 @@
#include <glib-object.h> #include <glib-object.h>
#include "net-device-simple.h" #include "net-device.h"
G_BEGIN_DECLS G_BEGIN_DECLS
#define NET_TYPE_DEVICE_ETHERNET (net_device_ethernet_get_type ()) #define NET_TYPE_DEVICE_ETHERNET (net_device_ethernet_get_type ())
G_DECLARE_FINAL_TYPE (NetDeviceEthernet, net_device_ethernet, NET, DEVICE_ETHERNET, NetDeviceSimple) G_DECLARE_FINAL_TYPE (NetDeviceEthernet, net_device_ethernet, NET, DEVICE_ETHERNET, NetDevice)
NetDeviceEthernet *net_device_ethernet_new (CcPanel *panel, NetDeviceEthernet *net_device_ethernet_new (CcPanel *panel,
GCancellable *cancellable, GCancellable *cancellable,

59
panels/network/net-device-simple.c
View file

@ -31,7 +31,7 @@
#include "net-device-simple.h" #include "net-device-simple.h"
typedef struct struct _NetDeviceSimple
{ {
GtkBuilder *builder; GtkBuilder *builder;
GtkBox *box; GtkBox *box;
@ -41,18 +41,16 @@ typedef struct
GtkSeparator *separator; GtkSeparator *separator;
gboolean updating_device; gboolean updating_device;
} NetDeviceSimplePrivate; };
G_DEFINE_TYPE_WITH_PRIVATE (NetDeviceSimple, net_device_simple, NET_TYPE_DEVICE) G_DEFINE_TYPE (NetDeviceSimple, net_device_simple, NET_TYPE_DEVICE)
void void
net_device_simple_set_show_separator (NetDeviceSimple *self, net_device_simple_set_show_separator (NetDeviceSimple *self,
gboolean show_separator) gboolean show_separator)
{ {
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
/* add widgets to size group */ /* add widgets to size group */
gtk_widget_set_visible (GTK_WIDGET (priv->separator), show_separator); gtk_widget_set_visible (GTK_WIDGET (self->separator), show_separator);
} }
static GtkWidget * static GtkWidget *
@ -61,11 +59,10 @@ device_simple_proxy_add_to_stack (NetObject *object,
GtkSizeGroup *heading_size_group) GtkSizeGroup *heading_size_group)
{ {
NetDeviceSimple *self = NET_DEVICE_SIMPLE (object); NetDeviceSimple *self = NET_DEVICE_SIMPLE (object);
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
/* add widgets to size group */ /* add widgets to size group */
gtk_stack_add_named (stack, GTK_WIDGET (priv->box), net_object_get_id (object)); gtk_stack_add_named (stack, GTK_WIDGET (self->box), net_object_get_id (object));
return GTK_WIDGET (priv->box); return GTK_WIDGET (self->box);
} }
static void static void
@ -73,9 +70,7 @@ update_off_switch_from_device_state (GtkSwitch *sw,
NMDeviceState state, NMDeviceState state,
NetDeviceSimple *self) NetDeviceSimple *self)
{ {
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self); self->updating_device = TRUE;
priv->updating_device = TRUE;
switch (state) { switch (state) {
case NM_DEVICE_STATE_UNMANAGED: case NM_DEVICE_STATE_UNMANAGED:
case NM_DEVICE_STATE_UNAVAILABLE: case NM_DEVICE_STATE_UNAVAILABLE:
@ -88,30 +83,29 @@ update_off_switch_from_device_state (GtkSwitch *sw,
gtk_switch_set_active (sw, TRUE); gtk_switch_set_active (sw, TRUE);
break; break;
} }
priv->updating_device = FALSE; self->updating_device = FALSE;
} }
static void static void
nm_device_simple_refresh_ui (NetDeviceSimple *self) nm_device_simple_refresh_ui (NetDeviceSimple *self)
{ {
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
NMDevice *nm_device; NMDevice *nm_device;
NMDeviceState state; NMDeviceState state;
nm_device = net_device_get_nm_device (NET_DEVICE (self)); nm_device = net_device_get_nm_device (NET_DEVICE (self));
/* set device kind */ /* set device kind */
g_object_bind_property (self, "title", priv->device_label, "label", 0); g_object_bind_property (self, "title", self->device_label, "label", 0);
/* set up the device on/off switch */ /* set up the device on/off switch */
state = nm_device_get_state (nm_device); state = nm_device_get_state (nm_device);
gtk_widget_set_visible (GTK_WIDGET (priv->device_off_switch), gtk_widget_set_visible (GTK_WIDGET (self->device_off_switch),
state != NM_DEVICE_STATE_UNAVAILABLE state != NM_DEVICE_STATE_UNAVAILABLE
&& state != NM_DEVICE_STATE_UNMANAGED); && state != NM_DEVICE_STATE_UNMANAGED);
update_off_switch_from_device_state (priv->device_off_switch, state, self); update_off_switch_from_device_state (self->device_off_switch, state, self);
/* set up the Options button */ /* set up the Options button */
gtk_widget_set_visible (GTK_WIDGET (priv->options_button), state != NM_DEVICE_STATE_UNMANAGED && g_find_program_in_path ("nm-connection-editor") != NULL); gtk_widget_set_visible (GTK_WIDGET (self->options_button), state != NM_DEVICE_STATE_UNMANAGED && g_find_program_in_path ("nm-connection-editor") != NULL);
} }
static void static void
@ -124,7 +118,6 @@ device_simple_refresh (NetObject *object)
static void static void
device_off_toggled (NetDeviceSimple *self) device_off_toggled (NetDeviceSimple *self)
{ {
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
const GPtrArray *acs; const GPtrArray *acs;
gboolean active; gboolean active;
gint i; gint i;
@ -132,10 +125,10 @@ device_off_toggled (NetDeviceSimple *self)
NMConnection *connection; NMConnection *connection;
NMClient *client; NMClient *client;
if (priv->updating_device) if (self->updating_device)
return; return;
active = gtk_switch_get_active (priv->device_off_switch); active = gtk_switch_get_active (self->device_off_switch);
if (active) { if (active) {
client = net_object_get_client (NET_OBJECT (self)); client = net_object_get_client (NET_OBJECT (self));
connection = net_device_get_find_connection (NET_DEVICE (self)); connection = net_device_get_find_connection (NET_DEVICE (self));
@ -194,9 +187,8 @@ static void
net_device_simple_finalize (GObject *object) net_device_simple_finalize (GObject *object)
{ {
NetDeviceSimple *self = NET_DEVICE_SIMPLE (object); NetDeviceSimple *self = NET_DEVICE_SIMPLE (object);
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
g_clear_object (&priv->builder); g_clear_object (&self->builder);
G_OBJECT_CLASS (net_device_simple_parent_class)->finalize (object); G_OBJECT_CLASS (net_device_simple_parent_class)->finalize (object);
} }
@ -216,11 +208,10 @@ net_device_simple_class_init (NetDeviceSimpleClass *klass)
static void static void
net_device_simple_init (NetDeviceSimple *self) net_device_simple_init (NetDeviceSimple *self)
{ {
NetDeviceSimplePrivate *priv = net_device_simple_get_instance_private (self);
g_autoptr(GError) error = NULL; g_autoptr(GError) error = NULL;
priv->builder = gtk_builder_new (); self->builder = gtk_builder_new ();
gtk_builder_add_from_resource (priv->builder, gtk_builder_add_from_resource (self->builder,
"/org/gnome/control-center/network/network-simple.ui", "/org/gnome/control-center/network/network-simple.ui",
&error); &error);
if (error != NULL) { if (error != NULL) {
@ -228,19 +219,19 @@ net_device_simple_init (NetDeviceSimple *self)
return; return;
} }
priv->box = GTK_BOX (gtk_builder_get_object (priv->builder, "box")); self->box = GTK_BOX (gtk_builder_get_object (self->builder, "box"));
priv->device_label = GTK_LABEL (gtk_builder_get_object (priv->builder, "device_label")); self->device_label = GTK_LABEL (gtk_builder_get_object (self->builder, "device_label"));
priv->device_off_switch = GTK_SWITCH (gtk_builder_get_object (priv->builder, "device_off_switch")); self->device_off_switch = GTK_SWITCH (gtk_builder_get_object (self->builder, "device_off_switch"));
priv->options_button = GTK_BUTTON (gtk_builder_get_object (priv->builder, "options_button")); self->options_button = GTK_BUTTON (gtk_builder_get_object (self->builder, "options_button"));
priv->separator = GTK_SEPARATOR (gtk_builder_get_object (priv->builder, "separator")); self->separator = GTK_SEPARATOR (gtk_builder_get_object (self->builder, "separator"));
/* setup simple combobox model */ /* setup simple combobox model */
g_signal_connect_swapped (priv->device_off_switch, "notify::active", g_signal_connect_swapped (self->device_off_switch, "notify::active",
G_CALLBACK (device_off_toggled), self); G_CALLBACK (device_off_toggled), self);
g_signal_connect_swapped (priv->options_button, "clicked", g_signal_connect_swapped (self->options_button, "clicked",
G_CALLBACK (edit_connection), self); G_CALLBACK (edit_connection), self);
gtk_widget_set_visible (GTK_WIDGET (priv->options_button), g_find_program_in_path ("nm-connection-editor") != NULL); gtk_widget_set_visible (GTK_WIDGET (self->options_button), g_find_program_in_path ("nm-connection-editor") != NULL);
} }
NetDeviceSimple * NetDeviceSimple *

7
panels/network/net-device-simple.h
View file

@ -29,12 +29,7 @@
G_BEGIN_DECLS G_BEGIN_DECLS
#define NET_TYPE_DEVICE_SIMPLE (net_device_simple_get_type ()) #define NET_TYPE_DEVICE_SIMPLE (net_device_simple_get_type ())
G_DECLARE_DERIVABLE_TYPE (NetDeviceSimple, net_device_simple, NET, DEVICE_SIMPLE, NetDevice) G_DECLARE_FINAL_TYPE (NetDeviceSimple, net_device_simple, NET, DEVICE_SIMPLE, NetDevice)
struct _NetDeviceSimpleClass
{
NetDeviceClass parent_class;
};
NetDeviceSimple *net_device_simple_new (CcPanel *panel, NetDeviceSimple *net_device_simple_new (CcPanel *panel,
GCancellable *cancellable, GCancellable *cancellable,