Hi Aki, On 2011-02-16 12:16, Aki Niemi wrote:
Curly brackets are either in both if and else, or in neither. I actually prefer not to have them here, since the first line is just a comment in the else statement. ;)
Yes you're right :)
This is not necessary, as a modem plugin is supposed to call g_isi_modem_set_device() to set the modem->device to whatever it needs to right after creating the GIsiModem instance. In other words, the assumption is that when the GIsiModem instace is actually used, all necessary internal data, such as flags and the remote device have already been set.
Tested and works fine, will submit a new patch for the u8500 plugin. This is not done in the n900 plugin though, perhaps not necessary ?
Regards Andreas _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono