From: Mikel Astiz <mikel.astiz@bmw-carit.de>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Subject: Re: [RFC obexd 00/10] Major API changes in obex-client
Date: Fri, 13 Jan 2012 16:53:25 +0100 [thread overview]
Message-ID: <4F105375.6010107@bmw-carit.de> (raw)
In-Reply-To: <CABBYNZLtFnm9RTBfn2s9hdtwWHF=hXQgP+p+cMR_Uch+ErvuyA@mail.gmail.com>
Hi Luiz,
> I will be doing some cleanups before we integrate this code, some are
> based on your mega patch that you have sent (actually it never made
> to the list due to its size), but I would like to separate the
> transport code to modules e.g. bluetooth like we did for obexd so the
> session just call e.g. .connect/.disconnect and the driver takes care
> of the transport details such as requesting a session and discovering
> records, this should simplify quite a lot session.c
I agree it would be convenient to have a simplified session.c. Right now
there is definitely too much in there.
I am about to finish a split of the mega-patch you mention, trying to
separate the internal changes from the dbus-related changes, as we
discussed. You may want to wait until I send this new proposal.
> The only problem is how we gonna interpret the address, by default it
> should be Bluetooth but perhaps we should support urls for other
> transport e.g. usb://, how about that?
Sounds fine to me.
Cheers,
Mikel
prev parent reply other threads:[~2012-01-13 15:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-24 14:08 [RFC obexd 00/10] Major API changes in obex-client Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 01/10] client-doc: reformatted to fit in 80 columns Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 02/10] client-doc: copyright statement added Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 03/10] client-doc: minor formatting changes Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 04/10] client-doc: wrap OPP into specific session type Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 05/10] client-doc: replace parameter dict with conventional ones Mikel Astiz
2011-11-24 18:24 ` Vinicius Costa Gomes
2011-11-25 11:52 ` Luiz Augusto von Dentz
2011-11-25 13:41 ` Mikel Astiz
2011-11-24 14:08 ` [RFC obexd 06/10] client-doc: remove agent in favour of transfer signals Mikel Astiz
2011-11-25 12:01 ` Luiz Augusto von Dentz
2011-11-24 14:08 ` [RFC obexd 07/10] client-doc: ObjectPush sessions return transports Mikel Astiz
2011-11-24 14:09 ` [RFC obexd 08/10] client-doc: FileTransfer " Mikel Astiz
2011-11-24 14:09 ` [RFC obexd 09/10] client-doc: PhonebookAccess " Mikel Astiz
2011-11-24 14:09 ` [RFC obexd 10/10] client-doc: Synchronization " Mikel Astiz
2011-11-24 18:11 ` [RFC obexd 00/10] Major API changes in obex-client Vinicius Costa Gomes
2012-01-13 14:24 ` Luiz Augusto von Dentz
2012-01-13 15:53 ` Mikel Astiz [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=4F105375.6010107@bmw-carit.de \
--to=mikel.astiz@bmw-carit.de \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.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).