[vorbis-dev] vorbis-tools CVS

Kenneth Arnold ken at arnoldnet.net
Wed Jun 26 13:54:45 PDT 2002


Coming out of a long (but definately needed) silence for a moment:

A few things for vorbis-tools, since I no longer have CVS access (I
think I just misplaced that key):

1. Before anything, put oggchain out of its misery. Please. It's a
pile of junk that should have never been committed. Any smidgen of
usefulness from it belongs in ogginfo anyway (and it's already there).

2. ./autogen.sh should check for the presence of gettext. Without it,
aclocal gives just a warning and ./configure fails with a cryptic
error message.

3. Quick patch to ./autogen.sh:

--- autogen.sh.~1.3.~   Thu Aug 23 13:39:55 2001
+++ autogen.sh  Wed Jun 26 16:01:23 2002
@@ -21,7 +21,7 @@
 (automake --version) < /dev/null > /dev/null 2>&1 || {
         echo
         echo "You must have automake installed to compile $package."
-       echo "Download the appropriate package for your system,
+       echo "Download the appropriate package for your system,"
        echo "or get the source from one of the GNU ftp sites"
        echo "listed in http://www.gnu.org/order/ftp.html"
         DIE=1
@@ -30,7 +30,7 @@
 (libtool --version) < /dev/null > /dev/null 2>&1 || {
        echo
        echo "You must have libtool installed to compile $package."
-       echo "Download the appropriate package for your system,
+       echo "Download the appropriate package for your system,"
        echo "or get the source from one of the GNU ftp sites"
        echo "listed in http://www.gnu.org/order/ftp.html"
        DIE=1

4. CFLAGS are huge. Could we possibly use config.h (currently empty)?

5. ogginfo2 notes / testing:
* doesn't like standard input
* doesn't give any output on tiny (1-byte) stream, not even error
* doesn't tell when headers are incomplete, and gives junk info for
the stream length and nan for the bitrate
* doesn't say anything if I put 1k of random data at the end of a
truncated stream, only 10k (then it says "Hole in data found.")
* output wishlist:
 - bitrate name is inconsistant: kb/s or kpbs
 - some section headers in the output have ':', others '...'
 - some sections of data are indented, some not
 - printf-style formatted output
 - line up the default output

My suggested output example, including a few other minor changes I
didn't note above:

--cut--
Found logical stream #1, serial '75b6cb6c', type 'vorbis'
Header information for stream #1:
        Version:  0
        Vendor:   Xiphophorus libVorbis I 20011231
        Channels: 2
        Rate:     44100 samples per second

        Nominal bitrate: 120.015 kb/s
        Upper bitrate:   not set
        Lower bitrate:   not set
User comments:
        ARTIST=<artist>
        ALBUM=<album>
        TITLE=<title>
        TRACKNUMBER=<tracknumber>
Data stream summary:
        Total data length: 3800912 bytes
        Playback length:   4m 26s
        Average bitrate:   114.238702 kb/s

Logical stream 1 ended
--cut--

Comments welcome. I considered indenting all the info for one logical
stream, but it seems like too much indentation. That might work better
if the indentation was reduced to 4 or 2 spaces, though. If people
like this, I can work on a patch.

Other than that, ogginfo2 seems to work fine on everything I've thrown
at it, which would be probably rc1 through rc3 streams.

Overall, lots of great work since I last checked up, and I'm
definately looking forward to the VP3 integration project. If I'm
lucky I might even have some time to help out!


-- 
Kenneth Arnold <ken at arnoldnet.net>
- "Know thyself."


-------------- next part --------------
A non-text attachment was scrubbed...
Name: part
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
Url : http://lists.xiph.org/pipermail/vorbis-dev/attachments/20020626/f3653853/part.pgp


More information about the Vorbis-dev mailing list