From: Johan Hedberg <johan.hedberg@gmail.com>
To: Syam Sidhardhan <s.syam@samsung.com>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH BlueZ 00/12] Redundant D-Bus error init, remove unwanted code, g_free missmatch etc
Date: Thu, 24 May 2012 11:10:40 +0300 [thread overview]
Message-ID: <20120524081040.GA18388@x220> (raw)
In-Reply-To: <1337788652-23697-1-git-send-email-s.syam@samsung.com>
Hi Syam,
On Wed, May 23, 2012, Syam Sidhardhan wrote:
> This series of patches resolves:
> Memory leak, free - g_free mismatch, redundant D-Bus error initilization
> and remove unwanted code.
>
> Syam Sidhardhan (12):
> monitor: Fix memory leak
> storage: Use g_free() instead of free()
> media: Remove redundant D-Bus error initilization
> telephony-maemo5: Remove redundant D-Bus error initilizations
> telephony-maemo6: Remove redundant D-Bus error initilization
> telephony-ofono: Remove redundant D-Bus error initilization
> agent: Remove redundant D-Bus error initilizations
> Remove redundant D-Bus error initilization in maemo6 plugin
> avctp: Fix NULL check after dereference
> adapter: Remove unwanted if check and code
> audio: Remove unwanted code from manager
> input: Remove unwanted codes from device
>
> audio/avctp.c | 3 ++-
> audio/manager.c | 2 --
> audio/media.c | 1 -
> audio/telephony-maemo5.c | 5 -----
> audio/telephony-maemo6.c | 1 -
> audio/telephony-ofono.c | 1 -
> input/device.c | 5 -----
> monitor/hcidump.c | 3 ++-
> plugins/maemo6.c | 1 -
> src/adapter.c | 3 ---
> src/agent.c | 5 -----
> src/storage.c | 6 +++---
> 12 files changed, 7 insertions(+), 29 deletions(-)
I've applied all patches except the telephony ones. Those do not buy us
very much but come with the expense of conflicting with the pending
telephony driver removal patches (destined for 5.0). I also applied one
extra patch from myself to do the agent variable removal suggested by
Vinicius.
Johan
prev parent reply other threads:[~2012-05-24 8:10 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-23 15:57 [PATCH BlueZ 00/12] Redundant D-Bus error init, remove unwanted code, g_free missmatch etc Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 01/12] monitor: Fix memory leak Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 02/12] storage: Use g_free() instead of free() Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 03/12] media: Remove redundant D-Bus error initilization Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 04/12] telephony-maemo5: Remove redundant D-Bus error initilizations Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 05/12] telephony-maemo6: Remove redundant D-Bus error initilization Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 06/12] telephony-ofono: " Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 07/12] agent: Remove redundant D-Bus error initilizations Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 08/12] Remove redundant D-Bus error initilization in maemo6 plugin Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 09/12] avctp: Fix NULL check after dereference Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 10/12] adapter: Remove unwanted if check and code Syam Sidhardhan
2012-05-23 18:02 ` Vinicius Costa Gomes
2012-05-23 15:57 ` [PATCH BlueZ 11/12] audio: Remove unwanted code from manager Syam Sidhardhan
2012-05-23 15:57 ` [PATCH BlueZ 12/12] input: Remove unwanted codes from device Syam Sidhardhan
2012-05-24 8:10 ` Johan Hedberg [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120524081040.GA18388@x220 \
--to=johan.hedberg@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=s.syam@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).