From 82e74a33661d235029f413eff2b21c4e29e057bc Mon Sep 17 00:00:00 2001 From: Florian Jung Date: Sat, 8 Jan 2011 17:58:48 +0100 Subject: Minor fixes and improvements The following has been changed - Note::set_param now sets orig-values and does an apply_pfactor() - Note::reattack now also reattacks the filter's envelope - oscval[i]=0 has been removed as it's unneccessary in Note::get_sample() - in Note::get_sample: fm- and out-sums are calculated in 2SCALE now and scaled down (via >>SCALE) afterwards (instead of while every single add-operation) - the filter-envelope is now inited with the shorter constructor - todo-list updated --- TODO | 22 ++++++++++++---------- synth/note.cpp | 30 +++++++++++++++--------------- 2 files changed, 27 insertions(+), 25 deletions(-) diff --git a/TODO b/TODO index e031b3f..f8155f6 100644 --- a/TODO +++ b/TODO @@ -10,32 +10,34 @@ TODO für den synth o envelope, filter, ggf. auch alles aus program.o im hauptprogramm lassen? d.h. via init funktionspointer übergeben oder virtuelle interfaceklassen benutzen (für envelope/filter z.B.) - o wave auf int*[] mit wave[i]=new int[] umbauen + x wave auf int*[] mit wave[i]=new int[] umbauen -!!!o BUG: bei Note::set_param sollte statt dem eigentlichen param +!!!x BUG: bei Note::set_param sollte statt dem eigentlichen param der orig gesetzt werden und dann neu berechnet werden! -!!!o BUG: bei release und reattack: BEIDE male die filter-envelope beachten! +!!!x BUG: bei release und reattack: BEIDE male die filter-envelope beachten! o KSL mit powf und floats statt mit double umschreiben o statt lfo-nummer direkten zugriff auf curr_lfo angeben? - o oscval-nullen kann in get_sample() weggelassen werden + x oscval-nullen kann in get_sample() weggelassen werden o bei tremolo (und vibrato?): eventuell nicht prüfen, obs aktiviert ist, sondern zur not einfach *1 rechnen? - o in get_sample(), beim aufaddieren zum out (und vmtl auf fm-)wert: + x in get_sample(), beim aufaddieren zum out (und vmtl auch fm-)wert: erst nach dem addieren scalen, statt für jeden faktor einzeln o wenn sync global aus ist, kann auch osc.sync genullt werden o beim default_program vielleicht auch ein optimiertes objekt benutzen? - o filter_envelope könnte mit anderem ctor geinitet werden (weniger schreibarbeit) - o notes compilieren und als .so-datei laden + x filter_envelope könnte mit anderem ctor geinitet werden (weniger schreibarbeit) + x notes compilieren und als .so-datei laden o programme on-the-fly (um)laden - o parser-klasse durch parse-funktion ersetzen + x parser-klasse durch parse-funktion ersetzen - o reattack a) garnicht, b) immer, c) nur wenn schon in releasephase + x reattack a) garnicht, b) immer, c) nur wenn schon in releasephase --> legatopedal! o bei envelopes: releasephase abschalten (bei sustain bleiben) + o soft-pedal + o RAM aufräumen? o jedes programm eigene LFOs? @@ -56,7 +58,7 @@ TODO für den synth o max_pitchbend per controller setzen? o nur auf bestimmte channels reagieren - o diverse pedale (soft, sostenuto, halte, legato (?)) + x diverse pedale (soft, sostenuto, halte, legato (?)) (o)lfo-maxima getrennt regeln. nää (o)bei filter-envelopes: ksr/ksl? nää. diff --git a/synth/note.cpp b/synth/note.cpp index a0d6226..86e4b2b 100644 --- a/synth/note.cpp +++ b/synth/note.cpp @@ -72,13 +72,7 @@ Note::Note(int n, float v, program_t &prg, jack_nframes_t pf, fixed_t pb, int pr if (filter_params.enabled) { - filter_envelope=new Envelope( - filter_params.env_settings.attack, - filter_params.env_settings.decay, - filter_params.env_settings.sustain, - filter_params.env_settings.release, - filter_params.env_settings.hold ); - + filter_envelope=new Envelope(filter_params.env_settings); filter_update_counter=filter_update_frames; } @@ -170,8 +164,8 @@ void Note::set_param(const parameter_t &p, fixed_t v) //ACHTUNG: case KSL: oscillator[p.osc].ksl=float(v)/ONE; break; case FACTOR: oscillator[p.osc].factor=v; break; - case MODULATION: oscillator[p.osc].fm_strength[p.index]=v*pfactor.fm[p.osc][p.index] >>SCALE; break; - case OUTPUT: oscillator[p.osc].output=v*pfactor.out[p.osc] >>SCALE; break; + case MODULATION: orig.oscillator[p.osc].fm_strength[p.index]=v; apply_pfactor(); break; + case OUTPUT: orig.oscillator[p.osc].output=v; apply_pfactor(); break; case TREMOLO: oscillator[p.osc].tremolo_depth=v; break; case TREM_LFO: oscillator[p.osc].tremolo_lfo=v; break; case VIBRATO: oscillator[p.osc].vibrato_depth=v; break; @@ -180,7 +174,7 @@ void Note::set_param(const parameter_t &p, fixed_t v) //ACHTUNG: case SYNC: oscillator[p.osc].sync=(v!=0); break; case FILTER_ENABLED: output_note("NOTE: cannot enable filter in playing notes"); break; - case FILTER_ENV_AMOUNT: filter_params.env_amount=float(v*pfactor.filter_env)/ONE/ONE; break; + case FILTER_ENV_AMOUNT: orig.filter_params.env_amount=float(v)/ONE; apply_pfactor(); break; case FILTER_ATTACK: if (filter_params.enabled) @@ -217,8 +211,8 @@ void Note::set_param(const parameter_t &p, fixed_t v) //ACHTUNG: output_note("NOTE: cannot set filter-hold when filter is disabled"); break; - case FILTER_OFFSET: filter_params.freqfactor_offset=float(v*pfactor.filter_offset)/ONE/ONE; break; - case FILTER_RESONANCE: filter_params.resonance=float(v*pfactor.filter_res)/ONE/ONE; break; + case FILTER_OFFSET: orig.filter_params.freqfactor_offset=float(v)/ONE; apply_pfactor(); break; + case FILTER_RESONANCE: orig.filter_params.resonance=float(v)/ONE; apply_pfactor(); break; case FILTER_TREMOLO: filter_params.trem_strength=v; break; case FILTER_TREM_LFO: filter_params.trem_lfo=v; break; @@ -268,6 +262,9 @@ void Note::reattack() { for (int i=0;ireattack(); + + if (filter_params.enabled) + filter_envelope->reattack(); } @@ -340,12 +337,13 @@ fixed_t Note::get_sample() for (i=0;i>SCALE; + fm+=old_oscval[j]*oscillator[i].fm_strength[j]; + fm=fm>>SCALE; + //phase increases in one second, i.e. in samp_rate frames, by the osc's freq if (oscillator[i].vibrato_depth!=0) oscillator[i].phase+=( (curr_lfo[oscillator[i].vibrato_lfo][oscillator[i].vibrato_depth]*actual_freq >>SCALE)*oscillator[i].factor/samp_rate)>>SCALE; @@ -368,9 +366,11 @@ fixed_t Note::get_sample() oscval[i]=oscval[i]* curr_lfo[oscillator[i].tremolo_lfo][oscillator[i].tremolo_depth] >> SCALE; if (oscillator[i].output!=0) - out+=oscillator[i].output*oscval[i] >>SCALE; + out+=oscillator[i].output*oscval[i]; } + out=out>>SCALE; + if (filter_params.enabled) { filter_update_counter++; -- cgit v1.2.1