diff options
-rw-r--r-- | Alc/effects/chorus.c | 11 | ||||
-rw-r--r-- | Alc/effects/dedicated.c | 11 | ||||
-rw-r--r-- | Alc/effects/distortion.c | 11 | ||||
-rw-r--r-- | Alc/effects/echo.c | 11 | ||||
-rw-r--r-- | Alc/effects/equalizer.c | 11 | ||||
-rw-r--r-- | Alc/effects/flanger.c | 11 | ||||
-rw-r--r-- | Alc/effects/modulator.c | 11 | ||||
-rw-r--r-- | Alc/effects/null.c | 16 | ||||
-rw-r--r-- | Alc/effects/reverb.c | 11 | ||||
-rw-r--r-- | OpenAL32/Include/alAuxEffectSlot.h | 15 | ||||
-rw-r--r-- | OpenAL32/Include/alMain.h | 5 |
11 files changed, 28 insertions, 96 deletions
diff --git a/Alc/effects/chorus.c b/Alc/effects/chorus.c index c74bf0b5..22241af5 100644 --- a/Alc/effects/chorus.c +++ b/Alc/effects/chorus.c @@ -243,9 +243,9 @@ static ALvoid ALchorusState_Process(ALchorusState *state, ALuint SamplesToDo, co ProcessSinusoid(state, SamplesToDo, SamplesIn, SamplesOut); } -static ALeffectStateFactory *ALchorusState_getCreator(void) +static void ALchorusState_Delete(ALchorusState *state) { - return STATIC_CAST(ALeffectStateFactory, &ChorusFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALchorusState); @@ -267,13 +267,6 @@ static ALeffectState *ALchorusStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALchorusStateFactory_destroy(ALeffectState *effect) -{ - ALchorusState *state = STATIC_UPCAST(ALchorusState, ALeffectState, effect); - ALchorusState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALchorusStateFactory); diff --git a/Alc/effects/dedicated.c b/Alc/effects/dedicated.c index 6b3ce1ae..184fe292 100644 --- a/Alc/effects/dedicated.c +++ b/Alc/effects/dedicated.c @@ -85,9 +85,9 @@ static ALvoid ALdedicatedState_Process(ALdedicatedState *state, ALuint SamplesTo } } -static ALeffectStateFactory *ALdedicatedState_getCreator(void) +static void ALdedicatedState_Delete(ALdedicatedState *state) { - return STATIC_CAST(ALeffectStateFactory, &DedicatedFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALdedicatedState); @@ -108,13 +108,6 @@ ALeffectState *ALdedicatedStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALdedicatedStateFactory_destroy(ALeffectState *effect) -{ - ALdedicatedState *state = STATIC_UPCAST(ALdedicatedState, ALeffectState, effect); - ALdedicatedState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALdedicatedStateFactory); diff --git a/Alc/effects/distortion.c b/Alc/effects/distortion.c index e92fb9a8..cc6669af 100644 --- a/Alc/effects/distortion.c +++ b/Alc/effects/distortion.c @@ -234,9 +234,9 @@ static ALvoid ALdistortionState_Process(ALdistortionState *state, ALuint Samples } } -static ALeffectStateFactory *ALdistortionState_getCreator(void) +static void ALdistortionState_Delete(ALdistortionState *state) { - return STATIC_CAST(ALeffectStateFactory, &DistortionFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALdistortionState); @@ -263,13 +263,6 @@ static ALeffectState *ALdistortionStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALdistortionStateFactory_destroy(ALeffectState *effect) -{ - ALdistortionState *state = STATIC_UPCAST(ALdistortionState, ALeffectState, effect); - ALdistortionState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALdistortionStateFactory); diff --git a/Alc/effects/echo.c b/Alc/effects/echo.c index 8f679c41..533461ac 100644 --- a/Alc/effects/echo.c +++ b/Alc/effects/echo.c @@ -175,9 +175,9 @@ static ALvoid ALechoState_Process(ALechoState *state, ALuint SamplesToDo, const state->Offset = offset; } -static ALeffectStateFactory *ALechoState_getCreator(void) +static void ALechoState_Delete(ALechoState *state) { - return STATIC_CAST(ALeffectStateFactory, &EchoFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALechoState); @@ -205,13 +205,6 @@ ALeffectState *ALechoStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALechoStateFactory_destroy(ALeffectState *effect) -{ - ALechoState *state = STATIC_UPCAST(ALechoState, ALeffectState, effect); - ALechoState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALechoStateFactory); diff --git a/Alc/effects/equalizer.c b/Alc/effects/equalizer.c index adcd40f4..3d5f35d5 100644 --- a/Alc/effects/equalizer.c +++ b/Alc/effects/equalizer.c @@ -270,9 +270,9 @@ static ALvoid ALequalizerState_Process(ALequalizerState *state, ALuint SamplesTo } } -static ALeffectStateFactory *ALequalizerState_getCreator(void) +static void ALequalizerState_Delete(ALequalizerState *state) { - return STATIC_CAST(ALeffectStateFactory, &EqualizerFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALequalizerState); @@ -305,13 +305,6 @@ ALeffectState *ALequalizerStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALequalizerStateFactory_destroy(ALeffectState *effect) -{ - ALequalizerState *state = STATIC_UPCAST(ALequalizerState, ALeffectState, effect); - ALequalizerState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALequalizerStateFactory); diff --git a/Alc/effects/flanger.c b/Alc/effects/flanger.c index de98a06d..12e16de2 100644 --- a/Alc/effects/flanger.c +++ b/Alc/effects/flanger.c @@ -243,9 +243,9 @@ static ALvoid ALflangerState_Process(ALflangerState *state, ALuint SamplesToDo, ProcessSinusoid(state, SamplesToDo, SamplesIn, SamplesOut); } -static ALeffectStateFactory *ALflangerState_getCreator(void) +static void ALflangerState_Delete(ALflangerState *state) { - return STATIC_CAST(ALeffectStateFactory, &FlangerFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALflangerState); @@ -267,13 +267,6 @@ ALeffectState *ALflangerStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALflangerStateFactory_destroy(ALeffectState *effect) -{ - ALflangerState *state = STATIC_UPCAST(ALflangerState, ALeffectState, effect); - ALflangerState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALflangerStateFactory); diff --git a/Alc/effects/modulator.c b/Alc/effects/modulator.c index e48a6d94..3f9ff8a5 100644 --- a/Alc/effects/modulator.c +++ b/Alc/effects/modulator.c @@ -197,9 +197,9 @@ static ALvoid ALmodulatorState_Process(ALmodulatorState *state, ALuint SamplesTo } } -static ALeffectStateFactory *ALmodulatorState_getCreator(void) +static void ALmodulatorState_Delete(ALmodulatorState *state) { - return STATIC_CAST(ALeffectStateFactory, &ModulatorFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALmodulatorState); @@ -222,13 +222,6 @@ static ALeffectState *ALmodulatorStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALmodulatorStateFactory_destroy(ALeffectState *effect) -{ - ALmodulatorState *state = STATIC_UPCAST(ALmodulatorState, ALeffectState, effect); - ALmodulatorState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALmodulatorStateFactory); diff --git a/Alc/effects/null.c b/Alc/effects/null.c index 76d650da..3d617b64 100644 --- a/Alc/effects/null.c +++ b/Alc/effects/null.c @@ -60,12 +60,10 @@ static ALvoid ALnullState_Process(ALnullState *state, ALuint samplesToDo, const (void)samplesOut; } -/* This returns the ALeffectStateFactory that creates these ALeffectState - * object types. - */ -static ALeffectStateFactory *ALnullState_getCreator(void) +/* This frees the memory used by the object, after it has been destructed. */ +static void ALnullState_Delete(ALnullState *state) { - return STATIC_CAST(ALeffectStateFactory, &NullFactory); + free(state); } /* Define the forwards and the ALeffectState vtable for this type. */ @@ -85,14 +83,6 @@ ALeffectState *ALnullStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -/* Destroys (destructs and frees) the ALeffectState. */ -static ALvoid ALnullStateFactory_destroy(ALeffectState *effect) -{ - ALnullState *state = STATIC_UPCAST(ALnullState, ALeffectState, effect); - ALnullState_Destruct(state); - free(state); -} - /* Define the ALeffectStateFactory vtable for this type. */ DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALnullStateFactory); diff --git a/Alc/effects/reverb.c b/Alc/effects/reverb.c index b634dd1c..2421225d 100644 --- a/Alc/effects/reverb.c +++ b/Alc/effects/reverb.c @@ -1174,9 +1174,9 @@ static ALvoid ALreverbState_Destruct(ALreverbState *State) State->SampleBuffer = NULL; } -static ALeffectStateFactory *ALreverbState_getCreator(void) +static void ALreverbState_Delete(ALreverbState *state) { - return STATIC_CAST(ALeffectStateFactory, &ReverbFactory); + free(state); } DEFINE_ALEFFECTSTATE_VTABLE(ALreverbState); @@ -1274,13 +1274,6 @@ static ALeffectState *ALreverbStateFactory_create(void) return STATIC_CAST(ALeffectState, state); } -static ALvoid ALreverbStateFactory_destroy(ALeffectState *effect) -{ - ALreverbState *state = STATIC_UPCAST(ALreverbState, ALeffectState, effect); - ALreverbState_Destruct(state); - free(state); -} - DEFINE_ALEFFECTSTATEFACTORY_VTABLE(ALreverbStateFactory); diff --git a/OpenAL32/Include/alAuxEffectSlot.h b/OpenAL32/Include/alAuxEffectSlot.h index ff88fe7b..d7062ff7 100644 --- a/OpenAL32/Include/alAuxEffectSlot.h +++ b/OpenAL32/Include/alAuxEffectSlot.h @@ -18,7 +18,8 @@ struct ALeffectStateVtable { ALboolean (*const DeviceUpdate)(ALeffectState *state, ALCdevice *device); ALvoid (*const Update)(ALeffectState *state, ALCdevice *device, const ALeffectslot *slot); ALvoid (*const Process)(ALeffectState *state, ALuint samplesToDo, const ALfloat *restrict samplesIn, ALfloat (*restrict samplesOut)[BUFFERSIZE]); - ALeffectStateFactory *(*const getCreator)(void); + + void (*const Delete)(ALeffectState *state); }; struct ALeffectState { @@ -29,7 +30,6 @@ struct ALeffectState { #define ALeffectState_DeviceUpdate(a,b) ((a)->vtbl->DeviceUpdate((a),(b))) #define ALeffectState_Update(a,b,c) ((a)->vtbl->Update((a),(b),(c))) #define ALeffectState_Process(a,b,c,d) ((a)->vtbl->Process((a),(b),(c),(d))) -#define ALeffectState_getCreator(a) ((a)->vtbl->getCreator()) #define DEFINE_ALEFFECTSTATE_VTABLE(T) \ static ALvoid T##_ALeffectState_Destruct(ALeffectState *state) \ @@ -40,21 +40,20 @@ static ALvoid T##_ALeffectState_Update(ALeffectState *state, ALCdevice *device, { T##_Update(STATIC_UPCAST(T, ALeffectState, state), device, slot); } \ static ALvoid T##_ALeffectState_Process(ALeffectState *state, ALuint samplesToDo, const ALfloat *restrict samplesIn, ALfloat (*restrict samplesOut)[BUFFERSIZE]) \ { T##_Process(STATIC_UPCAST(T, ALeffectState, state), samplesToDo, samplesIn, samplesOut); } \ -static ALeffectStateFactory* T##_ALeffectState_getCreator(void) \ -{ return T##_getCreator(); } \ +static ALvoid T##_ALeffectState_Delete(ALeffectState *state) \ +{ T##_Delete(STATIC_UPCAST(T, ALeffectState, state)); } \ \ static const struct ALeffectStateVtable T##_ALeffectState_vtable = { \ T##_ALeffectState_Destruct, \ T##_ALeffectState_DeviceUpdate, \ T##_ALeffectState_Update, \ T##_ALeffectState_Process, \ - T##_ALeffectState_getCreator, \ + T##_ALeffectState_Delete, \ } struct ALeffectStateFactoryVtable { ALeffectState *(*const create)(void); - ALvoid (*const destroy)(ALeffectState *state); }; struct ALeffectStateFactory { @@ -62,17 +61,13 @@ struct ALeffectStateFactory { }; #define ALeffectStateFactory_create(p) ((p)->vtbl->create()) -#define ALeffectStateFactory_destroy(p,a) ((p)->vtbl->destroy((a))) #define DEFINE_ALEFFECTSTATEFACTORY_VTABLE(T) \ static ALeffectState* T##_ALeffectStateFactory_create(void) \ { return T##_create(); } \ -static ALvoid T##_ALeffectStateFactory_destroy(ALeffectState *state) \ -{ T##_destroy(state); } \ \ static const struct ALeffectStateFactoryVtable T##_ALeffectStateFactory_vtable = { \ T##_ALeffectStateFactory_create, \ - T##_ALeffectStateFactory_destroy \ } diff --git a/OpenAL32/Include/alMain.h b/OpenAL32/Include/alMain.h index ca0df2ad..a7c79b2a 100644 --- a/OpenAL32/Include/alMain.h +++ b/OpenAL32/Include/alMain.h @@ -70,7 +70,10 @@ static const union { #define SET_VTABLE1(T1, obj) ((obj)->vtbl = &(T1##_vtable)) #define SET_VTABLE2(T1, T2, obj) SET_VTABLE1(T1##_##T2, STATIC_CAST(T2, (obj))) -#define DELETE_OBJ(obj) ((obj)->vtbl->getCreator()->vtbl->destroy((obj))) +#define DELETE_OBJ(obj) do { \ + (obj)->vtbl->Destruct((obj)); \ + (obj)->vtbl->Delete((obj)); \ +} while(0) #ifdef _WIN32 |