From: Fabien Chevalier <fchevalier@silicom.fr>
To: Brad Midgley <bmidgley@xmission.com>,
Johan Hedberg <johan.hedberg@nokia.com>,
Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
thiagossantos@gmail.com, Marcel Holtmann <marcel@holtmann.org>
Cc: BlueZ development <bluez-devel@lists.sourceforge.net>
Subject: [PATCH] newapi patch v1
Date: Fri, 19 Oct 2007 19:26:56 +0200 [thread overview]
Message-ID: <4718E8E0.5050608@silicom.fr> (raw)
[-- Attachment #1: Type: text/plain, Size: 1380 bytes --]
All, here is the first version of the patch that moves to the newapi
usage... yeheee !!
As nobody suggested a new name, i've just chose one, which is
"btaudioservice.h" :-). Of course this can be changed if somebody
prefers a better name :-)
This patch does not bring any new feature by itself, it's goal is only
to get rid from the ipc.h stuff. However is is already quite big, but i
really can't make it smaller without breaking stuff that is known to
work today :-(
I did some basic testing of the patch, which is:
- play a file using a2dp, seek during playing (audacious)
- play a file using a2dp, seek during playing (rhythmnbox)
- play a file using SCO, seek during playing (audacious)
- record and play file using aplay & arecord.
I didn't notice any regression, but i don't want to fool anybody : such
a big patch will inevitably bring its set of regressions, it's just that
i haven't found them yet :-)
Due to the fairly big nature of the patch, i would suggest to wait for
3.21 before moving to the new api, this will give us the usual time
between two bluez releases to :
- fix regressions introduced (my job :-) )
- upgrading the gstreamer plugin to use the new api ( i'd prefer
thiagoss to work on this one (with my support of course), as i've never
put my nose on this part of the code )
As usual, comments are welcome. ;-)
Cheers,
Fabien
[-- Attachment #2: fchevalier.vcf --]
[-- Type: text/x-vcard, Size: 253 bytes --]
begin:vcard
fn:Fabien CHEVALIER
n:CHEVALIER;Fabien
org:SILICOM
adr:;;4 rue de Jouanet; RENNES ATALANTE;;35700;FRANCE
email;internet:fchevalier@silicom.fr
title:Software & Studies Engineer
tel;work:+33 (0) 2 99 84 17 17
version:2.1
end:vcard
next reply other threads:[~2007-10-19 17:26 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-19 17:26 Fabien Chevalier [this message]
2007-10-19 17:30 ` [Bluez-devel] [PATCH] newapi patch v1 Fabien Chevalier
2007-10-19 17:30 ` Fabien Chevalier
2007-10-19 23:35 ` Brad Midgley
2007-10-23 18:01 ` Fabien Chevalier
2007-10-23 18:28 ` Marcel Holtmann
2007-10-24 12:12 ` Fabien Chevalier
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=4718E8E0.5050608@silicom.fr \
--to=fchevalier@silicom.fr \
--cc=bluez-devel@lists.sourceforge.net \
--cc=bmidgley@xmission.com \
--cc=johan.hedberg@nokia.com \
--cc=luiz.dentz@gmail.com \
--cc=marcel@holtmann.org \
--cc=thiagossantos@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