[retro-gtk/gtk-cleanup: 8/10] core-view: Use autocleanups
- From: Adrien Plazas <aplazas src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [retro-gtk/gtk-cleanup: 8/10] core-view: Use autocleanups
- Date: Sun, 24 May 2020 12:40:18 +0000 (UTC)
commit bbe67782ceed0695883c1bf30004d480026c3e25
Author: Adrien Plazas <kekun plazas laposte net>
Date: Sun May 24 14:35:26 2020 +0200
core-view: Use autocleanups
This makes the code safer and more concise.
retro-gtk/retro-core-view.c | 22 ++++++++--------------
1 file changed, 8 insertions(+), 14 deletions(-)
---
diff --git a/retro-gtk/retro-core-view.c b/retro-gtk/retro-core-view.c
index 4292beb..8c8a237 100644
--- a/retro-gtk/retro-core-view.c
+++ b/retro-gtk/retro-core-view.c
@@ -128,7 +128,7 @@ grab (RetroCoreView *self,
{
GdkSeat *seat;
GdkDisplay *display;
- GdkCursor *cursor;
+ g_autoptr (GdkCursor) cursor = NULL;
GdkScreen *screen = NULL;
GdkMonitor *monitor;
GdkRectangle monitor_geometry;
@@ -137,11 +137,8 @@ grab (RetroCoreView *self,
g_assert (window != NULL);
g_assert (event != NULL);
- if (self->grabbed_device != NULL)
- g_object_unref (self->grabbed_device);
-
- if (self->grabbed_screen != NULL)
- g_object_unref (self->grabbed_screen);
+ g_clear_object (&self->grabbed_device);
+ g_clear_object (&self->grabbed_screen);
self->grabbed_device = g_object_ref (device);
seat = gdk_device_get_seat (device);
@@ -161,8 +158,6 @@ grab (RetroCoreView *self,
recenter_pointer (self);
g_signal_emit (self, signals[SIG_CONTROLLER_STATE_CHANGED_SIGNAL], 0);
-
- g_object_unref (cursor);
}
static void
@@ -653,7 +648,6 @@ retro_core_view_set_as_default_controller (RetroCoreView *self,
RetroCore *core)
{
RetroControllerType type;
- RetroController *controller;
guint64 capabilities;
g_return_if_fail (RETRO_IS_CORE_VIEW (self));
@@ -664,12 +658,12 @@ retro_core_view_set_as_default_controller (RetroCoreView *self,
for (type = RETRO_CONTROLLER_TYPE_NONE;
type < RETRO_CONTROLLER_TYPE_COUNT;
type++) {
- if ((capabilities & (1 << type)) == 0)
- continue;
+ if ((capabilities & (1 << type)) != 0) {
+ g_autoptr (RetroController) controller =
+ retro_core_view_as_controller (self, type);
- controller = retro_core_view_as_controller (self, type);
- retro_core_set_default_controller (core, type, controller);
- g_object_unref (controller);
+ retro_core_set_default_controller (core, type, controller);
+ }
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]