On Sat, Apr 1, 2017 at 5:57 AM, Dan Williams <d...@redhat.com> wrote:
>> +MMFlowControl
>> +mm_broadband_modem_get_connected_flow_control (MMBroadbandModem
>> *self)
>> +{
>> +    return self->priv->modem_flow_control;
>> +}
>
> This part is kinda icky; and mm-broadband-bearer has no other case of
> calling a mm_broadband_modem...() function.  Could we perhaps pass it
> as a property or something into the bearer on creation?  It's not like
> it's going to change over the bearer lifetime.

Yes, that is a very good idea, I'll make it a MMBroadbandBearer
property given during object creation.


-- 
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