[libvirt] [dbus PATCH] connect: fix g_free order in virtDBusConnectFree

[Ján Tomko]

On Tue, Jul 03, 2018 at 02:18:02PM +0200, Pavel Hrdina wrote:
>On Mon, Jul 02, 2018 at 11:42:08AM -0400, Anya Harter wrote:
>> so that g_free(connect->nodeDevPath) line appears in alphabetical order
>> with the rest of the lines
>>
>> Signed-off-by: Anya Harter <aharter at redhat.com>
>> ---
>>  src/connect.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>In general we don't send/merge patches that just move code without any
>functional reason because it pollutes git history but in this case I
>would say it's ok.
>

What about the gchar *nodeDevPath; declaration in connect.h?
It is also in a different order.

/me runs

Jano

>Reviewed-by: Pavel Hrdina <phrdina at redhat.com> and pushed.



>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180703/3045ae81/attachment-0001.sig>