diff options
| author | Charles Kerr <charles.kerr@canonical.com> | 2012-05-24 19:00:51 -0500 |
|---|---|---|
| committer | Charles Kerr <charles.kerr@canonical.com> | 2012-05-24 19:00:51 -0500 |
| commit | 9e188d4b15e27ec26dd139fee61a4af3c522a5a8 (patch) | |
| tree | 4b6dc34f147777f81b948f009685aa3a240e4596 /src | |
| parent | 2643aa2da220f14f28744a0f2403a8e68364239a (diff) | |
| download | ayatana-indicator-power-9e188d4b15e27ec26dd139fee61a4af3c522a5a8.tar.gz ayatana-indicator-power-9e188d4b15e27ec26dd139fee61a4af3c522a5a8.tar.bz2 ayatana-indicator-power-9e188d4b15e27ec26dd139fee61a4af3c522a5a8.zip | |
remove the 'default' clause from get_property() and set_property().
Coverage testing isn't reaching them... glib is weeding out these invalid property keys before the device.c functions are ever reached. Nevertheless, leaving out a 'default' clause in a switch statement feels very unnatural to me. *twitch* *twitch*
Diffstat (limited to 'src')
| -rw-r--r-- | src/device.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/src/device.c b/src/device.c index 80b3208..4b983e9 100644 --- a/src/device.c +++ b/src/device.c @@ -173,10 +173,6 @@ get_property (GObject * o, guint prop_id, GValue * value, GParamSpec * pspec) case PROP_TIME: g_value_set_uint64 (value, priv->time); break; - - default: - G_OBJECT_WARN_INVALID_PROPERTY_ID (o, prop_id, pspec); - break; } } @@ -213,10 +209,6 @@ set_property (GObject * o, guint prop_id, const GValue * value, GParamSpec * psp case PROP_TIME: priv->time = g_value_get_uint64(value); break; - - default: - G_OBJECT_WARN_INVALID_PROPERTY_ID (o, prop_id, pspec); - break; } } |