<div dir="ltr">Please see the attachment for the patch from git format-patch.<div><br></div><div>Thanks</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 22, 2014 at 8:52 AM, 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 class="">On 22/07/14 17:13, Yunlian Jiang wrote:<br>
> Hi Aleksander,<br>
>    Please see below for a updated patch, please let me know if there<br>
> is any problem.<br>
><br>
>    Thanks<br>
><br>
<br>
</div>Yeah, the problem is that it is not a directly applicable git patch.<br>
Usually there are 2 ways to send patches:<br>
<br>
 * git format-patch: which will create an external file for each commit<br>
you want to send, so that you can *attach* the patches to the email. So<br>
for example imagine that you have a local branch with 2 commits on top<br>
of the master branch, you could run "git format-match master" and that<br>
will give you 2 files to attach to the email.<br>
<br>
 * git send-email: with this method you can directly send the patches to<br>
e.g. this mailing list. This method requires a bit more setup as you'll<br>
need to configure your SMTP account and such, but is way more powerful<br>
in the long run, especially as each patch is itself an email, and we can<br>
review them separately by replying to each of the emails.<br>
<br>
If you're just starting with this, I'd suggest you first try git<br>
"format-patch" and attach the patches to the email.<br>
<div class=""><br>
<br>
> This fixes a memory leak in test-message-parser.<br>
><br>
> ---<br>
><br>
>  src/libmbim-glib/test/test-message-parser.c | 1 +<br>
><br>
>  1 file changed, 1 insertion(+)<br>
><br>
><br>
> diff --git a/src/libmbim-glib/test/test-message-parser.c<br>
> b/src/libmbim-glib/test/test-message-parser.c<br>
><br>
> index 6878f45..a71c8c5 100644<br>
><br>
> --- a/src/libmbim-glib/test/test-message-parser.c<br>
><br>
> +++ b/src/libmbim-glib/test/test-message-parser.c<br>
><br>
> @@ -653,6 +653,7 @@ test_message_parser_basic_connect_register_state (void)<br>
><br>
>                                                     MBIM_DATA_CLASS_HSUPA));<br>
><br>
>      g_assert_cmpuint (current_cellular_class, ==, MBIM_CELLULAR_CLASS_GSM);<br>
><br>
>      g_assert_cmpstr (provider_id, ==, "26006");<br>
><br>
> +    g_free (provider_id);<br>
><br>
>      g_assert (provider_name == NULL);<br>
><br>
>      g_assert (roaming_text == NULL);<br>
><br>
>      g_assert_cmpuint (registration_flag, ==,<br>
> MBIM_REGISTRATION_FLAG_PACKET_SERVICE_AUTOMATIC_ATTACH);<br>
><br>
<br>
<br>
--<br>
</div>Aleksander<br>
<a href="https://aleksander.es" target="_blank">https://aleksander.es</a><br>
</blockquote></div><br></div>