From: Jean-Marc Valin <jean-marc.valin@usherbrooke.ca>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org,
James Courtier-Dutton <James@superbug.co.uk>
Subject: Re: [Alsa-devel] Quality resampling code for libasound
Date: Thu, 22 Mar 2007 00:39:44 +1100 [thread overview]
Message-ID: <460135A0.20903@usherbrooke.ca> (raw)
In-Reply-To: <s5hmz26buws.wl%tiwai@suse.de>
> Yeah, this is an interesting question. We often see similar conflicts
> in the maintenance of linux kernel tree, too. The unified coding
> style gives clearly better readability of the whole tree. OTOH, if a
> code chunk is derived from another tree, it requires more additional
> backport effort.
I think the main question is: do you plan on making many alsa-specific
changes to that code? resampler.c is pretty much as close as you get to
pure signal processing code (no "system" stuff), so I don't think many
people will actually want to mess with it and any improvements might as
well be merged in Speex first to be tested for a while. Of course, if
the coding rules are simple, an other approach could be to automatically
re-indent the code when syncing. Of course, it still means you need to
keep the same code as Speex has. In any case, I'd recommend not forking
that code too early because some of the planned improvements (e.g. SSE
optimisations) could be a pain to backport.
> In the ideal situation, we may link to speex resampler as a dynamic
> library -- e.g. adding a configure option to let user choose the
> built-in or system-wide library resampler code. Does libspeex provide
> this code snip as exported?
Not sure I understand what you mean here. If you're asking whether the
symbols for the resampler are exported in the shared library, then the
answer is yes -- though the resampler is only in the svn version of
Speex for now. The first release to include it will be 1.2beta2, which
I'm planning to release in a few weeks.
Jean-Marc
next prev parent reply other threads:[~2007-03-21 13:39 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-19 4:01 Quality resampling code for libasound Jean-Marc Valin
2007-02-19 12:23 ` Takashi Iwai
2007-02-19 12:54 ` Jean-Marc Valin
2007-02-19 13:10 ` Takashi Iwai
2007-02-19 13:38 ` Jean-Marc Valin
2007-02-19 13:47 ` Takashi Iwai
2007-02-20 5:53 ` Jean-Marc Valin
2007-02-20 6:48 ` Jean-Marc Valin
2007-02-20 10:45 ` Takashi Iwai
2007-02-20 11:54 ` Jean-Marc Valin
2007-02-20 12:13 ` Takashi Iwai
2007-02-20 12:38 ` Jean-Marc Valin
2007-02-20 21:50 ` Fernando Lopez-Lezcano
2007-02-20 22:07 ` James Courtier-Dutton
2007-02-20 22:45 ` Jean-Marc Valin
2007-02-21 2:10 ` Jean-Marc Valin
2007-02-21 9:18 ` Jean-Marc Valin
2007-02-22 11:38 ` Takashi Iwai
2007-02-22 11:48 ` Jean-Marc Valin
2007-02-22 12:07 ` Takashi Iwai
2007-02-28 2:10 ` Jean-Marc Valin
2007-03-07 22:25 ` Takashi Iwai
2007-03-08 0:59 ` Jean-Marc Valin
2007-03-09 16:08 ` Takashi Iwai
2007-03-20 13:22 ` Jean-Marc Valin
2007-03-20 13:52 ` Takashi Iwai
2007-03-20 14:07 ` Jean-Marc Valin
2007-03-20 14:19 ` Takashi Iwai
[not found] ` <4600539A.2070804@usherbrooke.ca>
2007-03-21 11:23 ` [Alsa-devel] " Takashi Iwai
2007-03-21 13:08 ` Jean-Marc Valin
2007-03-21 13:14 ` Takashi Iwai
2007-03-21 13:39 ` Jean-Marc Valin [this message]
2007-03-21 13:59 ` Takashi Iwai
2007-03-21 14:19 ` Jean-Marc Valin
2007-03-21 14:53 ` Takashi Iwai
2007-03-21 15:17 ` Jean-Marc Valin
2007-03-21 15:43 ` Takashi Iwai
2007-03-21 15:27 ` Rene Herman
2007-03-21 15:34 ` Jean-Marc Valin
2007-03-21 16:38 ` Rene Herman
2007-03-21 16:57 ` Takashi Iwai
2007-03-21 17:24 ` Takashi Iwai
2007-02-21 14:48 ` Jean-Marc Valin
2007-02-19 13:11 ` Prakash Punnoor
2007-02-19 13:36 ` Adam Tlałka
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=460135A0.20903@usherbrooke.ca \
--to=jean-marc.valin@usherbrooke.ca \
--cc=James@superbug.co.uk \
--cc=alsa-devel@alsa-project.org \
--cc=tiwai@suse.de \
/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