Commit graph

2008 commits

Author SHA1 Message Date
jjceresa
c6b2547215 fix typos 2019-09-10 23:22:28 +02:00
jjceresa
f23b3f9855 change value returned by fluid_list_copy_linked_mod() 2019-09-10 19:10:57 +02:00
jjceresa
16f07aa592 rename future API in comments 2019-09-10 16:51:56 +02:00
jjceresa
635321165c change return of fluid_list_check_linked_mod()
- change return value of fluid_check_linked_mod_path()
- change return value of fluid_list_check_linked_mod()
2019-09-10 16:45:22 +02:00
jjceresa
c6c3204a25 Merge branch 'linked-modulators' of https://github.com/FluidSynth/fluidsynth into linked-modulators 2019-09-10 14:53:49 +02:00
jjceresa
dc2a5190c4 consolidate fluid_linked_branch_test_identity()
- fix grammar in comments.
- Add overflow/underflow checks on branch_level table
2019-09-10 14:48:06 +02:00
jjceresa
8d34e25474 Allow linked modulators having amount set to 0.
- Later, this will allow adding a complex modulator
  to a voice in mode FLUID_VOICE_OVERWRITE.
2019-09-10 14:26:39 +02:00
derselbst
dcf0d8d110 fix random unit test failure 2019-08-06 17:36:25 +02:00
derselbst
b9dafa8dad Merge branch 'master' into linked-modulators 2019-08-06 17:33:44 +02:00
Tom M
5c795791c1
Suppress abort() dialog on windows (#549) 2019-08-06 17:29:49 +02:00
jjceresa
6ff22b30a1 first letter of warning is uppercase 2019-08-05 18:10:25 +02:00
jjceresa
1fe53ea1ad allows fluid_list_check_linked_mod() doing minimum safe check.
-useful for performance reason, when the caller already know
 that modulators are valid.
2019-08-05 17:45:40 +02:00
jjceresa
1025b772e2 consolidate fluid_list_copy_linked_mod() dependency.
Behaviour remains the same.
- make  fluid_list_copy_linked_mod() only dependant of
  fluid_check_linked_mod_path().

1)This leads with these two lower functions being only required for
doing minimun safe checking/copying of linked modulators.

2)Higher lever functions are:
- fluid_zone_check_mod().
- fluid_list_check_linked_mod().
- fluid_linked_mod_test_identity().
2019-08-05 15:47:03 +02:00
jjceresa
d27647e92e rename zone_xxx variable to list_xxx 2019-08-05 15:05:39 +02:00
jjceresa
9ee6b460cb more comments about path state variable.
- FLUID_PATH_CUR renamed FLUID_PATH_CURRENT.
- details about "isolated" linked path.
2019-08-05 14:44:11 +02:00
jjceresa
7de863af9a fix comments: typos, grammar, details 2019-08-05 13:16:48 +02:00
jjceresa
1c400c64d3 fix tab 2019-08-04 09:02:48 +02:00
jjceresa
c9078ba46f invalid modulators have amount forced to 0 2019-08-04 08:59:15 +02:00
jjceresa
a87a037b9a add function fluid_mod_is_linked() 2019-08-03 14:16:42 +02:00
jjceresa
5f4f118060 Make input list_mod left unmodified.
- fluid_list_check_linked_mod() doens't modifiy list_mod.
  This makes the function appropriate to be called by API
  fluid_voice_add_mod(),fluid_synth_add_default_mod().
2019-08-03 13:14:05 +02:00
derselbst
5019b2a1a9 Merge branch 'master' into linked-modulators 2019-08-02 14:16:40 +02:00
derselbst
d8bbd56fea Restore original libinstpatch pkgconfig module name 2019-08-02 14:06:54 +02:00
derselbst
6a6015f047 Fix build if -Denable-fpe-check=1 on windows 2019-08-02 13:53:14 +02:00
derselbst
7e2ed650ca Make travis builds verbose 2019-08-02 13:53:14 +02:00
derselbst
775beba985 Add additional windows CI builds 2019-08-02 13:53:14 +02:00
derselbst
e1dc5d8f68 Correctly define DEBUG macro 2019-08-02 13:21:21 +02:00
jjceresa
a56670ad33 fix comments 2019-08-02 11:22:27 +02:00
jjceresa
57a5e2e54d prepare fluid_zone_check_linked_mod
fluid_zone_check_linked_mod() is prepared to be called from API functions
fluid_voice_add_mod(), fluid_synth_add_default_mod().
The intend when calling these API is:
 1) add a complex modulator as well a simple modulator.
 2) keeping input parameter 'mod' list intact (i.e
    any modulators shouldn't removed).

The changes are:
- fluid_zone_check_linked_mod() renamed fluid_list_check_linked_mod().
- fluid_zone_copy_linked_mod() renamed fluid_list_copy_linked_mod().
- Calling fluid_list_copy_linked_mod() is integrated in
  fluid_list_check_linked_mod(). Now fluid_list_check_linked_mod()
  have the possibility:
  1) to check only the input list_mod (if linked_mod is NULL).
  2) if linked_mod is not NULL, the function clones linked modulators from
     lis_mod to linked_mod list.

