diff options
author | Jacki <jacki@thejackimonster.de> | 2024-01-19 00:48:49 +0100 |
---|---|---|
committer | Jacki <jacki@thejackimonster.de> | 2024-01-19 00:48:49 +0100 |
commit | 68dc9951534988d9ac21358f51ee72c0a81e2a9d (patch) | |
tree | 47e6dc1f642fa279d586f74e02c18ff3a7ed1f34 | |
parent | 8b9553de982477e80ae9465e1eafb50eaa532a7c (diff) | |
download | messenger-gtk-68dc9951534988d9ac21358f51ee72c0a81e2a9d.tar.gz messenger-gtk-68dc9951534988d9ac21358f51ee72c0a81e2a9d.zip |
Add documentation comments to header and code cleanup
Signed-off-by: Jacki <jacki@thejackimonster.de>
-rw-r--r-- | src/application.c | 2 | ||||
-rw-r--r-- | src/contact.h | 2 | ||||
-rw-r--r-- | src/request.c | 12 | ||||
-rw-r--r-- | src/request.h | 58 | ||||
-rw-r--r-- | src/ui/new_contact.c | 2 | ||||
-rw-r--r-- | src/ui/settings.c | 2 |
6 files changed, 77 insertions, 1 deletions
diff --git a/src/application.c b/src/application.c index d98d327..afde604 100644 --- a/src/application.c +++ b/src/application.c | |||
@@ -351,6 +351,8 @@ _request_background_callback(GObject *source_object, | |||
351 | portal, result, &error | 351 | portal, result, &error |
352 | ); | 352 | ); |
353 | 353 | ||
354 | request_drop(request); | ||
355 | |||
354 | if (error) { | 356 | if (error) { |
355 | g_printerr("ERROR: %s\n", error->message); | 357 | g_printerr("ERROR: %s\n", error->message); |
356 | g_error_free(error); | 358 | g_error_free(error); |
diff --git a/src/contact.h b/src/contact.h index a923444..33c20c7 100644 --- a/src/contact.h +++ b/src/contact.h | |||
@@ -63,7 +63,7 @@ contact_destroy_info(struct GNUNET_CHAT_Contact *contact); | |||
63 | */ | 63 | */ |
64 | void | 64 | void |
65 | contact_set_last_message_to_info(const struct GNUNET_CHAT_Contact *contact, | 65 | contact_set_last_message_to_info(const struct GNUNET_CHAT_Contact *contact, |
66 | void *message); | 66 | void *message); |
67 | 67 | ||
68 | /** | 68 | /** |
69 | * Returns the latest join/leave UI message handle of | 69 | * Returns the latest join/leave UI message handle of |
diff --git a/src/request.c b/src/request.c index 21ceabd..3560d8f 100644 --- a/src/request.c +++ b/src/request.c | |||
@@ -118,6 +118,18 @@ request_cleanup(MESSENGER_Request *request) | |||
118 | } | 118 | } |
119 | 119 | ||
120 | void | 120 | void |
121 | request_drop(MESSENGER_Request *request) | ||
122 | { | ||
123 | if (request->application->requests) | ||
124 | request->application->requests = g_list_remove( | ||
125 | request->application->requests, | ||
126 | request | ||
127 | ); | ||
128 | |||
129 | request_delete(request); | ||
130 | } | ||
131 | |||
132 | void | ||
121 | request_delete(MESSENGER_Request *request) | 133 | request_delete(MESSENGER_Request *request) |
122 | { | 134 | { |
123 | request_cleanup(request); | 135 | request_cleanup(request); |
diff --git a/src/request.h b/src/request.h index 53b8dde..18507ae 100644 --- a/src/request.h +++ b/src/request.h | |||
@@ -36,29 +36,87 @@ typedef struct MESSENGER_Request { | |||
36 | gpointer user_data; | 36 | gpointer user_data; |
37 | } MESSENGER_Request; | 37 | } MESSENGER_Request; |
38 | 38 | ||
39 | /** | ||
40 | * Creates a new request for the messsenger | ||
41 | * application for a certain permission. | ||
42 | * | ||
43 | * The request object will automatically be | ||
44 | * added to the list of the messenger application. | ||
45 | * | ||
46 | * @param application Messenger application | ||
47 | * @param cancellable Cancellable object (optional) | ||
48 | * @param user_data User data (optional) | ||
49 | * @return New request object | ||
50 | */ | ||
39 | MESSENGER_Request* | 51 | MESSENGER_Request* |
40 | request_new(MESSENGER_Application *application, | 52 | request_new(MESSENGER_Application *application, |
41 | GCancellable *cancellable, | 53 | GCancellable *cancellable, |
42 | gpointer user_data); | 54 | gpointer user_data); |
43 | 55 | ||
56 | /** | ||
57 | * Creates a new request for the messsenger | ||
58 | * application for a background permission. | ||
59 | * | ||
60 | * @param application Messenger application | ||
61 | * @param flags Background flags | ||
62 | * @param callback Callback | ||
63 | * @param user_data User data | ||
64 | * @return New background request object | ||
65 | */ | ||
44 | MESSENGER_Request* | 66 | MESSENGER_Request* |
45 | request_new_background(MESSENGER_Application *application, | 67 | request_new_background(MESSENGER_Application *application, |
46 | XdpBackgroundFlags flags, | 68 | XdpBackgroundFlags flags, |
47 | GAsyncReadyCallback callback, | 69 | GAsyncReadyCallback callback, |
48 | gpointer user_data); | 70 | gpointer user_data); |
49 | 71 | ||
72 | /** | ||
73 | * Creates a new request for the messsenger | ||
74 | * application for a camera permission. | ||
75 | * | ||
76 | * @param application Messenger application | ||
77 | * @param flags Camera flags | ||
78 | * @param callback Callback | ||
79 | * @param user_data User data | ||
80 | * @return New camera request object | ||
81 | */ | ||
50 | MESSENGER_Request* | 82 | MESSENGER_Request* |
51 | request_new_camera(MESSENGER_Application *application, | 83 | request_new_camera(MESSENGER_Application *application, |
52 | XdpCameraFlags flags, | 84 | XdpCameraFlags flags, |
53 | GAsyncReadyCallback callback, | 85 | GAsyncReadyCallback callback, |
54 | gpointer user_data); | 86 | gpointer user_data); |
55 | 87 | ||
88 | /** | ||
89 | * Cancel a request object if possible. | ||
90 | * | ||
91 | * @param request Request object | ||
92 | */ | ||
56 | void | 93 | void |
57 | request_cancel(MESSENGER_Request *request); | 94 | request_cancel(MESSENGER_Request *request); |
58 | 95 | ||
96 | /** | ||
97 | * Cleanup a request object and its owned | ||
98 | * resources. | ||
99 | * | ||
100 | * @param request Request object | ||
101 | */ | ||
59 | void | 102 | void |
60 | request_cleanup(MESSENGER_Request *request); | 103 | request_cleanup(MESSENGER_Request *request); |
61 | 104 | ||
105 | /** | ||
106 | * Drop a request object from the messenger | ||
107 | * application list of requests. | ||
108 | * | ||
109 | * @param request Request object | ||
110 | */ | ||
111 | void | ||
112 | request_drop(MESSENGER_Request *request); | ||
113 | |||
114 | /** | ||
115 | * Delete a request object and its owned | ||
116 | * resources. | ||
117 | * | ||
118 | * @param request Request object | ||
119 | */ | ||
62 | void | 120 | void |
63 | request_delete(MESSENGER_Request *request); | 121 | request_delete(MESSENGER_Request *request); |
64 | 122 | ||
diff --git a/src/ui/new_contact.c b/src/ui/new_contact.c index ab3e63d..c300801 100644 --- a/src/ui/new_contact.c +++ b/src/ui/new_contact.c | |||
@@ -291,6 +291,8 @@ _request_camera_callback(GObject *source_object, | |||
291 | portal, result, &error | 291 | portal, result, &error |
292 | ); | 292 | ); |
293 | 293 | ||
294 | request_drop(request); | ||
295 | |||
294 | if (error) { | 296 | if (error) { |
295 | g_printerr("ERROR: %s\n", error->message); | 297 | g_printerr("ERROR: %s\n", error->message); |
296 | g_error_free(error); | 298 | g_error_free(error); |
diff --git a/src/ui/settings.c b/src/ui/settings.c index 0c4e2ba..e05da47 100644 --- a/src/ui/settings.c +++ b/src/ui/settings.c | |||
@@ -59,6 +59,8 @@ _request_background_callback(GObject *source_object, | |||
59 | portal, result, &error | 59 | portal, result, &error |
60 | ); | 60 | ); |
61 | 61 | ||
62 | request_drop(request); | ||
63 | |||
62 | if (error) { | 64 | if (error) { |
63 | g_printerr("ERROR: %s\n", error->message); | 65 | g_printerr("ERROR: %s\n", error->message); |
64 | g_error_free(error); | 66 | g_error_free(error); |