Subject: Re: [Bluez-devel] Re: a2play code that is working
From: Marcel Holtmann <marcel@holtmann.org>
To: bluez-devel@lists.sourceforge.net
In-Reply-To: <426E70F6.4060700@xmission.com>
References: <20050426122127.57962.qmail@web8302.mail.in.yahoo.com>
	 <1114527365.10706.288.camel@pegasus> <426E6857.6010204@xmission.com>
	 <426E70F6.4060700@xmission.com>
Content-Type: text/plain
Message-Id: <1114534644.10706.316.camel@pegasus>
Mime-Version: 1.0
Sender: bluez-devel-admin@lists.sourceforge.net
Errors-To: bluez-devel-admin@lists.sourceforge.net
Reply-To: bluez-devel@lists.sourceforge.net
Date: Tue, 26 Apr 2005 18:57:24 +0200

Hi Brad,

> How about if I move the declaration for struct sbc_frame into sbc.h and 
> change that last argument to sbc_encode to be an sbc_frame *
> 
> We wouldn't expose the data in the private struct, but just reuse the 
> struct for communicating sbc info back to the caller...

this is a bad idea, because then we can never change the internals of
the decoder/encoder. One of the goals should be to get a SBC codec that
uses only integer operation.

The sbc_struct already contains "rate" and "channels" and we should
simply add the rest of them. I was do lazy to do in the first place and
actually because I had no use for them.

Regards

Marcel


-------------------------------------------------------
SF.Net email is sponsored by: Tell us your software development plans!
Take this survey and enter to win a one-year sub to SourceForge.net
Plus IDC's 2005 look-ahead and a copy of this survey
Click here to start!  http://www.idcswdc.com/cgi-bin/survey?id=105hix
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel