<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 7, 2013 at 2:55 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@lanedo.com" target="_blank">aleksander@lanedo.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 02/10/13 23:14, Thieu Le wrote:<br>
> Added libmm-glib methods, updated mmcli and simple status and added gtk-doc documentation.<br>
><br>
<br>
</div>Just saw that I re-reviewed a previous version of this patch :) Still,<br>
the comments I said apply to this one as well, so please consider them.<br></blockquote><div> </div><div>I have addressed the following code review comments.  Did I miss something on your list or was there another email with more comments?</div>
<div><br></div><div><br></div><div>On Sat, Sep 28, 2013 at 7:49 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@lanedo.com" target="_blank">aleksander@lanedo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><span style="color:rgb(34,34,34)">Please also update:</span><br></div><br> * libmm-glib with the methods needed to access the new property.<br> * mmcli to show the property value when showing the modem status info<br>
(e.g. mmcli -m 0).<br> * the gtk-doc documentation with the new enum and methods.<br></blockquote></div><div> </div></div></div></div>