Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12190 - Add the "warble" codec test program

Attached to Project: Rockbox
Opened by Sean Bartell (wtachi) - Tuesday, 12 July 2011, 08:13 GMT+2
Last edited by Nils Wallménius (nls) - Saturday, 03 March 2012, 16:55 GMT+2
Task Type Patches
Category Codecs
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This is my codec test program introduced at <http://www.rockbox.org/mail/archive/rockbox-dev-archive-2011-06/0067.shtml>. It can be used to play files with SDL or convert them to WAV. There's more information in the program's help and at the above link.

This patch depends on FS#12189. This version is slightly modified from the mailing list version. In particular, most of the changes to reduce dependencies have been postponed.

The test program is named "warble", while the library will be named "librbcodec". The location of the library needs to be discussed; for now, I put it and the test program in /rbcodec.
   0001-Add-the-warble-test-program.patch (29.9 KiB) 
 b/apps/metadata/metadata_common.c |   11 
 b/rbcodec/Makefile                |   70 +++
 b/rbcodec/autoconf.h              |   17 
 b/rbcodec/warble.c                |  826 ++++++++++++++++++++++++++++++++++++++
 4 files changed, 921 insertions(+), 3 deletions(-)

This task depends upon

Closed by  Nils Wallménius (nls)
Saturday, 03 March 2012, 16:55 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  comitted the updated patch in http://gerrit.rockbox.org/r/#/c/135/
Comment by Dominik Riebeling (bluebrother) - Tuesday, 12 July 2011, 08:31 GMT+2
Why not put the test program in a folder below utils/? We already have a couple of utilitites in that location that aren't necessary for building Rockbox so this sounds like a reasonable place to me.

(oh, and if you avoid putting <> around URLs Flyspray will link them automatically)
Comment by Sean Bartell (wtachi) - Tuesday, 12 July 2011, 21:25 GMT+2
I figured I should put all the librbcodec stuff in one directory, since theoretically it's independent from the rest of Rockbox.
Comment by Sean Bartell (wtachi) - Wednesday, 27 July 2011, 23:16 GMT+2
Updated to use /lib/rbcodec.
   0001-Add-the-warble-test-program-v2.patch (30 KiB) 
 b/apps/metadata/metadata_common.c |   11 
 b/lib/rbcodec/test/Makefile       |   70 +++
 b/lib/rbcodec/test/autoconf.h     |   17 
 b/lib/rbcodec/test/warble.c       |  826 ++++++++++++++++++++++++++++++++++++++
 4 files changed, 921 insertions(+), 3 deletions(-)

Comment by Sean Bartell (wtachi) - Wednesday, 17 August 2011, 18:56 GMT+2
Updated CFLAGS to make libgme build.
   0002-Add-the-warble-test-program-v3.patch (30.2 KiB) 
 b/apps/metadata/metadata_common.c |   11 
 b/lib/rbcodec/test/Makefile       |   71 +++
 b/lib/rbcodec/test/autoconf.h     |   17 
 b/lib/rbcodec/test/warble.c       |  827 ++++++++++++++++++++++++++++++++++++++
 4 files changed, 923 insertions(+), 3 deletions(-)

Loading...