Mercurial > SDL_sound_CoreAudio
annotate decoders/ogg.c @ 468:cd46c97e58bc
Fixed broken VOC decoder...would always dereference NULL during VOC_open().
author | Ryan C. Gordon <icculus@icculus.org> |
---|---|
date | Sat, 25 Oct 2003 12:38:57 +0000 |
parents | 30f96c853462 |
children | c66080364dff 50bb9a6cebfe |
rev | line source |
---|---|
24 | 1 /* |
2 * SDL_sound -- An abstract sound format decoding API. | |
3 * Copyright (C) 2001 Ryan C. Gordon. | |
4 * | |
5 * This library is free software; you can redistribute it and/or | |
6 * modify it under the terms of the GNU Lesser General Public | |
7 * License as published by the Free Software Foundation; either | |
8 * version 2.1 of the License, or (at your option) any later version. | |
9 * | |
10 * This library is distributed in the hope that it will be useful, | |
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
13 * Lesser General Public License for more details. | |
14 * | |
15 * You should have received a copy of the GNU Lesser General Public | |
16 * License along with this library; if not, write to the Free Software | |
17 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA | |
18 */ | |
19 | |
20 /* | |
21 * Ogg Vorbis decoder for SDL_sound. | |
22 * | |
23 * This driver handles .OGG audio files, and depends on libvorbisfile to | |
24 * do the actual decoding work. libvorbisfile is part of libvorbis, which | |
25 * is part of the Ogg Vorbis project. | |
26 * | |
27 * Ogg Vorbis: http://www.xiph.org/ogg/vorbis/ | |
28 * vorbisfile documentation: http://www.xiph.org/ogg/vorbis/doc/vorbisfile/ | |
29 * | |
184
47cc2de2ae36
Changed reference to "LICENSE" file to "COPYING".
Ryan C. Gordon <icculus@icculus.org>
parents:
149
diff
changeset
|
30 * Please see the file COPYING in the source's root directory. |
24 | 31 * |
32 * This file written by Ryan C. Gordon. (icculus@clutteredmind.org) | |
33 */ | |
34 | |
106
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
35 #if HAVE_CONFIG_H |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
36 # include <config.h> |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
37 #endif |
100
6d9fdec2f708
added config.h, added --enable-debug flag, various other changes to the build system
fingolfin
parents:
63
diff
changeset
|
38 |
104
103cfcb3c014
Updated to fix build system problem.
Ryan C. Gordon <icculus@icculus.org>
parents:
100
diff
changeset
|
39 #ifdef SOUND_SUPPORTS_OGG |
103cfcb3c014
Updated to fix build system problem.
Ryan C. Gordon <icculus@icculus.org>
parents:
100
diff
changeset
|
40 |
24 | 41 #include <stdio.h> |
42 #include <stdlib.h> | |
43 #include <string.h> | |
44 #include <math.h> | |
106
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
45 |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
46 #include "SDL_sound.h" |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
47 |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
48 #define __SDL_SOUND_INTERNAL__ |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
49 #include "SDL_sound_internal.h" |
40de367eb59e
Changing my include structure to do this right.
Ryan C. Gordon <icculus@icculus.org>
parents:
104
diff
changeset
|
50 |
443 | 51 #include <vorbis/codec.h> |
52 #include <vorbis/vorbisfile.h> | |
24 | 53 |
54 | |
47
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
55 static int OGG_init(void); |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
56 static void OGG_quit(void); |
24 | 57 static int OGG_open(Sound_Sample *sample, const char *ext); |
58 static void OGG_close(Sound_Sample *sample); | |
59 static Uint32 OGG_read(Sound_Sample *sample); | |
221
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
60 static int OGG_rewind(Sound_Sample *sample); |
306
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
61 static int OGG_seek(Sound_Sample *sample, Uint32 ms); |
24 | 62 |
149
1df5c106504e
Decoders can now list multiple file extensions.
Ryan C. Gordon <icculus@icculus.org>
parents:
122
diff
changeset
|
63 static const char *extensions_ogg[] = { "OGG", NULL }; |
24 | 64 const Sound_DecoderFunctions __Sound_DecoderFunctions_OGG = |
65 { | |
66 { | |
149
1df5c106504e
Decoders can now list multiple file extensions.
Ryan C. Gordon <icculus@icculus.org>
parents:
122
diff
changeset
|
67 extensions_ogg, |
24 | 68 "Ogg Vorbis audio through VorbisFile", |
69 "Ryan C. Gordon <icculus@clutteredmind.org>", | |
70 "http://www.icculus.org/SDL_sound/" | |
71 }, | |
72 | |
221
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
73 OGG_init, /* init() method */ |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
74 OGG_quit, /* quit() method */ |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
75 OGG_open, /* open() method */ |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
76 OGG_close, /* close() method */ |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
77 OGG_read, /* read() method */ |
306
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
78 OGG_rewind, /* rewind() method */ |
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
79 OGG_seek /* seek() method */ |
24 | 80 }; |
81 | |
82 | |
47
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
83 static int OGG_init(void) |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
84 { |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
85 return(1); /* always succeeds. */ |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
86 } /* OGG_init */ |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
87 |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
88 |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
89 static void OGG_quit(void) |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
90 { |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
91 /* it's a no-op. */ |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
92 } /* OGG_quit */ |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
93 |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
94 |
ea58bc3b15d7
Added init() and quit() methods.
Ryan C. Gordon <icculus@icculus.org>
parents:
24
diff
changeset
|
95 |
24 | 96 /* |
97 * These are callbacks from vorbisfile that let them read data from | |
98 * a RWops... | |
99 */ | |
100 | |
63
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
101 static size_t RWops_ogg_read(void *ptr, size_t size, size_t nmemb, void *datasource) |
24 | 102 { |
103 return((size_t) SDL_RWread((SDL_RWops *) datasource, ptr, size, nmemb)); | |
104 } /* RWops_ogg_read */ | |
105 | |
122 | 106 static int RWops_ogg_seek(void *datasource, ogg_int64_t offset, int whence) |
24 | 107 { |
108 return(SDL_RWseek((SDL_RWops *) datasource, offset, whence)); | |
109 } /* RWops_ogg_seek */ | |
110 | |
63
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
111 static int RWops_ogg_close(void *datasource) |
24 | 112 { |
113 /* do nothing; SDL_sound will delete the RWops at a higher level. */ | |
114 return(0); /* this is success in fclose(), so I guess that's okay. */ | |
115 } /* RWops_ogg_close */ | |
116 | |
63
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
117 static long RWops_ogg_tell(void *datasource) |
24 | 118 { |
119 return((long) SDL_RWtell((SDL_RWops *) datasource)); | |
120 } /* RWops_ogg_tell */ | |
121 | |
122 static const ov_callbacks RWops_ogg_callbacks = | |
123 { | |
124 RWops_ogg_read, | |
125 RWops_ogg_seek, | |
126 RWops_ogg_close, | |
127 RWops_ogg_tell | |
128 }; | |
129 | |
130 | |
131 /* Return a human readable version of an VorbisFile error code... */ | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
132 #if (defined DEBUG_CHATTER) |
24 | 133 static const char *ogg_error(int errnum) |
134 { | |
135 switch(errnum) | |
136 { | |
137 case OV_EREAD: | |
138 return("i/o error"); | |
139 case OV_ENOTVORBIS: | |
140 return("not a vorbis file"); | |
141 case OV_EVERSION: | |
142 return("Vorbis version mismatch"); | |
143 case OV_EBADHEADER: | |
144 return("invalid Vorbis bitstream header"); | |
145 case OV_EFAULT: | |
146 return("internal logic fault in Vorbis library"); | |
147 } /* switch */ | |
148 | |
149 return("unknown error"); | |
150 } /* ogg_error */ | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
151 #endif |
24 | 152 |
153 static __inline__ void output_ogg_comments(OggVorbis_File *vf) | |
154 { | |
155 #if (defined DEBUG_CHATTER) | |
156 int i; | |
157 vorbis_comment *vc = ov_comment(vf, -1); | |
158 | |
159 if (vc == NULL) | |
160 return; | |
161 | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
162 SNDDBG(("OGG: vendor == [%s].\n", vc->vendor)); |
24 | 163 for (i = 0; i < vc->comments; i++) |
164 { | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
165 SNDDBG(("OGG: user comment [%s].\n", vc->user_comments[i])); |
24 | 166 } /* for */ |
167 #endif | |
168 } /* output_ogg_comments */ | |
169 | |
170 | |
171 static int OGG_open(Sound_Sample *sample, const char *ext) | |
172 { | |
173 int rc; | |
174 OggVorbis_File *vf; | |
175 vorbis_info *info; | |
176 Sound_SampleInternal *internal = (Sound_SampleInternal *) sample->opaque; | |
177 | |
178 vf = (OggVorbis_File *) malloc(sizeof (OggVorbis_File)); | |
179 BAIL_IF_MACRO(vf == NULL, ERR_OUT_OF_MEMORY, 0); | |
180 | |
181 rc = ov_open_callbacks(internal->rw, vf, NULL, 0, RWops_ogg_callbacks); | |
182 if (rc != 0) | |
183 { | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
184 SNDDBG(("OGG: can't grok data. reason: [%s].\n", ogg_error(rc))); |
24 | 185 free(vf); |
186 BAIL_MACRO("OGG: Not valid Ogg Vorbis data.", 0); | |
187 } /* if */ | |
188 | |
189 info = ov_info(vf, -1); | |
190 if (info == NULL) | |
191 { | |
192 ov_clear(vf); | |
193 free(vf); | |
194 BAIL_MACRO("OGG: failed to retrieve bitstream info", 0); | |
195 } /* if */ | |
196 | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
197 SNDDBG(("OGG: Accepting data stream.\n")); |
24 | 198 |
199 output_ogg_comments(vf); | |
59
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
200 SNDDBG(("OGG: bitstream version == (%d).\n", info->version)); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
201 SNDDBG(("OGG: bitstream channels == (%d).\n", info->channels)); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
202 SNDDBG(("OGG: bitstream sampling rate == (%ld).\n", info->rate)); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
203 SNDDBG(("OGG: seekable == {%s}.\n", ov_seekable(vf) ? "TRUE" : "FALSE")); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
204 SNDDBG(("OGG: number of logical bitstreams == (%ld).\n", ov_streams(vf))); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
205 SNDDBG(("OGG: serial number == (%ld).\n", ov_serialnumber(vf, -1))); |
cd91e1857b42
Changed _D macro to SNDDBG, and fixed a warning if compiling without debug
Ryan C. Gordon <icculus@icculus.org>
parents:
47
diff
changeset
|
206 SNDDBG(("OGG: total seconds of sample == (%f).\n", ov_time_total(vf, -1))); |
24 | 207 |
208 internal->decoder_private = vf; | |
327
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
209 sample->flags = SOUND_SAMPLEFLAG_CANSEEK; |
24 | 210 sample->actual.rate = (Uint32) info->rate; |
211 sample->actual.channels = (Uint8) info->channels; | |
212 | |
213 /* | |
214 * Since we might have more than one logical bitstream in the OGG file, | |
215 * and these bitstreams may be in different formats, we might be | |
216 * converting two or three times: once in vorbisfile, once again in | |
217 * SDL_sound, and perhaps a third time to get it to the sound device's | |
218 * format. That's wickedly inefficient. | |
219 * | |
220 * To combat this a little, if the user specified a desired format, we | |
221 * claim that to be the "actual" format of the collection of logical | |
222 * bitstreams. This means that VorbisFile will do a conversion as | |
223 * necessary, and SDL_sound will not. If the user didn't specify a | |
224 * desired format, then we pretend the "actual" format is something that | |
225 * OGG files are apparently commonly encoded in. | |
226 */ | |
227 sample->actual.format = (sample->desired.format == 0) ? | |
228 AUDIO_S16LSB : sample->desired.format; | |
229 return(1); | |
230 } /* OGG_open */ | |
231 | |
232 | |
233 static void OGG_close(Sound_Sample *sample) | |
234 { | |
235 Sound_SampleInternal *internal = (Sound_SampleInternal *) sample->opaque; | |
236 OggVorbis_File *vf = (OggVorbis_File *) internal->decoder_private; | |
237 ov_clear(vf); | |
238 free(vf); | |
239 } /* OGG_close */ | |
240 | |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
241 /* Note: According to the Vorbis documentation: |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
242 * "ov_read() will decode at most one vorbis packet per invocation, |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
243 * so the value returned will generally be less than length." |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
244 * Due to this, for buffer sizes like 16384, SDL_Sound was always getting |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
245 * an underfilled buffer and always setting the EAGAIN flag. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
246 * Since the SDL_Sound API implies that the entire buffer |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
247 * should be filled unless EOF, additional code has been added |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
248 * to this function to call ov_read() until the buffer is filled. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
249 * However, there may still be some corner cases where the buffer |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
250 * cannot be entirely filled. So be aware. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
251 */ |
24 | 252 static Uint32 OGG_read(Sound_Sample *sample) |
253 { | |
254 int rc; | |
255 int bitstream; | |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
256 Sound_SampleInternal *internal = (Sound_SampleInternal *) sample->opaque; |
24 | 257 OggVorbis_File *vf = (OggVorbis_File *) internal->decoder_private; |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
258 |
24 | 259 rc = ov_read(vf, internal->buffer, internal->buffer_size, |
260 ((sample->actual.format & 0x1000) ? 1 : 0), /* bigendian? */ | |
261 ((sample->actual.format & 0xFF) / 8), /* bytes per sample point */ | |
262 ((sample->actual.format & 0x8000) ? 1 : 0), /* signed data? */ | |
263 &bitstream); | |
264 | |
265 /* Make sure the read went smoothly... */ | |
266 if (rc == 0) | |
267 sample->flags |= SOUND_SAMPLEFLAG_EOF; | |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
268 |
24 | 269 else if (rc < 0) |
270 sample->flags |= SOUND_SAMPLEFLAG_ERROR; | |
271 | |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
272 /* If the buffer isn't filled, keep trying to fill it |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
273 * until no more data can be grabbed */ |
24 | 274 else if ((Uint32) rc < internal->buffer_size) |
433
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
275 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
276 /* Creating a pointer to the buffer that denotes where to start |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
277 * writing new data. */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
278 Uint8* buffer_start_point = NULL; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
279 int total_bytes_read = rc; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
280 int bytes_remaining = internal->buffer_size - rc; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
281 |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
282 /* Keep grabbing data until something prevents |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
283 * us from getting more. (Could be EOF, |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
284 * packets are too large to fit in remaining |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
285 * space, or an error.) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
286 */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
287 while( (rc > 0) && (bytes_remaining > 0) ) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
288 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
289 /* Set buffer pointer to end of last write */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
290 /* All the messiness is to get rid of the warning for |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
291 * dereferencing a void* |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
292 */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
293 buffer_start_point = &(((Uint8*)internal->buffer)[total_bytes_read]); |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
294 rc = ov_read(vf, buffer_start_point, bytes_remaining, |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
295 ((sample->actual.format & 0x1000) ? 1 : 0), /* bigendian? */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
296 ((sample->actual.format & 0xFF) / 8), /* bytes per sample point */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
297 ((sample->actual.format & 0x8000) ? 1 : 0), /* signed data? */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
298 &bitstream); |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
299 /* Make sure rc > 0 because we don't accidently want |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
300 * to change the counters if there was an error |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
301 */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
302 if(rc > 0) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
303 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
304 total_bytes_read += rc; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
305 bytes_remaining = bytes_remaining - rc; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
306 } |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
307 } |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
308 /* I think the minimum read size is 2, though I'm |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
309 * not sure about this. (I've hit cases where I |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
310 * couldn't read less than 4.) What I don't want to do is |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
311 * accidently claim we hit EOF when the reason rc == 0 |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
312 * is because the requested amount of data was smaller |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
313 * than the minimum packet size. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
314 * For now, I will be conservative |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
315 * and not set the EOF flag, and let the next call to |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
316 * this function figure it out. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
317 * I think the ERROR flag is safe to set because |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
318 * it looks like OGG simply returns 0 if the |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
319 * read size is too small. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
320 * And in most cases for sensible buffer sizes, |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
321 * this fix will fill the buffer, |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
322 * so we can set the EAGAIN flag without worrying |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
323 * that it will always be set. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
324 */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
325 if(rc < 0) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
326 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
327 sample->flags |= SOUND_SAMPLEFLAG_ERROR; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
328 } |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
329 else if(rc == 0) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
330 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
331 /* Do nothing for now until there is a better solution */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
332 /* sample->flags |= SOUND_SAMPLEFLAG_EOF; */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
333 } |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
334 |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
335 /* Test for a buffer underrun. It should occur less frequently |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
336 * now, but it still may happen and not necessarily mean |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
337 * anything useful. */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
338 if ((Uint32) total_bytes_read < internal->buffer_size) |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
339 { |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
340 sample->flags |= SOUND_SAMPLEFLAG_EAGAIN; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
341 } |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
342 /* change rc to the total bytes read so function |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
343 * can return the correct value. |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
344 */ |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
345 rc = total_bytes_read; |
f1c16afd9ebe
Read until buffer is full in Sound_Decode() rather than one packet per call.
Ryan C. Gordon <icculus@icculus.org>
parents:
377
diff
changeset
|
346 } |
24 | 347 |
348 return((Uint32) rc); | |
349 } /* OGG_read */ | |
350 | |
221
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
351 |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
352 static int OGG_rewind(Sound_Sample *sample) |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
353 { |
231
d3dc34315ac7
Rewind method implemented by Torbj�rn.
Ryan C. Gordon <icculus@icculus.org>
parents:
221
diff
changeset
|
354 Sound_SampleInternal *internal = (Sound_SampleInternal *) sample->opaque; |
d3dc34315ac7
Rewind method implemented by Torbj�rn.
Ryan C. Gordon <icculus@icculus.org>
parents:
221
diff
changeset
|
355 OggVorbis_File *vf = (OggVorbis_File *) internal->decoder_private; |
d3dc34315ac7
Rewind method implemented by Torbj�rn.
Ryan C. Gordon <icculus@icculus.org>
parents:
221
diff
changeset
|
356 |
d3dc34315ac7
Rewind method implemented by Torbj�rn.
Ryan C. Gordon <icculus@icculus.org>
parents:
221
diff
changeset
|
357 BAIL_IF_MACRO(ov_raw_seek(vf, 0) < 0, ERR_IO_ERROR, 0); |
d3dc34315ac7
Rewind method implemented by Torbj�rn.
Ryan C. Gordon <icculus@icculus.org>
parents:
221
diff
changeset
|
358 return(1); |
221
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
359 } /* OGG_rewind */ |
c9772a9f5271
Initial implementation or stubs for rewind method. Other cleanups.
Ryan C. Gordon <icculus@icculus.org>
parents:
184
diff
changeset
|
360 |
306
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
361 |
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
362 static int OGG_seek(Sound_Sample *sample, Uint32 ms) |
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
363 { |
327
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
364 Sound_SampleInternal *internal = (Sound_SampleInternal *) sample->opaque; |
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
365 OggVorbis_File *vf = (OggVorbis_File *) internal->decoder_private; |
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
366 double timepos = (((double) ms) / 1000.0); |
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
367 BAIL_IF_MACRO(ov_time_seek(vf, timepos) < 0, ERR_IO_ERROR, 0); |
aa7f6a110971
Added seek implementation.
Ryan C. Gordon <icculus@icculus.org>
parents:
306
diff
changeset
|
368 return(1); |
306
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
369 } /* OGG_seek */ |
c97be6e1bd27
Added framework for Sound_Seek() support.
Ryan C. Gordon <icculus@icculus.org>
parents:
231
diff
changeset
|
370 |
63
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
371 #endif /* SOUND_SUPPORTS_OGG */ |
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
372 |
9669aa13d3e0
Changes in preparation for autoconf, and the RWops wrappers changed to
Ryan C. Gordon <icculus@icculus.org>
parents:
59
diff
changeset
|
373 |
24 | 374 /* end of ogg.c ... */ |
375 |