-unit test test_modulator_links.c is updated with
fluid_zone_check_linked_mod() remamed to fluid_list_check_linked_mod() and
linked_mod parameter to NULL. The behaviour is the same.
2019-07-31 18:05:12 +02:00
jjceresa
50493e9efe add comments 2019-07-31 15:34:27 +02:00
jjceresa
56bc0bd44c make CI builds happy.
-remove #ifdef DEBUG,#endif around fluid_dump_modulator.
2019-07-30 18:08:29 +02:00
Tom M
df0a84ec53
Add a debug CI build using MSVC
and increase verbosity of build log
2019-07-30 12:53:17 +02:00
jjceresa
7101d72d0e forget to remove comments in b2db11c 2019-07-30 12:14:46 +02:00
jjceresa
fb148e6e7d add DEBUG macro for MSVC when fluidsynth compiled with enable-debug options 2019-07-29 23:35:38 +02:00
jjceresa
539d31bd4f add braces 2019-07-29 22:54:03 +02:00
jjceresa
07047039f7 Fix Travis CI build
- make use of fluid_dump_modulator() in fluid_mod.c
- make use of string literal for printf.
- allow test_fluid_zone_check_mod to be executed with no arguments.
- use of TEST_SUCCESS macro for each test.
- fix minor comments typos.
2019-07-29 17:16:00 +02:00
jjceresa
d752414342 Add comments in tests tables.
Fix Travis CI complaint.
Add return code FLUID_FAILED, FLUID_OK.
2019-07-29 00:50:12 +02:00
jjceresa
817661be52 fix comments 2019-07-28 03:52:12 +02:00
jjceresa
aefa4feaab add test of fluid_zone_check_mod() 2019-07-28 03:15:21 +02:00
jjceresa
a20b0a8674 replace add_amount parameter by test_mode 2019-07-27 21:50:12 +02:00
jjceresa
b2db11c5a2 Simplify identification of linked modulators.
- When removing modulators in list_mod list, all linked modulators
  must be removed regardless they are valid or invalid modulators.
This commit is just a simplification, fluid_zone_check_remove_mod()
behaves as before.
2019-07-27 15:33:05 +02:00
jjceresa
f65dea1fe7 Move checks of modulators with source GC none.
- These checks are moved from soundfont loader to fluid_mod_check_sources().
  This ensure these checks will also done when adding modulators by
  calling the API functions fluid_voice_add_mod() or
  fluid_synth_add_default_mod().
2019-07-26 23:05:28 +02:00
jjceresa
e71a3e0e9b Add comments
- add comments to describe why a circular path is detected as
  "isolated  path".
- fix ordering in 6.4 test title.
2019-07-24 18:38:41 +02:00
derselbst
9847df9f48 add even more circular complex modulator test cases 2019-07-24 16:40:55 +02:00
Carlo Bramini
d6c51cd2cc Aid constant folding in fluid_rev.c (#547) 2019-07-16 17:47:25 +02:00
jjceresa
95efd7e503 Initialize linked modulator list
- initialize linked modulator list to NULL. (just in case the caller
  have not done this).
- fix comments.
2019-07-14 14:09:58 +02:00
jjceresa
1dbd7450fe fix more comments:
- in fluid_check_linked_mod_path().
- in fluid_zone_copy_linked_mod().
2019-07-14 13:50:18 +02:00
jjceresa
c23c891d18 fix comments in fluid_check_linked_mod_path() 2019-07-14 12:27:35 +02:00
jjceresa
fb95fce202 add test 9 to test_modulators_links.c 2019-07-14 03:46:07 +02:00
jjceresa
a0929a25de remove #if #endif 2019-07-14 03:17:03 +02:00
jjceresa
956f1992e2 Add test to test_modulator_links.c:
- complete test 6.
- add circular path tests 6.0, 6.1.
- fix test 8 title.
2019-07-14 03:07:27 +02:00