Vorbis dev - Oct 2000 - ogg123 Patch

> Ralph Giles pointed me here for my ogg123 patch. I had mail'd the
authors
> (listed in ogg123.1) and Xiph about this but I recieved no response - and
here
> is probably a better place for it.
If I went to me, I've been on vacation and out of touch :-)  Got back this 
weekend.

Ogg123 is not my code, so I kindly ask one of its maintainers to review this
patch.  If no one gets back to me in a reasonable amount of time, I'll
review
it and apply it (or delegate with confirmation).

Monty

--- >8 ----
List archives:  http://www.xiph.org/archives/
Ogg project homepage: http://www.xiph.org/ogg/
To unsubscribe from this list, send a message to
'vorbis-dev-request@xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is
needed.
Unsubscribe messages sent to the list will be ignored/filtered.

Hi all,

Ralph Giles pointed me here for my ogg123 patch. I had mail'd the authors
(listed in ogg123.1) and Xiph about this but I recieved no response - and here
is probably a better place for it.

I decided to work on ogg123 when I wanted to work on Ogg Vorbis support in
Nautilus (if you don't know what it is, it is the new Graphical File Manager
for GNOME). Currently Nautilus has mp3 support (through mpg123) - I wanted to
use ogg123 for Ogg Vorbis support but it lacked some features (which I have
implemented with this patch).

1) I made it respect the '--quiet' flag. This is so we can easily parse
the
time given in the '--verbose' option. This is simple and straightforward

2) I implemented a '--skip n' (or '-k n') - in mpg123 you can
skip 'frames'
using the '-k n' argument. This is the same thing, except you skip
seconds. If
you provide an 'n' that is <= 0 or >= file size it will
automatically pretend
that you set the 'n' to be the total file time (like mpg123 does). Also
- it
'consistently' skips the same amount for all the specified files, so
'ogg123
-k 5 ~/*.ogg' will skip the first 5 seconds of /all/ files - Again this is
for
mpg123 compatiblity purposes. The inspiration behind this patch is so that we
can enable seeking (hey, now people can write graphical front-ends to ogg123!)

I've tested it on my system and works fine :) 

Regards,
Ali


<HR NOSHADE>
<UL>
<LI>text/plain attachment: ogg_improve.diff
</UL>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ogg_improve.diff
Type: application/octet-stream
Size: 9076 bytes
Desc: not available
Url :
http://lists.xiph.org/pipermail/vorbis-dev/attachments/20001002/bbabbe8a/ogg_improve-0001.obj