All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jukka Saunamaki <jukka.saunamaki@nokia.com>
To: ofono@ofono.org
Subject: Re: [PATCH 1/8] gprs-provision: add driver API header
Date: Wed, 19 Jan 2011 07:58:55 +0200	[thread overview]
Message-ID: <1295416735.17394.4.camel@jsaunama-desktop> (raw)
In-Reply-To: <1295362115.3873.190.camel@aeonflux>

[-- Attachment #1: Type: text/plain, Size: 634 bytes --]

Hi Marcel,

> > +/*
> > + * Callback from provisioning plugin.
> > + * settings: list of struct ofono_gprs_provision_data
> > + *
> > + * It is responsibility of callback function to free settings-list
> > + * settings-list elements must be freed with ofono_gprs_provision_data_free()
> > + */
> > +typedef void (*ofono_gprs_provision_cb_t)(GSList *settings, void *userdata);
> 
> so our general rule is no GLib types in oFono public APIs.

OK. Do you have any preferences for the best type here? Linked list of
'struct ofono_gprs_provision_data's, array of structs + count, array of
pointers (+count)?

--Jukka



  reply	other threads:[~2011-01-19  5:58 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-18  8:09 [gprs-provision PATCHv4 0/8] plugin API for provisioning of GPRS context settings Jukka Saunamaki
2011-01-18  8:09 ` [PATCH 1/8] gprs-provision: add driver API header Jukka Saunamaki
2011-01-18 14:48   ` Marcel Holtmann
2011-01-19  5:58     ` Jukka Saunamaki [this message]
2011-01-18  8:09 ` [PATCH 2/8] gprs-provision: add new atom type Jukka Saunamaki
2011-01-18  8:09 ` [PATCH 3/8] gprs-provision: add driver API sources Jukka Saunamaki
2011-01-18 14:55   ` Marcel Holtmann
2011-01-18  8:09 ` [PATCH 4/8] gprs-provision: probe gprs_provision drivers Jukka Saunamaki
2011-01-18  8:09 ` [PATCH 5/8] gprs: add gprs context provisioning Jukka Saunamaki
2011-01-18  8:09 ` [PATCH 6/8] sim: getters for mcc and mnc definition Jukka Saunamaki
2011-01-18  8:09 ` [PATCH 7/8] sim: getters for mcc and mnc implementation Jukka Saunamaki
2011-01-18 14:44   ` Marcel Holtmann
2011-01-18  8:09 ` [PATCH 8/8] gprs-provision: add example context provisioning driver Jukka Saunamaki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1295416735.17394.4.camel@jsaunama-desktop \
    --to=jukka.saunamaki@nokia.com \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.