On 18 October 2017 at 16:53, walter harms <[email protected]> wrote: > > > save one indentlevel in IceProtocolSetup by early check and > remove a lost free() check > Signed-off-by: Walter Harms <[email protected]> > Thanks for the poke Walter. Patches 1 + 2 were applied back in Sept and having similar patches got me confused a bit ;-)
> @@ -235,53 +236,49 @@ IceProtocolSetup ( > free (reply.protocol_error.error_message); > } > > - if (iceConn->protosetup_to_you->my_auth_indices) > - free (iceConn->protosetup_to_you->my_auth_indices); > + Unnecessary newline. > + free (iceConn->protosetup_to_you->my_auth_indices); For posterity - can you please split this out? Thanks Emil _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
