On Mon, May 22, 2017 at 7:15 PM, Aleksander Morgado
<aleksan...@aleksander.es> wrote:
>>> +static void
>>> +set_unsolicited_events_handlers (MMBroadbandModemCinterion *self,
>>> +                                 gboolean                   enable)
>>> +{
>>> +    MMPortSerialAt *ports[2];
>>> +    guint           i;
>>> +
>>> +    ports[0] = mm_base_modem_peek_port_primary   (MM_BASE_MODEM (self));
>>> +    ports[1] = mm_base_modem_peek_port_secondary (MM_BASE_MODEM (self));
>>> +
>>> +    /* Enable unsolicited events in given port */
>>> +    for (i = 0; i < 2; i++) {
>>
>> G_N_ELEMENTS()?
>>
>
> Ah, yeah, we could do that. This was copy pasted from some other
> plugin, so if you don't mind I'll push a separate patch fixing the
> same thing in several places.

Pushed already a patch to git master changing this in all plugins that
were not using G_N_ELEMENTS.

-- 
Aleksander
https://aleksander.es
_______________________________________________
ModemManager-devel mailing list
ModemManager-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/modemmanager-devel

Reply via email to