Hi Johan,
----- Original Message -----
From: "Johan Hedberg" <johan.hedberg@xxxxxxxxx>
To: "Syam Sidhardhan" <syamsidhardh@xxxxxxxxx>
Cc: "Syam Sidhardhan" <s.syam@xxxxxxxxxxx>;
<linux-bluetooth@xxxxxxxxxxxxxxx>
Sent: Monday, December 05, 2011 2:25 AM
Subject: Re: [PATCH 1/3] Fix dbus reply memory leak
Hi Syam,
On Sun, Dec 04, 2011, Syam Sidhardhan wrote:
>Your commit message uses the word leak in singular form but there are
>multiple fixes in this patch, i.e. the commit message is misleading. In
>this case I'd split the patch into four separate ones:
>
>telephony-maemo5: Fix D-Bus reply memory leaks
>cups: Fix D-Bus reply memory leaks
>agent: Fix D-Bus reply memory leaks
>mpris-player: Fix D-Bus reply memory leaks
>
>Johan
Yes, you are correct. You can split it into multiple patches.
Thanks in advance.
Maybe I was a bit unclear: I'm expecting *you* to do this split and
resend the patches. Thanks :)
Johan
Ok :-),
I'll split it and send.
Regadrs,
Syam
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
