[PATCH] libmm-glib,sms: fix get|dup_data() introspection annotations

Colin Helliwell colin.helliwell at ln-systems.com
Wed Sep 27 15:10:16 UTC 2017


> On 26 September 2017 at 14:23 Aleksander Morgado <aleksander at aleksander.es> wrote:
> 
> We need to specify explicitly that the return type is an array of
> 
> guint8 elements.
> 
> ---
> 
> Hey Colin,
> 
> Could you apply this patch and see if the generated introspection support allows you to use the API? It should work as g_bytes_get_data(), i.e.:
> https://lazka.github.io/pgi-docs/#GLib-2.0/classes/Bytes.html#GLib.Bytes.get_data
> 
> ---
>  libmm-glib/mm-sms.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libmm-glib/mm-sms.c b/libmm-glib/mm-sms.c
> index a49deb88..0467b6c1 100644
> --- a/libmm-glib/mm-sms.c
> +++ b/libmm-glib/mm-sms.c
> @@ -131,7 +131,7 @@ mm_sms_dup_text (MMSms *self)
>  *
> 
> *   Gets the message data.
> *
> 
>     *   *   Returns: (transfer none): The message data, or %NULL if it doesn't contain any (e.g. contains text instead).
> 
>     *   *   Returns: (transfer none) (array length=data_len) (element-type guint8): The message data, or %NULL if it doesn't contain any (e.g. contains text instead).
> */
> const guint8 *
> mm_sms_get_data (MMSms *self,
> @@ -158,7 +158,7 @@ mm_sms_get_data (MMSms *self,
> *
> 
> *   Gets the message data.
> *
> 
>     *   *   Returns: (transfer full): The message data, or %NULL if it doesn't contain any (e.g. contains text instead). The returned value should be freed with g_free().
> 
>     *   *   Returns: (transfer full) (array length=data_len) (element-type guint8): The message data, or %NULL if it doesn't contain any (e.g. contains text instead). The returned value should be freed with g_free().
> */
> guint8 *
> mm_sms_dup_data (MMSms *self,
> --
> 2.14.1


LGTM - can it be pushed to Master?


More information about the ModemManager-devel mailing list