aboutsummaryrefslogtreecommitdiffstats
path: root/OpenAL32/alSource.c
Commit message (Collapse)AuthorAgeFilesLines
* Move the current buffer queue entry and play position to the voiceChris Robinson2017-02-271-91/+100
| | | | | | | | | | | | | | This has a couple behavioral changes. First and biggest is that querying AL_BUFFERS_PROCESSED from a source will always return all buffers processed when in an AL_STOPPED state. Previously all buffers would be set as processed when first becoming stopped, but newly queued buffers would *not* be indicated as processed. That old behavior was not compliant with the spec, which unequivocally states "On a source in the AL_STOPPED state, all buffers are processed." Secondly, querying AL_BUFFER on an AL_STREAMING source will now always return 0. Previously it would return the current "active" buffer in the queue, but there's no basis for that in the spec.
* Ensure a non-playing or -paused source does not use a mixing voiceChris Robinson2017-02-251-43/+55
|
* Improve handling of source state readsChris Robinson2017-02-241-71/+88
| | | | | | | This avoids using seq_cst for loading the source state when either inside the mixer, or otherwise protected from inconsistencies with async updates. It also fixes potential race conditions with getting the source offset just as a source stops.
* Dynamically allocate the ALsource Send[] arrayChris Robinson2017-02-211-22/+33
|
* Make the voices' Send[] array dynamically sizedChris Robinson2017-02-211-8/+8
| | | | | The voices are still all allocated in one chunk to avoid memory fragmentation. But they're accessed as an array of pointers since the size isn't static.
* Reorganize ALvoice membersChris Robinson2017-02-151-3/+3
| | | | | This places the Send[] array at the end of the struct, making it easier to handle dynamically.
* Make ALsourceProps' Send array dynamically sizedChris Robinson2017-02-141-15/+6
| | | | | | ALsourceProps' Send[] array is placed at the end of the struct, and given an indeterminate size. Extra space is allocated at the end of each struct given the number of auxiliary sends set for the device.
* Fix build with non-C11 atomicsChris Robinson2017-02-131-2/+2
|
* Make the source state atomicChris Robinson2017-02-131-26/+20
| | | | | Since it's modified by the mixer when playback is ended, a plain struct member isn't safe.
* Remove a couple context lock wrapper functionsChris Robinson2017-02-071-17/+17
|
* Avoid duplicating code using a macroChris Robinson2016-12-211-6/+1
|
* Use separate macros for atomics that don't take a memory orderChris Robinson2016-12-201-29/+32
|
* Fix an infinite loopChris Robinson2016-11-231-0/+1
|
* Only send source updates for sources that have updatedChris Robinson2016-11-231-1/+10
|
* Clean up finding a source's voiceChris Robinson2016-11-221-26/+29
|
* Improve seqlock behaviorChris Robinson2016-11-211-24/+32
|
* Stop using almemory_order_consumeChris Robinson2016-11-171-1/+1
|
* Remove the upper limit from AL_MIN_GAIN and AL_MAX_GAINChris Robinson2016-08-291-2/+2
| | | | As per the current AL_SOFT_gain_clamp_ex proposal.
* Allow sources to play while alcSuspendContext is in effectChris Robinson2016-08-261-1/+1
| | | | | | | | | | | | | | This appears to be how Creative's Windows drivers handle it, and is necessary for at least the Windows version of UT2k4 (otherwise it tries to play a source while suspended, checks and sees it's stopped, then kills it before it's given a chance to start playing). Consequently, the internal properties it gets mixed with are determined by what the source properties are at the time of the play call, and the listener properties at the time of the suspend call. This does not change alDeferUpdatesSOFT, which will still hold the play state change until alProcessUpdatesSOFT.
* Make a function definition staticChris Robinson2016-08-231-1/+1
|
* Hold updates for both listener and source updatesChris Robinson2016-08-231-11/+0
|
* Don't pass the context's distance model as the source'sChris Robinson2016-08-231-10/+7
|
* Avoid checking DeferUpdates for each source state changeChris Robinson2016-08-081-8/+24
|
* Don't store the looping state in the voiceChris Robinson2016-07-311-9/+20
| | | | | Certain operations on the buffer queue depend on the loop state to behave properly, so it should not be deferred until the async voice update occurs.
* Move the input channel array out of the DirectParams and SendParamsChris Robinson2016-07-131-3/+3
|
* Reorder some source fieldsChris Robinson2016-07-071-19/+22
|
* Use separate arrays for UIntMap keys and valuesChris Robinson2016-07-041-2/+2
|
* Ensure voices has been updated once before mixing themChris Robinson2016-06-161-1/+8
| | | | | | | | Sometimes the mixer is temporarily prevented from applying updates, when multiple sources need to be updated simultaneously for example, but does not prevent mixing. If the mixer runs during that time and a voice was just started, it would've mixed the voice without any internal properties being set for it.
* Make a function staticChris Robinson2016-06-011-6/+7
|
* Hold the effectslot map lock while handling itChris Robinson2016-05-291-0/+3
|
* Avoid the mixer lock when getting the plain source offsetChris Robinson2016-05-281-11/+15
| | | | i.e. without the latency
* Avoid an explicit mixer lock for getting the source offset and latencyChris Robinson2016-05-281-27/+72
| | | | | The only mixer locking involved is with the backend, as determined by it's ability to get the device clock and latency atomically.
* Change the backend getLatency method to return the clock time tooChris Robinson2016-05-281-3/+8
| | | | | | This will also allow backends to better synchronize the tracked clock time with the device output latency, without necessarily needing to lock if the backend API can allow for it.
* Remove a couple unneeded functionsChris Robinson2016-05-251-4/+7
|
* Avoid using realloc in a number of placesChris Robinson2016-05-211-1/+3
|
* Make the source position calues atomicChris Robinson2016-05-191-19/+20
|
* Avoid redundantly storing distance model settingsChris Robinson2016-05-171-6/+9
|
* Don't store the source's update method with the voiceChris Robinson2016-05-161-4/+0
|
* Avoid separate updates to sources that should apply togetherChris Robinson2016-05-151-1/+12
|
* Provide asynchronous property updates for sourcesChris Robinson2016-05-141-132/+320
| | | | | | | | | | | | | | | | | | | | | | | | | This necessitates a change in how source updates are handled. Rather than just being able to update sources when a dependent object state is changed (e.g. a listener gain change), now all source updates must be proactively provided. Consequently, apps that do not utilize any deferring (AL_SOFT_defer_updates or alcSuspendContext/alcProcessContext) may utilize more CPU since it'll be filling out more update containers for the mixer thread to use. The upside is that there's less blocking between the app's calling thread and the mixer thread, particularly for vectors and other multi-value properties (filters and sends). Deferring behavior when used is also improved, since updates that shouldn't be applied yet are simply not provided. And when they are provided, the mixer doesn't have to ignore them, meaning the actual deferring of a context doesn't have to synchrnously force an update -- the process call will send any pending updates, which the mixer will apply even if another deferral occurs before the mixer runs, because it'll still be there waiting on the next mixer invocation. There is one slight bug introduced by this commit. When a listener change is made, or changes to multiple sources while updates are being deferred, it is possible for the mixer to run while the sources are prepping their updates, causing some of the source updates to be seen before the other. This will be fixed in short order.
* Hold the effect and filter maps while handling effects and filtersChris Robinson2016-05-121-1/+10
|
* Hold the buffer map lock while handling the bufferChris Robinson2016-05-101-1/+11
|
* Hold the source map lock while handling itChris Robinson2016-05-101-0/+68
|
* Use the source's offset type to determine if there's an offsetChris Robinson2016-05-091-6/+11
|
* Remove unnecessary code for the now-unused write offsetChris Robinson2016-04-251-53/+17
|
* Add support for AL_EXT_SOURCE_RADIUSChris Robinson2016-04-251-6/+27
|
* Drop support for AL_SOFT_buffer_samples and AL_SOFT_buffer_sub_dataChris Robinson2016-04-241-49/+0
| | | | | | Unfortunately they conflict with AL_EXT_SOURCE_RADIUS, as AL_SOURCE_RADIUS and AL_BYTE_RW_OFFSETS_SOFT share the same source property value. A replacement for AL_SOFT_buffer_samples will eventually be made.
* Move the aligned malloc functions to the common libChris Robinson2016-03-291-0/+1
|
* Implement AL_EXT_STEREO_ANGLES supportChris Robinson2016-03-251-0/+36
|
* Calculate HRTF stepping params right before mixingChris Robinson2016-02-141-1/+0
| | | | | This means we track the current params and the target params, rather than the target params and the stepping. This closer matches the non-HRTF mixers.