diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 421534741e6b8ac4d30cabce27b752e83b2234e2..eeff37511e189ee26e1603e50fcbc5f0941a94ba 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -232,7 +232,7 @@ build-doc: - git clean -dfx - 'echo "Build opts: ${BUILD_OPTS}"' - meson setup ${BUILD_OPTS} _build - - tools/check-doc _build + - meson compile -C _build - mv _build/docs/phosh-0/ _reference/ artifacts: paths: diff --git a/src/auth.c b/src/auth.c index 939a9d1cb060ed14a6c8f154732247283abd0014..08b184f9d5ff2e3b923344ad384b95dccbea5577 100644 --- a/src/auth.c +++ b/src/auth.c @@ -170,11 +170,11 @@ phosh_auth_new (void) void -phosh_auth_authenticate_async_start (PhoshAuth *self, - const char *number, - GCancellable *cancellable, - GAsyncReadyCallback callback, - gpointer callback_data) +phosh_auth_authenticate_async (PhoshAuth *self, + const char *number, + GCancellable *cancellable, + GAsyncReadyCallback callback, + gpointer callback_data) { GTask *task; @@ -186,9 +186,9 @@ phosh_auth_authenticate_async_start (PhoshAuth *self, gboolean -phosh_auth_authenticate_async_finish (PhoshAuth *self, - GAsyncResult *result, - GError **error) +phosh_auth_authenticate_finish (PhoshAuth *self, + GAsyncResult *result, + GError **error) { g_return_val_if_fail (g_task_is_valid (result, self), FALSE); return g_task_propagate_boolean (G_TASK (result), error); diff --git a/src/auth.h b/src/auth.h index e977c111a68ab0142ef7f6b1435d75a892c91792..b6c5cb875aad0a3ae5ed47a7156e2c1029b89bf1 100644 --- a/src/auth.h +++ b/src/auth.h @@ -16,12 +16,12 @@ G_DECLARE_FINAL_TYPE (PhoshAuth, phosh_auth, PHOSH, AUTH, GObject) GObject *phosh_auth_new (void); -void phosh_auth_authenticate_async_start (PhoshAuth *self, - const char *number, - GCancellable *cancellable, - GAsyncReadyCallback callback, - gpointer user_data); -gboolean phosh_auth_authenticate_async_finish (PhoshAuth *self, - GAsyncResult *result, - GError **error); +void phosh_auth_authenticate_async (PhoshAuth *self, + const char *number, + GCancellable *cancellable, + GAsyncReadyCallback callback, + gpointer user_data); +gboolean phosh_auth_authenticate_finish (PhoshAuth *self, + GAsyncResult *result, + GError **error); diff --git a/src/lockscreen.c b/src/lockscreen.c index fb5c1a2bd98a8471d67e6209f8543be395dca1c2..311b8094a5982b63c71df5fb6ea819bcede8f182 100644 --- a/src/lockscreen.c +++ b/src/lockscreen.c @@ -276,7 +276,7 @@ auth_async_cb (PhoshAuth *auth, GAsyncResult *result, PhoshLockscreen *self) gboolean authenticated; priv = phosh_lockscreen_get_instance_private (self); - authenticated = phosh_auth_authenticate_async_finish (auth, result, &error); + authenticated = phosh_auth_authenticate_finish (auth, result, &error); if (error != NULL) { g_warning ("Auth failed unexpected: %s", error->message); return; @@ -394,11 +394,11 @@ submit_cb (PhoshLockscreen *self) if (priv->auth == NULL) priv->auth = PHOSH_AUTH (phosh_auth_new ()); - phosh_auth_authenticate_async_start (priv->auth, - input, - NULL, - (GAsyncReadyCallback)auth_async_cb, - g_object_ref (self)); + phosh_auth_authenticate_async (priv->auth, + input, + NULL, + (GAsyncReadyCallback)auth_async_cb, + g_object_ref (self)); } diff --git a/src/meson.build b/src/meson.build index 07fb7cced2ae565a41a2323d09bdb0e0964ac16e..da7d46d831616d59ceab86d3a6fb2f014a6b3a61 100644 --- a/src/meson.build +++ b/src/meson.build @@ -474,6 +474,7 @@ if enable_introspection includes : ['Gcr-3', 'Gio-2.0', 'Gtk-3.0', 'GnomeDesktop-3.0', 'Handy-1', 'NM-1.0'], extra_args : phosh_gir_extra_args, dependencies : phosh_static_lib_dep, + fatal_warnings : true, ) endif diff --git a/src/monitor/head-priv.h b/src/monitor/head-priv.h index 9d598333928cce2ee3d2ea360c6d745f1a48fd99..d116cdc3375d427a9d43db7e0e83ae53e742d2d8 100644 --- a/src/monitor/head-priv.h +++ b/src/monitor/head-priv.h @@ -51,7 +51,7 @@ struct _PhoshHead { PhoshHeadMode *mode; GPtrArray *modes; - struct pending { + struct PhoshHeadStatePending { int32_t x, y; enum wl_output_transform transform; PhoshHeadMode *mode; diff --git a/src/settings/audio-devices.c b/src/settings/audio-devices.c index 7749c3af80d435e53a7033ed15a7f559b1662a17..7f2546b6314f6fdb5b8df3fb486f70e99ca2fd3d 100644 --- a/src/settings/audio-devices.c +++ b/src/settings/audio-devices.c @@ -312,9 +312,16 @@ phosh_audio_devices_init (PhoshAudioDevices *self) g_signal_connect_swapped (self->devices, "items-changed", G_CALLBACK (on_items_changed), self); } - +/** + * phosh_audio_devices_new: + * @mixer_control: A new GvcMixerControl + * @is_input: Whether this is this an input + * + * Gets a new audio devices object which exposes the currently known + * input or output devices as a list model. + */ PhoshAudioDevices * -phosh_audio_devices_new (GvcMixerControl *mixer_control, gboolean is_input) +phosh_audio_devices_new (gpointer mixer_control, gboolean is_input) { return g_object_new (PHOSH_TYPE_AUDIO_DEVICES, "mixer-control", mixer_control, diff --git a/src/settings/audio-devices.h b/src/settings/audio-devices.h index da9be099b81a3b732c13e0a03da716ab1c51ce00..20b2c5d90d7d186596ec7968c42fcaef326296a9 100644 --- a/src/settings/audio-devices.h +++ b/src/settings/audio-devices.h @@ -16,7 +16,7 @@ G_BEGIN_DECLS G_DECLARE_FINAL_TYPE (PhoshAudioDevices, phosh_audio_devices, PHOSH, AUDIO_DEVICES, GObject) -PhoshAudioDevices *phosh_audio_devices_new (GvcMixerControl *mixer, - gboolean is_input); +PhoshAudioDevices *phosh_audio_devices_new (gpointer mixer_control, + gboolean is_input); G_END_DECLS diff --git a/tools/check-doc b/tools/check-doc deleted file mode 100755 index 33e02898c4cc96975e7ba0e11546e4e1625b590c..0000000000000000000000000000000000000000 --- a/tools/check-doc +++ /dev/null @@ -1,15 +0,0 @@ -#!/bin/bash -# -# Build docs and choke on gi-docgen warnings - -DIR="${1:-_build}" -LOG="${DIR}/docs/doc-build.log" - -meson compile -C "${DIR}" docs/phosh-doc |& tee "${LOG}" - -if grep -vE '('\ -'src/dbus/phosh-[a-z\-]+-dbus.h:[0-9]+: Warning: Phosh: Unknown namespace for identifier .pending.'\ -'|src/settings/audio-devices.h:[0-9]+: Warning: Phosh: phosh_audio_devices_new: argument mixer: Unresolved type: .GvcMixerControl'\ -')' "${LOG}" | grep -i 'warning:'; then - exit 1 -fi