<div dir="ltr">Thanks, I have another set of patches to fix another typo, from MBIM_CID_DEVICE_SERVICE_SUBSCRIBER_LIST to MBIM_CID_DEVICE_SERVICE_SUBSCRIBE_LIST.<div><br></div><div>But we need to align libmbim and ModemManager. How would you prefer to do that? Do you prefer a version bump in libmbim?<div>

<br></div><div>Ben</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 18, 2014 at 11:39 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 18/02/14 20:27, Ben Chan wrote:<br>
> This patch renames 'MBIM_REGISTRATION_FLAG_MANUAL_PACKET_SERVICE_AUTOMATIC_ATTACH' in<br>
> MbimRegistrationFlag to 'MBIM_REGISTRATION_FLAG_PACKET_SERVICE_AUTOMATIC_ATTACH' to be<br>
> consistent with the MBIM specification.<br>
> ---<br>
> Hi Aleksander,<br>
><br>
> I think this rename shouldn't be too disruptive as neither libmbim nor<br>
> ModemManager currently uses<br>
> 'MBIM_REGISTRATION_FLAG_MANUAL_PACKET_SERVICE_AUTOMATIC_ATTACH' in code.<br>
><br>
<br>
</div>Yeah, let's just fix it, even if it's an API break. I'll add a warning<br>
in the next release.<br>
<br>
BTW, I fixed the commit message to use "libmbim" instead of "libmm".<br>
<br>
Pushed, thanks.<br>
<div><div class="h5"><br>
> Thanks,<br>
> Ben<br>
><br>
>  src/libmbim-glib/mbim-enums.h | 8 ++++----<br>
>  1 file changed, 4 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/src/libmbim-glib/mbim-enums.h b/src/libmbim-glib/mbim-enums.h<br>
> index 6275537..38e12f0 100644<br>
> --- a/src/libmbim-glib/mbim-enums.h<br>
> +++ b/src/libmbim-glib/mbim-enums.h<br>
> @@ -449,14 +449,14 @@ typedef enum {<br>
>   * MbimRegistrationFlag:<br>
>   * @MBIM_REGISTRATION_FLAG_NONE: None.<br>
>   * @MBIM_REGISTRATION_FLAG_MANUAL_SELECTION_NOT_AVAILABLE: Network doesn't support manual network selection.<br>
> - * @MBIM_REGISTRATION_FLAG_MANUAL_PACKET_SERVICE_AUTOMATIC_ATTACH: Modem should auto-attach to the network after registration.<br>
> + * @MBIM_REGISTRATION_FLAG_PACKET_SERVICE_AUTOMATIC_ATTACH: Modem should auto-attach to the network after registration.<br>
>   *<br>
>   * Registration flags.<br>
>   */<br>
>  typedef enum {<br>
> -    MBIM_REGISTRATION_FLAG_NONE                                   = 0,<br>
> -    MBIM_REGISTRATION_FLAG_MANUAL_SELECTION_NOT_AVAILABLE         = 1 << 0,<br>
> -    MBIM_REGISTRATION_FLAG_MANUAL_PACKET_SERVICE_AUTOMATIC_ATTACH = 1 << 2,<br>
> +    MBIM_REGISTRATION_FLAG_NONE                            = 0,<br>
> +    MBIM_REGISTRATION_FLAG_MANUAL_SELECTION_NOT_AVAILABLE  = 1 << 0,<br>
> +    MBIM_REGISTRATION_FLAG_PACKET_SERVICE_AUTOMATIC_ATTACH = 1 << 2,<br>
>  } MbimRegistrationFlag;<br>
><br>
>  /*****************************************************************************/<br>
><br>
<br>
<br>
--<br>
</div></div>Aleksander<br>
<a href="https://aleksander.es" target="_blank">https://aleksander.es</a><br>
</blockquote></div><br></div>