zdoom/fluidsynth
0
0
Fork 0
mirror of https://github.com/ZDoom/fluidsynth.git synced 2025-02-22 20:01:04 +00:00
Code Issues Projects Releases Packages Wiki Activity

remove redundant struct _fluid_sfont_info_t

Browse source 
integrate its members into fluid_sfont_t
This commit is contained in:
derselbst 2018-04-07 22:17:08 +02:00
parent 6ab1c7fde3
commit 4ede11e71f
3 changed files with 89 additions and 169 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/sfloader/fluid_sfont.h
View file

@ -123,6 +123,8 @@ void fluid_sfont_set_iteration_next(fluid_sfont_t* sfont, fluid_sfont_iteration_
struct _fluid_sfont_t {
void* data; /**< User defined data */
unsigned int id; /**< SoundFont ID */
int refcount; /**< SoundFont reference count (1 if no presets referencing it) */
int bankofs; /**< Bank offset */
fluid_sfont_free_t free;

236
src/synth/fluid_synth.c
View file

@ -24,7 +24,6 @@
#include "fluid_tuning.h"
#include "fluid_settings.h"
#include "fluid_sfont.h"
#include "fluid_hash.h"
#include "fluid_defsfont.h"
#ifdef TRAP_ON_FPE
@ -95,8 +94,6 @@ static int fluid_synth_render_blocks(fluid_synth_t* synth, int blockcount);
static fluid_voice_t* fluid_synth_free_voice_by_kill_LOCAL(fluid_synth_t* synth);
static void fluid_synth_kill_by_exclusive_class_LOCAL(fluid_synth_t* synth,
fluid_voice_t* new_voice);
static fluid_sfont_info_t *new_fluid_sfont_info (fluid_synth_t *synth,
fluid_sfont_t *sfont);
static int fluid_synth_sfunload_callback(void* data, unsigned int msec);
void fluid_synth_release_voice_on_same_note_LOCAL(fluid_synth_t* synth,
int chan, int key);
@ -688,9 +685,6 @@ new_fluid_synth(fluid_settings_t *settings)
/* as soon as the synth is created it starts playing. */
synth->state = FLUID_SYNTH_PLAYING;
synth->sfont_info = NULL;
synth->sfont_hash = new_fluid_hashtable (NULL, NULL);
synth->noteid = 0;
synth->fromkey_portamento = INVALID_NOTE; /* disable portamento */
@ -847,7 +841,7 @@ delete_fluid_synth(fluid_synth_t* synth)
{
int i, k;
fluid_list_t *list;
fluid_sfont_info_t* sfont_info;
fluid_sfont_t* sfont;
fluid_sfloader_t* loader;
fluid_mod_t* default_mod;
fluid_mod_t* mod;
@ -887,18 +881,11 @@ delete_fluid_synth(fluid_synth_t* synth)
delete_fluid_rvoice_eventhandler(synth->eventhandler);
/* delete all the SoundFonts */
for (list = synth->sfont_info; list; list = fluid_list_next (list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
fluid_sfont_delete_internal (sfont_info->sfont);
FLUID_FREE (sfont_info);
for (list = synth->sfont; list; list = fluid_list_next (list)) {
sfont = fluid_list_get (list);
fluid_sfont_delete_internal (sfont);
}
delete_fluid_list(synth->sfont_info);
/* Delete the SoundFont info hash */
delete_fluid_hashtable (synth->sfont_hash);
delete_fluid_list(synth->sfont);
/* delete all the SoundFont loaders */
@ -2233,20 +2220,20 @@ fluid_synth_get_preset(fluid_synth_t* synth, unsigned int sfontnum,
unsigned int banknum, unsigned int prognum)
{
fluid_preset_t *preset = NULL;
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
/* 128 indicates an "unset" operation" */
if (prognum == FLUID_UNSET_PROGRAM) return NULL;
for (list = synth->sfont_info; list; list = fluid_list_next (list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
for (list = synth->sfont; list; list = fluid_list_next (list)) {
sfont = fluid_list_get (list);
if (fluid_sfont_get_id (sfont_info->sfont) == sfontnum)
if (fluid_sfont_get_id (sfont) == sfontnum)
{
preset = fluid_sfont_get_preset (sfont_info->sfont,
banknum - sfont_info->bankofs, prognum);
if (preset) sfont_info->refcount++; /* Add reference to SoundFont */
preset = fluid_sfont_get_preset (sfont,
banknum - sfont->bankofs, prognum);
if (preset) sfont->refcount++; /* Add reference to SoundFont */
break;
}
}
@ -2265,17 +2252,17 @@ fluid_synth_get_preset_by_sfont_name(fluid_synth_t* synth, const char *sfontname
unsigned int banknum, unsigned int prognum)
{
fluid_preset_t *preset = NULL;
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
for (list = synth->sfont_info; list; list = fluid_list_next (list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
for (list = synth->sfont; list; list = fluid_list_next (list)) {
sfont = fluid_list_get (list);
if (FLUID_STRCMP (fluid_sfont_get_name (sfont_info->sfont), sfontname) == 0)
if (FLUID_STRCMP (fluid_sfont_get_name (sfont), sfontname) == 0)
{
preset = fluid_sfont_get_preset (sfont_info->sfont,
banknum - sfont_info->bankofs, prognum);
if (preset) sfont_info->refcount++; /* Add reference to SoundFont */
preset = fluid_sfont_get_preset (sfont,
banknum - sfont->bankofs, prognum);
if (preset) sfont->refcount++; /* Add reference to SoundFont */
break;
}
}
@ -2293,17 +2280,17 @@ fluid_synth_find_preset(fluid_synth_t* synth, unsigned int banknum,
unsigned int prognum)
{
fluid_preset_t *preset = NULL;
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
for (list = synth->sfont_info; list; list = fluid_list_next (list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
for (list = synth->sfont; list; list = fluid_list_next (list)) {
sfont = fluid_list_get (list);
preset = fluid_sfont_get_preset (sfont_info->sfont,
banknum - sfont_info->bankofs, prognum);
preset = fluid_sfont_get_preset (sfont,
banknum - sfont->bankofs, prognum);
if (preset)
{
sfont_info->refcount++; /* Add reference to SoundFont */
sfont->refcount++; /* Add reference to SoundFont */
break;
}
}
@ -3675,7 +3662,7 @@ fluid_synth_add_sfloader(fluid_synth_t* synth, fluid_sfloader_t* loader)
fluid_synth_api_enter(synth);
/* Test if sfont is already loaded */
if (synth->sfont_info == NULL)
if (synth->sfont == NULL)
synth->loaders = fluid_list_prepend(synth->loaders, loader);
fluid_synth_api_exit(synth);
}
@ -3694,7 +3681,6 @@ fluid_synth_add_sfloader(fluid_synth_t* synth, fluid_sfloader_t* loader)
int
fluid_synth_sfload(fluid_synth_t* synth, const char* filename, int reset_presets)
{
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
fluid_sfloader_t *loader;
@ -3712,17 +3698,10 @@ fluid_synth_sfload(fluid_synth_t* synth, const char* filename, int reset_presets
sfont = fluid_sfloader_load(loader, filename);
if (sfont != NULL) {
sfont_info = new_fluid_sfont_info (synth, sfont);
if (!sfont_info)
{
fluid_sfont_delete_internal (sfont);
FLUID_API_RETURN(FLUID_FAILED);
}
sfont->refcount++;
sfont->id = sfont_id = ++synth->sfont_id;
synth->sfont_info = fluid_list_prepend(synth->sfont_info, sfont_info); /* prepend to list */
fluid_hashtable_insert (synth->sfont_hash, sfont, sfont_info); /* Hash sfont->sfont_info */
synth->sfont = fluid_list_prepend(synth->sfont, sfont); /* prepend to list */
/* reset the presets for all channels if requested */
if (reset_presets) fluid_synth_program_reset(synth);
@ -3735,28 +3714,6 @@ fluid_synth_sfload(fluid_synth_t* synth, const char* filename, int reset_presets
FLUID_API_RETURN(FLUID_FAILED);
}
/* Create a new SoundFont info structure, free with FLUID_FREE */
static fluid_sfont_info_t *
new_fluid_sfont_info (fluid_synth_t *synth, fluid_sfont_t *sfont)
{
fluid_sfont_info_t *sfont_info;
sfont_info = FLUID_NEW (fluid_sfont_info_t);
if (!sfont_info)
{
FLUID_LOG(FLUID_ERR, "Out of memory");
return NULL;
}
sfont_info->sfont = sfont;
sfont_info->synth = synth;
sfont_info->refcount = 1; /* Start with refcount of 1 for owning synth */
sfont_info->bankofs = 0;
return (sfont_info);
}
/**
* Unload a SoundFont.
* @param synth FluidSynth instance
@ -3767,19 +3724,19 @@ new_fluid_sfont_info (fluid_synth_t *synth, fluid_sfont_t *sfont)
int
fluid_synth_sfunload(fluid_synth_t* synth, unsigned int id, int reset_presets)
{
fluid_sfont_info_t *sfont_info = NULL;
fluid_sfont_t *sfont = NULL;
fluid_list_t *list;
fluid_return_val_if_fail (synth != NULL, FLUID_FAILED);
fluid_synth_api_enter(synth);
/* remove the SoundFont from the list */
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont_info = (fluid_sfont_info_t*) fluid_list_get(list);
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont = fluid_list_get(list);
if (fluid_sfont_get_id (sfont_info->sfont) == id)
if (fluid_sfont_get_id (sfont) == id)
{
synth->sfont_info = fluid_list_remove (synth->sfont_info, sfont_info);
synth->sfont = fluid_list_remove (synth->sfont, sfont);
break;
}
}
@ -3793,8 +3750,8 @@ fluid_synth_sfunload(fluid_synth_t* synth, unsigned int id, int reset_presets)
if (reset_presets) fluid_synth_program_reset (synth);
else fluid_synth_update_presets (synth);
/* -- Remove synth->sfont_info list's reference to SoundFont */
fluid_synth_sfont_unref (synth, sfont_info->sfont);
/* -- Remove synth->sfont list's reference to SoundFont */
fluid_synth_sfont_unref (synth, sfont);
FLUID_API_RETURN(FLUID_OK);
}
@ -3803,30 +3760,17 @@ fluid_synth_sfunload(fluid_synth_t* synth, unsigned int id, int reset_presets)
void
fluid_synth_sfont_unref (fluid_synth_t *synth, fluid_sfont_t *sfont)
{
fluid_sfont_info_t *sfont_info;
int refcount = 0;
fluid_return_if_fail (sfont != NULL); /* Shouldn't happen, programming error if so */
sfont_info = fluid_hashtable_lookup (synth->sfont_hash, sfont);
sfont->refcount--; /* -- Remove the sfont list's reference */
if (sfont_info)
if (sfont->refcount == 0) /* No more references? - Attempt delete */
{
sfont_info->refcount--; /* -- Remove the sfont_info list's reference */
refcount = sfont_info->refcount;
if (refcount == 0) /* Remove SoundFont from hash if no more references */
fluid_hashtable_remove (synth->sfont_hash, sfont_info->sfont);
}
fluid_return_if_fail (sfont_info != NULL); /* Shouldn't happen, programming error if so */
if (refcount == 0) /* No more references? - Attempt delete */
if (fluid_sfont_delete_internal (sfont) == 0) /* SoundFont loader can block SoundFont unload */
{
if (fluid_sfont_delete_internal (sfont_info->sfont) == 0) /* SoundFont loader can block SoundFont unload */
{
FLUID_FREE (sfont_info);
FLUID_LOG (FLUID_DBG, "Unloaded SoundFont");
} /* spin off a timer thread to unload the sfont later (SoundFont loader blocked unload) */
else new_fluid_timer (100, fluid_synth_sfunload_callback, sfont_info, TRUE, TRUE, FALSE);
else new_fluid_timer (100, fluid_synth_sfunload_callback, sfont, TRUE, TRUE, FALSE);
}
}
@ -3835,11 +3779,10 @@ fluid_synth_sfont_unref (fluid_synth_t *synth, fluid_sfont_t *sfont)
static int
fluid_synth_sfunload_callback(void* data, unsigned int msec)
{
fluid_sfont_info_t *sfont_info = (fluid_sfont_info_t *)data;
fluid_sfont_t *sfont = data;
if (fluid_sfont_delete_internal (sfont_info->sfont) == 0)
if (fluid_sfont_delete_internal (sfont) == 0)
{
FLUID_FREE (sfont_info);
FLUID_LOG (FLUID_DBG, "Unloaded SoundFont");
return FALSE;
}
@ -3855,32 +3798,31 @@ fluid_synth_sfunload_callback(void* data, unsigned int msec)
int
fluid_synth_sfreload(fluid_synth_t* synth, unsigned int id)
{
char filename[1024];
fluid_sfont_info_t *sfont_info, *old_sfont_info;
char *filename = NULL;
fluid_sfont_t *sfont;
fluid_sfloader_t* loader;
fluid_list_t *list;
int index;
int index, ret = FLUID_FAILED;
fluid_return_val_if_fail (synth != NULL, FLUID_FAILED);
fluid_synth_api_enter(synth);
/* Search for SoundFont and get its index */
for (list = synth->sfont_info, index = 0; list; list = fluid_list_next (list), index++) {
old_sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
if (fluid_sfont_get_id (old_sfont_info->sfont) == id) break;
for (list = synth->sfont, index = 0; list; list = fluid_list_next (list), index++) {
sfont = fluid_list_get (list);
if (fluid_sfont_get_id (sfont) == id) break;
}
if (!list) {
FLUID_LOG(FLUID_ERR, "No SoundFont with id = %d", id);
FLUID_API_RETURN(FLUID_FAILED);
goto exit;
}
/* keep a copy of the SoundFont's filename */
FLUID_STRCPY (filename, fluid_sfont_get_name (old_sfont_info->sfont));
filename = FLUID_STRDUP(fluid_sfont_get_name (sfont));
if (fluid_synth_sfunload (synth, id, FALSE) != FLUID_OK)
FLUID_API_RETURN(FLUID_FAILED);
if (filename == NULL || fluid_synth_sfunload (synth, id, FALSE) != FLUID_OK)
goto exit;
/* MT Note: SoundFont loader list will not change */
@ -3891,26 +3833,22 @@ fluid_synth_sfreload(fluid_synth_t* synth, unsigned int id)
if (sfont != NULL) {
sfont->id = id;
sfont->refcount++;
sfont_info = new_fluid_sfont_info (synth, sfont);
if (!sfont_info)
{
fluid_sfont_delete_internal (sfont);
FLUID_API_RETURN(FLUID_FAILED);
}
synth->sfont_info = fluid_list_insert_at(synth->sfont_info, index, sfont_info); /* insert the sfont at the same index */
fluid_hashtable_insert (synth->sfont_hash, sfont, sfont_info); /* Hash sfont->sfont_info */
synth->sfont = fluid_list_insert_at(synth->sfont, index, sfont); /* insert the sfont at the same index */
/* reset the presets for all channels */
fluid_synth_update_presets(synth);
FLUID_API_RETURN(sfont->id);
ret = id;
goto exit;
}
}
FLUID_LOG(FLUID_ERR, "Failed to load SoundFont \"%s\"", filename);
FLUID_API_RETURN(FLUID_FAILED);
exit:
FLUID_FREE(filename);
FLUID_API_RETURN(ret);
}
/**
@ -3922,20 +3860,15 @@ fluid_synth_sfreload(fluid_synth_t* synth, unsigned int id)
int
fluid_synth_add_sfont(fluid_synth_t* synth, fluid_sfont_t* sfont)
{
fluid_sfont_info_t *sfont_info;
unsigned int sfont_id;
fluid_return_val_if_fail (synth != NULL, FLUID_FAILED);
fluid_return_val_if_fail (sfont != NULL, FLUID_FAILED);
fluid_synth_api_enter(synth);
sfont_info = new_fluid_sfont_info (synth, sfont);
if (!sfont_info)
FLUID_API_RETURN(FLUID_FAILED);
sfont->id = sfont_id = ++synth->sfont_id;
synth->sfont_info = fluid_list_prepend (synth->sfont_info, sfont_info); /* prepend to list */
fluid_hashtable_insert (synth->sfont_hash, sfont, sfont_info); /* Hash sfont->sfont_info */
synth->sfont = fluid_list_prepend (synth->sfont, sfont); /* prepend to list */
/* reset the presets for all channels */
fluid_synth_program_reset (synth);
@ -3957,7 +3890,7 @@ fluid_synth_add_sfont(fluid_synth_t* synth, fluid_sfont_t* sfont)
void
fluid_synth_remove_sfont(fluid_synth_t* synth, fluid_sfont_t* sfont)
{
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont_tmp;
fluid_list_t *list;
fluid_return_if_fail (synth != NULL);
@ -3965,15 +3898,12 @@ fluid_synth_remove_sfont(fluid_synth_t* synth, fluid_sfont_t* sfont)
fluid_synth_api_enter(synth);
/* remove the SoundFont from the list */
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont_info = (fluid_sfont_info_t*) fluid_list_get(list);
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont_tmp = fluid_list_get(list);
if (sfont_info->sfont == sfont)
if (sfont_tmp == sfont)
{
synth->sfont_info = fluid_list_remove (synth->sfont_info, sfont_info);
/* Remove from SoundFont hash regardless of refcount (SoundFont delete is up to caller) */
fluid_hashtable_remove (synth->sfont_hash, sfont_info->sfont);
synth->sfont = fluid_list_remove (synth->sfont, sfont_tmp);
break;
}
}
@ -3995,7 +3925,7 @@ fluid_synth_sfcount(fluid_synth_t* synth)
fluid_return_val_if_fail (synth != NULL, 0);
fluid_synth_api_enter(synth);
count = fluid_list_size (synth->sfont_info);
count = fluid_list_size (synth->sfont);
FLUID_API_RETURN(count);
}
@ -4016,8 +3946,8 @@ fluid_synth_get_sfont(fluid_synth_t* synth, unsigned int num)
fluid_return_val_if_fail (synth != NULL, NULL);
fluid_synth_api_enter(synth);
list = fluid_list_nth (synth->sfont_info, num);
if (list) sfont = ((fluid_sfont_info_t *)fluid_list_get (list))->sfont;
list = fluid_list_nth (synth->sfont, num);
if (list) sfont = fluid_list_get (list);
FLUID_API_RETURN(sfont);
}
@ -4039,8 +3969,8 @@ fluid_synth_get_sfont_by_id(fluid_synth_t* synth, unsigned int id)
fluid_return_val_if_fail (synth != NULL, NULL);
fluid_synth_api_enter(synth);
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont = ((fluid_sfont_info_t *)fluid_list_get (list))->sfont;
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont = fluid_list_get (list);
if (fluid_sfont_get_id (sfont) == id)
break;
}
@ -4068,8 +3998,8 @@ fluid_synth_get_sfont_by_name(fluid_synth_t* synth, const char *name)
fluid_return_val_if_fail (name != NULL, NULL);
fluid_synth_api_enter(synth);
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont = ((fluid_sfont_info_t *)fluid_list_get (list))->sfont;
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont = fluid_list_get (list);
if (FLUID_STRCMP(fluid_sfont_get_name(sfont), name) == 0)
break;
}
@ -5347,18 +5277,18 @@ fluid_synth_stop_LOCAL (fluid_synth_t *synth, unsigned int id)
int
fluid_synth_set_bank_offset(fluid_synth_t* synth, int sfont_id, int offset)
{
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
fluid_return_val_if_fail (synth != NULL, FLUID_FAILED);
fluid_synth_api_enter(synth);
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont = fluid_list_get (list);
if (fluid_sfont_get_id (sfont_info->sfont) == (unsigned int)sfont_id)
if (fluid_sfont_get_id (sfont) == (unsigned int)sfont_id)
{
sfont_info->bankofs = offset;
sfont->bankofs = offset;
break;
}
}
@ -5381,19 +5311,19 @@ fluid_synth_set_bank_offset(fluid_synth_t* synth, int sfont_id, int offset)
int
fluid_synth_get_bank_offset(fluid_synth_t* synth, int sfont_id)
{
fluid_sfont_info_t *sfont_info;
fluid_sfont_t *sfont;
fluid_list_t *list;
int offset = 0;
fluid_return_val_if_fail (synth != NULL, 0);
fluid_synth_api_enter(synth);
for (list = synth->sfont_info; list; list = fluid_list_next(list)) {
sfont_info = (fluid_sfont_info_t *)fluid_list_get (list);
for (list = synth->sfont; list; list = fluid_list_next(list)) {
sfont = fluid_list_get (list);
if (fluid_sfont_get_id (sfont_info->sfont) == (unsigned int)sfont_id)
if (fluid_sfont_get_id (sfont) == (unsigned int)sfont_id)
{
offset = sfont_info->bankofs;
offset = sfont->bankofs;
break;
}
}

14
src/synth/fluid_synth.h
View file

@ -73,17 +73,6 @@ enum fluid_synth_status
#define SYNTH_REVERB_CHANNEL 0
#define SYNTH_CHORUS_CHANNEL 1
/*
* Structure used for sfont_info field in #fluid_synth_t for each loaded
* SoundFont with the SoundFont instance and additional fields.
*/
typedef struct _fluid_sfont_info_t {
fluid_sfont_t *sfont; /**< Loaded SoundFont */
fluid_synth_t *synth; /**< Parent synth */
int refcount; /**< SoundFont reference count (0 if no presets referencing it) */
int bankofs; /**< Bank offset */
} fluid_sfont_info_t;
/*
* fluid_synth_t
*
@ -124,8 +113,7 @@ struct _fluid_synth_t
fluid_overflow_prio_t overflow; /**< parameters for overflow priority (aka voice-stealing) */
fluid_list_t *loaders; /**< the SoundFont loaders */
fluid_list_t *sfont_info; /**< List of fluid_sfont_info_t for each loaded SoundFont (remains until SoundFont is unloaded) */
fluid_hashtable_t *sfont_hash; /**< Hash of fluid_sfont_t->fluid_sfont_info_t (remains until SoundFont is deleted) */
fluid_list_t *sfont; /**< List of fluid_sfont_info_t for each loaded SoundFont (remains until SoundFont is unloaded) */
unsigned int sfont_id; /**< Incrementing ID assigned to each loaded SoundFont */
float gain; /**< master gain */