<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 11 July 2016 at 14:09, 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 id=":3o" class="a3s aXjCH m155d9dd0f36953de">You shouldn't be passing the return of g_regex_new() directly to<br>
mm_port_serial_at_add_unsolicited_msg_handler(), as that method takes<br>
its own reference of the GRegex, and therefore the regex would be<br>
leaking afterwards. You should g_regex_new() and store it in a local<br>
variable, pass that variable to<br>
mm_port_serial_at_add_unsolicited_msg_handler() and then<br>
g_regex_unref() the variable.</div></blockquote></div><br><br><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Oh, sorry, I missed that point in the first review​</div></div></div>