Re: [PATCH 1/3] Fix dbus reply memory leak

Hi Syam,

On Sun, Dec 04, 2011, Syam Sidhardhan wrote:

quoted

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